PR/59129 CVS commit: [netbsd-10] src

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

The following reply was made to PR kern/59129; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59129 CVS commit: [netbsd-10] src
Date: Sun, 25 Oct 2026 11:18:51 +0000

 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.