2023-09-13 15:14:29

by Alexander Lobakin

[permalink] [raw]
Subject: Re: [PATCH] tracing/user_events: fix long size check in the ABI selftest

From: Alexander Lobakin <[email protected]>
Date: Mon, 11 Sep 2023 18:09:35 +0200

> The current check for 64-bit architecture is double-bugged.
> First of all, %BITS_PER_LONG is not available in the userspace,
> the underscored version from <asm/bitsperlong.h> must be used.
> The following check:
>
> #if BITS_PER_LONG == 0
> #error
> #endif
>
> triggers the error in this source file -- the macro is undefined and
> thus is implicitly evaluated to 0.
> Next, %BITS_PER_LONG means "bits", not "bytes". In the Linux kernel,
> it can be 32 or 64, never 8. Given that the tests guarded by that check
> are meant to be run on a 64-bit system, the correct value would be 64.
>
> Prefix the macro name and fix the value it's compared to.
>
> Fixes: 60b1af8de8c1 ("tracing/user_events: Add ABI self-test")
> Cc: [email protected] # 6.4+
> Reviewed-by: Ira Weiny <[email protected]>
> Signed-off-by: Alexander Lobakin <[email protected]>

Patchwork says "Accepted", but I don't see it anywhere on korg O_o

[...]

Thanks,
Olek