Drop any recursed taking of Giant once and for all at the top of
kern_reboot(). The shutdown path is now safe to run without Giant. Discussed with: kib@ Sponsored by: Netflix
This commit is contained in:
parent
112b88e391
commit
f0d847af61
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=331362
@ -366,6 +366,17 @@ kern_reboot(int howto)
|
||||
{
|
||||
static int once = 0;
|
||||
|
||||
/*
|
||||
* Normal paths here don't hold Giant, but we can wind up here
|
||||
* unexpectedly with it held. Drop it now so we don't have to
|
||||
* drop and pick it up elsewhere. The paths it is locking will
|
||||
* never be returned to, and it is preferable to preclude
|
||||
* deadlock than to lock against code that won't ever
|
||||
* continue.
|
||||
*/
|
||||
while (mtx_owned(&Giant))
|
||||
mtx_unlock(&Giant);
|
||||
|
||||
#if defined(SMP)
|
||||
/*
|
||||
* Bind us to the first CPU so that all shutdown code runs there. Some
|
||||
|
Loading…
Reference in New Issue
Block a user