lkml.org 
[lkml]   [2007]   [Sep]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [Resend][PATCH -mm] Hibernation: Enter platform hibernation state in a consistent way (rev. 4)
Date
On Saturday, 15 September 2007 01:37, Andrew Morton wrote:
> On Wed, 12 Sep 2007 13:14:08 +0200
> "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
>
> > + if (!hibernation_ops)
> > + return -ENOSYS;
> > +
> > + /*
> > + * We have cancelled the power transition by running
> > + * hibernation_ops->finish() before saving the image, so we should let
> > + * the firmware know that we're going to enter the sleep state after all
> > + */
> > + error = hibernation_ops->start();
> > + if (error)
> > + return error;
> > +
> > + suspend_console();
> > + error = device_suspend(PMSG_SUSPEND);
> > + if (error)
> > + return error;
> > +
> > + error = hibernation_ops->prepare();
> > + if (error)
> > + goto Resume_devices;
> > +
> > + error = disable_nonboot_cpus();
> > + if (error)
> > + goto Finish;
> > +
> > + local_irq_disable();
> > + error = device_power_down(PMSG_SUSPEND);
> > + if (!error) {
> > + hibernation_ops->enter();
> > + /* We should never get here */
> > + while (1);
> > }
> > + local_irq_enable();
> > +
>
> Confused. afacit there's no way for the caller of this function to know
> whether or not suspend_console() was called, so the error recovery doesn't
> know whether or not to run resume_console(). How does all that work?

Well, if this fails, we try to power off and if that fails, we halt.

Resuming devices in case of an error is needed so that kernel_power_off()
can work, but the consoles need not be resumed for that.

Still, now that I think of it, the resuming of consoles in case of an error
will probably be useful in diagnostics. I'll post a fix on top of this patch.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-09-15 02:21    [W:0.037 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site