From patchwork Sun Dec 20 04:55:34 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicholas Piggin X-Patchwork-Id: 11983781 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.5 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7A133C4361B for ; Sun, 20 Dec 2020 04:55:50 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 2169723A5E for ; Sun, 20 Dec 2020 04:55:50 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2169723A5E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id B68ED6B0068; Sat, 19 Dec 2020 23:55:49 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id B19836B006C; Sat, 19 Dec 2020 23:55:49 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9E2806B006E; Sat, 19 Dec 2020 23:55:49 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0204.hostedemail.com [216.40.44.204]) by kanga.kvack.org (Postfix) with ESMTP id 83FF76B0068 for ; Sat, 19 Dec 2020 23:55:49 -0500 (EST) Received: from smtpin22.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 422318249980 for ; Sun, 20 Dec 2020 04:55:49 +0000 (UTC) X-FDA: 77612448018.22.anger03_0a05bd32744c Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin22.hostedemail.com (Postfix) with ESMTP id 287CF18038E67 for ; Sun, 20 Dec 2020 04:55:49 +0000 (UTC) X-HE-Tag: anger03_0a05bd32744c X-Filterd-Recvd-Size: 6435 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by imf33.hostedemail.com (Postfix) with ESMTP for ; Sun, 20 Dec 2020 04:55:48 +0000 (UTC) Received: by mail-pl1-f174.google.com with SMTP id e2so3822505plt.12 for ; Sat, 19 Dec 2020 20:55:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=S0i+SiU+HSwLByUxXA2fjHZNAtDQPgxviRKysSaSS3w=; b=i8gXCGtTmntMbjJF+dcARSh3FSXoytamZ1YK2mrv8qsRVKw9tHR7Y7Zarxlplra4YD eyYmwItiWgWa1dh6votrLxg3Kgzu0xBlIzbZIQQ8g7T7L6e+90drgkZrcOdoUt4iMd6l NY299oxJc3+kXrncmocritYske70k6woY2N2yg+h3eeqBqgWgwvvyXLWlr3N8OcKYUyp 2qb+vCJ3jfNwCCFV/WS3eGaCdaEED7S1B1h6zyuCPdRuh6JjbZY8h8G5qxkRbg/QGISY daRzvefmTL7tkVAoTY+T/yRDdiLyqguZlhfs71Tag5FE0rBO6MVdwfBnQJfHB0jMl72M svng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=S0i+SiU+HSwLByUxXA2fjHZNAtDQPgxviRKysSaSS3w=; b=BGs0V6cSj7GFdCDBu1l4xIwtRBL12ah6RxyLhY0M+gUKOifvFc8Gs+kbPdJiB2cWHg VeTSEol5zcuxda3XhWCCZmRwkt9Cv2eadija/sFOHyG7ugiF2TLnQwWNlnNVZTQZO2bI zsJUpsGZpqKhfshG8D3ZIVLLIwm6eEjaJ+qoWvj10fmHEBwBu1Y+l4ycg3kyJcLwCexm AXS+KFBR8zHHBxGJDmh5p/2UwG5yEPlMrdeObLOUFvR0PViUrtb+YuG1HikxAsI2oMHY 0Qg9CmJcMInrNTFs+BDjDJnIgBgwSG9/eszptMvUZxqC3UOZbuUo68tQ2sDmIKLiwPJB VL2Q== X-Gm-Message-State: AOAM533rz7HLEjtUILgMXXfuFNyhM5XwwdNXwyQfsvuWiPhu20uWvozB o0+pb26nLS6/4mdVjbn/DMu+SuOd0hA= X-Google-Smtp-Source: ABdhPJx8xhUTo1at8PlNARbDVpR3p/TT1iGAX/C6QjTlYX0wqujyS56OlrJBL5wGh42KW0UWAjF2pQ== X-Received: by 2002:a17:902:6903:b029:da:f458:798c with SMTP id j3-20020a1709026903b02900daf458798cmr11291063plk.68.1608440147810; Sat, 19 Dec 2020 20:55:47 -0800 (PST) Received: from bobo.ibm.com (193-116-97-30.tpgi.com.au. [193.116.97.30]) by smtp.gmail.com with ESMTPSA id l13sm13632529pgq.51.2020.12.19.20.55.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 19 Dec 2020 20:55:47 -0800 (PST) From: Nicholas Piggin To: linux-mm@kvack.org Cc: Nicholas Piggin , Linus Torvalds , Bibo Mao Subject: [PATCH v3 2/3] mm/cow: optimise pte accessed bit handling in fork Date: Sun, 20 Dec 2020 14:55:34 +1000 Message-Id: <20201220045535.848591-3-npiggin@gmail.com> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20201220045535.848591-1-npiggin@gmail.com> References: <20201220045535.848591-1-npiggin@gmail.com> MIME-Version: 1.0 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: fork clears dirty/accessed bits from new ptes in the child. This logic has existed since mapped page reclaim was done by scanning ptes when it may have been quite important. Today with physical based pte scanning, there is less reason to clear these bits, so this patch avoids clearing the accessed bit in the child. Any accessed bit is treated similarly to many, with the difference today with > 1 referenced bit causing the page to be activated, while 1 bit causes it to be kept. This patch causes pages shared by fork(2) to be more readily activated, but this heuristic is very fuzzy anyway -- a page can be accessed by multiple threads via a single pte and be just as important as one that is accessed via multiple ptes, for example. In the end I don't believe fork(2) is a significant driver of page reclaim behaviour that this should matter too much. This and the following change eliminate a major source of faults that powerpc/radix requires to set dirty/accessed bits in ptes, speeding up a fork/exit microbenchmark by about 5% on POWER9 (16600 -> 17500 fork/execs per second). Skylake appears to have a micro-fault overhead too -- a test which allocates 4GB anonymous memory, reads each page, then forks, and times the child reading a byte from each page. The first pass over the pages takes about 1000 cycles per page, the second pass takes about 27 cycles (TLB miss). With no additional minor faults measured due to either child pass, and the page array well exceeding TLB capacity, the large cost must be micro faults caused by setting the accessed bit. Signed-off-by: Nicholas Piggin --- mm/huge_memory.c | 2 -- mm/memory.c | 1 - mm/vmscan.c | 5 +++++ 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/mm/huge_memory.c b/mm/huge_memory.c index 87da60c583a9..f2ca0326b5af 100644 --- a/mm/huge_memory.c +++ b/mm/huge_memory.c @@ -1115,7 +1115,6 @@ int copy_huge_pmd(struct mm_struct *dst_mm, struct mm_struct *src_mm, pmdp_set_wrprotect(src_mm, addr, src_pmd); pmd = pmd_wrprotect(pmd); } - pmd = pmd_mkold(pmd); set_pmd_at(dst_mm, addr, dst_pmd, pmd); ret = 0; @@ -1225,7 +1224,6 @@ int copy_huge_pud(struct mm_struct *dst_mm, struct mm_struct *src_mm, pudp_set_wrprotect(src_mm, addr, src_pud); pud = pud_mkold(pud_wrprotect(pud)); } - pud = pud_mkold(pud); set_pud_at(dst_mm, addr, dst_pud, pud); ret = 0; diff --git a/mm/memory.c b/mm/memory.c index 990e5d704c08..dd1f364d8ca3 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -886,7 +886,6 @@ copy_present_pte(struct vm_area_struct *dst_vma, struct vm_area_struct *src_vma, */ if (vm_flags & VM_SHARED) pte = pte_mkclean(pte); - pte = pte_mkold(pte); /* * Make sure the _PAGE_UFFD_WP bit is cleared if the new VMA diff --git a/mm/vmscan.c b/mm/vmscan.c index 257cba79a96d..604ead623842 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1012,6 +1012,11 @@ static enum page_references page_check_references(struct page *page, * Note: the mark is set for activated pages as well * so that recently deactivated but used pages are * quickly recovered. + * + * Note: fork() will copy referenced bit from parent + * to child ptes, despite not having been accessed by + * the child. This is to avoid micro-faults on initial + * access. */ SetPageReferenced(page);