Subject: Re: panic on suspend?
To: None <port-i386@NetBSD.org>
From: Takahiro Kambe <taca@back-street.net>
List: port-i386
Date: 09/13/2007 10:33:56
In message <alpine.NEB.0.999.0709071226080.24256@discovery.invisible.ca>
	on Fri, 7 Sep 2007 12:27:01 -0400 (EDT),
	"Jared D. McNeill" <jmcneill@invisible.ca> wrote:
> 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().
Hmm.  I don't know it.  Since above change fixed crash at suspend,
something wrong with spl handling?

Best regards.

-- 
Takahiro Kambe <taca@back-street.net>