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: "gnats-bugs%NetBSD.org@localhost" <gnats-bugs%NetBSD.org@localhost>
Cc: 
Subject: Re: PR/54734 CVS commit: src/sys/arch/sparc/sparc
Date: Fri, 15 May 2020 15:08:22 +0200

 On 14.05.2020 14:26, Kamil Rytarowski wrote:
 > 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/=
 2020.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/=
 2020.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/b=
 racket/build/2020.03.27.11.15.33-sparc/src/sys/kern/kern_condvar.c", line =
 486
 >>   2020.03.28.20.13.13/test.log.gz:    getrusage_maxrss: [ 5143.9754705]=
  panic: kernel diagnostic assertion "cv_is_valid(cv)" failed: file "/tmp/b=
 racket/build/2020.03.28.20.13.13-sparc/src/sys/kern/kern_condvar.c", line =
 486
 >>
 >
 > 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
 >
 
 As informed offlist, I confused PR/54734 with PR/55076.
 
 PR/55076 was fixed; PR/53736 is unrelated bug.