[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <612b7e64af3872fde837740404888a03a2c7b6a0.camel@kernel.org>
Date: Sun, 16 Nov 2025 17:23:15 -0500
From: Jeff Layton <jlayton@...nel.org>
To: Stephen Rothwell <sfr@...b.auug.org.au>, Christian Brauner
<brauner@...nel.org>
Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Linux Next
Mailing List <linux-next@...r.kernel.org>
Subject: Re: linux-next: build failure after merge of the vfs-brauner tree
On Mon, 2025-11-17 at 08:43 +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the vfs-brauner tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> In file included from samples/check-exec/inc.c:16:
> usr/include/linux/fcntl.h:88:9: error: unknown type name 'uint32_t'
> 88 | uint32_t d_flags; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:7:1: note: 'uint32_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> 6 | #include <linux/openat2.h>
> +++ |+#include <stdint.h>
> 7 |
> usr/include/linux/fcntl.h:89:9: error: unknown type name 'uint16_t'
> 89 | uint16_t d_type; /* F_RDLCK, F_WRLCK, F_UNLCK */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:89:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> usr/include/linux/fcntl.h:90:9: error: unknown type name 'uint16_t'
> 90 | uint16_t __pad; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:90:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> In file included from samples/vfs/test-statx.c:23:
> usr/include/linux/fcntl.h:88:9: error: unknown type name 'uint32_t'
> 88 | uint32_t d_flags; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:7:1: note: 'uint32_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> 6 | #include <linux/openat2.h>
> +++ |+#include <stdint.h>
> 7 |
> usr/include/linux/fcntl.h:89:9: error: unknown type name 'uint16_t'
> 89 | uint16_t d_type; /* F_RDLCK, F_WRLCK, F_UNLCK */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:89:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> usr/include/linux/fcntl.h:90:9: error: unknown type name 'uint16_t'
> 90 | uint16_t __pad; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:90:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> In file included from usr/include/linux/watch_queue.h:6,
> from samples/watch_queue/watch_test.c:19:
> usr/include/linux/fcntl.h:88:9: error: unknown type name 'uint32_t'
> 88 | uint32_t d_flags; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:7:1: note: 'uint32_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> 6 | #include <linux/openat2.h>
> +++ |+#include <stdint.h>
> 7 |
> usr/include/linux/fcntl.h:89:9: error: unknown type name 'uint16_t'
> 89 | uint16_t d_type; /* F_RDLCK, F_WRLCK, F_UNLCK */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:89:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> usr/include/linux/fcntl.h:90:9: error: unknown type name 'uint16_t'
> 90 | uint16_t __pad; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:90:9: note: 'uint16_t' is defined in header '<stdint.h>'; this is probably fixable by adding '#include <stdint.h>'
> In file included from ./usr/include/linux/eventfd.h:5,
> from <command-line>:
> usr/include/linux/fcntl.h:88:9: error: unknown type name 'uint32_t'
> 88 | uint32_t d_flags; /* Must be 0 */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:89:9: error: unknown type name 'uint16_t'
> 89 | uint16_t d_type; /* F_RDLCK, F_WRLCK, F_UNLCK */
> | ^~~~~~~~
> usr/include/linux/fcntl.h:90:9: error: unknown type name 'uint16_t'
> 90 | uint16_t __pad; /* Must be 0 */
> | ^~~~~~~~
>
> Caused by commit
>
> 1602bad16d7d ("vfs: expose delegation support to userland")
>
> I have used the vfs-brauner tree from next-20251114 for today.
Thanks Stephen. This patch should fix it.
Christian, you can either fold this into 1602bad16d7d, or I can send it
separately. Let me know which you prefer.
Thanks,
----------------8<----------------------
vfs: add needed headers for new struct delegation definition
The definition of struct delegation uses stdint.h integer types. Add the
necessary headers to ensure that always works.
Fixes: 1602bad16d7d ("vfs: expose delegation support to userland")
Signed-off-by: Jeff Layton <jlayton@...nel.org>
---
include/uapi/linux/fcntl.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/include/uapi/linux/fcntl.h b/include/uapi/linux/fcntl.h
index 008fac15e573..5e277fd955aa 100644
--- a/include/uapi/linux/fcntl.h
+++ b/include/uapi/linux/fcntl.h
@@ -4,6 +4,11 @@
#include <asm/fcntl.h>
#include <linux/openat2.h>
+#ifdef __KERNEL__
+#include <linux/types.h>
+#else
+#include <stdint.h>
+#endif
#define F_SETLEASE (F_LINUX_SPECIFIC_BASE + 0)
#define F_GETLEASE (F_LINUX_SPECIFIC_BASE + 1)
--
2.51.1
Powered by blists - more mailing lists