On 6/27/25 2:08 PM, Ben Zong-You Xie wrote: > glibc does not define SYS_futex for 32-bit architectures using 64-bit > time_t e.g. riscv32, therefore this test fails to compile since it does not > find SYS_futex in C library headers. Define SYS_futex as SYS_futex_time64 > in this situation to ensure successful compilation and compatibility. > > Signed-off-by: Cynthia Huang <cynt...@andestech.com> > Signed-off-by: Ben Zong-You Xie <ben...@andestech.com> Reviewed-by: Muhammad Usama Anjum <usama.an...@collabora.com>
> --- > Changes since v1: > - Fix the SOB chain > > v1 : > https://lore.kernel.org/all/20250527093536.3646143-1-ben...@andestech.com/ > --- > tools/testing/selftests/futex/include/futextest.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/tools/testing/selftests/futex/include/futextest.h > b/tools/testing/selftests/futex/include/futextest.h > index ddbcfc9b7bac..7a5fd1d5355e 100644 > --- a/tools/testing/selftests/futex/include/futextest.h > +++ b/tools/testing/selftests/futex/include/futextest.h > @@ -47,6 +47,17 @@ typedef volatile u_int32_t futex_t; > FUTEX_PRIVATE_FLAG) > #endif > > +/* > + * SYS_futex is expected from system C library, in glibc some 32-bit > + * architectures (e.g. RV32) are using 64-bit time_t, therefore it doesn't > have > + * SYS_futex defined but just SYS_futex_time64. Define SYS_futex as > + * SYS_futex_time64 in this situation to ensure the compilation and the > + * compatibility. > + */ > +#if !defined(SYS_futex) && defined(SYS_futex_time64) > +#define SYS_futex SYS_futex_time64 > +#endif > + > /** > * futex() - SYS_futex syscall wrapper > * @uaddr: address of first futex > -- > 2.34.1
