In title: maybe s/to 64-bits/for compat64/? Maybe you can drop the first "compat" to avoid the repetition, e.g. "Update defines for compat64".
On 17/01/2023 11:00, Teo Couprie Diaz wrote:
Some compat defines are still using 32-bits values, update them to a 64-bits one (COMPAT_OFF_T_MAX) or use the native value in COMPAT64 (COMPAT_RLIM_INFINITY).
It looks like that parenthesis pertains to "use the native value" so it should be just after it, otherwise it looks like only the latter is for compat64.
Kevin
Signed-off-by: Teo Couprie Diaz teo.coupriediaz@arm.com
include/asm-generic/compat.h | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/include/asm-generic/compat.h b/include/asm-generic/compat.h index 8aa32efdeee8..f4ef0518470e 100644 --- a/include/asm-generic/compat.h +++ b/include/asm-generic/compat.h @@ -7,12 +7,20 @@ #endif #ifndef COMPAT_RLIM_INFINITY +#ifdef CONFIG_COMPAT64 +#define COMPAT_RLIM_INFINITY RLIM_INFINITY +#else #define COMPAT_RLIM_INFINITY 0xffffffff -#endif +#endif /* CONFIG_COMPAT64 */ +#endif /* COMPAT_RLIM_INFINITY */ #ifndef COMPAT_OFF_T_MAX +#ifdef CONFIG_COMPAT64 +#define COMPAT_OFF_T_MAX LONG_MAX +#else #define COMPAT_OFF_T_MAX 0x7fffffff -#endif +#endif /* CONFIG_COMPAT64 */ +#endif /* COMPAT_OFF_T_MAX */ #ifndef compat_arg_u64 #ifndef CONFIG_CPU_BIG_ENDIAN