cvs commit: src/sys/vm vm_fault.c

Brian Fundakowski Feldman green at FreeBSD.org
Sat Aug 21 21:10:28 PDT 2004


On Sat, Aug 21, 2004 at 11:39:55PM -0400, Brian Fundakowski Feldman wrote:
> On Sat, Aug 21, 2004 at 10:30:03PM -0400, Robert Watson wrote:
> > 
> > On Sat, 21 Aug 2004, Brian Fundakowski Feldman wrote:
> > 
> > > > > #1. Lock order reversal.  Giant is acquired after the map read lock.
> > > > 
> > > > No, that is the order it has been in for months.  The other functions
> > > > that acquire a map lock and Giant do the same.
> > > 
> > > Giant can never be acquired after another lock has been acquired :-(
> > 
> > Actually, that's not quite accurate.  Take a look at the relationship
> > between lockmgr/sx locks and Giant, for example.
> 
> Point taken.  This is the first time I've seen Giant in a lock order
> reversal with vm locks, though, and I've been running WITNESS for years.

Also, it was the system_map lock, so it was a mutex, not an sx.

/Goes back to trying to figure out wtf portupgrade -rR kde\* keeps causing
 hangs, but only in X, not at the command line, and not seemingly
 dependant on AGP/X driver.

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green at FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\


More information about the cvs-src mailing list