Re: [PATCH] mm/memfd: remove redundant casts

[David Hildenbrand <david@redhat.com>]

On 31.08.25 21:34, Andrew Morton wrote:
> On Sun, 31 Aug 2025 01:47:48 -1000 Joey Pabalinas <joeypabalinas@gmail.com> wrote:
> 
>> MFD_ALL_FLAGS is already an unsigned int. Remove redundant casts to
>> unsigned int.
> 
> lgtm.
> 
> It's rather annoying that the MFD_ALL_FLAGS definition is so far away
> from the other MFD_* definitions.  What do peope think of this little
> cleanup?
> 
> From: Andrew Morton <akpm@linux-foundation.org>
> Subject: memfd: move MFD_ALL_FLAGS definition to memfd.h
> Date: Sun Aug 31 12:29:57 PM PDT 2025
> 
> It's not part of the UAPI, but putting it here is better from a
> maintainability POV.
> 
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>   include/uapi/linux/memfd.h |    2 ++
>   mm/memfd.c                 |    2 --
>   2 files changed, 2 insertions(+), 2 deletions(-)
> 
> --- a/mm/memfd.c~memfd-move-mfd_all_flags-definition-to-memfdh
> +++ a/mm/memfd.c
> @@ -313,8 +313,6 @@ long memfd_fcntl(struct file *file, unsi
>   #define MFD_NAME_PREFIX_LEN (sizeof(MFD_NAME_PREFIX) - 1)
>   #define MFD_NAME_MAX_LEN (NAME_MAX - MFD_NAME_PREFIX_LEN)
>   
> -#define MFD_ALL_FLAGS (MFD_CLOEXEC | MFD_ALLOW_SEALING | MFD_HUGETLB | MFD_NOEXEC_SEAL | MFD_EXEC)
> -
>   static int check_sysctl_memfd_noexec(unsigned int *flags)
>   {
>   #ifdef CONFIG_SYSCTL
> --- a/include/uapi/linux/memfd.h~memfd-move-mfd_all_flags-definition-to-memfdh
> +++ a/include/uapi/linux/memfd.h
> @@ -12,6 +12,8 @@
>   #define MFD_NOEXEC_SEAL		0x0008U
>   /* executable */
>   #define MFD_EXEC		0x0010U
> +#define MFD_ALL_FLAGS		(MFD_CLOEXEC | MFD_ALLOW_SEALING | \
> +				 MFD_HUGETLB | MFD_NOEXEC_SEAL | MFD_EXEC)

No strong opinion.

When we are exposing this through uapi, wondering if we should rename it 
to something else that includes "MASK".

Quickly search for similar instances. For mmap we have the internal 
"LEGACY_MAP_MASK", but that's also a bit different.

There is this oddity that we store the hugetlb size in the upper bits of 
flags, masked by (MFD_HUGE_MASK << MFD_HUGE_SHIFT).

MFD_FLAGS_MASK maybe? not sure.

-- 
Cheers

David / dhildenb