]> Gentwo Git Trees - linux/.git/commit
samples: work around glibc redefining some of our defines wrong
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 21 Nov 2025 17:29:02 +0000 (09:29 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 21 Nov 2025 17:29:02 +0000 (09:29 -0800)
commita48f822908982353c3256e35a089e9e7d0d61580
treec1f985e74c38446a59e83d855156ff296fad082e
parentfd95357fd8c6778ac7dea6c57a19b8b182b6e91f
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 <torvalds@linux-foundation.org>
samples/vfs/test-statx.c
samples/watch_queue/watch_test.c