Subject: Re: panic on suspend?
To: Takahiro Kambe <taca@back-street.net>
From: Jared D. McNeill <jmcneill@invisible.ca>
List: port-i386
Date: 09/07/2007 12:27:01
On Sat, 8 Sep 2007, Takahiro Kambe wrote:
> This change make suspend/resume succeeded.  :-D
[snip]
> 			dopowerhooks(PWR_SOFTSUSPEND);
> -			s = splhigh();
> 			dopowerhooks(PWR_SUSPEND);
> 			acpi_md_sleep(state);
> 			dopowerhooks(PWR_RESUME);
> -			splx(s);
> 			dopowerhooks(PWR_SOFTRESUME);

This is incorrect in the old powerhooks framework -- PWR_SUSPEND and 
PWR_RESUME are documented to run at splhigh().

Cheers,
Jared