From patchwork Mon Oct 24 20:48:29 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Xu X-Patchwork-Id: 13018121 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id D86D1C38A2D for ; Mon, 24 Oct 2022 20:48:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6D86A94000A; Mon, 24 Oct 2022 16:48:48 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6616D940007; Mon, 24 Oct 2022 16:48:48 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4DBA094000A; Mon, 24 Oct 2022 16:48:48 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 3FC36940007 for ; Mon, 24 Oct 2022 16:48:48 -0400 (EDT) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 0327F120658 for ; Mon, 24 Oct 2022 20:48:47 +0000 (UTC) X-FDA: 80057031936.12.98EF80F Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by imf10.hostedemail.com (Postfix) with ESMTP id 82735C0037 for ; Mon, 24 Oct 2022 20:48:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1666644527; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=DtjgD11+A1aHTX9ijK54hqLSilN13SGt8oWF8m3PTwE=; b=g+kXHmVdZmYgqDCs7ecssQ5k8XOpC7ISftPliqqsVL8otvfQaurMySN2qXMaRDNLsPb/iE P/dXP7P3Y4h9pzbgj120+2CguahZyvxMv7gCIxj+4tCN3YivC1JB+WR/jsyDC2u/uGXd6O UZv2yYzr52TV+Kyhl/UETuyHdR1WePI= Received: from mail-qv1-f71.google.com (mail-qv1-f71.google.com [209.85.219.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-590-RRbvioxaMm21INrJ06RBHQ-1; Mon, 24 Oct 2022 16:48:45 -0400 X-MC-Unique: RRbvioxaMm21INrJ06RBHQ-1 Received: by mail-qv1-f71.google.com with SMTP id c6-20020ad44306000000b004bb8352cb4cso1118516qvs.14 for ; Mon, 24 Oct 2022 13:48:45 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=DtjgD11+A1aHTX9ijK54hqLSilN13SGt8oWF8m3PTwE=; b=thF+uw3xIReOjx2yPSfFcGENI8pIRy4d61R1aazc+4yA+hLy1ExAX3Elj2FyKERNaC T7SpvWfp/EdrTa1L4J37HZRS9OyjPLZxMDwPyh9VSLuCKWnDrTMy5zIRVb5jkl5Mhp2E tySUwXTju7cmk/1mtUIam9JQ/ZnojGxPTeExldn+ABX/aaguwlq6qWIdHasJuoPzbMzA 9H/nzo4jGExLgNPSdD2jPKXhYbX+8FEJehg0XOjGlp9fJalRWQR0RLospYYTAiLLagWO 8XJA4FOgYcVlpWEt2eTSQk1IbSGWOIu+V99qIw9Fnnjb+YHlS6CJyc6Nn8rKJFqfIMHf N5NA== X-Gm-Message-State: ACrzQf0ElbZvXDCt3n0JOEpVUwvtT7DH2bwvAT3K46SDublo+2wNxQyE pmZ3nGSRl3ATwx7/HzO7pi3+Y6zQvPESEBrbtaKU6Zy+dNNPAHhieaLKVNn9hJ4nYKpOmPXmR0r GAJRZr35k8kQ= X-Received: by 2002:ac8:5c82:0:b0:39c:d508:f1a8 with SMTP id r2-20020ac85c82000000b0039cd508f1a8mr28841672qta.142.1666644524963; Mon, 24 Oct 2022 13:48:44 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4oC3D5ScenLf0AO4sZWnMdtFGxOo0lGq1CGbaHFx86JYXFD9MW/8lK3Zu73fLeqWmyNTJMwg== X-Received: by 2002:a05:620a:12ec:b0:6ee:9e71:190 with SMTP id f12-20020a05620a12ec00b006ee9e710190mr23862434qkl.527.1666644514574; Mon, 24 Oct 2022 13:48:34 -0700 (PDT) Received: from x1n.redhat.com (bras-base-aurron9127w-grc-46-70-31-27-79.dsl.bell.ca. [70.31.27.79]) by smtp.gmail.com with ESMTPSA id d13-20020ac8544d000000b0038d9555b580sm498531qtq.44.2022.10.24.13.48.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Oct 2022 13:48:34 -0700 (PDT) From: Peter Xu To: linux-kernel@vger.kernel.org, linux-mm@kvack.org Cc: Huang Ying , peterx@redhat.com, Andrew Morton , HORIGUCHI NAOYA , Miaohe Lin , David Hildenbrand , Andrea Arcangeli Subject: [PATCH RFC 1/2] mm: Always compile in pte markers Date: Mon, 24 Oct 2022 16:48:29 -0400 Message-Id: <20221024204830.1342169-2-peterx@redhat.com> X-Mailer: git-send-email 2.37.3 In-Reply-To: <20221024204830.1342169-1-peterx@redhat.com> References: <20221024204830.1342169-1-peterx@redhat.com> MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-type: text/plain ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1666644527; a=rsa-sha256; cv=none; b=Yni7z2IkuWbr9GVYoiANUsot8uD5IfvzbrUqU9q4uWJcfSqsM91x3zquDpX/lO4Jy0++b1 1PBpvLcgF4dokJLoGvNMPSFHDwjuiilS0UBz8ezy5TV4LN/Plj+ZQNc28gUXQQgftTpS1b 8iKiDTXLTRSlkDHf+LSl4/NLl1LWR4g= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=g+kXHmVd; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf10.hostedemail.com: domain of peterx@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=peterx@redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1666644527; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=DtjgD11+A1aHTX9ijK54hqLSilN13SGt8oWF8m3PTwE=; b=AEmwm0KGK/d37/SzkHLss4tgudvZOOu5tVsEFLN1fVm5Bn2z8C2O6WWCsSXgLJgeXoaKPv 7ecZ/Pbl+GB07pWeTWxlVN/jf3xV62ukEZULwGvQPEojWF+RQbe1mLoh/C0Uc4D4WuSsR0 e8KVji6DfvpywvlEy8YSbBYDv+KDlws= X-Rspamd-Queue-Id: 82735C0037 Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=g+kXHmVd; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf10.hostedemail.com: domain of peterx@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=peterx@redhat.com X-Rspamd-Server: rspam12 X-Rspam-User: X-Stat-Signature: t8uc7fnc7up5wauycdcsgzfijc9cs4uh X-HE-Tag: 1666644527-383581 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: The PTE markers code is tiny and now it's enabled for most of the distributions. It's fine to keep it as-is, but to make a broader use of it (e.g. replacing read error swap entry) it needs to be there always otherwise we need special code path to take care of !PTE_MARKER case. It'll be easier just make pte marker always exist. Use this chance to extend its usage to anonymous too by simply touching up some of the old comments, because it'll be used for anonymous pages in the follow up patches. Signed-off-by: Peter Xu Reviewed-by: Miaohe Lin --- include/linux/swap.h | 10 +++------- include/linux/swapops.h | 31 ------------------------------- mm/Kconfig | 7 ------- mm/memory.c | 7 +++---- 4 files changed, 6 insertions(+), 49 deletions(-) diff --git a/include/linux/swap.h b/include/linux/swap.h index 369d7799205d..211aeca9bfa7 100644 --- a/include/linux/swap.h +++ b/include/linux/swap.h @@ -60,17 +60,13 @@ static inline int current_is_kswapd(void) SWP_MIGRATION_NUM + SWP_DEVICE_NUM + \ SWP_PTE_MARKER_NUM) /* - * PTE markers are used to persist information onto PTEs that are mapped with - * file-backed memories. As its name "PTE" hints, it should only be applied to - * the leaves of pgtables. + * PTE markers are used to persist information onto PTEs that otherwise + * should be a none pte. As its name "PTE" hints, it should only be + * applied to the leaves of pgtables. */ -#ifdef CONFIG_PTE_MARKER #define SWP_PTE_MARKER_NUM 1 #define SWP_PTE_MARKER (MAX_SWAPFILES + SWP_HWPOISON_NUM + \ SWP_MIGRATION_NUM + SWP_DEVICE_NUM) -#else -#define SWP_PTE_MARKER_NUM 0 -#endif /* * Unaddressable device memory support. See include/linux/hmm.h and diff --git a/include/linux/swapops.h b/include/linux/swapops.h index 86b95ccb81bb..41a6e7b23415 100644 --- a/include/linux/swapops.h +++ b/include/linux/swapops.h @@ -412,8 +412,6 @@ typedef unsigned long pte_marker; #define PTE_MARKER_UFFD_WP BIT(0) #define PTE_MARKER_MASK (PTE_MARKER_UFFD_WP) -#ifdef CONFIG_PTE_MARKER - static inline swp_entry_t make_pte_marker_entry(pte_marker marker) { return swp_entry(SWP_PTE_MARKER, marker); @@ -434,32 +432,6 @@ static inline bool is_pte_marker(pte_t pte) return is_swap_pte(pte) && is_pte_marker_entry(pte_to_swp_entry(pte)); } -#else /* CONFIG_PTE_MARKER */ - -static inline swp_entry_t make_pte_marker_entry(pte_marker marker) -{ - /* This should never be called if !CONFIG_PTE_MARKER */ - WARN_ON_ONCE(1); - return swp_entry(0, 0); -} - -static inline bool is_pte_marker_entry(swp_entry_t entry) -{ - return false; -} - -static inline pte_marker pte_marker_get(swp_entry_t entry) -{ - return 0; -} - -static inline bool is_pte_marker(pte_t pte) -{ - return false; -} - -#endif /* CONFIG_PTE_MARKER */ - static inline pte_t make_pte_marker(pte_marker marker) { return swp_entry_to_pte(make_pte_marker_entry(marker)); @@ -477,9 +449,6 @@ static inline pte_t make_pte_marker(pte_marker marker) * memory, kernel-only memory (including when the system is during-boot), * non-ram based generic file-system. It's fine to be used even there, but the * extra pte marker check will be pure overhead. - * - * For systems configured with !CONFIG_PTE_MARKER this will be automatically - * optimized to pte_none(). */ static inline int pte_none_mostly(pte_t pte) { diff --git a/mm/Kconfig b/mm/Kconfig index 57e1d8c5b505..4b28800d9be1 100644 --- a/mm/Kconfig +++ b/mm/Kconfig @@ -1107,17 +1107,10 @@ config HAVE_ARCH_USERFAULTFD_MINOR help Arch has userfaultfd minor fault support -config PTE_MARKER - bool - - help - Allows to create marker PTEs for file-backed memory. - config PTE_MARKER_UFFD_WP bool "Userfaultfd write protection support for shmem/hugetlbfs" default y depends on HAVE_ARCH_USERFAULTFD_WP - select PTE_MARKER help Allows to create marker PTEs for userfaultfd write protection diff --git a/mm/memory.c b/mm/memory.c index 25b12d1a7db0..71114f05a68c 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -3699,11 +3699,10 @@ static vm_fault_t handle_pte_marker(struct vm_fault *vmf) unsigned long marker = pte_marker_get(entry); /* - * PTE markers should always be with file-backed memories, and the - * marker should never be empty. If anything weird happened, the best - * thing to do is to kill the process along with its mm. + * PTE markers should never be empty. If anything weird happened, + * the best thing to do is to kill the process along with its mm. */ - if (WARN_ON_ONCE(vma_is_anonymous(vmf->vma) || !marker)) + if (WARN_ON_ONCE(!marker)) return VM_FAULT_SIGBUS; if (pte_marker_entry_uffd_wp(entry))