cvs commit: src/sys/dev/acpica acpi.c

Nate Lawson nate at root.org
Sun Dec 5 14:16:47 PST 2004


M. Warner Losh wrote:
> In message: <200412050135.iB51ZJJl040909 at repoman.freebsd.org>
>             Nate Lawson <njl at FreeBSD.org> writes:
> :   If we are resuming non-MPSAFE drivers, they need Giant held for them.
> 
> Generally, when walking the device tree, one must have Giant.  I've
> been meaning to add asserts for this in the tree...  I think that we
> have enough things giant is pushed out of that it is time...

I'd like that.  I think the GIANT_REQUIRED asserts should go in the 
root_bus suspend/resume/shutdown methods, not in acpi.  This will cover 
everyone unless a driver drops Giant and doesn't reacquire it (unlikely).


-- 
Nate


More information about the cvs-src mailing list