PR/59327 CVS commit: src/tests/kernel

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

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

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59327 CVS commit: src/tests/kernel
Date: Mon, 21 Apr 2025 12:06:08 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Mon Apr 21 12:06:08 UTC 2025
 
 Modified Files:
 	src/tests/kernel: t_signal_and_sp.c
 	src/tests/kernel/arch/hppa: contextspfunc.S
 	src/tests/kernel/arch/x86_64: threadspfunc.S
 
 Log Message:
 t_signal_and_sp: Fix threadsp test to actually test the thread sp.
 
 Copypasta error had it testing alignment of signalsp instead, and
 since it was always null, that always passed.  Mark it xfail on mips
 now as originally expected.
 
 While here, deal with some other issues:
 
 - Test was failing on riscv for me because I haven't updated this
   `current' VM in a while so it didn't have the fix for 57721
   (pthread_attr_setstack incorrectly adjusts address as if for guard
   page).  Don't mark it xfail.
 
 - Fix amd64 threadspfunc.S to adjust rsp like in all the other amd64
   stubs so it's congruent to 0 mod 16, not congruent to 8 mod 16.
 
 - Fix hppa contextspfunc.S to use separate registers for separate
   purposes at the same time, instead of expecting addil to preserve
   %r1 AND yield a result we use later in %r1.
 
 PR kern/59327: user stack pointer is not aligned properly
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.9 -r1.10 src/tests/kernel/t_signal_and_sp.c
 cvs rdiff -u -r1.1 -r1.2 src/tests/kernel/arch/hppa/contextspfunc.S
 cvs rdiff -u -r1.1 -r1.2 src/tests/kernel/arch/x86_64/threadspfunc.S
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.