memfd: move MFD_ALL_FLAGS definition to memfd.h

akpm00 · akpm00 · commit 3fe5a59e6c26 · 2025-09-09T23:38:30.000-07:00
It's not part of the UAPI, but putting it here is better from a
maintainability POV.
Cc: Baolin Wang &lt;baolin.wang@linux.alibaba.com&gt;
Cc: Hugh Dickins &lt;hughd@google.com&gt;
Cc: Joey Pabalinas &lt;joeypabalinas@gmail.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
diff --git a/include/uapi/linux/memfd.h b/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)
 
 /*
  * Huge page size encoding when MFD_HUGETLB is specified, and a huge page
diff --git a/mm/memfd.c b/mm/memfd.c
@@ -313,8 +313,6 @@ long memfd_fcntl(struct file *file, unsigned int cmd, unsigned int arg)
 #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

Original file line number	Diff line number	Diff line change
`@@ -313,8 +313,6 @@ long memfd_fcntl(struct file *file, unsigned int cmd, unsigned int arg)`
`313`	`313`	`#define MFD_NAME_PREFIX_LEN (sizeof(MFD_NAME_PREFIX) - 1)`
`314`	`314`	`#define MFD_NAME_MAX_LEN (NAME_MAX - MFD_NAME_PREFIX_LEN)`
`315`	`315`
`316`		`-#define MFD_ALL_FLAGS (MFD_CLOEXEC \| MFD_ALLOW_SEALING \| MFD_HUGETLB \| MFD_NOEXEC_SEAL \| MFD_EXEC)`
`317`		`-`
`318`	`316`	`static int check_sysctl_memfd_noexec(unsigned int *flags)`
`319`	`317`	`{`
`320`	`318`	`#ifdef CONFIG_SYSCTL`