On Tuesday, 17 October 2006 08:41, Stefan Seyfried wrote:
> On Mon, Oct 16, 2006 at 09:45:03PM +0200, Rafael J. Wysocki wrote:
> > On Monday, 16 October 2006 21:21, Stefan Seyfried wrote:
> > > On Mon, Oct 16, 2006 at 09:06:07PM +0200, R. J. Wysocki wrote:
> > > > On Monday, 16 October 2006 20:55, Stefan Seyfried wrote:
> > > > > Hi,
> > > > > 
> > > > > If nobody objects, i'd send this to LKML:
> > > > > Maybe somebody with a recent -mm kernel can test if it still applies 
> > > > > there :-)
> > > > 
> > > > I have a very similar patch, but with two differences, that applies to 
> > > > -mm
> > > > and has been tested (on one box):
> > > 
> > > ok, even better. My patch is still compiling, but i have no machine here 
> > > where
> > > i can test its effectiveness (on a thinkpad, you see the moon blink after
> > > pm_ops->prepare(), but i only get to one tomorrow ;-)
> > 
> > When you can, please test my patch and I'll post it to LKML if it works.
> 
> I did not forget this one, but will test later today.

OK

> > Do you mean platform_finish is also needed when platform_prepare fails?
> 
> Probably not. The other sleep modes do (in main.c:84):
> 
>         if (pm_ops->prepare) {
>                 if ((error = pm_ops->prepare(state)))
>                         goto Thaw;
>         }
> 
>         suspend_console();
>         if ((error = device_suspend(PMSG_SUSPEND))) {
>                 printk(KERN_ERR "Some devices failed to suspend\n");
>                 goto Finish;
>         }
>         return 0;
>  Finish:
>         if (pm_ops->finish)
>                 pm_ops->finish(state);
>  Thaw:
>         thaw_processes();
>  Enable_cpu:
>         enable_nonboot_cpus();
>         pm_restore_console();
>         return error;
> 
> So this looks ok. No, i do not want to look at ACPI spec :-)

\metoo


-- 
You never change things by fighting the existing reality.
                R. Buckminster Fuller

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Suspend-devel mailing list
Suspend-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/suspend-devel

Reply via email to