CVS commit: [netbsd-10] src

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Sun Oct 25 11:18:51 UTC 2026

Modified Files:
        src/sys/kern [netbsd-10]: sys_futex.c
        src/sys/sys [netbsd-10]: futex.h
        src/tests/lib/libc/sys [netbsd-10]: t_futex_ops.c

Log Message:
Pull up following revision(s) (requested by riastradh in ticket #1398):

        sys/kern/sys_futex.c: revision 1.23
        sys/kern/sys_futex.c: revision 1.24
        sys/kern/sys_futex.c: revision 1.25
        tests/lib/libc/sys/t_futex_ops.c: revision 1.11
        tests/lib/libc/sys/t_futex_ops.c: revision 1.12
        tests/lib/libc/sys/t_futex_ops.c: revision 1.13
        tests/lib/libc/sys/t_futex_ops.c: revision 1.14
        sys/sys/futex.h: revision 1.6
        sys/sys/futex.h: revision 1.7

sys/futex.h: Nix trailing whitespace.

No functional change intended.

t_futex_ops: Spruce up diagnostics.
- Use RL to report errno for any syscalls that set it.
- Sprinkle messages to show intermediate quantities.

Preparation for:
PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP

t_futex_ops: Relax various ATF_REQUIRE_* to ATF_CHECK_*.

This makes the test output more usable for browsing diagnostics when
multi-part tests partially fail.

Preparation for:
PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP

t_futex_ops: Test sign-extension of WAKE_OP oparg/cmparg.
PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP

futex(2): Sign-extend FUTEX_WAKE_OP oparg/cmparg as Linux does.

Also mask off bits in the FUTEX_OP macro as Linux does so that passing
negative arguments works like in Linux.

PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP

futex(2): Fix some comments to match the usual argument order.

No functional change intenteded.

Prompted by:
PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP

futex(2): Rename various parameters to clarify correspondence.

No functional change intended.

I have spent way too much time puzzling over what val/val2/val3 mean
for each operation; let's just give them all meaningful names and
write down the correspondence in the dispatch switch in do_futex.

Prompted by how much time I spent scratching my head for:
PR kern/59129: futex(3): missing sign extension in FUTEX_WAKE_OP


To generate a diff of this commit:
cvs rdiff -u -r1.18.4.1 -r1.18.4.2 src/sys/kern/sys_futex.c
cvs rdiff -u -r1.5 -r1.5.4.1 src/sys/sys/futex.h
cvs rdiff -u -r1.5.10.1 -r1.5.10.2 src/tests/lib/libc/sys/t_futex_ops.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.