Re: CVS commit: src/sys/uvm

[Mindaugas Rasiukevicius <rmind%netbsd.org@localhost>]

yamt%mwd.biglobe.ne.jp@localhost (YAMAMOTO Takashi) wrote:
> hi,
> 
> > Module Name:        src
> > Committed By:       hannken
> > Date:               Mon Jun 27 15:56:37 UTC 2011
> > 
> > Modified Files:
> >     src/sys/uvm: uvm_amap.c
> > 
> > Log Message:
> > amap_copy(): Keep the source amap locked until its lock has been copied.
> 
> btw, this code seems to assume that uvm_anfree does not release the lock
> even temporarily while the comment on uvm_anfree1 says the opposite.

Right.  Fix would be to always drop the lock in uvm_anfree() and call it
in the last point.  I will write a patch tomorrow.

> 
> YAMAMOTO Takashi

-- 
Mindaugas