From patchwork Mon Oct 29 06:29:59 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ashish Mhetre X-Patchwork-Id: 10658787 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 6ACFD14DE for ; Mon, 29 Oct 2018 06:31:46 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 56EF4294FB for ; Mon, 29 Oct 2018 06:31:46 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 493592950F; Mon, 29 Oct 2018 06:31:46 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 9CC33294FB for ; Mon, 29 Oct 2018 06:31:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=3X82q4kJCHeiXNc7oIVGzj5bqbR1YHJ+KSYfTEEzIa8=; b=nzoZqPrLDNRrZx 6X2a7DgVj3j32OE/S/tNTS9Ef+ts/cfu5nUV9wBIQfDi28YyM68grQ61jn3IQIzmizde2IgbppRDm mMRj15lPa6yWoZdionu69TY9FH0p4w367h4wDh+m5bmRDaZ4pxcX1d+Gm8QWs++LyJk1XGioLBuCf e/rpmUYnZK19T2xUfo5vbBt9ZGOcbismLF0EuXg+Bx9XXoGZsldJ215FoWGlMZ/gMN0kpfv3D4Pn8 0Z6GOCPnvNXh2Pxb+3x+ZFB9eepkQe1gTMTFTHHXi8YUutKXyJkLHNW7euz8T35F0101JHo660jri gsrvf884KTwMU8P1VD6g==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gH15N-0006RY-Oy; Mon, 29 Oct 2018 06:31:37 +0000 Received: from hqemgate14.nvidia.com ([216.228.121.143]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gH15K-0006QO-9R for linux-arm-kernel@lists.infradead.org; Mon, 29 Oct 2018 06:31:35 +0000 Received: from hqpgpgate101.nvidia.com (Not Verified[216.228.121.13]) by hqemgate14.nvidia.com (using TLS: TLSv1.2, DES-CBC3-SHA) id ; Sun, 28 Oct 2018 23:31:10 -0700 Received: from HQMAIL106.nvidia.com ([172.20.161.6]) by hqpgpgate101.nvidia.com (PGP Universal service); Sun, 28 Oct 2018 23:31:22 -0700 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Sun, 28 Oct 2018 23:31:22 -0700 Received: from HQMAIL101.nvidia.com (172.20.187.10) by HQMAIL106.nvidia.com (172.18.146.12) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Mon, 29 Oct 2018 06:31:22 +0000 Received: from hqnvemgw01.nvidia.com (172.20.150.20) by HQMAIL101.nvidia.com (172.20.187.10) with Microsoft SMTP Server (TLS) id 15.0.1395.4 via Frontend Transport; Mon, 29 Oct 2018 06:31:22 +0000 Received: from amhetre.nvidia.com (Not Verified[10.24.229.42]) by hqnvemgw01.nvidia.com with Trustwave SEG (v7, 5, 8, 10121) id ; Sun, 28 Oct 2018 23:31:21 -0700 From: Ashish Mhetre To: , , , Subject: [PATCH V2] arm64: Don't flush tlb while clearing the accessed bit Date: Mon, 29 Oct 2018 11:59:59 +0530 Message-ID: <1540794599-30922-1-git-send-email-amhetre@nvidia.com> X-Mailer: git-send-email 2.7.4 X-NVConfidentiality: public MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1540794670; bh=kP6gRkxdVxPNw0KVZ/gw4PC00mKxd4cqx2ODoYa5Fso=; h=X-PGP-Universal:From:To:CC:Subject:Date:Message-ID:X-Mailer: X-NVConfidentiality:MIME-Version:Content-Type; b=cCX0XkJnodU6KQYn9O+YSaZQutJBpcJuz9VIAo6/mykkdf3MNuPuvwBWi6RGhxtAh dWqL4Ul8tMc4E6om6PE/gp24qEWDnsmG3nO82oBm2H627Tl9p3qIwsBEJ6HdIPy6s4 RsT4FI7jkBWWyQO0pWESROX0QJVCibu3gAUou9hNM69/aO1EuECVT+ms3EylG3N2da 0JTbxwa62hy53mVmwyts35s4fg4nPGpoiDYLWILeq1YNJ9IcNCdRk+kywb8fIkGfVt oFNcFODaSSdkU/b69lu5S5dnCK0b2rghC4I+wEL78Jh3p13hgr3Ql3JLS3CfvUJoRf SbPj2sqwfN00A== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181028_233134_342636_CE2D95AD X-CRM114-Status: UNSURE ( 8.12 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Ashish Mhetre , linux-kernel@vger.kernel.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP From: Alex Van Brunt Accessed bit is used to age a page and in generic implementation there is flush_tlb while clearing the accessed bit. Flushing a TLB is overhead on ARM64 as access flag faults don't get translation table entries cached into TLB's. Flushing TLB is not necessary for this. Clearing the accessed bit without flushing TLB doesn't cause data corruption on ARM64. [It may cause incorrect page aging but chances of that should be relatively low.] In our case with this patch, speed of reading from fast NVMe/SSD through PCIe got improved by 10% ~ 15% and writing got improved by 20% ~ 40%. So for performance optimisation don't flush TLB when clearing the accessed bit on ARM64. x86 made the same optimization even though their TLB invalidate is much faster as it doesn't broadcast to other CPUs. Signed-off-by: Alex Van Brunt Signed-off-by: Ashish Mhetre --- v2: Added comments about why flushing is not needed while clearing accessed bit arch/arm64/include/asm/pgtable.h | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h index 2ab2031..33e1940 100644 --- a/arch/arm64/include/asm/pgtable.h +++ b/arch/arm64/include/asm/pgtable.h @@ -652,6 +652,22 @@ static inline int ptep_test_and_clear_young(struct vm_area_struct *vma, return __ptep_test_and_clear_young(ptep); } +#define __HAVE_ARCH_PTEP_CLEAR_YOUNG_FLUSH +static inline int ptep_clear_flush_young(struct vm_area_struct *vma, + unsigned long address, pte_t *ptep) +{ + /* + * Flushing a TLB is overhead on ARM64 as access flag faults don't get + * translation table entries cached into TLB's. Flushing TLB is not + * necessary for this. Clearing the accessed bit without flushing TLB + * doesn't cause data corruption on ARM64.[ It may cause imcorrect page + * aging but chances of this should be comparatively low. ] + * So as a performance optimization don't flush the TLB when clearing + * the accessed bit. + */ + return ptep_test_and_clear_young(vma, address, ptep); +} + #ifdef CONFIG_TRANSPARENT_HUGEPAGE #define __HAVE_ARCH_PMDP_TEST_AND_CLEAR_YOUNG static inline int pmdp_test_and_clear_young(struct vm_area_struct *vma,