From: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
To: Arnd Bergmann <arnd@kernel.org>
Cc: linux-mm@kvack.org, Arnd Bergmann <arnd@arndb.de>,
"Jason A. Donenfeld" <Jason@zx2c4.com>,
Alexander Viro <viro@zeniv.linux.org.uk>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Andreas Larsson <andreas@gaisler.com>,
Andrew Morton <akpm@linux-foundation.org>,
Ard Biesheuvel <ardb@kernel.org>,
Christian Brauner <brauner@kernel.org>,
Christoph Hellwig <hch@lst.de>,
Christophe Leroy <christophe.leroy@csgroup.eu>,
Damien Le Moal <dlemoal@kernel.org>,
David Hildenbrand <david@redhat.com>,
Greg Ungerer <gerg@linux-m68k.org>, Helge Deller <deller@gmx.de>,
Kees Cook <kees@kernel.org>,
"Liam R . Howlett" <Liam.Howlett@oracle.com>,
Matt Turner <mattst88@gmail.com>,
Max Filippov <jcmvbkbc@gmail.com>,
Michael Ellerman <mpe@ellerman.id.au>,
Michal Hocko <mhocko@suse.com>,
Nicholas Piggin <npiggin@gmail.com>,
Richard Henderson <richard.henderson@linaro.org>,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Vladimir Murzin <vladimir.murzin@arm.com>,
Vlastimil Babka <vbabka@suse.cz>,
linux-stm32@st-md-mailman.stormreply.com,
linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org,
linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
linux-arch@vger.kernel.org
Subject: Re: [PATCH 4/5] asm-generic: use asm-generic/mman-common.h on parisc and alpha
Date: Wed, 9 Oct 2024 15:31:01 +0100 [thread overview]
Message-ID: <1cf8c09f-6b36-4650-a017-707f4bc41ca7@lucifer.local> (raw)
In-Reply-To: <20240925210615.2572360-5-arnd@kernel.org>
On Wed, Sep 25, 2024 at 09:06:14PM +0000, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> These two architectures each have their own set of MAP_* flags, like
> powerpc, mips and others do. In addition, the msync() flags are also
> different, here both define the same flags but in a different order.
> Finally, alpha also has a custom MADV_DONTNEED flag for madvise.
>
> Make the generic MADV_DONTNEED and MS_* definitions conditional on
> them already being defined and then include the common header
> header from both architectures, to remove the bulk of the contents.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Lovely! Look at all that red :) Great work!
Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> ---
> arch/alpha/include/uapi/asm/mman.h | 68 +++-----------------------
> arch/parisc/include/uapi/asm/mman.h | 66 +------------------------
> include/uapi/asm-generic/mman-common.h | 5 ++
> 3 files changed, 13 insertions(+), 126 deletions(-)
>
> diff --git a/arch/alpha/include/uapi/asm/mman.h b/arch/alpha/include/uapi/asm/mman.h
> index 1f1c03c047ce..fc8b74aa3f89 100644
> --- a/arch/alpha/include/uapi/asm/mman.h
> +++ b/arch/alpha/include/uapi/asm/mman.h
> @@ -2,18 +2,6 @@
> #ifndef __ALPHA_MMAN_H__
> #define __ALPHA_MMAN_H__
>
> -#define PROT_READ 0x1 /* page can be read */
> -#define PROT_WRITE 0x2 /* page can be written */
> -#define PROT_EXEC 0x4 /* page can be executed */
> -#ifndef PROT_SEM /* different on mips and xtensa */
> -#define PROT_SEM 0x8 /* page may be used for atomic ops */
> -#endif
> -/* 0x10 reserved for arch-specific use */
> -/* 0x20 reserved for arch-specific use */
> -#define PROT_NONE 0x0 /* page can not be accessed */
> -#define PROT_GROWSDOWN 0x01000000 /* mprotect flag: extend change to start of growsdown vma */
> -#define PROT_GROWSUP 0x02000000 /* mprotect flag: extend change to end of growsup vma */
> -
> /* 0x01 - 0x03 are defined in linux/mman.h */
> #define MAP_TYPE 0x0f /* Mask for type of mapping (OSF/1 is _wrong_) */
> #define MAP_FIXED 0x100 /* Interpret addr exactly */
> @@ -43,62 +31,18 @@
> #define MCL_ONFAULT 32768 /* lock all pages that are faulted in */
>
> /*
> - * Flags for mlock
> - */
> -#define MLOCK_ONFAULT 0x01 /* Lock pages in range after they are faulted in, do not prefault */
> -
> -/*
> - * Flags for msync
> + * Flags for msync, order is different from all others
> */
> #define MS_ASYNC 1 /* sync memory asynchronously */
> #define MS_SYNC 2 /* synchronous memory sync */
> #define MS_INVALIDATE 4 /* invalidate the caches */
>
> -#define MADV_NORMAL 0 /* no further special treatment */
> -#define MADV_RANDOM 1 /* expect random page references */
> -#define MADV_SEQUENTIAL 2 /* expect sequential page references */
> -#define MADV_WILLNEED 3 /* will need these pages */
> -#define MADV_DONTNEED 6 /* don't need these pages */
> +/*
> + * Flags for madvise, 1 through 3 are normal
> + */
> /* originally MADV_SPACEAVAIL 5 */
> +#define MADV_DONTNEED 6 /* don't need these pages */
>
> -/* common parameters: try to keep these consistent across architectures */
> -#define MADV_FREE 8 /* free pages only if memory pressure */
> -#define MADV_REMOVE 9 /* remove these pages & resources */
> -#define MADV_DONTFORK 10 /* don't inherit across fork */
> -#define MADV_DOFORK 11 /* do inherit across fork */
> -
> -#define MADV_MERGEABLE 12 /* KSM may merge identical pages */
> -#define MADV_UNMERGEABLE 13 /* KSM may not merge identical pages */
> -
> -#define MADV_HUGEPAGE 14 /* Worth backing with hugepages */
> -#define MADV_NOHUGEPAGE 15 /* Not worth backing with hugepages */
> -
> -#define MADV_DONTDUMP 16 /* Explicity exclude from the core dump,
> - overrides the coredump filter bits */
> -#define MADV_DODUMP 17 /* Clear the MADV_DONTDUMP flag */
> -
> -#define MADV_WIPEONFORK 18 /* Zero memory on fork, child only */
> -#define MADV_KEEPONFORK 19 /* Undo MADV_WIPEONFORK */
> -
> -#define MADV_COLD 20 /* deactivate these pages */
> -#define MADV_PAGEOUT 21 /* reclaim these pages */
> -
> -#define MADV_POPULATE_READ 22 /* populate (prefault) page tables readable */
> -#define MADV_POPULATE_WRITE 23 /* populate (prefault) page tables writable */
> -
> -#define MADV_DONTNEED_LOCKED 24 /* like DONTNEED, but drop locked pages too */
> -
> -#define MADV_COLLAPSE 25 /* Synchronous hugepage collapse */
> -
> -#define MADV_HWPOISON 100 /* poison a page for testing */
> -#define MADV_SOFT_OFFLINE 101 /* soft offline page for testing */
> -
> -/* compatibility flags */
> -#define MAP_FILE 0
> -
> -#define PKEY_DISABLE_ACCESS 0x1
> -#define PKEY_DISABLE_WRITE 0x2
> -#define PKEY_ACCESS_MASK (PKEY_DISABLE_ACCESS |\
> - PKEY_DISABLE_WRITE)
> +#include <asm-generic/mman-common.h>
>
> #endif /* __ALPHA_MMAN_H__ */
> diff --git a/arch/parisc/include/uapi/asm/mman.h b/arch/parisc/include/uapi/asm/mman.h
> index 1cd5d816d4cf..3732950a5cd8 100644
> --- a/arch/parisc/include/uapi/asm/mman.h
> +++ b/arch/parisc/include/uapi/asm/mman.h
> @@ -2,19 +2,6 @@
> #ifndef __PARISC_MMAN_H__
> #define __PARISC_MMAN_H__
>
> -
> -#define PROT_READ 0x1 /* page can be read */
> -#define PROT_WRITE 0x2 /* page can be written */
> -#define PROT_EXEC 0x4 /* page can be executed */
> -#ifndef PROT_SEM /* different on mips and xtensa */
> -#define PROT_SEM 0x8 /* page may be used for atomic ops */
> -#endif
> -/* 0x10 reserved for arch-specific use */
> -/* 0x20 reserved for arch-specific use */
> -#define PROT_NONE 0x0 /* page can not be accessed */
> -#define PROT_GROWSDOWN 0x01000000 /* mprotect flag: extend change to start of growsdown vma */
> -#define PROT_GROWSUP 0x02000000 /* mprotect flag: extend change to end of growsup vma */
> -
> /* 0x01 - 0x03 are defined in linux/mman.h */
> #define MAP_TYPE 0x2b /* Mask for type of mapping, includes bits 0x08 and 0x20 */
> #define MAP_FIXED 0x04 /* Interpret addr exactly */
> @@ -43,61 +30,12 @@
> #define MCL_ONFAULT 4 /* lock all pages that are faulted in */
>
> /*
> - * Flags for mlock
> - */
> -#define MLOCK_ONFAULT 0x01 /* Lock pages in range after they are faulted in, do not prefault */
> -
> -/*
> - * Flags for msync
> + * Flags for msync, order is different from all others
> */
> #define MS_SYNC 1 /* synchronous memory sync */
> #define MS_ASYNC 2 /* sync memory asynchronously */
> #define MS_INVALIDATE 4 /* invalidate the caches */
>
> -#define MADV_NORMAL 0 /* no further special treatment */
> -#define MADV_RANDOM 1 /* expect random page references */
> -#define MADV_SEQUENTIAL 2 /* expect sequential page references */
> -#define MADV_WILLNEED 3 /* will need these pages */
> -#define MADV_DONTNEED 4 /* don't need these pages */
> -
> -/* common parameters: try to keep these consistent across architectures */
> -#define MADV_FREE 8 /* free pages only if memory pressure */
> -#define MADV_REMOVE 9 /* remove these pages & resources */
> -#define MADV_DONTFORK 10 /* don't inherit across fork */
> -#define MADV_DOFORK 11 /* do inherit across fork */
> -
> -#define MADV_MERGEABLE 12 /* KSM may merge identical pages */
> -#define MADV_UNMERGEABLE 13 /* KSM may not merge identical pages */
> -
> -#define MADV_HUGEPAGE 14 /* Worth backing with hugepages */
> -#define MADV_NOHUGEPAGE 15 /* Not worth backing with hugepages */
> -
> -#define MADV_DONTDUMP 16 /* Explicity exclude from the core dump,
> - overrides the coredump filter bits */
> -#define MADV_DODUMP 17 /* Clear the MADV_DONTDUMP flag */
> -
> -#define MADV_WIPEONFORK 18 /* Zero memory on fork, child only */
> -#define MADV_KEEPONFORK 19 /* Undo MADV_WIPEONFORK */
> -
> -#define MADV_COLD 20 /* deactivate these pages */
> -#define MADV_PAGEOUT 21 /* reclaim these pages */
> -
> -#define MADV_POPULATE_READ 22 /* populate (prefault) page tables readable */
> -#define MADV_POPULATE_WRITE 23 /* populate (prefault) page tables writable */
> -
> -#define MADV_DONTNEED_LOCKED 24 /* like DONTNEED, but drop locked pages too */
> -
> -#define MADV_COLLAPSE 25 /* Synchronous hugepage collapse */
> -
> -#define MADV_HWPOISON 100 /* poison a page for testing */
> -#define MADV_SOFT_OFFLINE 101 /* soft offline page for testing */
> -
> -/* compatibility flags */
> -#define MAP_FILE 0
> -
> -#define PKEY_DISABLE_ACCESS 0x1
> -#define PKEY_DISABLE_WRITE 0x2
> -#define PKEY_ACCESS_MASK (PKEY_DISABLE_ACCESS |\
> - PKEY_DISABLE_WRITE)
> +#include <asm-generic/mman-common.h>
>
> #endif /* __PARISC_MMAN_H__ */
> diff --git a/include/uapi/asm-generic/mman-common.h b/include/uapi/asm-generic/mman-common.h
> index 2911dd14ef2a..81a14ed99197 100644
> --- a/include/uapi/asm-generic/mman-common.h
> +++ b/include/uapi/asm-generic/mman-common.h
> @@ -27,15 +27,20 @@
> /*
> * Flags for msync
> */
> +#ifndef MS_ASYNC /* different order on alpha and parisc */
> #define MS_ASYNC 1 /* sync memory asynchronously */
> #define MS_INVALIDATE 2 /* invalidate the caches */
> #define MS_SYNC 4 /* synchronous memory sync */
> +#endif
>
> #define MADV_NORMAL 0 /* no further special treatment */
> #define MADV_RANDOM 1 /* expect random page references */
> #define MADV_SEQUENTIAL 2 /* expect sequential page references */
> #define MADV_WILLNEED 3 /* will need these pages */
> +/* 4 through 6 are different on alpha */
> +#ifndef MADV_DONTNEED
> #define MADV_DONTNEED 4 /* don't need these pages */
> +#endif
Rather nice to have these differences both de-duplicated and documented
here...
>
> /* common parameters: try to keep these consistent across architectures */
> #define MADV_FREE 8 /* free pages only if memory pressure */
> --
> 2.39.2
>
next prev parent reply other threads:[~2024-10-09 14:32 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-25 21:06 [PATCH 0/5] asm-generic: clean up asm/mman.h Arnd Bergmann
2024-09-25 21:06 ` [PATCH 1/5] asm-generic: cosmetic updates to uapi/asm/mman.h Arnd Bergmann
2024-09-26 9:21 ` Helge Deller
2024-09-26 9:41 ` Arnd Bergmann
2024-10-09 14:16 ` Lorenzo Stoakes
2024-09-25 21:06 ` [PATCH 2/5] asm-generic: move MAP_* flags from mman-common.h to mman.h Arnd Bergmann
2024-10-09 14:25 ` Lorenzo Stoakes
2024-09-25 21:06 ` [PATCH 3/5] asm-generic: use asm-generic/mman-common.h on mips and xtensa Arnd Bergmann
2024-10-05 17:13 ` Richard Henderson
2024-09-25 21:06 ` [PATCH 4/5] asm-generic: use asm-generic/mman-common.h on parisc and alpha Arnd Bergmann
2024-10-05 17:15 ` Richard Henderson
2024-10-09 14:31 ` Lorenzo Stoakes [this message]
2024-09-25 21:06 ` [PATCH 5/5] [RFC] mm: Remove MAP_UNINITIALIZED support Arnd Bergmann
2024-09-26 8:46 ` David Hildenbrand
2024-09-26 13:54 ` Arnd Bergmann
2024-10-09 14:32 ` Lorenzo Stoakes
2024-09-26 9:18 ` Helge Deller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1cf8c09f-6b36-4650-a017-707f4bc41ca7@lucifer.local \
--to=lorenzo.stoakes@oracle.com \
--cc=Jason@zx2c4.com \
--cc=Liam.Howlett@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=alexandre.torgue@foss.st.com \
--cc=andreas@gaisler.com \
--cc=ardb@kernel.org \
--cc=arnd@arndb.de \
--cc=arnd@kernel.org \
--cc=brauner@kernel.org \
--cc=christophe.leroy@csgroup.eu \
--cc=david@redhat.com \
--cc=deller@gmx.de \
--cc=dlemoal@kernel.org \
--cc=gerg@linux-m68k.org \
--cc=hch@lst.de \
--cc=jcmvbkbc@gmail.com \
--cc=kees@kernel.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-parisc@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mattst88@gmail.com \
--cc=mhocko@suse.com \
--cc=mpe@ellerman.id.au \
--cc=npiggin@gmail.com \
--cc=richard.henderson@linaro.org \
--cc=tsbogend@alpha.franken.de \
--cc=vbabka@suse.cz \
--cc=viro@zeniv.linux.org.uk \
--cc=vladimir.murzin@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox