[PATCH v5] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-04 Thread Terry Tritton
: Terry Tritton --- Changes in v5: - Fixed checkpatch errors Changes in v4: - Change to use __kernel_timespec as suggested by tglx Changes in v3: - Fix signed-off-by chain but for real this time Changes in v2: - Fix signed-off-by chain tools/testing/selftests/futex/include/futex2test.h | 8

Re: [PATCH v4] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-04 Thread Terry Tritton
> However, please check the results ./scripts/checkpatch.pl, there are a > lot of codestyle errors: Whoops! Can't believe I forgot to run check patch! Thanks for letting me know.

[PATCH v2] selftests/futex: Add futex_numa to .gitignore

2025-07-04 Thread Terry Tritton
futex_numa was never added to the .gitignore file. Add it. Fixes: 9140f57c1c13 ("futex,selftests: Add another FUTEX2_NUMA selftest") Signed-off-by: Terry Tritton --- Changes in v2: - Add Fixes tag tools/testing/selftests/futex/functional/.gitignore | 1 + 1 file changed, 1 insertio

[PATCH v4] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-04 Thread Terry Tritton
: Terry Tritton --- Changes in v4: - Change to use __kernel_timespec as suggested by tglx Changes in v3: - Fix signed-off-by chain but for real this time Changes in v2: - Fix signed-off-by chain tools/testing/selftests/futex/include/futex2test.h | 8 +++- 1 file changed, 7 insertions(+), 1

[PATCH v3] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-02 Thread Terry Tritton
Futex_waitv can not accept old_timespec32 struct, so userspace should convert it from 32bit to 64bit before syscall in 32bit compatible mode. This fix is based off [1] Link: https://lore.kernel.org/all/20231203235117.29677-1-we...@suse.com/ [1] Originally-by: Wei Gao Signed-off-by: Terry

Re: [PATCH v2] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-02 Thread Terry Tritton
> > > Signed-off-by: Wei Gao > > > Signed-off-by: Terry Tritton > > > > This is still wrong. > > > > If it is based on someone else work, then you need to attribute it > > Originally-by and omit the Signed-off-by of the original author. > >

Re: [PATCH v2] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-02 Thread Terry Tritton
> > Signed-off-by: Wei Gao > > Signed-off-by: Terry Tritton > > This is still wrong. > > If it is based on someone else work, then you need to attribute it > Originally-by and omit the Signed-off-by of the original author. > > If you just picked it up and ado

Re: [PATCH v2] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-01 Thread Terry Tritton
Sorry forgot to save the change log before sending. v2: fix Signed-off-by chain On Tue, 1 Jul 2025 at 15:23, Terry Tritton wrote: > > Futex_waitv can not accept old_timespec32 struct, so userspace should > convert it from 32bit to 64bit before syscall in 32bit compatible mode. > &

[PATCH v2] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-07-01 Thread Terry Tritton
Futex_waitv can not accept old_timespec32 struct, so userspace should convert it from 32bit to 64bit before syscall in 32bit compatible mode. This fix is based off [1] Link: https://lore.kernel.org/all/20231203235117.29677-1-we...@suse.com/ [1] Signed-off-by: Wei Gao Signed-off-by: Terry

[PATCH] selftests/futex: Add futex_numa to .gitignore

2025-06-09 Thread Terry Tritton
futex_numa was never added to the .gitignore file. Add it. Signed-off-by: Terry Tritton --- tools/testing/selftests/futex/functional/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/futex/functional/.gitignore b/tools/testing/selftests/futex/functional

[PATCH] selftests/futex: Convert 32bit timespec struct to 64bit version for 32bit compatibility mode

2025-06-09 Thread Terry Tritton
Futex_waitv can not accept old_timespec32 struct, so userspace should convert it from 32bit to 64bit before syscall in 32bit compatible mode. This fix is based off [1] Link: https://lore.kernel.org/all/20231203235117.29677-1-we...@suse.com/ [1] Signed-off-by: Terry Tritton Signed-off-by: Wei

Re: [PATCH] selftests/timers: Fix integer overlow errors on 32 bit systems

2025-06-04 Thread Terry Tritton
> So this seems to be undoing commit 80fa614e2fbc ("selftests: timers: > Remove local NSEC_PER_SEC and USEC_PER_SEC defines") Thanks John, I somehow missed 80fa614e2fbc and was wondering how this had got in. > Would it make more sense to fix the NSEC_PER_SEC definition in time64.h to a > LL? I wa

[PATCH] selftests/timers: Fix integer overlow errors on 32 bit systems

2025-06-03 Thread Terry Tritton
The use of NSEC_PER_SEC (10L) as defined in include/vdso/time64.h causes several integer overflow warnings and test errors on 32 bit architectures. Use a long long instead of long to prevent integer overflow when converting seconds to nanoseconds. Signed-off-by: Terry Tritton

[PATCH] selftests/seccomp: fix negative_ENOSYS tracer tests on arm32

2025-05-09 Thread Terry Tritton
between the forked processes the tracer is returning the KSFT_SKIP value set by the tracee which is non 0. Remove the setting of the _metadata.exit_code in teardown_trace_fixture. Fixes: 24cf65a62266 ("selftests/harness: Share _metadata between forked processes") Signed-off-by: Ter