From mboxrd@z Thu Jan 1 00:00:00 1970 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 988FAC433EF for ; Tue, 25 Jan 2022 16:53:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 15D2F6B00B5; Tue, 25 Jan 2022 11:53:40 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 10CE96B00B6; Tue, 25 Jan 2022 11:53:40 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F3D4F6B00B7; Tue, 25 Jan 2022 11:53:39 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (relay029.a.hostedemail.com [64.99.140.29]) by kanga.kvack.org (Postfix) with ESMTP id E3EE96B00B5 for ; Tue, 25 Jan 2022 11:53:39 -0500 (EST) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A1B0322174 for ; Tue, 25 Jan 2022 16:53:39 +0000 (UTC) X-FDA: 79069405758.03.E6BAFE3 Received: from mail.sf-mail.de (mail.sf-mail.de [116.202.16.50]) by imf15.hostedemail.com (Postfix) with ESMTP id 5E0A6A0087 for ; Tue, 25 Jan 2022 16:53:37 +0000 (UTC) Received: (qmail 10734 invoked from network); 25 Jan 2022 16:52:27 -0000 Received: from p200300cf0713e20076d435fffeb7be92.dip0.t-ipconnect.de ([2003:cf:713:e200:76d4:35ff:feb7:be92]:48794 HELO eto.sf-tec.de) (auth=eike@sf-mail.de) by mail.sf-mail.de (Qsmtpd 0.38dev) with (TLS_AES_256_GCM_SHA384 encrypted) ESMTPSA for ; Tue, 25 Jan 2022 17:52:27 +0100 From: Rolf Eike Beer To: linux-mm@kvack.org, Anshuman Khandual Cc: linux-kernel@vger.kernel.org, hch@infradead.org, akpm@linux-foundation.org, Anshuman Khandual , "James E.J. Bottomley" , linux-parisc@vger.kernel.org Subject: Re: [RFC V1 21/31] parisc/mm: Enable ARCH_HAS_VM_GET_PAGE_PROT Date: Tue, 25 Jan 2022 17:53:27 +0100 Message-ID: <11909976.O9o76ZdvQC@eto.sf-tec.de> In-Reply-To: <1643029028-12710-22-git-send-email-anshuman.khandual@arm.com> References: <1643029028-12710-1-git-send-email-anshuman.khandual@arm.com> <1643029028-12710-22-git-send-email-anshuman.khandual@arm.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart5535662.DvuYhMxLoT"; micalg="pgp-sha1"; protocol="application/pgp-signature" X-Stat-Signature: ufy513ks7hc38s1csje1z41epu8t7nn5 X-Rspam-User: nil Authentication-Results: imf15.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf15.hostedemail.com: domain of eike-kernel@sf-tec.de designates 116.202.16.50 as permitted sender) smtp.mailfrom=eike-kernel@sf-tec.de X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 5E0A6A0087 X-HE-Tag: 1643129617-246497 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: --nextPart5535662.DvuYhMxLoT Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Anshuman Khandual wrote: > This defines and exports a platform specific custom vm_get_page_prot() via > subscribing ARCH_HAS_VM_GET_PAGE_PROT. Subsequently all __SXXX and __PXXX > macros can be dropped which are no longer needed. > > Cc: "James E.J. Bottomley" > Cc: linux-parisc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Anshuman Khandual > --- > arch/parisc/Kconfig | 1 + > arch/parisc/include/asm/pgtable.h | 20 --------------- > arch/parisc/mm/init.c | 41 +++++++++++++++++++++++++++++++ > 3 files changed, 42 insertions(+), 20 deletions(-) > > diff --git a/arch/parisc/Kconfig b/arch/parisc/Kconfig > index 43c1c880def6..de512f120b50 100644 > --- a/arch/parisc/Kconfig > +++ b/arch/parisc/Kconfig > @@ -10,6 +10,7 @@ config PARISC > select ARCH_HAS_ELF_RANDOMIZE > select ARCH_HAS_STRICT_KERNEL_RWX > select ARCH_HAS_UBSAN_SANITIZE_ALL > + select ARCH_HAS_VM_GET_PAGE_PROT > select ARCH_NO_SG_CHAIN > select ARCH_SUPPORTS_HUGETLBFS if PA20 > select ARCH_SUPPORTS_MEMORY_FAILURE > diff --git a/arch/parisc/include/asm/pgtable.h > b/arch/parisc/include/asm/pgtable.h index 3e7cf882639f..80d99b2b5913 100644 > --- a/arch/parisc/include/asm/pgtable.h > +++ b/arch/parisc/include/asm/pgtable.h > @@ -269,26 +269,6 @@ extern void __update_cache(pte_t pte); > * pages. > */ > > - /*xwr*/ > -#define __P000 PAGE_NONE > -#define __P001 PAGE_READONLY > -#define __P010 __P000 /* copy on write */ > -#define __P011 __P001 /* copy on write */ > -#define __P100 PAGE_EXECREAD > -#define __P101 PAGE_EXECREAD > -#define __P110 __P100 /* copy on write */ > -#define __P111 __P101 /* copy on write */ > - > -#define __S000 PAGE_NONE > -#define __S001 PAGE_READONLY > -#define __S010 PAGE_WRITEONLY > -#define __S011 PAGE_SHARED > -#define __S100 PAGE_EXECREAD > -#define __S101 PAGE_EXECREAD > -#define __S110 PAGE_RWX > -#define __S111 PAGE_RWX > - > - > extern pgd_t swapper_pg_dir[]; /* declared in init_task.c */ > > /* initial page tables for 0-8MB for kernel */ > diff --git a/arch/parisc/mm/init.c b/arch/parisc/mm/init.c > index 1ae31db9988f..c8316e97e1a2 100644 > --- a/arch/parisc/mm/init.c > +++ b/arch/parisc/mm/init.c > @@ -866,3 +866,44 @@ void flush_tlb_all(void) > spin_unlock(&sid_lock); > } > #endif > + > +pgprot_t vm_get_page_prot(unsigned long vm_flags) > +{ > + switch (vm_flags & (VM_READ | VM_WRITE | VM_EXEC | VM_SHARED)) { > + case VM_NONE: > + return PAGE_NONE; > + case VM_READ: > + return PAGE_READONLY; > + case VM_WRITE: > + return PAGE_NONE; > + case VM_READ | VM_WRITE: > + return PAGE_READONLY; This looks extremely strange. It probably is correct when it comes to CoW, how about including the comment that was in the original definitions for the cases where CoW is expected? > + case VM_EXEC: > + return PAGE_EXECREAD; > + case VM_EXEC | VM_READ: > + return PAGE_EXECREAD; > + case VM_EXEC | VM_WRITE: > + return PAGE_EXECREAD; > + case VM_EXEC | VM_READ | VM_WRITE: > + return PAGE_EXECREAD; > + case VM_SHARED: > + return PAGE_NONE; > + case VM_SHARED | VM_READ: > + return PAGE_READONLY; > + case VM_SHARED | VM_WRITE: > + return PAGE_WRITEONLY; > + case VM_SHARED | VM_READ | VM_WRITE: > + return PAGE_SHARED; > + case VM_SHARED | VM_EXEC: > + return PAGE_EXECREAD; > + case VM_SHARED | VM_EXEC | VM_READ: > + return PAGE_EXECREAD; > + case VM_SHARED | VM_EXEC | VM_WRITE: > + return PAGE_RWX; > + case VM_SHARED | VM_EXEC | VM_READ | VM_WRITE: > + return PAGE_RWX; > + default: > + BUILD_BUG(); > + } > +} > +EXPORT_SYMBOL(vm_get_page_prot); --nextPart5535662.DvuYhMxLoT Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- iF0EABECAB0WIQSaYVDeqwKa3fTXNeNcpIk+abn8TgUCYfArBwAKCRBcpIk+abn8 Tn3fAJ9mCiqlAWz0LSE02l6g56q9bKoqEgCfTTU/KTI/Xf7FeIAR89tBwIQgI/A= =Hgfz -----END PGP SIGNATURE----- --nextPart5535662.DvuYhMxLoT--