Skip to content

Commit a48f822

Browse files
committed
samples: work around glibc redefining some of our defines wrong
Apparently as of version 2.42, glibc headers define AT_RENAME_NOREPLACE and some of the other flags for renameat2() and friends in <stdio.h>. Which would all be fine, except for inexplicable reasons glibc decided to define them _differently_ from the kernel definitions, which then makes some of our sample code that includes both kernel headers and user space headers unhappy, because the compiler will (correctly) complain about redefining things. Now, mixing kernel headers and user space headers is always a somewhat iffy proposition due to namespacing issues, but it's kind of inevitable in our sample and selftest code. And this is just glibc being stupid. Those defines come from the kernel, glibc is exposing the kernel interfaces, and glibc shouldn't make up some random new expressions for these values. It's not like glibc headers changed the actual result values, but they arbitrarily just decided to use a different expression to describe those values. The kernel just does #define AT_RENAME_NOREPLACE 0x0001 while glibc does # define RENAME_NOREPLACE (1 << 0) # define AT_RENAME_NOREPLACE RENAME_NOREPLACE instead. Same value in the end, but very different macro definition. For absolutely no reason. This has since been fixed in the glibc development tree, so eventually we'll end up with the canonical expressions and no clashes. But in the meantime the broken headers are in the glibc-2.42 release and have made it out into distributions. Do a minimal work-around to make the samples build cleanly by just undefining the affected macros in between the user space header include and the kernel header includes. Signed-off-by: Linus Torvalds <[email protected]>
1 parent fd95357 commit a48f822

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

samples/vfs/test-statx.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,12 @@
1919
#include <time.h>
2020
#include <sys/syscall.h>
2121
#include <sys/types.h>
22+
23+
// Work around glibc header silliness
24+
#undef AT_RENAME_NOREPLACE
25+
#undef AT_RENAME_EXCHANGE
26+
#undef AT_RENAME_WHITEOUT
27+
2228
#include <linux/stat.h>
2329
#include <linux/fcntl.h>
2430
#define statx foo

samples/watch_queue/watch_test.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,12 @@
1616
#include <errno.h>
1717
#include <sys/ioctl.h>
1818
#include <limits.h>
19+
20+
// Work around glibc header silliness
21+
#undef AT_RENAME_NOREPLACE
22+
#undef AT_RENAME_EXCHANGE
23+
#undef AT_RENAME_WHITEOUT
24+
1925
#include <linux/watch_queue.h>
2026
#include <linux/unistd.h>
2127
#include <linux/keyctl.h>

0 commit comments

Comments
 (0)