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 A7F1AC7115E for ; Thu, 19 Jun 2025 08:42:33 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 36DCA6B0093; Thu, 19 Jun 2025 04:42:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 344B86B0095; Thu, 19 Jun 2025 04:42:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 281EF6B0096; Thu, 19 Jun 2025 04:42:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 182216B0093 for ; Thu, 19 Jun 2025 04:42:33 -0400 (EDT) Received: from smtpin22.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id A3F1010123E for ; Thu, 19 Jun 2025 08:42:32 +0000 (UTC) X-FDA: 83571508944.22.5ABE57D Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf06.hostedemail.com (Postfix) with ESMTP id 0896A18000D for ; Thu, 19 Jun 2025 08:42:30 +0000 (UTC) Authentication-Results: imf06.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=SErdEUma; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf06.hostedemail.com: domain of brauner@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=brauner@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1750322551; a=rsa-sha256; cv=none; b=xtHE7zKMkvPe/3WrgD2qD31ThrUWmRNz4HQSw1yNsNgGQrnMdnuzjqKE44xFG3+ileS+TJ RZZgcrm1O3OXzSrtgA/S5Ab7P/ZHjNBpurCSyVxOcNEDiHesQaXziRwSxnuMxW/mlek0ra MZyU1T1jlB1irkTnppaxREYMBEO0DgQ= ARC-Authentication-Results: i=1; imf06.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=SErdEUma; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf06.hostedemail.com: domain of brauner@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=brauner@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1750322551; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=kWumWeDPcZU7fhri+6IT3usAKUMUHf511Di79l7CmcQ=; b=Vj8Tc4zohNV1zJ7V4KfbDX1TB0+rRzuGEcNlJ1Kf2yWpEXf78kMLmvJdkEJ5+9pm+XR5Yo xdwKmpiMjXkpBcsVLhAaD3bBXb4gfm1BbmwaxIOQ/DNUE8I68aKhReJMfEIKFZ46PtzyiK l9EwvkQWEebvQQ5OkyU8J6quqzNRQBY= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 65EE85C64CA; Thu, 19 Jun 2025 08:40:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 04FBBC4CEEA; Thu, 19 Jun 2025 08:42:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1750322549; bh=3VFuG2FOQta3qfo/tXfeyceWpM2CtNcoqXoxG/UtwWc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=SErdEUmaRiXUtAdabhdTNYYV8q5kEViKHeMvbdP07i8SdJqnyGDweje7/hf5ob2r1 Y7GakfurOZd/kO55Uuejfbjb1S87B2AsLMEOZS16N3PqMiLkRzq0uK6SYLU7qgvTU/ SVw+UbeIJFOnBUY3xSh5j8mzLsb/i9WbBp9W6Nyzn5AswOiCpHExDyjKEa4xghVxhq qVMVSR8Ah8c9+Gp0UD1wpCI2KA3aEvgMXWU2aB9vqWItkHQLyOH3Ng4vCX7N4MSO38 GNkOCm/RoBAbDnsH25JuklGSA+93rTqS16MvDDHIq0NGhvg1unBUyT6Up6WtuKaRQA 0XljRO/7NmUxw== Date: Thu, 19 Jun 2025 10:42:14 +0200 From: Christian Brauner To: Lorenzo Stoakes Cc: Andrew Morton , Russell King , Catalin Marinas , Will Deacon , Madhavan Srinivasan , Michael Ellerman , Nicholas Piggin , Christophe Leroy , "David S . Miller" , Andreas Larsson , Jarkko Sakkinen , Dave Hansen , Thomas Gleixner , Ingo Molnar , Borislav Petkov , "H . Peter Anvin" , Andy Lutomirski , Peter Zijlstra , Alexander Viro , Jan Kara , Kees Cook , Peter Xu , David Hildenbrand , Zi Yan , Baolin Wang , "Liam R . Howlett" , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Xu Xin , Chengming Zhou , Hugh Dickins , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Rik van Riel , Harry Yoo , Dan Williams , Matthew Wilcox , Steven Rostedt , Masami Hiramatsu , Mathieu Desnoyers , Jason Gunthorpe , John Hubbard , Muchun Song , Oscar Salvador , Jann Horn , Pedro Falcato , Johannes Weiner , Qi Zheng , Shakeel Butt , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, kvm@vger.kernel.org, sparclinux@vger.kernel.org, linux-sgx@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, nvdimm@lists.linux.dev, linux-trace-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] mm: change vm_get_page_prot() to accept vm_flags_t argument Message-ID: <20250619-unwiederholbar-addition-6875c99fe08d@brauner> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Rspam-User: X-Rspamd-Queue-Id: 0896A18000D X-Rspamd-Server: rspam10 X-Stat-Signature: j97ajzzqiy6coyzior4ifsxg4ijj8843 X-HE-Tag: 1750322550-484150 X-HE-Meta: U2FsdGVkX1+Qq60GyZnWVc2aN+G3hNHrCDOKQy7emvGDMqFlykPKSGSrUp41oqkxjbLgYhxS3vxC0RvviFDXxqtG3sAeIqVHClYn9aXspUuoeYuyq9DmY2gC6ZRltbFgroMHbc951gbKWyOOGCeMQ6MPYlyD7C1hwMCuT5RQQxOX9od0Bh4LFo85ZnvQzVptBWgf+x90GfUx0Z203HL8t07qDG7znSoiV95sbb7QeEudAzxpGGORetbuf0fg8hRdODqerQ7ALuu2Li50JRCp4eRCrv8gf2goVh01/PIjCQYMxjv/QvdiFRjFtOE14VUvAqsedd8VACDoS/L5W9uhbw8TBnJ9F1mX8n8ionkrDeBstuIxo24H6VJxkrOdF3CLFXKpkRIdPeWBBmSS0NSf0yuGaYQnZE5BYuX01Zfjo7+dIGJPGyQ1hFB7RycJVTEUNyPhtBLqhKZl+V2nH688Fy0EhfOS7Vbc+iEKIkq7SonHbOsb7RfNwO8RXWfkOjJ7lROK+RDBbj1p6EYrUCnFDjSybn1M7gPw4+IfP58Ep/Rs8WLdo0umis2YPeseMY5HzQHVpf/DK29LSBJ81O0PH5WGZmBDs+s16cSfvLjknzwvkIeTym6PPvCA/v43sSILI/3ovnjjMkdA8PhjGgauinTo7rN6NnSuyKcGBG4t4cKzDWDN33J9yx5EeaXlduC/WLonol3qmdthKN51p0ritwtQ28xlNyjW5fdJ7IM4kvv7rngU/kJLSZPN38/OwREkQ7NPzfEGRfSeK8iXIPnp7f1FPP5B/gCQKKrjMUFTUqUQyiqhwtKqUAeGG1BIpXX2ZC+q+laTT5kI1cv4yHwNfMv1NY/9Gn18nVymamIsVtZxqog88+yBeOwFMNcNrVkRPWwGdWdmqkUXA4X/k4n74FHo4S6isquQH/KQLKWHbkgj/78j5q6atewBNbveLmvH4C6/oAO8zAT0vbvm9Ac ekRcL1sD gRNvSNMOgzbtN0Vd2KgwYuIOrw6mjtvYHyjuVDNVbzLqI8tgrcffXFk4p+eVDUzAGUtQsp5z2TdmaKpaFbsXgjeeLwXErw/IM9ckDdE1eE+A0PSv45xXgVnjRSj22u4T/0IvPErT9a7ljcoKIVdyINNcj+5sjkfSSPALGO6FmXesxYHWEfxp12TbjWLnFxg7igYxg8GCMHgXIWOcyUEYBpKix8AnYNgOmuvU++8bcfK7r9kS1tZeqxHMk41HmaI0xcVhhoD4FF1EatrtZmwMIawZIKBzokqK0Y3b2DrstsschQaxWBHHJXnCrzhSV/LqNFUqRkwKFoke5mLx7uiX/6uewkxBWcsL93wMj+yQ/SPJCFIuu8o1yRPPu2znKGVol+AEhavS/iXoVv30= 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: List-Subscribe: List-Unsubscribe: On Wed, Jun 18, 2025 at 08:42:52PM +0100, Lorenzo Stoakes wrote: > We abstract the type of the VMA flags to vm_flags_t, however in may places > it is simply assumed this is unsigned long, which is simply incorrect. > > At the moment this is simply an incongruity, however in future we plan to > change this type and therefore this change is a critical requirement for > doing so. > > Overall, this patch does not introduce any functional change. > > Signed-off-by: Lorenzo Stoakes > --- > arch/arm64/mm/mmap.c | 2 +- > arch/powerpc/include/asm/book3s/64/pkeys.h | 3 ++- > arch/sparc/mm/init_64.c | 2 +- > arch/x86/mm/pgprot.c | 2 +- > include/linux/mm.h | 4 ++-- > include/linux/pgtable.h | 2 +- > tools/testing/vma/vma_internal.h | 2 +- > 7 files changed, 9 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/mm/mmap.c b/arch/arm64/mm/mmap.c > index c86c348857c4..08ee177432c2 100644 > --- a/arch/arm64/mm/mmap.c > +++ b/arch/arm64/mm/mmap.c > @@ -81,7 +81,7 @@ static int __init adjust_protection_map(void) > } > arch_initcall(adjust_protection_map); > > -pgprot_t vm_get_page_prot(unsigned long vm_flags) > +pgprot_t vm_get_page_prot(vm_flags_t vm_flags) > { > ptdesc_t prot; > > diff --git a/arch/powerpc/include/asm/book3s/64/pkeys.h b/arch/powerpc/include/asm/book3s/64/pkeys.h > index 5b178139f3c0..6f2075636591 100644 > --- a/arch/powerpc/include/asm/book3s/64/pkeys.h > +++ b/arch/powerpc/include/asm/book3s/64/pkeys.h > @@ -4,8 +4,9 @@ > #define _ASM_POWERPC_BOOK3S_64_PKEYS_H > > #include > +#include > > -static inline u64 vmflag_to_pte_pkey_bits(u64 vm_flags) > +static inline u64 vmflag_to_pte_pkey_bits(vm_flags_t vm_flags) If you change vm_flags_t to u64 you probably want to compile with some of these integer truncation options when you're doing the conversion. Because otherwise you risk silently truncating the upper 32bits when assigning to a 32bit variable. We've had had a patch series that almost introduced a very subtle bug when it tried to add the first flag outside the 32bit range in the lookup code a while ago. That series never made it but it just popped back into my head when I read your series. Acked-by: Christian Brauner