Re: PR/54734 CVS commit: src/sys/arch/sparc/sparc

[Kamil Rytarowski <n54%gmx.com@localhost>]

The following reply was made to PR port-sparc/54734; it has been noted by GNATS.

From: Kamil Rytarowski <n54%gmx.com@localhost>
To: Andreas Gustafsson <gson%gson.org@localhost>, kamil%NetBSD.org@localhost
Cc: gnats-bugs%netbsd.org@localhost, ad%netbsd.org@localhost
Subject: Re: PR/54734 CVS commit: src/sys/arch/sparc/sparc
Date: Thu, 14 May 2020 14:26:29 +0200

 On 14.05.2020 10:07, Andreas Gustafsson wrote:
 > Kamil Rytarowski committed:
 >>  Module Name:	src
 >>  Committed By:	kamil
 >>  Date:		Mon May 11 18:38:27 UTC 2020
 >>
 >>  Modified Files:
 >>  	src/sys/arch/sparc/sparc: process_machdep.c
 >>
 >>  Log Message:
 >>  Reject setting unaligned Program Counter
 >>
 >>  Patch taken from OpenBSD.
 >>
 >>  Tested in qemu.
 >>
 >>  Fixes PR port-sparc/54734 by Andreas Gustafsson
 >>
 >>
 >>  To generate a diff of this commit:
 >>  cvs rdiff -u -r1.19 -r1.20 src/sys/arch/sparc/sparc/process_machdep.c
 >
 > The sparc test runs are still panicing in the same test case both on
 > babylon5 and my own testbed.  The panic message changed from "panic:
 > TAILQ_PREREMOVE" to "panic: kernel diagnostic assertion
 > "cv_is_valid(cv)" in late March, so maybe there were two different
 > bugs and the commit did fix the original one, but since the subject
 > line of the PR is still applies, I'll reopen it rather than filing a
 > new one.
 >
 > For logs, see:
 >
 >   http://releng.netbsd.org/b5reports/sparc/commits-2020.05.html#2020.05.=
 11.18.38.26
 >
 > Grep output from babylon5 showing when the panic message changed:
 >
 >   2020.03.18.19.34.15/test.log.gz:    getrusage_maxrss: [ 4995.8360525] =
 panic: TAILQ_PREREMOVE head 0xf0002008 elm 0xf075f080 /tmp/bracket/build/2=
 020.03.18.19.34.15-sparc/src/sys/kern/kern_sleepq.c:119
 >   2020.03.24.22.09.50/test.log.gz:    getrusage_maxrss: [ 5060.4379790] =
 panic: TAILQ_PREREMOVE head 0xf0002008 elm 0xf0761080 /tmp/bracket/build/2=
 020.03.24.22.09.50-sparc/src/sys/kern/kern_sleepq.c:119
 >   2020.03.27.11.15.33/test.log.gz:    getrusage_maxrss: [ 4793.0443690] =
 panic: kernel diagnostic assertion "cv_is_valid(cv)" failed: file "/tmp/br=
 acket/build/2020.03.27.11.15.33-sparc/src/sys/kern/kern_condvar.c", line 4=
 86
 >   2020.03.28.20.13.13/test.log.gz:    getrusage_maxrss: [ 5143.9754705] =
 panic: kernel diagnostic assertion "cv_is_valid(cv)" failed: file "/tmp/br=
 acket/build/2020.03.28.20.13.13-sparc/src/sys/kern/kern_condvar.c", line 4=
 86
 >
 
 Please reopen as a new PR. That one is unrelated in
 lib/libc/sys/t_getrusage:
 
 lib/libc/sys/t_getrusage (284/855): 6 test cases
     getrusage_err: [0.120000s] Passed.
     getrusage_maxrss: [ 5370.1468210] panic: kernel diagnostic assertion
 "cv_is_valid(cv)" failed: file
 "/tmp/bracket/build/2020.05.11.18.38.26-sparc/src/sys/kern/kern_condvar.c"=
 ,
 line 455