Fixes for 6.9

Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/queue-6.9/series b/queue-6.9/series
index d0f62d9f84a49543141c62afeda223970329d7ec..c2673df873d948664e5d1e63ab2bdaefc76a1b67 100644 (file)
--- a/queue-6.9/series
+++ b/queue-6.9/series
@@ -193,3 +193,4 @@ ata-libata-core-add-ata_horkage_nolpm-for-all-crucial-bx-ssd1-models.patch
 ata-libata-core-fix-double-free-on-error.patch
 ftruncate-pass-a-signed-offset.patch
 syscalls-fix-compat_sys_io_pgetevents_time64-usage.patch
+syscalls-fix-sys_fanotify_mark-prototype.patch

diff --git a/queue-6.9/syscalls-fix-sys_fanotify_mark-prototype.patch b/queue-6.9/syscalls-fix-sys_fanotify_mark-prototype.patch
new file mode 100644 (file)
index 0000000..4369d6a
--- /dev/null
+++ b/queue-6.9/syscalls-fix-sys_fanotify_mark-prototype.patch
@@ -0,0 +1,58 @@
+From d3552d776d71bc8384c7c4913e58eedc615f11e5 Mon Sep 17 00:00:00 2001
+From: Sasha Levin <sashal@kernel.org>
+Date: Sat, 29 Jun 2024 21:48:41 +0200
+Subject: syscalls: fix sys_fanotify_mark prototype
+
+From: Arnd Bergmann <arnd@arndb.de>
+
+[ Upstream commit 63e2f40c9e3187641afacde4153f54b3ee4dbc8c ]
+
+My earlier fix missed an incorrect function prototype that shows up on
+native 32-bit builds:
+
+In file included from fs/notify/fanotify/fanotify_user.c:14:
+include/linux/syscalls.h:248:25: error: conflicting types for 'sys_fanotify_mark'; have 'long int(int,  unsigned int,  u32,  u32,  int,  const char *)' {aka 'long int(int,  unsigned int,  unsigned int,  unsigned int,  int,  const char *)'}
+ 1924 | SYSCALL32_DEFINE6(fanotify_mark,
+      | ^~~~~~~~~~~~~~~~~
+include/linux/syscalls.h:862:17: note: previous declaration of 'sys_fanotify_mark' with type 'long int(int,  unsigned int,  u64,  int, const char *)' {aka 'long int(int,  unsigned int,  long long unsigned int,  int,  const char *)'}
+
+On x86 and powerpc, the prototype is also wrong but hidden in an #ifdef,
+so it never caused problems.
+
+Add another alternative declaration that matches the conditional function
+definition.
+
+Fixes: 403f17a33073 ("parisc: use generic sys_fanotify_mark implementation")
+Cc: stable@vger.kernel.org
+Reported-by: Guenter Roeck <linux@roeck-us.net>
+Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
+Reported-by: kernel test robot <lkp@intel.com>
+Signed-off-by: Arnd Bergmann <arnd@arndb.de>
+Signed-off-by: Sasha Levin <sashal@kernel.org>
+---
+ include/linux/syscalls.h | 6 ++++++
+ 1 file changed, 6 insertions(+)
+
+diff --git a/include/linux/syscalls.h b/include/linux/syscalls.h
+index 1f3d710afe4d7..ac3db59a74e23 100644
+--- a/include/linux/syscalls.h
++++ b/include/linux/syscalls.h
+@@ -858,9 +858,15 @@ asmlinkage long sys_prlimit64(pid_t pid, unsigned int resource,
+                               const struct rlimit64 __user *new_rlim,
+                               struct rlimit64 __user *old_rlim);
+ asmlinkage long sys_fanotify_init(unsigned int flags, unsigned int event_f_flags);
++#if defined(CONFIG_ARCH_SPLIT_ARG64)
++asmlinkage long sys_fanotify_mark(int fanotify_fd, unsigned int flags,
++                                unsigned int mask_1, unsigned int mask_2,
++                              int dfd, const char  __user * pathname);
++#else
+ asmlinkage long sys_fanotify_mark(int fanotify_fd, unsigned int flags,
+                                 u64 mask, int fd,
+                                 const char  __user *pathname);
++#endif
+ asmlinkage long sys_name_to_handle_at(int dfd, const char __user *name,
+                                     struct file_handle __user *handle,
+                                     int __user *mnt_id, int flag);
+-- 
+2.43.0
+