cvs commit: src/sys/vm vm_fault.c
Brian Fundakowski Feldman
green at FreeBSD.org
Sat Aug 21 20:39:59 PDT 2004
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.
Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\
<> green at FreeBSD.org \ The Power to Serve! \
Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\
More information about the cvs-src