Re: Suspend, followed by immediate resume

["Rafael J. Wysocki" <rjw@sisk.pl>]

On Saturday, 3 of May 2008, Jay Cliburn wrote:
> On Sat, 3 May 2008 01:49:44 +0200
> "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> 
> > On Friday, 2 of May 2008, Jay Cliburn wrote:
> [...]
> > > I poked around in BIOS and changed the behavior somewhat.  There's a
> > > BIOS setting called "Suspend Mode" with three options:
> > > 
> > > 	S1 (POS) only
> > > 	S3 only
> > > 	Auto
> > > 
> > > When set to "S3 only" or "Auto," the machine spontaneously resumes
> > > following a suspend.  When set to "S1 (POS) only" the machine stays
> > > asleep.
> > > 
> > > What is "POS" an acronym for?
> > 
> > No idea.
> 
> Wikipedia to the rescue.  http://en.wikipedia.org/wiki/Acpi
> 
> S1 == POS == Power On Standby
> 
> I don't think I want to use this mode.  The cpu fan runs continuously at
> max while suspended and the system won't resume; I have to whack it
> with the reset button to get it going again (with a reboot).

That's bad.

> > > Does it make sense to leave the thing set to "S1 only?"
> > 
> > I guess it does.  Still, can you please post a boot log with it set
> > this way?
> 
> Okay, it's attached.

Well, it says all ACPI sleep states are supported.
 
> BTW, I'm still using the modified version of acpi_pm_enter().

That shouldn't matter a lot, I think you can remove the comment.

> diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
> index 71183ee..ccc760a 100644
> --- a/drivers/acpi/sleep/main.c
> +++ b/drivers/acpi/sleep/main.c
> @@ -129,7 +129,9 @@ static int acpi_pm_enter(suspend_state_t pm_state)
>         }
>  
>         local_irq_save(flags);
> +/*
>         acpi_enable_wakeup_device(acpi_state);
> +*/
>         switch (acpi_state) {
>         case ACPI_STATE_S1:
>                 barrier();
> 
> 
> 

Thanks,
Rafael