Re: PR/38792 CVS commit: src/sys

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

On Fri, Apr 16, 2010 at 03:25:02AM +0000, Mindaugas Rasiukevicius wrote:
> The following reply was made to PR kern/38792; it has been noted by GNATS.
> 
> From: Mindaugas Rasiukevicius <rmind%netbsd.org@localhost>
> To: gnats-bugs%gnats.NetBSD.org@localhost
> Cc: 
> Subject: PR/38792 CVS commit: src/sys
> Date: Fri, 16 Apr 2010 03:21:50 +0000
> 
>  Module Name: src
>  Committed By:        rmind
>  Date:                Fri Apr 16 03:21:49 UTC 2010
>  
>  Modified Files:
>       src/sys/kern: kern_synch.c
>       src/sys/sys: sched.h
>       src/sys/uvm: uvm_extern.h uvm_glue.c uvm_meter.c
>  
>  Log Message:
>  - Merge sched_pstats() and uvm_meter()/uvm_loadav().  Avoids double loop
>    through all LWPs and duplicate locking overhead.
>  
>  - Move sched_pstats() from soft-interrupt context to process 0 main loop.
>    Avoids blocking effect on real-time threads.  Mostly fixes PR/38792.
>  
>  Note: it might be worth to move the loop above PRI_PGDAEMON.  Also,
>  sched_pstats() might be cleaned-up slightly.
>  
>  
>  To generate a diff of this commit:
>  cvs rdiff -u -r1.280 -r1.281 src/sys/kern/kern_synch.c
>  cvs rdiff -u -r1.71 -r1.72 src/sys/sys/sched.h
>  cvs rdiff -u -r1.162 -r1.163 src/sys/uvm/uvm_extern.h
>  cvs rdiff -u -r1.144 -r1.145 src/sys/uvm/uvm_glue.c
>  cvs rdiff -u -r1.51 -r1.52 src/sys/uvm/uvm_meter.c
>  
>  Please note that diffs are not public domain; they are subject to the
>  copyright notices on the relevant files.

Is kernel_lock now held across sched_pstats, as a result of running in lwp0?
It may have been acquired in configure() / configure2()?