Re: CVS commit: src/sys/kern

[Andrew Doran <ad%netbsd.org@localhost>]

Hi,

On Sat, Nov 30, 2019 at 04:52:38PM +0900, Rin Okuyama wrote:
> On 2019/11/30 5:50, Andrew Doran wrote:
> > Module Name:	src
> > Committed By:	ad
> > Date:		Fri Nov 29 20:50:54 UTC 2019
> > 
> > Modified Files:
> > 	src/sys/kern: kern_rwlock.c
> > 
> > Log Message:
> > A couple more tweaks to avoid reading the lock word.
> > 
> > 
> > To generate a diff of this commit:
> > cvs rdiff -u -r1.56 -r1.57 src/sys/kern/kern_rwlock.c
> > 
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
> 
> Hi,
> 
> After this commit, GENERIC64 kernel on evbarm-aarch64 does no longer
> boot multiuser anymore with rwlock related panics:
> 
> 	panic: kernel diagnostic assertion "vm_map_locked_p(map)" failed: file "../../../../uvm/uvm_map.c", line 1315
> 
> or
> 
> 	panic: kernel diagnostic assertion "rw_write_held(&map->lock)" failed: file "../../../../uvm/uvm_map.c", line 701
> 
> By reverting kern_rwlock.c to rev 1.56, panics disappear as far as
> I can see.

Hmm, it works fine on amd64 and looks OK but me, but I have backed it out
for the time being.

Thank you,
Andrew