Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Friday, 22 of February 2008, [EMAIL PROTECTED] wrote: > On Fri, 22 Feb 2008, Rafael J. Wysocki wrote: > > > On Friday, 22 of February 2008, Mark Lord wrote: > >> Rafael J. Wysocki wrote: > >>> On Friday, 22 of February 2008, Mark Lord wrote: > [EMAIL PROTECTED] wrote: > .. > > I've been watching for kexec hibernate for a little while now, and the > > last I saw was that acpi was incompatible with the kexec hibernate (but > > the suspend folks were still claiming that devices needed to be put in > > the 'right mode' not just powered off. I've been waiting to see this > > resolved. > .. > > Yeah, exactly. What's so special about poweroff on hibernation? > Why even bother with the special "S4" state there? > >>> > >>> (1) To be able to wake up with the help of devices that can't wake > >>> the system up from S5 (power off) > >>> (2) To handle some platform devices appropriately over the cycle > >> .. > >> > >> That's the theory. I've read about it, but have yet to imagine > >> any real-life situation where it applies. > >> > >> But this isn't my speciality, so.. do you have experience with any real > >> examples? > > > > Yup. The fan in my notebook behaves incorrectly after a resume from > > hibernation if S5 is entered instead of S4 during it. > > so if you power off your laptop the fan doesn't work when you turn it back > on? No, it works fine then. > > I don't know why exactly it happens, but that's how it goes. > > > > Also, some machines are reported to behave incorrectly after a "shutdown" > > mode hibernation, while the same machines work just fine after a "platform" > > mode hibernation. So at least for these machines it seems to matter. > > given that we don't have a pure "shutdown" option available to try I don't > see how this can be said to have been tested. Yes, we have. > currently any attempts to do a shutdown type hibernate are tangled in the > other code that is there for the suspend modes. this makes it _very_ hard > to say that the hardware requires something as opposed to the strong > possibility that the software is doing something wrong. How is it tangled exactly? > there are also a _lot_ of people who are not able to reliably use the > existing "platform" mode hibernation, so it's not a fair statement to say > that it's the 'right' thing to do. If you want to make it an option, fine. > But please give those of us who don't care about these other wakeup > options, and who want to be able to use other OS's while linux is stopped > an option as well. There is such an option. Put # echo shutdown > /sys/power/disk into the init scripts and it will do the trick. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Fri, 22 Feb 2008, Rafael J. Wysocki wrote: On Friday, 22 of February 2008, Mark Lord wrote: Rafael J. Wysocki wrote: On Friday, 22 of February 2008, Mark Lord wrote: [EMAIL PROTECTED] wrote: .. I've been watching for kexec hibernate for a little while now, and the last I saw was that acpi was incompatible with the kexec hibernate (but the suspend folks were still claiming that devices needed to be put in the 'right mode' not just powered off. I've been waiting to see this resolved. .. Yeah, exactly. What's so special about poweroff on hibernation? Why even bother with the special "S4" state there? (1) To be able to wake up with the help of devices that can't wake the system up from S5 (power off) (2) To handle some platform devices appropriately over the cycle .. That's the theory. I've read about it, but have yet to imagine any real-life situation where it applies. But this isn't my speciality, so.. do you have experience with any real examples? Yup. The fan in my notebook behaves incorrectly after a resume from hibernation if S5 is entered instead of S4 during it. so if you power off your laptop the fan doesn't work when you turn it back on? I don't know why exactly it happens, but that's how it goes. Also, some machines are reported to behave incorrectly after a "shutdown" mode hibernation, while the same machines work just fine after a "platform" mode hibernation. So at least for these machines it seems to matter. given that we don't have a pure "shutdown" option available to try I don't see how this can be said to have been tested. currently any attempts to do a shutdown type hibernate are tangled in the other code that is there for the suspend modes. this makes it _very_ hard to say that the hardware requires something as opposed to the strong possibility that the software is doing something wrong. there are also a _lot_ of people who are not able to reliably use the existing "platform" mode hibernation, so it's not a fair statement to say that it's the 'right' thing to do. If you want to make it an option, fine. But please give those of us who don't care about these other wakeup options, and who want to be able to use other OS's while linux is stopped an option as well. David Lang - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Friday, 22 of February 2008, Mark Lord wrote: > Rafael J. Wysocki wrote: > > On Friday, 22 of February 2008, Mark Lord wrote: > >> [EMAIL PROTECTED] wrote: > >> .. > >>> I've been watching for kexec hibernate for a little while now, and the > >>> last I saw was that acpi was incompatible with the kexec hibernate (but > >>> the suspend folks were still claiming that devices needed to be put in > >>> the 'right mode' not just powered off. I've been waiting to see this > >>> resolved. > >> .. > >> > >> Yeah, exactly. What's so special about poweroff on hibernation? > >> Why even bother with the special "S4" state there? > > > > (1) To be able to wake up with the help of devices that can't wake > > the system up from S5 (power off) > > (2) To handle some platform devices appropriately over the cycle > .. > > That's the theory. I've read about it, but have yet to imagine > any real-life situation where it applies. > > But this isn't my speciality, so.. do you have experience with any real > examples? Yup. The fan in my notebook behaves incorrectly after a resume from hibernation if S5 is entered instead of S4 during it. I don't know why exactly it happens, but that's how it goes. Also, some machines are reported to behave incorrectly after a "shutdown" mode hibernation, while the same machines work just fine after a "platform" mode hibernation. So at least for these machines it seems to matter. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Rafael J. Wysocki wrote: On Friday, 22 of February 2008, Mark Lord wrote: [EMAIL PROTECTED] wrote: .. I've been watching for kexec hibernate for a little while now, and the last I saw was that acpi was incompatible with the kexec hibernate (but the suspend folks were still claiming that devices needed to be put in the 'right mode' not just powered off. I've been waiting to see this resolved. .. Yeah, exactly. What's so special about poweroff on hibernation? Why even bother with the special "S4" state there? (1) To be able to wake up with the help of devices that can't wake the system up from S5 (power off) (2) To handle some platform devices appropriately over the cycle .. That's the theory. I've read about it, but have yet to imagine any real-life situation where it applies. But this isn't my speciality, so.. do you have experience with any real examples? Thanks! I want a real full poweroff, or at least I think I do. Why wouldn't I? You may want that, some people may not want it. We are supposed to handle S4, the BIOS/platform may expect us to do that, so IMO this is a good enough reason to do it. Especially that we can. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Friday, 22 of February 2008, Mark Lord wrote: > [EMAIL PROTECTED] wrote: > .. > > I've been watching for kexec hibernate for a little while now, and the > > last I saw was that acpi was incompatible with the kexec hibernate (but > > the suspend folks were still claiming that devices needed to be put in > > the 'right mode' not just powered off. I've been waiting to see this > > resolved. > .. > > Yeah, exactly. What's so special about poweroff on hibernation? > Why even bother with the special "S4" state there? (1) To be able to wake up with the help of devices that can't wake the system up from S5 (power off) (2) To handle some platform devices appropriately over the cycle > I want a real full poweroff, or at least I think I do. Why wouldn't I? > > You may want that, some people may not want it. We are supposed to handle S4, the BIOS/platform may expect us to do that, so IMO this is a good enough reason to do it. Especially that we can. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
[EMAIL PROTECTED] wrote: .. I've been watching for kexec hibernate for a little while now, and the last I saw was that acpi was incompatible with the kexec hibernate (but the suspend folks were still claiming that devices needed to be put in the 'right mode' not just powered off. I've been waiting to see this resolved. .. Yeah, exactly. What's so special about poweroff on hibernation? Why even bother with the special "S4" state there? I want a real full poweroff, or at least I think I do. Why wouldn't I? - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Rafael J. Wysocki wrote: No. Again, if there are devices that wake us up from S4, but not from S5, they need to be handled differently in the *enter S4* case (hibernation) and in the *enter S5* case (powering off the system). .. Something I've never understood, is why we would ever want to bother with *S4* at all? I actually like hibernation (great for travelling), but I treat it as if it were a complete power-off (S5?). I pull batteries, unplug drives, boot other operating systems, etc.. And when I put it all back together again with the Linux disk inserted, I fully expect it to "resume" from the hibernation of 3 months ago. And it does. Why would I ever want anything less than a full poweroff for hibernation Thanks. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi! > It's "snapshot-and-restore", and my opinion is that: > > - it should *never* call "suspend()"/"resume()" at all (that should be >reserved purely for suspend-to-RAM and has real power management >issues!) Hmm, entering S4 seems like good place to call suspend() for... unless you want separate freeze()/unfreeze(), suspend()/resume(), suspend_s4() and halt() callbacks. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Fri, Feb 22, 2008 at 7:45 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > On Thursday, February 21, 2008 2:11 pm Rafael J. Wysocki wrote: > > Below is a patch that should work around the issue. Please try it and let > > me know if it helps. > > I ended up applying the below patch instead, so it would build, and > unfortunately it still hung at suspend time. I encountered the same patching problem, but realized that it was due to earlier patch that you had wanted me to test, so if you revert your patch back to the current git, Rafael's patch will apply and compile cleanly. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Friday, 22 of February 2008, Jesse Barnes wrote: > On Thursday, February 21, 2008 2:11 pm Rafael J. Wysocki wrote: > > Below is a patch that should work around the issue. Please try it and let > > me know if it helps. > > I ended up applying the below patch instead, so it would build, and > unfortunately it still hung at suspend time. Hmm. > So at this point, the known workarounds to the hang at suspend time are to > remove the device power down call or to boot with 'no_console_suspend'. > The 'screen turns green' problem is fixed by the extra 'inb' added in the > patch below (at least for me). That is suspicious (see below). > > diff --git a/drivers/char/drm/i915_drv.c b/drivers/char/drm/i915_drv.c > index 35758a6..35b5a60 100644 > --- a/drivers/char/drm/i915_drv.c > +++ b/drivers/char/drm/i915_drv.c > @@ -27,6 +27,7 @@ > * > */ > > +#include > #include "drmP.h" > #include "drm.h" > #include "i915_drm.h" > @@ -222,6 +223,7 @@ static void i915_restore_vga(struct drm_device *dev) > dev_priv->saveGR[0x18]); > > /* Attribute controller registers */ > + inb(st01); /* switch back to index mode */ > for (i = 0; i < 20; i++) > i915_write_ar(st01, i, dev_priv->saveAR[i], 0); > inb(st01); /* switch back to index mode */ > @@ -249,6 +251,9 @@ static int i915_suspend(struct drm_device *dev) > return -ENODEV; > } > > + if (in_hibernation_power_off()) > + return 0; > + This thing should make i915_suspend() a noop in the last phase of hibernation, so if it still only works when you remove the pci_set_power_state(dev->pdev, PCI_D3hot), then I don't get it. Can you please try the pach below instead? Thanks, Rafael On Thursday, February 21, 2008 2:11 pm Rafael J. Wysocki wrote: > Below is a patch that should work around the issue. Please try it and let > me know if it helps. I ended up applying the below patch instead, so it would build, and unfortunately it still hung at suspend time. So at this point, the known workarounds to the hang at suspend time are to remove the device power down call or to boot with 'no_console_suspend'. The 'screen turns green' problem is fixed by the extra 'inb' added in the patch below (at least for me). Jesse --- drivers/char/drm/i915_drv.c |5 +++-- include/linux/suspend.h |2 ++ kernel/power/disk.c | 10 +- 3 files changed, 14 insertions(+), 3 deletions(-) Index: linux-2.6/drivers/char/drm/i915_drv.c === --- linux-2.6.orig/drivers/char/drm/i915_drv.c +++ linux-2.6/drivers/char/drm/i915_drv.c @@ -27,6 +27,7 @@ * */ +#include #include "drmP.h" #include "drm.h" #include "i915_drm.h" @@ -222,6 +223,7 @@ static void i915_restore_vga(struct drm_ dev_priv->saveGR[0x18]); /* Attribute controller registers */ + inb(st01); /* switch back to index mode */ for (i = 0; i < 20; i++) i915_write_ar(st01, i, dev_priv->saveAR[i], 0); inb(st01); /* switch back to index mode */ @@ -366,9 +368,8 @@ static int i915_suspend(struct drm_devic i915_save_vga(dev); - if (state.event == PM_EVENT_SUSPEND) { + if (state.event == PM_EVENT_SUSPEND && !in_hibernation_power_off()) { /* Shut down the device */ - pci_disable_device(dev->pdev); pci_set_power_state(dev->pdev, PCI_D3hot); } Index: linux-2.6/include/linux/suspend.h === --- linux-2.6.orig/include/linux/suspend.h +++ linux-2.6/include/linux/suspend.h @@ -209,6 +209,7 @@ extern unsigned long get_safe_page(gfp_t extern void hibernation_set_ops(struct platform_hibernation_ops *ops); extern int hibernate(void); +extern bool in_hibernation_power_off(void); #else /* CONFIG_HIBERNATION */ static inline int swsusp_page_is_forbidden(struct page *p) { return 0; } static inline void swsusp_set_page_free(struct page *p) {} @@ -216,6 +217,7 @@ static inline void swsusp_unset_page_fre static inline void hibernation_set_ops(struct platform_hibernation_ops *ops) {} static inline int hibernate(void) { return -ENOSYS; } +static inline bool in_hibernation_power_off(void) { return false; } #endif /* CONFIG_HIBERNATION */ #ifdef CONFIG_PM_SLEEP Index: linux-2.6/kernel/power/disk.c === --- linux-2.6.orig/kernel/power/disk.c +++ linux-2.6/kernel/power/disk.c @@ -24,7 +24,7 @@ #include "power.h" - +static bool entering_sleep_state; static int noresume = 0; static char resume_file[256] = CONFIG_PM_STD_PARTITION; dev_t swsusp_resume_device; @@ -381,6 +381,7 @@ int hibernation_platform_enter(void) if (!hibernation_ops) return -ENOSYS; + entering_sleep_state = true; /* * We have cancelled the power trans
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, February 21, 2008 2:11 pm Rafael J. Wysocki wrote: > Below is a patch that should work around the issue. Please try it and let > me know if it helps. I ended up applying the below patch instead, so it would build, and unfortunately it still hung at suspend time. So at this point, the known workarounds to the hang at suspend time are to remove the device power down call or to boot with 'no_console_suspend'. The 'screen turns green' problem is fixed by the extra 'inb' added in the patch below (at least for me). Jesse diff --git a/drivers/char/drm/i915_drv.c b/drivers/char/drm/i915_drv.c index 35758a6..35b5a60 100644 --- a/drivers/char/drm/i915_drv.c +++ b/drivers/char/drm/i915_drv.c @@ -27,6 +27,7 @@ * */ +#include #include "drmP.h" #include "drm.h" #include "i915_drm.h" @@ -222,6 +223,7 @@ static void i915_restore_vga(struct drm_device *dev) dev_priv->saveGR[0x18]); /* Attribute controller registers */ + inb(st01); /* switch back to index mode */ for (i = 0; i < 20; i++) i915_write_ar(st01, i, dev_priv->saveAR[i], 0); inb(st01); /* switch back to index mode */ @@ -249,6 +251,9 @@ static int i915_suspend(struct drm_device *dev) return -ENODEV; } + if (in_hibernation_power_off()) + return 0; + pci_save_state(dev->pdev); pci_read_config_byte(dev->pdev, LBB, &dev_priv->saveLBB); @@ -364,7 +369,6 @@ static int i915_suspend(struct drm_device *dev) i915_save_vga(dev); /* Shut down the device */ - pci_disable_device(dev->pdev); pci_set_power_state(dev->pdev, PCI_D3hot); return 0; diff --git a/include/linux/suspend.h b/include/linux/suspend.h index 1d7d4c5..58d9f67 100644 --- a/include/linux/suspend.h +++ b/include/linux/suspend.h @@ -209,6 +209,7 @@ extern unsigned long get_safe_page(gfp_t gfp_mask); extern void hibernation_set_ops(struct platform_hibernation_ops *ops); extern int hibernate(void); +extern bool in_hibernation_power_off(void); #else /* CONFIG_HIBERNATION */ static inline int swsusp_page_is_forbidden(struct page *p) { return 0; } static inline void swsusp_set_page_free(struct page *p) {} @@ -216,6 +217,7 @@ static inline void swsusp_unset_page_free(struct page *p) {} static inline void hibernation_set_ops(struct platform_hibernation_ops *ops) {} static inline int hibernate(void) { return -ENOSYS; } +static inline bool in_hibernation_power_off(void) { return false; } #endif /* CONFIG_HIBERNATION */ #ifdef CONFIG_PM_SLEEP diff --git a/kernel/power/disk.c b/kernel/power/disk.c index 859a8e5..d842bf0 100644 --- a/kernel/power/disk.c +++ b/kernel/power/disk.c @@ -24,7 +24,7 @@ #include "power.h" - +static bool entering_sleep_state; static int noresume = 0; static char resume_file[256] = CONFIG_PM_STD_PARTITION; dev_t swsusp_resume_device; @@ -381,6 +381,7 @@ int hibernation_platform_enter(void) if (!hibernation_ops) return -ENOSYS; + entering_sleep_state = true; /* * We have cancelled the power transition by running * hibernation_ops->finish() before saving the image, so we should let @@ -412,6 +413,7 @@ int hibernation_platform_enter(void) } local_irq_enable(); + entering_sleep_state = false; /* * We don't need to reenable the nonboot CPUs or resume consoles, since * the system is going to be halted anyway. @@ -427,6 +429,12 @@ int hibernation_platform_enter(void) return error; } +bool in_hibernation_power_off(void) +{ + return entering_sleep_state; +} +EXPORT_SYMBOL(in_hibernation_power_off); + /** * power_down - Shut the machine down for hibernation. * - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Jesse Barnes wrote: > On Thursday, February 21, 2008 8:27 am Rafael J. Wysocki wrote: > > On Thursday, 21 of February 2008, Jeff Chua wrote: > > > On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> > wrote: > > > > On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > > > > > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes > > > > > <[EMAIL PROTECTED]> > > > > > > > > wrote: > > > > > > Ok, can you give this patch a try with the 'platform' method? It > > > > > > should at least tell us what ACPI would like the device to do at > > > > > > suspend time, but it probably won't fix the hang. > > > > > > It says "calling pci_set_power_state with 3". Then after all then it > > > still hangs, and then resume with Mr Green. > > > > > > PM: Syncing filesystems ... done. > > > Freezing user space processes ... (elapsed 0.00 seconds) done. > > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > > > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > > > PM: Freed 0 kbytes in 0.20 seconds (0.00 MB/s) > > > ACPI: Preparing to enter system sleep state S4 > > > Suspending console(s) > > > sd 0:0:0:0: [sda] Synchronizing SCSI cache > > > drm_sysfs_suspend > > > ACPI: PCI interrupt for device :00:02.0 disabled > > > calling pci_set_power_state with 3 > > > > So it returns the right value. > > > > Jeff, Jesse, please check one thing for me. Below is a patch that should work around the issue. Please try it and let me know if it helps. Thanks, Rafael --- drivers/char/drm/i915_drv.c |3 +++ include/linux/suspend.h |2 ++ kernel/power/disk.c |9 - 3 files changed, 13 insertions(+), 1 deletion(-) Index: linux-2.6/include/linux/suspend.h === --- linux-2.6.orig/include/linux/suspend.h +++ linux-2.6/include/linux/suspend.h @@ -209,6 +209,7 @@ extern unsigned long get_safe_page(gfp_t extern void hibernation_set_ops(struct platform_hibernation_ops *ops); extern int hibernate(void); +extern bool in_hibernation_power_off(void); #else /* CONFIG_HIBERNATION */ static inline int swsusp_page_is_forbidden(struct page *p) { return 0; } static inline void swsusp_set_page_free(struct page *p) {} @@ -216,6 +217,7 @@ static inline void swsusp_unset_page_fre static inline void hibernation_set_ops(struct platform_hibernation_ops *ops) {} static inline int hibernate(void) { return -ENOSYS; } +static inline bool in_hibernation_power_off(void) { return false; } #endif /* CONFIG_HIBERNATION */ #ifdef CONFIG_PM_SLEEP Index: linux-2.6/kernel/power/disk.c === --- linux-2.6.orig/kernel/power/disk.c +++ linux-2.6/kernel/power/disk.c @@ -24,7 +24,7 @@ #include "power.h" - +static bool entering_sleep_state; static int noresume = 0; static char resume_file[256] = CONFIG_PM_STD_PARTITION; dev_t swsusp_resume_device; @@ -381,6 +381,7 @@ int hibernation_platform_enter(void) if (!hibernation_ops) return -ENOSYS; + entering_sleep_state = true; /* * We have cancelled the power transition by running * hibernation_ops->finish() before saving the image, so we should let @@ -412,6 +413,7 @@ int hibernation_platform_enter(void) } local_irq_enable(); + entering_sleep_state = false; /* * We don't need to reenable the nonboot CPUs or resume consoles, since * the system is going to be halted anyway. @@ -427,6 +429,11 @@ int hibernation_platform_enter(void) return error; } +bool in_hibernation_power_off(void) +{ + return entering_sleep_state; +} + /** * power_down - Shut the machine down for hibernation. * Index: linux-2.6/drivers/char/drm/i915_drv.c === --- linux-2.6.orig/drivers/char/drm/i915_drv.c +++ linux-2.6/drivers/char/drm/i915_drv.c @@ -247,6 +247,9 @@ static int i915_suspend(struct drm_devic return -ENODEV; } + if (in_hibernation_power_off()) + return 0; + pci_save_state(dev->pdev); pci_read_config_byte(dev->pdev, LBB, &dev_priv->saveLBB); - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Jesse Barnes wrote: > On Thursday, February 21, 2008 8:27 am Rafael J. Wysocki wrote: > > On Thursday, 21 of February 2008, Jeff Chua wrote: > > > On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> > wrote: > > > > On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > > > > > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes > > > > > <[EMAIL PROTECTED]> > > > > > > > > wrote: > > > > > > Ok, can you give this patch a try with the 'platform' method? It > > > > > > should at least tell us what ACPI would like the device to do at > > > > > > suspend time, but it probably won't fix the hang. > > > > > > It says "calling pci_set_power_state with 3". Then after all then it > > > still hangs, and then resume with Mr Green. > > > > > > PM: Syncing filesystems ... done. > > > Freezing user space processes ... (elapsed 0.00 seconds) done. > > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > > > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > > > PM: Freed 0 kbytes in 0.20 seconds (0.00 MB/s) > > > ACPI: Preparing to enter system sleep state S4 > > > Suspending console(s) > > > sd 0:0:0:0: [sda] Synchronizing SCSI cache > > > drm_sysfs_suspend > > > ACPI: PCI interrupt for device :00:02.0 disabled > > > calling pci_set_power_state with 3 > > > > So it returns the right value. > > > > Jeff, Jesse, please check one thing for me. > > > > Please boot 2.6.25-rc2 (or better, the current head of the Linus' tree) > > with no_console_suspend and try to do the following: > > > > # echo 8 > /proc/sys/kernel/printk > > # echo core > /sys/power/pm_test > > # echo disk > /sys/power/state > > > > (that will run a test of the freeze/unfreeze code without creating the > > image) and then > > That comes back for me, without creating the green screen. There's a long > delay between it saying "entering S4" and actually resuming back to my > console though. There's an intentional 5 sec. wait. If the delay is longer that 5 sec., that's a bit strange. > > # echo mem > /sys/power/state > > > > (that will run a test of the suspend/resume code without actually > > suspending). > > > > I'd like to know if that works. > > This also works (after doing the echo disk > ...) above. That's what I wanted to know, thanks. > There's still a delay between "entering S3" and the resume to my console > though. If that's 5 sec., it's fine. Please apply the appended patch and try to hibernate. I wonder if you get the reboot or it hangs earlier. Thanks, Rafael --- kernel/power/disk.c |7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) Index: linux-2.6/kernel/power/disk.c === --- linux-2.6.orig/kernel/power/disk.c +++ linux-2.6/kernel/power/disk.c @@ -405,11 +405,7 @@ int hibernation_platform_enter(void) local_irq_disable(); error = device_power_down(PMSG_SUSPEND); - if (!error) { - hibernation_ops->enter(); - /* We should never get here */ - while (1); - } + mdelay(1000); local_irq_enable(); /* @@ -424,6 +420,7 @@ int hibernation_platform_enter(void) resume_console(); Close: hibernation_ops->end(); + kernel_restart(NULL); return error; } - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, February 21, 2008 8:27 am Rafael J. Wysocki wrote: > On Thursday, 21 of February 2008, Jeff Chua wrote: > > On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > > > > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes > > > > <[EMAIL PROTECTED]> > > > > > > wrote: > > > > > Ok, can you give this patch a try with the 'platform' method? It > > > > > should at least tell us what ACPI would like the device to do at > > > > > suspend time, but it probably won't fix the hang. > > > > It says "calling pci_set_power_state with 3". Then after all then it > > still hangs, and then resume with Mr Green. > > > > PM: Syncing filesystems ... done. > > Freezing user space processes ... (elapsed 0.00 seconds) done. > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > > PM: Freed 0 kbytes in 0.20 seconds (0.00 MB/s) > > ACPI: Preparing to enter system sleep state S4 > > Suspending console(s) > > sd 0:0:0:0: [sda] Synchronizing SCSI cache > > drm_sysfs_suspend > > ACPI: PCI interrupt for device :00:02.0 disabled > > calling pci_set_power_state with 3 > > So it returns the right value. > > Jeff, Jesse, please check one thing for me. > > Please boot 2.6.25-rc2 (or better, the current head of the Linus' tree) > with no_console_suspend and try to do the following: > > # echo 8 > /proc/sys/kernel/printk > # echo core > /sys/power/pm_test > # echo disk > /sys/power/state > > (that will run a test of the freeze/unfreeze code without creating the > image) and then That comes back for me, without creating the green screen. There's a long delay between it saying "entering S4" and actually resuming back to my console though. > # echo mem > /sys/power/state > > (that will run a test of the suspend/resume code without actually > suspending). > > I'd like to know if that works. This also works (after doing the echo disk > ...) above. There's still a delay between "entering S3" and the resume to my console though. Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Jeff Chua wrote: > On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > > > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes <[EMAIL PROTECTED]> > > wrote: > > > > Ok, can you give this patch a try with the 'platform' method? It > > should > > > > at least tell us what ACPI would like the device to do at suspend time, > > > > but it probably won't fix the hang. > > It says "calling pci_set_power_state with 3". Then after all then it > still hangs, and then resume with Mr Green. > > PM: Syncing filesystems ... done. > Freezing user space processes ... (elapsed 0.00 seconds) done. > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > PM: Freed 0 kbytes in 0.20 seconds (0.00 MB/s) > ACPI: Preparing to enter system sleep state S4 > Suspending console(s) > sd 0:0:0:0: [sda] Synchronizing SCSI cache > drm_sysfs_suspend > ACPI: PCI interrupt for device :00:02.0 disabled > calling pci_set_power_state with 3 So it returns the right value. Jeff, Jesse, please check one thing for me. Please boot 2.6.25-rc2 (or better, the current head of the Linus' tree) with no_console_suspend and try to do the following: # echo 8 > /proc/sys/kernel/printk # echo core > /sys/power/pm_test # echo disk > /sys/power/state (that will run a test of the freeze/unfreeze code without creating the image) and then # echo mem > /sys/power/state (that will run a test of the suspend/resume code without actually suspending). I'd like to know if that works. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wed, 20 Feb 2008, Linus Torvalds wrote: Well, it seems like we'll have to fix drivers in either case, and isn't a kexec approach fundamentally more sound and simple, design-wise? Rafael pointed out some problems with properly setting wakeup states, but I think that could be overcome... I don't personally mind kexec at all, but on the other hand, I don't care about suspend-to-disk in the first place. I do know that some people really don't want it, and I suspect that they have valid reasons. Ranging from memory use to simply just performance. I've been watching for kexec hibernate for a little while now, and the last I saw was that acpi was incompatible with the kexec hibernate (but the suspend folks were still claiming that devices needed to be put in the 'right mode' not just powered off. I've been waiting to see this resolved. David Lang - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 05:05:32PM +1100, Nigel Cunningham wrote: > Hi Greg. > > Greg KH wrote: >> On Thu, Feb 21, 2008 at 12:17:06PM +1100, Nigel Cunningham wrote: >>> Hi. >>> >>> Greg KH wrote: On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: > Hi. > > Matthew Garrett wrote: >> On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: >>> - people keep talking about hibernating to an ext3 fs mounted on fuse >>> as a limitation of the freezer. To do that with kexec, you're still >>> going to have to bmap the ext3 fs and pass the block list (in which >>> case we can also do it without kexec) or umount all the ext3/fuse >>> part and remount in the kexec'd kernel. Sort of defeats the purpose, >>> doesn't it? >> No, with a freezer-based model you can basically *never* suspend to >> anything related to FUSE or a userspace USB device or anything >> involving userspace iSCSI initiators or whatever. Sure, there are >> cases where moving away from the current model doesn't buy you >> anything, but that doesn't mean that the current model is a good >> thing. It's not. The freezer is a fundamentally broken concept. > Putting drivers and filesystems in userspace is the fundamentally > broken concept. Not just when it comes to the freezer. The whole idea > is inherently racy. Racy with regards to other things becides trying to suspend a machine? If so, what? >>> That depends on what sort of tangled web you want to weave. >> Lots of them :) >> We have tanks running Linux using userspace USB drivers for vision >> control systems (scary, I know...) They seem to be successfully running >> for many years now, and I'm interested in making sure those kinds of >> things keep working. >> We also have laser welding robots with userspace PCI drivers in car >> manufacturing plants. And other laser cutting robots slicing wood in >> patterns moving at a rate of over 3 meters a second. Again, with >> userspace drivers and Linux. >> Those users would also love to know of any potential problems you know >> of for this situation. >>> Low memory situations is one other situation that occurs to me >>> quickly, especially (though not only) if your ability to swap were to >>> depend upon a userspace driver and/or filesystem. >> Sure, swap over a userspace filesystem or driver isn't a sane idea. And >> neither is swaping over NFS over a PPP connection attached to a USB to >> serial device. Yes, it's possible, and all in the kernel, but not a >> wise decision. >> Other than foolish configurations, if you come up with other issues >> surrounding userspace drivers that could cause problems, please let me >> know. > > A simple OOM condition isn't an issue? Surely a driver stalling because > some of its memory gets swapped out just before it goes to use it would be > a problem if it resulted in getting the length of a cut wrong or caused > some distorted vision or a late turn :> > > Am I missing something? Maybe these drivers mlock memory to avoid those > issues or something like that? I think the mlock their memory to prevent this from happening, it's not hard when you control all the applications on the box :) thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi Greg. Greg KH wrote: On Thu, Feb 21, 2008 at 12:17:06PM +1100, Nigel Cunningham wrote: Hi. Greg KH wrote: On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: Hi. Matthew Garrett wrote: On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: - people keep talking about hibernating to an ext3 fs mounted on fuse as a limitation of the freezer. To do that with kexec, you're still going to have to bmap the ext3 fs and pass the block list (in which case we can also do it without kexec) or umount all the ext3/fuse part and remount in the kexec'd kernel. Sort of defeats the purpose, doesn't it? No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. Putting drivers and filesystems in userspace is the fundamentally broken concept. Not just when it comes to the freezer. The whole idea is inherently racy. Racy with regards to other things becides trying to suspend a machine? If so, what? That depends on what sort of tangled web you want to weave. Lots of them :) We have tanks running Linux using userspace USB drivers for vision control systems (scary, I know...) They seem to be successfully running for many years now, and I'm interested in making sure those kinds of things keep working. We also have laser welding robots with userspace PCI drivers in car manufacturing plants. And other laser cutting robots slicing wood in patterns moving at a rate of over 3 meters a second. Again, with userspace drivers and Linux. Those users would also love to know of any potential problems you know of for this situation. Low memory situations is one other situation that occurs to me quickly, especially (though not only) if your ability to swap were to depend upon a userspace driver and/or filesystem. Sure, swap over a userspace filesystem or driver isn't a sane idea. And neither is swaping over NFS over a PPP connection attached to a USB to serial device. Yes, it's possible, and all in the kernel, but not a wise decision. Other than foolish configurations, if you come up with other issues surrounding userspace drivers that could cause problems, please let me know. A simple OOM condition isn't an issue? Surely a driver stalling because some of its memory gets swapped out just before it goes to use it would be a problem if it resulted in getting the length of a cut wrong or caused some distorted vision or a late turn :> Am I missing something? Maybe these drivers mlock memory to avoid those issues or something like that? Regards, Nigel - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 12:17:06PM +1100, Nigel Cunningham wrote: > Hi. > > Greg KH wrote: >> On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: >>> Hi. >>> >>> Matthew Garrett wrote: On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: > - people keep talking about hibernating to an ext3 fs mounted on fuse > as a limitation of the freezer. To do that with kexec, you're still > going to have to bmap the ext3 fs and pass the block list (in which > case we can also do it without kexec) or umount all the ext3/fuse part > and remount in the kexec'd kernel. Sort of defeats the purpose, doesn't > it? No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. >>> Putting drivers and filesystems in userspace is the fundamentally broken >>> concept. Not just when it comes to the freezer. The whole idea is >>> inherently racy. >> Racy with regards to other things becides trying to suspend a machine? >> If so, what? > > That depends on what sort of tangled web you want to weave. Lots of them :) We have tanks running Linux using userspace USB drivers for vision control systems (scary, I know...) They seem to be successfully running for many years now, and I'm interested in making sure those kinds of things keep working. We also have laser welding robots with userspace PCI drivers in car manufacturing plants. And other laser cutting robots slicing wood in patterns moving at a rate of over 3 meters a second. Again, with userspace drivers and Linux. Those users would also love to know of any potential problems you know of for this situation. > Low memory situations is one other situation that occurs to me > quickly, especially (though not only) if your ability to swap were to > depend upon a userspace driver and/or filesystem. Sure, swap over a userspace filesystem or driver isn't a sane idea. And neither is swaping over NFS over a PPP connection attached to a USB to serial device. Yes, it's possible, and all in the kernel, but not a wise decision. Other than foolish configurations, if you come up with other issues surrounding userspace drivers that could cause problems, please let me know. thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes <[EMAIL PROTECTED]> > wrote: > > > Ok, can you give this patch a try with the 'platform' method? It should > > > at least tell us what ACPI would like the device to do at suspend time, > > > but it probably won't fix the hang. It says "calling pci_set_power_state with 3". Then after all then it still hangs, and then resume with Mr Green. PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.00 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. PM: Shrinking memory... ^H-^Hdone (0 pages freed) PM: Freed 0 kbytes in 0.20 seconds (0.00 MB/s) ACPI: Preparing to enter system sleep state S4 Suspending console(s) sd 0:0:0:0: [sda] Synchronizing SCSI cache drm_sysfs_suspend ACPI: PCI interrupt for device :00:02.0 disabled calling pci_set_power_state with 3 ACPI: PCI interrupt for device :00:1d.7 disabled ACPI: PCI interrupt for device :00:1d.3 disabled ACPI: PCI interrupt for device :00:1d.2 disabled ACPI: PCI interrupt for device :00:1d.1 disabled ACPI: PCI interrupt for device :00:1d.0 disabled ACPI: PCI interrupt for device :00:1b.0 disabled Disabling non-boot CPUs ... PM: Creating hibernation image: PM: Need to copy 25136 pages tick-braodcast: ignoring broadcast for offline CPU #1 PM: Writing back config space on device :00:02.0 at offset 1 (was 97, writing 93) ACPI: PCI Interrupt :00:1b.0[B] -> GSI 17 (level, low) -> IRQ 17 PCI: Setting latency timer of device :00:1b.0 to 64 PCI: Setting latency timer of device :00:1c.0 to 64 PCI: Setting latency timer of device :00:1c.1 to 64 ... Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 9:21 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > I hope those are just warning that can just be ignored. > > Oops again, should be dev->pdev. Silly DRM layer obfuscation. I was just about to write that the test didn't work. Both std str hangs even before attempting to suspend. Anyway, I'm compiling and rebooting now. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi. Matthew Garrett wrote: On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: Matthew Garrett wrote: No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. Putting drivers and filesystems in userspace is the fundamentally broken concept. Not just when it comes to the freezer. The whole idea is inherently racy. You can draw silly diagrams about how the freezer supposedly works in LCA slides and spread FUD as much as you like. In the end, though, it's not nearly as hit-and-miss as you say, and replacing the freezer with a kexec based freezer is only going to create as many problems as it removes. I'm really not interested in debating the matter. There are all sorts of potential uses for the freezer, but hibernation isn't one of them. We *need* to get rid of the freezer for suspend to RAM (because a band-aid to ensure atomicity is kind of pointless when the operation you're entering is inherently atomic), and once all the drivers are able to deal with that then it's trivial to get rid of it for hibernation as well. Arguing that the reality of userspace drivers is broken doesn't help here. It's what we have to work with. Re suspend to ram, I agree. No argument there. Re hibernation, I think your assertion that it will be trivial to get rid of it for hibernation is just plain wrong. Perhaps you don't understand the issues as well as you think you do. Re arguing that the reality of userspace drivers is broken doesn't help here: Yeah, I know. But sometimes if you point out broken ideas for long enough, people do actually listen. Or you learn. Or both. Frankly, I don't want to debate the issue either. What I really want is just to have a hibernation implementation that works, is flexibile, reliable and quick, and one that I don't have to keep maintaining. Unfortunately for me, most people seem to be more concerned with fixing hypothetical problems than with giving users something they can actually use. You're looking at a tiny amount of memory when compared to current systems. It's really not a problem. Please, quantify 'tiny'. In embedded, 5MB can be too much. I've worked on embedded solutions. I'm not pulling problems out of thin air. Then the in-kernel solution has already lost anyway, and I'm desperately unconcerned about out of tree stuff. I know. I'd submit it, or work on breaking it into pieces and submitting them one at a time, but that seems to me to be a waste of time. Nigel - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 5:19 pm Jeff Chua wrote: > On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > Oops, maybe this should just be pci_choose_state instead. > > And this change should just be reverted (leave it as PCI_D0). > > drivers/char/drm/i915_drv.c: In function 'i915_suspend': > drivers/char/drm/i915_drv.c:372: warning: passing argument 1 of > 'pci_choose_state' from incompatible pointer type > drivers/char/drm/i915_drv.c:373: warning: passing argument 1 of > 'pci_choose_state' from incompatible pointer type > > I hope those are just warning that can just be ignored. Oops again, should be dev->pdev. Silly DRM layer obfuscation. Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 8:39 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > Oops, maybe this should just be pci_choose_state instead. > And this change should just be reverted (leave it as PCI_D0). drivers/char/drm/i915_drv.c: In function 'i915_suspend': drivers/char/drm/i915_drv.c:372: warning: passing argument 1 of 'pci_choose_state' from incompatible pointer type drivers/char/drm/i915_drv.c:373: warning: passing argument 1 of 'pci_choose_state' from incompatible pointer type I hope those are just warning that can just be ignored. Ok, rebooting and will get back shortly. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi. Greg KH wrote: On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: Hi. Matthew Garrett wrote: On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: - people keep talking about hibernating to an ext3 fs mounted on fuse as a limitation of the freezer. To do that with kexec, you're still going to have to bmap the ext3 fs and pass the block list (in which case we can also do it without kexec) or umount all the ext3/fuse part and remount in the kexec'd kernel. Sort of defeats the purpose, doesn't it? No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. Putting drivers and filesystems in userspace is the fundamentally broken concept. Not just when it comes to the freezer. The whole idea is inherently racy. Racy with regards to other things becides trying to suspend a machine? If so, what? That depends on what sort of tangled web you want to weave. Low memory situations is one other situation that occurs to me quickly, especially (though not only) if your ability to swap were to depend upon a userspace driver and/or filesystem. Regards, Nigel - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: > Matthew Garrett wrote: > >No, with a freezer-based model you can basically *never* suspend to > >anything related to FUSE or a userspace USB device or anything involving > >userspace iSCSI initiators or whatever. Sure, there are cases where > >moving away from the current model doesn't buy you anything, but that > >doesn't mean that the current model is a good thing. It's not. The > >freezer is a fundamentally broken concept. > > Putting drivers and filesystems in userspace is the fundamentally broken > concept. Not just when it comes to the freezer. The whole idea is > inherently racy. You can draw silly diagrams about how the freezer > supposedly works in LCA slides and spread FUD as much as you like. In > the end, though, it's not nearly as hit-and-miss as you say, and > replacing the freezer with a kexec based freezer is only going to create > as many problems as it removes. I'm really not interested in debating the matter. There are all sorts of potential uses for the freezer, but hibernation isn't one of them. We *need* to get rid of the freezer for suspend to RAM (because a band-aid to ensure atomicity is kind of pointless when the operation you're entering is inherently atomic), and once all the drivers are able to deal with that then it's trivial to get rid of it for hibernation as well. Arguing that the reality of userspace drivers is broken doesn't help here. It's what we have to work with. > >You're looking at a tiny amount of memory when compared to current > >systems. It's really not a problem. > > Please, quantify 'tiny'. In embedded, 5MB can be too much. I've worked > on embedded solutions. I'm not pulling problems out of thin air. Then the in-kernel solution has already lost anyway, and I'm desperately unconcerned about out of tree stuff. -- Matthew Garrett | [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Jesse Barnes wrote: > On Wednesday, February 20, 2008 3:49 pm Rafael J. Wysocki wrote: > > > And just to confirm that, I just tested the current DRM modules against a > > > 2.6.23.15 kernel. > > > > In 2.6.23.x there's no second ->suspend() during hibernation, so no wonder. > > In 2.6.23 it's just: > ->suspend() > ->resume() ->shutdown() (that breaks wake up from S4 with many devices, including but not limited to the RTC wake alarm). > *S4* > ? > Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Linus Torvalds wrote: > > On Thu, 21 Feb 2008, Rafael J. Wysocki wrote: > > > > > Secondly, the one that people should use ("pci_choose_state()") doesn't > > > actually do what you claim it does. It does all kinds of wrong things, > > > and > > > doesn't even take the target state into account at all. So look again. > > > > Well, if platform_pci_choose_state() is defined, pci_choose_state() returns > > its result and on ACPI systems that points to acpi_pci_choose_state(), so in > > fact it does what I said (apart from the error path). > > Did you check closer? Yes, I did. > I repeat: acpi_pci_choose_state() (when called from pci_choose_state()) > doesn't even look at the target 'state'. It just blindly assumes that you > want the deepest sleep-state you can have. acpi_pm_device_sleep_state() (that is called by acpi_pci_choose_state()) takes the target state directly from the ACPI layer. We just want to get rid of the argument passed to ->suspend() eventually, but there may be many _suspend_ states available (eg. "mem" and "standby") and for each of them there may be different constraints on the device's state. We have to tell the driver which device states are possible in the target system sleep state. Right now we arbitrarily choose the one with the lowest power usage - for given target system sleep state. > Which happens to be correct for normal suspend, but means that if you want > to test other states (through '/sys/devices/.../power'), that sounds > broken. This interface is not available any more (ie. there's only "wakeup" in /sys/devices/.../power). > I didn't check any closer, but go check it yourself. The short and sweet: > acpi_pci_choose_state() totally ignores its 'state' argument. Do you > really think that's correct? Yes, I do. > But yes, "pci_choose_state()' effectively does that too, apart from > PM_EVENT_ON, which is never used. > > (But the whole and only point of pci_choose_state() was to do the > PM_EVENT_FREEZE thing differently, which it doesn't do, so I think the > real issue here is that the interface is really rather mis-designed) You're wrong, sorry. With PM_EVENT_FREEZE it wouldn't even be necessary. It's there, because potentially there are many possibilities with PM_EVENT_SUSPEND and in fact it shouldn't even be used with PM_EVENT_FREEZE. All of this is more or less orthogonal to the issue at hand, which boils down to the fact that we use the _suspend_ callbacks for hibernation and we shouldn't be doing that. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 4:35 pm Jeff Chua wrote: > On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > Ok, can you give this patch a try with the 'platform' method? It should > > at least tell us what ACPI would like the device to do at suspend time, > > but it probably won't fix the hang. > > I can't get it to compile. > > drivers/char/drm/i915_drv.c: In function 'i915_suspend': > drivers/char/drm/i915_drv.c:372: error: implicit declaration of > function 'acpi_pci_choose_state' Oops, maybe this should just be pci_choose_state instead. > drivers/char/drm/i915_drv.c: In function 'i915_resume': > drivers/char/drm/i915_drv.c:383: error: 'state' undeclared (first use > in this function) > drivers/char/drm/i915_drv.c:383: error: (Each undeclared identifier is > reported only once > drivers/char/drm/i915_drv.c:383: error: for each function it appears in.) And this change should just be reverted (leave it as PCI_D0). Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 11:40:06AM +1100, Nigel Cunningham wrote: > Hi. > > Matthew Garrett wrote: >> On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: >>> - people keep talking about hibernating to an ext3 fs mounted on fuse as >>> a limitation of the freezer. To do that with kexec, you're still going to >>> have to bmap the ext3 fs and pass the block list (in which case we can >>> also do it without kexec) or umount all the ext3/fuse part and remount in >>> the kexec'd kernel. Sort of defeats the purpose, doesn't it? >> No, with a freezer-based model you can basically *never* suspend to >> anything related to FUSE or a userspace USB device or anything involving >> userspace iSCSI initiators or whatever. Sure, there are cases where moving >> away from the current model doesn't buy you anything, but that doesn't >> mean that the current model is a good thing. It's not. The freezer is a >> fundamentally broken concept. > > Putting drivers and filesystems in userspace is the fundamentally broken > concept. Not just when it comes to the freezer. The whole idea is > inherently racy. Racy with regards to other things becides trying to suspend a machine? If so, what? thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi. Matthew Garrett wrote: On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: - people keep talking about hibernating to an ext3 fs mounted on fuse as a limitation of the freezer. To do that with kexec, you're still going to have to bmap the ext3 fs and pass the block list (in which case we can also do it without kexec) or umount all the ext3/fuse part and remount in the kexec'd kernel. Sort of defeats the purpose, doesn't it? No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. Putting drivers and filesystems in userspace is the fundamentally broken concept. Not just when it comes to the freezer. The whole idea is inherently racy. You can draw silly diagrams about how the freezer supposedly works in LCA slides and spread FUD as much as you like. In the end, though, it's not nearly as hit-and-miss as you say, and replacing the freezer with a kexec based freezer is only going to create as many problems as it removes. I also wonder about how much of a pain it's going to be setting up userspace for this kexec'd kernel. Will you need a separate partition just for it? If not, will the userspace be loaded into memory all the time (more memory wasted for normal use), or loaded from ordinary partitions at kexec time (how to do safely? - more info to transfer between kernels?). You're looking at a tiny amount of memory when compared to current systems. It's really not a problem. Please, quantify 'tiny'. In embedded, 5MB can be too much. I've worked on embedded solutions. I'm not pulling problems out of thin air. Regards, Nigel - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 5:37 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > Ok, can you give this patch a try with the 'platform' method? It should at > least tell us what ACPI would like the device to do at suspend time, but it > probably won't fix the hang. I can't get it to compile. drivers/char/drm/i915_drv.c: In function 'i915_suspend': drivers/char/drm/i915_drv.c:372: error: implicit declaration of function 'acpi_pci_choose_state' drivers/char/drm/i915_drv.c: In function 'i915_resume': drivers/char/drm/i915_drv.c:383: error: 'state' undeclared (first use in this function) drivers/char/drm/i915_drv.c:383: error: (Each undeclared identifier is reported only once drivers/char/drm/i915_drv.c:383: error: for each function it appears in.) make[3]: *** [drivers/char/drm/i915_drv.o] Error 1 make[2]: *** [drivers/char/drm] Error 2 make[1]: *** [drivers/char] Error 2 make: *** [drivers] Error 2 Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, 21 Feb 2008, Rafael J. Wysocki wrote: > > > Secondly, the one that people should use ("pci_choose_state()") doesn't > > actually do what you claim it does. It does all kinds of wrong things, and > > doesn't even take the target state into account at all. So look again. > > Well, if platform_pci_choose_state() is defined, pci_choose_state() returns > its result and on ACPI systems that points to acpi_pci_choose_state(), so in > fact it does what I said (apart from the error path). Did you check closer? I repeat: acpi_pci_choose_state() (when called from pci_choose_state()) doesn't even look at the target 'state'. It just blindly assumes that you want the deepest sleep-state you can have. Which happens to be correct for normal suspend, but means that if you want to test other states (through '/sys/devices/.../power'), that sounds broken. I didn't check any closer, but go check it yourself. The short and sweet: acpi_pci_choose_state() totally ignores its 'state' argument. Do you really think that's correct? But yes, "pci_choose_state()' effectively does that too, apart from PM_EVENT_ON, which is never used. (But the whole and only point of pci_choose_state() was to do the PM_EVENT_FREEZE thing differently, which it doesn't do, so I think the real issue here is that the interface is really rather mis-designed) I suspect most people who ever really looked and worked on this code had a specific device in mind, and I'm sure that all of the code individually always ends up making sense from the standpoint of some specific device driver. It's just that it never seems to make sense from a bigger issues standpoint, and often seems senseless from the standpoint of other devices of other types. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 3:49 pm Rafael J. Wysocki wrote: > > And just to confirm that, I just tested the current DRM modules against a > > 2.6.23.15 kernel. > > In 2.6.23.x there's no second ->suspend() during hibernation, so no wonder. In 2.6.23 it's just: ->suspend() ->resume() *S4* ? I ask because we still do the D3hot call in the DRM tree, so the hang should still occur unless the PM or ACPI core has changed. > I'll figure out how to work around this issue in the current mainline, but > a real fix will only be possible when we have separate callbacks for > hibernation. Ok, thanks. Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Linus Torvalds wrote: > > On Thu, 21 Feb 2008, Rafael J. Wysocki wrote: > > > > In fact we have acpi_pci_choose_state() that tells the driver which power > > state to put the device into in ->suspend(). If that is used, the device > > ends > > up in the state expected by to BIOS for S4. > > First off, nobody should *ever* use that directly anyway. Yes, sorry. > Secondly, the one that people should use ("pci_choose_state()") doesn't > actually do what you claim it does. It does all kinds of wrong things, and > doesn't even take the target state into account at all. So look again. Well, if platform_pci_choose_state() is defined, pci_choose_state() returns its result and on ACPI systems that points to acpi_pci_choose_state(), so in fact it does what I said (apart from the error path). > > No. Again, if there are devices that wake us up from S4, but not from S5, > > they need to be handled differently in the *enter S4* case (hibernation) and > > in the *enter S5* case (powering off the system). > > And again, what does this have to do with (the example I used) the > graphics hardware? Answer: nothing. The example I gave you we simply DO > THE WRONG THING FOR. > > Same thing for things like USB devices - where pci_choose_state() doesn't > work to begin with. Why do we call "suspend()" on such a thing when we > don't want to suspend it? We shouldn't. We should call "freeze/unfreeze" > (which are no-ops) and then finally perhaps "poweroff", and that final > stage might want to spin things down or similar. I'm already convinced, really. :-) Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 09:45:02AM +1100, Nigel Cunningham wrote: > - people keep talking about hibernating to an ext3 fs mounted on fuse as > a limitation of the freezer. To do that with kexec, you're still going > to have to bmap the ext3 fs and pass the block list (in which case we > can also do it without kexec) or umount all the ext3/fuse part and > remount in the kexec'd kernel. Sort of defeats the purpose, doesn't it? No, with a freezer-based model you can basically *never* suspend to anything related to FUSE or a userspace USB device or anything involving userspace iSCSI initiators or whatever. Sure, there are cases where moving away from the current model doesn't buy you anything, but that doesn't mean that the current model is a good thing. It's not. The freezer is a fundamentally broken concept. > I also wonder about how much of a pain it's going to be setting up > userspace for this kexec'd kernel. Will you need a separate partition > just for it? If not, will the userspace be loaded into memory all the > time (more memory wasted for normal use), or loaded from ordinary > partitions at kexec time (how to do safely? - more info to transfer > between kernels?). You're looking at a tiny amount of memory when compared to current systems. It's really not a problem. -- Matthew Garrett | [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, 21 Feb 2008, Rafael J. Wysocki wrote: > > In fact we have acpi_pci_choose_state() that tells the driver which power > state to put the device into in ->suspend(). If that is used, the device ends > up in the state expected by to BIOS for S4. First off, nobody should *ever* use that directly anyway. Secondly, the one that people should use ("pci_choose_state()") doesn't actually do what you claim it does. It does all kinds of wrong things, and doesn't even take the target state into account at all. So look again. > No. Again, if there are devices that wake us up from S4, but not from S5, > they need to be handled differently in the *enter S4* case (hibernation) and > in the *enter S5* case (powering off the system). And again, what does this have to do with (the example I used) the graphics hardware? Answer: nothing. The example I gave you we simply DO THE WRONG THING FOR. Same thing for things like USB devices - where pci_choose_state() doesn't work to begin with. Why do we call "suspend()" on such a thing when we don't want to suspend it? We shouldn't. We should call "freeze/unfreeze" (which are no-ops) and then finally perhaps "poweroff", and that final stage might want to spin things down or similar. But *none* of it has anything to do with suspend, and none of it has anything to do with pci_choose_state() (much less acpi_pci_choose_state) The fact is, we should let the driver decide, and we should make it clear to the driver writer what he is deciding about - rather than basically lie and say "suspend the device and put it into D3" even when that's the last thing it should ever do. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Jesse Barnes wrote: > On Wednesday, February 20, 2008 3:03 pm Jesse Barnes wrote: > > On Wednesday, February 20, 2008 2:32 pm Jesse Barnes wrote: > > > On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > > > > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > > > > So, next I'll try "shutdown" to see if it work. I was using > > > > > > "platform". > > > > > > > > > > Ok, that would be good to try. > > > > > > > > "shutdown" does power down properly. But still green on resume. > > > > > > > > > Looks like the AR registers are hosed, which is what I thought I > > > > > fixed... Can you attach your i915_drv.c file just so I can sanity > > > > > check it? > > > > > > > > Attached. > > > > > > Jeff, for the hang on suspend problem, I know suspect something else in > > > 2.6.25-rc2 caused that. > > > > Looks like 2.6.25-rc1 also had broken suspend (my test was broken). IIRC, > > Dave and I had it working at LCA using the out of tree DRM modules on > > 2.6.23.14 or 15... Maybe you could give that a try? > > And just to confirm that, I just tested the current DRM modules against a > 2.6.23.15 kernel. In 2.6.23.x there's no second ->suspend() during hibernation, so no wonder. I'll figure out how to work around this issue in the current mainline, but a real fix will only be possible when we have separate callbacks for hibernation. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 3:03 pm Jesse Barnes wrote: > On Wednesday, February 20, 2008 2:32 pm Jesse Barnes wrote: > > On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > > > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > > > So, next I'll try "shutdown" to see if it work. I was using > > > > > "platform". > > > > > > > > Ok, that would be good to try. > > > > > > "shutdown" does power down properly. But still green on resume. > > > > > > > Looks like the AR registers are hosed, which is what I thought I > > > > fixed... Can you attach your i915_drv.c file just so I can sanity > > > > check it? > > > > > > Attached. > > > > Jeff, for the hang on suspend problem, I know suspect something else in > > 2.6.25-rc2 caused that. > > Looks like 2.6.25-rc1 also had broken suspend (my test was broken). IIRC, > Dave and I had it working at LCA using the out of tree DRM modules on > 2.6.23.14 or 15... Maybe you could give that a try? And just to confirm that, I just tested the current DRM modules against a 2.6.23.15 kernel. It suspends to disk correctly (w/o a hang) and doesn't give me a green screen, so something in 2.6.25 must be causing that (even 2.6.25-rc1 seems to have the problem). Also, this patch against 2.6.25-rc1 seemed to prevent the 'green screen' problem. 2.6.25-rc2 already has part of it... Anyway, let me know how your testing goes. Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thursday, 21 of February 2008, Linus Torvalds wrote: > > On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: > > > > > > > > which may have four entry-points that can be illogically mapped to the > > > > suspend/resume ones like we do now, but they really have nothing to do > > > > with suspending/resuming. > > > > Apart from putting devices into the right low power states, that is. > > And by "right low power states" you mean "wrong low-power states", right? No, I don't. > The thing is, they really *are* the wrong states for 99% of all hardware. > > If you really have a piece of hardware that you want to have the > "->poweroff()" thing do the same as "->suspend()", then hey, just use the > same function (or better yet, use two different functions with a call to a > shared part). > > Because IT IS NOT TRUE that ->suspend() puts the devices in the "right > power state". The power states are likely to be totally different for S3 > and for poweroff, and they are going to differ in different ways depending > on the device type. In fact we have acpi_pci_choose_state() that tells the driver which power state to put the device into in ->suspend(). If that is used, the device ends up in the state expected by to BIOS for S4. > One example would be the one that started this version of the whole > discussion (shock horror! We're on subject!) ie when you do a system > shutdown, you generally do not even *want* to put individual devices into > low-power states at all, because the actual "power off the system" thing > will take care of it for you much better. No. Again, if there are devices that wake us up from S4, but not from S5, they need to be handled differently in the *enter S4* case (hibernation) and in the *enter S5* case (powering off the system). > So to take just something as simple as VGA as an example: you really do > not want to suspend that device, because you want to see the poweroff > messages until the very end. > > So that final device ->poweroff function really has absolutely *nothing* > in common with the device ->suspend[_late] functions, simply because > almost any sane driver would decide to do different things. Yes, it would. Still, the common thing is, it (ie. ->poweroff) _may_ want to put the device into a low power state different from D3. > Of course, we can continue to do the insane thing and just continue to use > inappropriate and misleadign function callback names, and then encodign > what the *real* action should be in the argument and/or in magic > system-wide state parameters. To clarify, I agree that we should use different callbacks for hibernation. I'm only saying that _in_ _general_ we may need the ->poweroff callback. > So in that sense, it's certainly totally the same thing whether we call it > ->shutdown or ->poweroff or ->eat_a_banana, since you could always just > look at the argument and other clues, and decide that *this* time, for > *this* kind of device, the "eat a banana" callback actually means that we > should power it off, but wouldn't it be a lot more logical to just make it > clear in the first place that they aren't called for the same reason at > all? > > I'd claim that it's much easier for everybody (and _especially_ for device > driver writers) to have > > static int my_shutdown(struct pci_device *dev, int state) > { > .. do something .. > } > > static int my_suspend(struct pci_device *dev, int state) > { > .. do something .. > } > > ... > .shutdown = my_shutdown, > .suspend = my_suspend, > ... > > than to have > > static int my_suspend(struct pci_device *dev, state) > { > .. common code .. > if (state == XYZZY) > ..special code.. > else > ..other case code.. > } > > ... > .suspend = my_suspend > ... > > even if the latter might be fewer lines. It doesn't really matter if it's > fewer, does it, if the alternate version is more obvious about what it > does? > > The other issue is that I've long wanted to make sure that when people fix > suspend-to-ram, they don't screw up suspend-to-disk by mistake and vice > versa. When a driver writer makes changes, he shouldn't have the kind of > illogical "oops, unintended consequences" issues in general. It should be > pretty damn obvious when he changes suspend code vs when he changes > snapshot/restore code. > > We've somewhat untangled that on the "core kernel" layer, but we've left > the driver confusion alone. Well, I agree with that. As I said before, that's mainly because I've been busy with other stuff recently. Now, with the Alex's help, I'm hoping to take care of it soon. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kern
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: > > > > > > which may have four entry-points that can be illogically mapped to the > > > suspend/resume ones like we do now, but they really have nothing to do > > > with suspending/resuming. > > Apart from putting devices into the right low power states, that is. And by "right low power states" you mean "wrong low-power states", right? The thing is, they really *are* the wrong states for 99% of all hardware. If you really have a piece of hardware that you want to have the "->poweroff()" thing do the same as "->suspend()", then hey, just use the same function (or better yet, use two different functions with a call to a shared part). Because IT IS NOT TRUE that ->suspend() puts the devices in the "right power state". The power states are likely to be totally different for S3 and for poweroff, and they are going to differ in different ways depending on the device type. One example would be the one that started this version of the whole discussion (shock horror! We're on subject!) ie when you do a system shutdown, you generally do not even *want* to put individual devices into low-power states at all, because the actual "power off the system" thing will take care of it for you much better. So to take just something as simple as VGA as an example: you really do not want to suspend that device, because you want to see the poweroff messages until the very end. So that final device ->poweroff function really has absolutely *nothing* in common with the device ->suspend[_late] functions, simply because almost any sane driver would decide to do different things. Of course, we can continue to do the insane thing and just continue to use inappropriate and misleadign function callback names, and then encodign what the *real* action should be in the argument and/or in magic system-wide state parameters. So in that sense, it's certainly totally the same thing whether we call it ->shutdown or ->poweroff or ->eat_a_banana, since you could always just look at the argument and other clues, and decide that *this* time, for *this* kind of device, the "eat a banana" callback actually means that we should power it off, but wouldn't it be a lot more logical to just make it clear in the first place that they aren't called for the same reason at all? I'd claim that it's much easier for everybody (and _especially_ for device driver writers) to have static int my_shutdown(struct pci_device *dev, int state) { .. do something .. } static int my_suspend(struct pci_device *dev, int state) { .. do something .. } ... .shutdown = my_shutdown, .suspend = my_suspend, ... than to have static int my_suspend(struct pci_device *dev, state) { .. common code .. if (state == XYZZY) ..special code.. else ..other case code.. } ... .suspend = my_suspend ... even if the latter might be fewer lines. It doesn't really matter if it's fewer, does it, if the alternate version is more obvious about what it does? The other issue is that I've long wanted to make sure that when people fix suspend-to-ram, they don't screw up suspend-to-disk by mistake and vice versa. When a driver writer makes changes, he shouldn't have the kind of illogical "oops, unintended consequences" issues in general. It should be pretty damn obvious when he changes suspend code vs when he changes snapshot/restore code. We've somewhat untangled that on the "core kernel" layer, but we've left the driver confusion alone. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 2:32 pm Jesse Barnes wrote: > On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > > So, next I'll try "shutdown" to see if it work. I was using > > > > "platform". > > > > > > Ok, that would be good to try. > > > > "shutdown" does power down properly. But still green on resume. > > > > > Looks like the AR registers are hosed, which is what I thought I > > > fixed... Can you attach your i915_drv.c file just so I can sanity check > > > it? > > > > Attached. > > Jeff, for the hang on suspend problem, I know suspect something else in > 2.6.25-rc2 caused that. Looks like 2.6.25-rc1 also had broken suspend (my test was broken). IIRC, Dave and I had it working at LCA using the out of tree DRM modules on 2.6.23.14 or 15... Maybe you could give that a try? Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Hi. Jesse Barnes wrote: Well, it seems like we'll have to fix drivers in either case, and isn't a kexec approach fundamentally more sound and simple, design-wise? Rafael pointed out some problems with properly setting wakeup states, but I think that could be overcome... No. AFAICS, kexec is going to be more complex and ugly in many ways. To summarise, a kexec based hibernation is going to need the following additional requirements to just replace what we already have: - get the original kernel to allocate storage while racing against the rest of the system (currently allocation is done post-atomic copy & post-freezing - no racing). This makes it potentially slower, too; - get the original kernel to transfer the information about what swap was allocated to the kexec'd kernel, probably together with a lot of other information (which pages are nosave etc). - get the original kernel to keep memory free for the kexec'd kernel which would otherwise be usable. Not a biggy on desktops or laptops, but think about embedded. - people keep talking about hibernating to an ext3 fs mounted on fuse as a limitation of the freezer. To do that with kexec, you're still going to have to bmap the ext3 fs and pass the block list (in which case we can also do it without kexec) or umount all the ext3/fuse part and remount in the kexec'd kernel. Sort of defeats the purpose, doesn't it? I also wonder about how much of a pain it's going to be setting up userspace for this kexec'd kernel. Will you need a separate partition just for it? If not, will the userspace be loaded into memory all the time (more memory wasted for normal use), or loaded from ordinary partitions at kexec time (how to do safely? - more info to transfer between kernels?). I'd love it if kexec really was the panacea to the freezer issues, but problems like these make me think it isn't a viable solution. Nigel - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, 20 of February 2008, Jesse Barnes wrote: > On Wednesday, February 20, 2008 1:13 pm Linus Torvalds wrote: > > On Wed, 20 Feb 2008, Jesse Barnes wrote: > > > The current callback system looks like this (according to Rafael and the > > > last time I looked): > > > ->suspend(PMSG_FREEZE) > > > ->resume() > > > ->suspend(PMSG_SUSPEND) > > > *enter S3 or power off* > > > ->resume() > > > > Yes, it's very messy. > > > > It's messy for a few different reasons: > > > > - the one you hit: a driver actually has a really hard time telling what > >PMSG_SUSPEND really means. In fact the driver can find out in which state to put the device into, depending on the target ACPI state which is known. > > - more importantly, we generally don't want to "suspend/resume" the > >hardware at all around a power-off, because we're going to resume with > >the state at the time of the PMSG_FREEZE, which means that the hardware > >has actually *changed* and been used in between! > > Exactly. > > > So the "->resume" really isn't a resume at all. It's much closer to a > > "->reset". > > Yeah, in the hibernate case this is definitely true. Agreed. > > Of course, the "solution" to this all right now is that we have to reset > > everything even if it *is* a suspend event, so it basically means that STR > > ends up using the much weaker model that snapshot-to-disk uses. > > > > The fundamental problem being that the two really have nothing > > what-so-ever to do with each other. They aren't even similar. Never were. > > > > > And in the long term we could have: > > > ->suspend() > > > *enter S3* > > > ->resume() > > > > Yes, apart from all the complexities (suspend_late/resume_early). So in > > reality it's more than that, but the suspend/resume things are clearly > > nesting, and they have the potential to actually keep state around > > (because we *know* this machine is not going to mess with the devices in > > between). > > Really, in the simple s3 case we still need early/late stuff? Yes, we do. There are devices that need to be suspended with interrupts off. > > IOW, here we actually can have as an option "assume the device is there > > when you return". That is, unless the user pulls out that pendrive while suspended, no? > > > or: > > > ->hibernate() > > > *kexec to another kernel to save image* > > > *power off* > > > ->return_from_hibernate() (or somesuch) > > > > Enough people don't trust kexec that I suspect the right thing simply is > > > > ->freeze() // stop dma, synchronize device state > > *snapshot* > > ->unfreeze(); // resume dma > > *save image* > > [ optionally ->poweroff() ] // do we really care? I'd say no We do, if there are devices that wake us up from S4 and don't wake us up from S5, for example. Plus this f*cking fan in my box that doesn't work after the resume if we don't do ->poweroff() ... > > *power off* > > ->restore() // reset device to the frozen one > > > > which may have four entry-points that can be illogically mapped to the > > suspend/resume ones like we do now, but they really have nothing to do > > with suspending/resuming. Apart from putting devices into the right low power states, that is. > Well, it seems like we'll have to fix drivers in either case, and isn't a > kexec approach fundamentally more sound and simple, design-wise? Rafael > pointed out some problems with properly setting wakeup states, but I think > that could be overcome... Your honor, I would like to register a differing opinion ... > > And notice how while "freeze/restore" kind of pairs like a > > "suspend/resume", it really shouldn't be expected to realistically restore > > the same state at all. The "restore" part is generally much better seen as > > a "reset hardware" than a "resume" thing. That's absolutely correct. > > Because we literally cannot trust *anything* about the state since we froze > > it - we might have booted a different OS in between etc. Very different from > > suspend/resume. > > Yeah, definitely. It has to be much more robust and deal with configuration > changes, etc. (within reason). Agreed. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > So, next I'll try "shutdown" to see if it work. I was using "platform". > > > > Ok, that would be good to try. > > "shutdown" does power down properly. But still green on resume. > > > Looks like the AR registers are hosed, which is what I thought I fixed... > > Can you attach your i915_drv.c file just so I can sanity check it? > > Attached. Jeff, for the hang on suspend problem, I know suspect something else in 2.6.25-rc2 caused that. Can you try the 2.6.25-rc1 version of i915_drv.c (in fact all of drivers/char/drm from 2.6.25-rc1) but in a 2.6.25-rc2 kernel? I ask because 2.6.25-rc1 suspends to disk just fine for me and resumes w/o a green screen, while 2.6.25-rc2 fails to suspend (hangs like you say) and gives me a green screen. Were there other changes in ACPI or the PM core that might have caused this I wonder? Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wed, 20 Feb 2008, Jesse Barnes wrote: > > Really, in the simple s3 case we still need early/late stuff? Absolutely. Two big reasons: - debuggability I know we don't do this correctly right now, but I want to be able to at least feel like we can some day actually do printk's etc through 99% of the suspend/resume cycle. It's a *huge* thing for debugging problems that happen in the wild, and one of the biggest issues is that we currently usualyl just get a "the machine died" message when suspend or resume doesn't work. Yes, doing printk's to the Intel management flash stuff can help a lot here, and I want that too, but I'd really like to shut down consoles individually rather than having the "big hammer" approach that shuts them up entirely over the whole suspend/resume sequence (or not at all, if you use "no_console_suspend"). And I'd *really* like to do things like VGA-console shutdown in the late phase (and resume early). - it's actually likely *much* simpler for some devices. Simple devices (and that includes things like PCI bridges etc, but also potentially USB host controllers etc) are things that can often be trivially suspended - all the complexity is really not in the controller itself, but beyond, in the bus that it actually drives. And the late-suspend/early-resume means that you don't have to worry about things like interrupts happening while you're suspended. Yes, putting the device into D3 will disable interrupts from that device too (unless there are bugs), *BUT* you may be sharing an interrupt line, and interrupts may be posted and delayed, so an earlier interrupt may well be pending etc. suspending late and resuming early just avoids those issues entirely. Sometimes these things interact. For example, firewire is certainly not trivial to suspend as a "subsystem" thing (ie all the devices behind the firewire bridge need to do magic things, like spinning down etc that obviously can not happen in the final "late" phase), but the firewire controller itself is likely trivial to suspend/resume and can easily be handled in the late/early routines. And guess what? It's also exactly what you want to happen in case you end up using the firewire RDMA as a debug aid. IOW, you want that firewire controller (and the PCI bridges) working really early, so that if a problem does happen when you resume some more complex device (say, one of the graphics chips that need X to really come alive), you can use the firewire rdma to read out the kernel log buffer from memory. > Well, it seems like we'll have to fix drivers in either case, and isn't a > kexec approach fundamentally more sound and simple, design-wise? Rafael > pointed out some problems with properly setting wakeup states, but I think > that could be overcome... I don't personally mind kexec at all, but on the other hand, I don't care about suspend-to-disk in the first place. I do know that some people really don't want it, and I suspect that they have valid reasons. Ranging from memory use to simply just performance. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 1:13 pm Linus Torvalds wrote: > On Wed, 20 Feb 2008, Jesse Barnes wrote: > > The current callback system looks like this (according to Rafael and the > > last time I looked): > > ->suspend(PMSG_FREEZE) > > ->resume() > > ->suspend(PMSG_SUSPEND) > > *enter S3 or power off* > > ->resume() > > Yes, it's very messy. > > It's messy for a few different reasons: > > - the one you hit: a driver actually has a really hard time telling what >PMSG_SUSPEND really means. > > - more importantly, we generally don't want to "suspend/resume" the >hardware at all around a power-off, because we're going to resume with >the state at the time of the PMSG_FREEZE, which means that the hardware >has actually *changed* and been used in between! Exactly. > So the "->resume" really isn't a resume at all. It's much closer to a > "->reset". Yeah, in the hibernate case this is definitely true. > Of course, the "solution" to this all right now is that we have to reset > everything even if it *is* a suspend event, so it basically means that STR > ends up using the much weaker model that snapshot-to-disk uses. > > The fundamental problem being that the two really have nothing > what-so-ever to do with each other. They aren't even similar. Never were. > > > And in the long term we could have: > > ->suspend() > > *enter S3* > > ->resume() > > Yes, apart from all the complexities (suspend_late/resume_early). So in > reality it's more than that, but the suspend/resume things are clearly > nesting, and they have the potential to actually keep state around > (because we *know* this machine is not going to mess with the devices in > between). Really, in the simple s3 case we still need early/late stuff? > IOW, here we actually can have as an option "assume the device is there > when you return". > > > or: > > ->hibernate() > > *kexec to another kernel to save image* > > *power off* > > ->return_from_hibernate() (or somesuch) > > Enough people don't trust kexec that I suspect the right thing simply is > > ->freeze() // stop dma, synchronize device state > *snapshot* > ->unfreeze(); // resume dma > *save image* > [ optionally ->poweroff() ] // do we really care? I'd say no > *power off* > ->restore() // reset device to the frozen one > > which may have four entry-points that can be illogically mapped to the > suspend/resume ones like we do now, but they really have nothing to do > with suspending/resuming. Well, it seems like we'll have to fix drivers in either case, and isn't a kexec approach fundamentally more sound and simple, design-wise? Rafael pointed out some problems with properly setting wakeup states, but I think that could be overcome... > And notice how while "freeze/restore" kind of pairs like a > "suspend/resume", it really shouldn't be expected to realistically restore > the same state at all. The "restore" part is generally much better seen as > a "reset hardware" than a "resume" thing. Because we literally cannot > trust *anything* about the state since we froze it - we might have booted > a different OS in between etc. Very different from suspend/resume. Yeah, definitely. It has to be much more robust and deal with configuration changes, etc. (within reason). Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > So, next I'll try "shutdown" to see if it work. I was using "platform". > > > > Ok, that would be good to try. > > "shutdown" does power down properly. But still green on resume. > > > Looks like the AR registers are hosed, which is what I thought I fixed... > > Can you attach your i915_drv.c file just so I can sanity check it? > > Attached. Ok, can you give this patch a try with the 'platform' method? It should at least tell us what ACPI would like the device to do at suspend time, but it probably won't fix the hang. Thanks, Jesse diff --git a/drivers/char/drm/i915_drv.c b/drivers/char/drm/i915_drv.c index 4048f39..d8aa2c9 100644 --- a/drivers/char/drm/i915_drv.c +++ b/drivers/char/drm/i915_drv.c @@ -366,11 +366,11 @@ static int i915_suspend(struct drm_device *dev, pm_message_t state) i915_save_vga(dev); - if (state.event == PM_EVENT_SUSPEND) { - /* Shut down the device */ - pci_disable_device(dev->pdev); - pci_set_power_state(dev->pdev, PCI_D3hot); - } + /* Ask ACPI which state the device should be put in */ + pci_disable_device(dev->pdev); + printk("calling pci_set_power_state with %d\n", + acpi_pci_choose_state(dev, state)); + pci_set_power_state(dev->pdev, acpi_pci_choose_state(dev, state)); return 0; } @@ -380,7 +380,7 @@ static int i915_resume(struct drm_device *dev) struct drm_i915_private *dev_priv = dev->dev_private; int i; - pci_set_power_state(dev->pdev, PCI_D0); + pci_set_power_state(dev->pdev, acpi_pci_choose_state(dev, state)); pci_restore_state(dev->pdev); if (pci_enable_device(dev->pdev)) return -1; - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Rafael J. Wysocki wrote: On Wednesday, 20 of February 2008, Linus Torvalds wrote: On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: I think we should export the target sleep state somehow. Yeah. By *not* using "->suspend()" for freezing or hibernate. Please, Rafael - just make the f*cking suspend-to-disk use other routines already. Okay, I think I'll just start sending patches for that, but rather not earlier than in the 2.6.27 time frame. No one else works on that and I've been busy with other things recently. Besides, I'm not even a full time kernel developer ... Rafael, If I can help, please say so. Regards, Alex. 99% of all hardware needs to do exactly *nothing* on suspend-to-disk, and the ones that really do need things tend to need to not do a whole lot. For example, the "freeze" action for USB (which is one of the hardest things to suspend) should literally be something like just setting the controller STOP bit, and waiting for it to have stopped. The "unfreeze" should be to just clear the stop bit, while the "restart" should be just a controller reset to use the current memory image. NONE OF THIS HAS ABSOLUTELY ANYTHING TO DO WITH SUSPEND. It never did. I've told people so for years. Maybe actually seeing the problems will make people realize. I think so. So please, we shouldn't call "->suspend[_late]" or "->resume[_early]" at all. Not with PMSG_FREEZE, not with PMSG_*anything*. Can we please get this fixed some day? Yes, we can (hopefully). Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wed, 20 Feb 2008, Jesse Barnes wrote: > > The current callback system looks like this (according to Rafael and the last > time I looked): > ->suspend(PMSG_FREEZE) > ->resume() > ->suspend(PMSG_SUSPEND) > *enter S3 or power off* > ->resume() Yes, it's very messy. It's messy for a few different reasons: - the one you hit: a driver actually has a really hard time telling what PMSG_SUSPEND really means. - more importantly, we generally don't want to "suspend/resume" the hardware at all around a power-off, because we're going to resume with the state at the time of the PMSG_FREEZE, which means that the hardware has actually *changed* and been used in between! that second case is very fundamental for things like USB devices, which in theory you can hold alive over a real suspend event (ie a STR event), but which absolutely MUST NOT be resumed over a suspend-to-disk event, because all the low-level request state is bogus! So the "->resume" really isn't a resume at all. It's much closer to a "->reset". Of course, the "solution" to this all right now is that we have to reset everything even if it *is* a suspend event, so it basically means that STR ends up using the much weaker model that snapshot-to-disk uses. The fundamental problem being that the two really have nothing what-so-ever to do with each other. They aren't even similar. Never were. > And in the long term we could have: > ->suspend() > *enter S3* > ->resume() Yes, apart from all the complexities (suspend_late/resume_early). So in reality it's more than that, but the suspend/resume things are clearly nesting, and they have the potential to actually keep state around (because we *know* this machine is not going to mess with the devices in between). IOW, here we actually can have as an option "assume the device is there when you return". > or: > ->hibernate() > *kexec to another kernel to save image* > *power off* > ->return_from_hibernate() (or somesuch) Enough people don't trust kexec that I suspect the right thing simply is ->freeze() // stop dma, synchronize device state *snapshot* ->unfreeze(); // resume dma *save image* [ optionally ->poweroff() ] // do we really care? I'd say no *power off* ->restore() // reset device to the frozen one which may have four entry-points that can be illogically mapped to the suspend/resume ones like we do now, but they really have nothing to do with suspending/resuming. And notice how while "freeze/restore" kind of pairs like a "suspend/resume", it really shouldn't be expected to realistically restore the same state at all. The "restore" part is generally much better seen as a "reset hardware" than a "resume" thing. Because we literally cannot trust *anything* about the state since we froze it - we might have booted a different OS in between etc. Very different from suspend/resume. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday 20 February 2008 at 3:29 pm, Linus Torvalds penned about "Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green." > Can we please get this fixed some day? I can't say I even come close to understand what's going on but getting s2ram to work on my Dell M4300 has been a nightmare. Even after writing up how to get it to work (posted on the suspend-devel list - but no one answered .. yet again), I'm having some quirks. If I had a bizillion $'s, I'd buy an M4300 for Linus and give him a million to get it to s2ram! :p Cheers, -- Pablo Sanchez - Blueoak Database Engineering, Inc Ph:819.459.1926 Toll free: 888.459.1926 Fax: 603.720.7723 (US) Text Page: [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, 20 of February 2008, Linus Torvalds wrote: > > On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: > > > > I think we should export the target sleep state somehow. > > Yeah. By *not* using "->suspend()" for freezing or hibernate. > > Please, Rafael - just make the f*cking suspend-to-disk use other routines > already. Okay, I think I'll just start sending patches for that, but rather not earlier than in the 2.6.27 time frame. No one else works on that and I've been busy with other things recently. Besides, I'm not even a full time kernel developer ... > 99% of all hardware needs to do exactly *nothing* on suspend-to-disk, and the > ones that really do need things tend to need to not do a whole lot. > > For example, the "freeze" action for USB (which is one of the hardest > things to suspend) should literally be something like just setting the > controller STOP bit, and waiting for it to have stopped. The "unfreeze" > should be to just clear the stop bit, while the "restart" should be just a > controller reset to use the current memory image. > > NONE OF THIS HAS ABSOLUTELY ANYTHING TO DO WITH SUSPEND. > > It never did. I've told people so for years. Maybe actually seeing the > problems will make people realize. I think so. > So please, we shouldn't call "->suspend[_late]" or "->resume[_early]" at > all. Not with PMSG_FREEZE, not with PMSG_*anything*. > > Can we please get this fixed some day? Yes, we can (hopefully). Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 12:29 pm Linus Torvalds wrote: > On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: > > I think we should export the target sleep state somehow. > > Yeah. By *not* using "->suspend()" for freezing or hibernate. > > Please, Rafael - just make the f*cking suspend-to-disk use other routines > already. 99% of all hardware needs to do exactly *nothing* on > suspend-to-disk, and the ones that really do need things tend to need to > not do a whole lot. In talking with Rafael on IRC about this, I think we're agreed that we need separate entry points. Even with a kexec based hibernate, we'll probably want ->hibernate callbacks so we don't end up shutting down the device. The current callback system looks like this (according to Rafael and the last time I looked): ->suspend(PMSG_FREEZE) ->resume() ->suspend(PMSG_SUSPEND) *enter S3 or power off* ->resume() The fact that we get suspend/resume called once before suspend again in the hibernate case is somewhat obnoxious, but it's even worse that we don't know what we're about to enter after ->suspend(PMSG_SUSPEND). So in the short term it would be nice to at least get the target state exported. And in the long term we could have: ->suspend() *enter S3* ->resume() or: ->hibernate() *kexec to another kernel to save image* *power off* ->return_from_hibernate() (or somesuch) Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wed, 20 Feb 2008, Rafael J. Wysocki wrote: > > I think we should export the target sleep state somehow. Yeah. By *not* using "->suspend()" for freezing or hibernate. Please, Rafael - just make the f*cking suspend-to-disk use other routines already. 99% of all hardware needs to do exactly *nothing* on suspend-to-disk, and the ones that really do need things tend to need to not do a whole lot. For example, the "freeze" action for USB (which is one of the hardest things to suspend) should literally be something like just setting the controller STOP bit, and waiting for it to have stopped. The "unfreeze" should be to just clear the stop bit, while the "restart" should be just a controller reset to use the current memory image. NONE OF THIS HAS ABSOLUTELY ANYTHING TO DO WITH SUSPEND. It never did. I've told people so for years. Maybe actually seeing the problems will make people realize. So please, we shouldn't call "->suspend[_late]" or "->resume[_early]" at all. Not with PMSG_FREEZE, not with PMSG_*anything*. Can we please get this fixed some day? Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, 20 of February 2008, Jesse Barnes wrote: > On Wednesday, February 20, 2008 11:18 am Jesse Barnes wrote: > > On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > > > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > > > So, next I'll try "shutdown" to see if it work. I was using > > > > > "platform". > > > > > > > > Ok, that would be good to try. > > > > > > "shutdown" does power down properly. But still green on resume. > > > > Ok, so Linus' theory about something later in the resume path trying to > > touch video is looking good. > > > > Rafael, is there anyway to prevent the device shutdown in the hibernate > > path? > > Given the way the PM core works, do we need to set a flag like this? I > really > hope there's a better way of doing this... I think we should export the target sleep state somehow. > diff --git a/drivers/char/drm/i915_drv.c b/drivers/char/drm/i915_drv.c > index 4048f39..a2d6242 100644 > --- a/drivers/char/drm/i915_drv.c > +++ b/drivers/char/drm/i915_drv.c > @@ -238,6 +238,13 @@ static void i915_restore_vga(struct drm_device *dev) > > } > > +/* > + * If we're doing a suspend to disk, we don't want to power off the device. > + * Unfortunately, the PM core doesn't tell us if we're headed for a regular > + * S3 state or that it's about to shut down the machine, so we use this flag. > + */ > +static int i915_hibernate; > + > static int i915_suspend(struct drm_device *dev, pm_message_t state) > { > struct drm_i915_private *dev_priv = dev->dev_private; > @@ -252,6 +259,9 @@ static int i915_suspend(struct drm_device *dev, > pm_message_t state) > if (state.event == PM_EVENT_PRETHAW) > return 0; > > + if (state.event == PM_EVENT_FREEZE) > + i915_hibernate = 1; > + > pci_save_state(dev->pdev); > pci_read_config_byte(dev->pdev, LBB, &dev_priv->saveLBB); > > @@ -366,7 +376,7 @@ static int i915_suspend(struct drm_device *dev, > pm_message_t state) > > i915_save_vga(dev); > > - if (state.event == PM_EVENT_SUSPEND) { > + if (!i915_hibernate) { > /* Shut down the device */ > pci_disable_device(dev->pdev); > pci_set_power_state(dev->pdev, PCI_D3hot); > @@ -385,6 +395,8 @@ static int i915_resume(struct drm_device *dev) > if (pci_enable_device(dev->pdev)) > return -1; > > + i915_hibernate = 0; > + > pci_write_config_byte(dev->pdev, LBB, dev_priv->saveLBB); > > /* Pipe & plane A info */ Then, the .resume() called after the image creation will clear the flag and I don't think it's safe to allow it to survive i915_resume() ... Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 11:18 am Jesse Barnes wrote: > On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > > So, next I'll try "shutdown" to see if it work. I was using > > > > "platform". > > > > > > Ok, that would be good to try. > > > > "shutdown" does power down properly. But still green on resume. > > Ok, so Linus' theory about something later in the resume path trying to > touch video is looking good. > > Rafael, is there anyway to prevent the device shutdown in the hibernate > path? Given the way the PM core works, do we need to set a flag like this? I really hope there's a better way of doing this... Thanks, Jesse diff --git a/drivers/char/drm/i915_drv.c b/drivers/char/drm/i915_drv.c index 4048f39..a2d6242 100644 --- a/drivers/char/drm/i915_drv.c +++ b/drivers/char/drm/i915_drv.c @@ -238,6 +238,13 @@ static void i915_restore_vga(struct drm_device *dev) } +/* + * If we're doing a suspend to disk, we don't want to power off the device. + * Unfortunately, the PM core doesn't tell us if we're headed for a regular + * S3 state or that it's about to shut down the machine, so we use this flag. + */ +static int i915_hibernate; + static int i915_suspend(struct drm_device *dev, pm_message_t state) { struct drm_i915_private *dev_priv = dev->dev_private; @@ -252,6 +259,9 @@ static int i915_suspend(struct drm_device *dev, pm_message_t state) if (state.event == PM_EVENT_PRETHAW) return 0; + if (state.event == PM_EVENT_FREEZE) + i915_hibernate = 1; + pci_save_state(dev->pdev); pci_read_config_byte(dev->pdev, LBB, &dev_priv->saveLBB); @@ -366,7 +376,7 @@ static int i915_suspend(struct drm_device *dev, pm_message_t state) i915_save_vga(dev); - if (state.event == PM_EVENT_SUSPEND) { + if (!i915_hibernate) { /* Shut down the device */ pci_disable_device(dev->pdev); pci_set_power_state(dev->pdev, PCI_D3hot); @@ -385,6 +395,8 @@ static int i915_resume(struct drm_device *dev) if (pci_enable_device(dev->pdev)) return -1; + i915_hibernate = 0; + pci_write_config_byte(dev->pdev, LBB, dev_priv->saveLBB); /* Pipe & plane A info */ - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 11:10 am Jeff Chua wrote: > On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > So, next I'll try "shutdown" to see if it work. I was using "platform". > > > > Ok, that would be good to try. > > "shutdown" does power down properly. But still green on resume. Ok, so Linus' theory about something later in the resume path trying to touch video is looking good. Rafael, is there anyway to prevent the device shutdown in the hibernate path? > > Looks like the AR registers are hosed, which is what I thought I fixed... > > Can you attach your i915_drv.c file just so I can sanity check it? > > Attached. Hm, looks right. Let me see if I can reproduce this on my T61. Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, Feb 21, 2008 at 02:49:39AM +0800, Jeff Chua wrote: > Here's an interesting discovery. After I found that "echo reboot > > /sys/power/disk" does reboot, I tried "echo shutdown > > /sys/power/disk", it does shutdown properly. > > With "platform" it refuses to shutdown. Both reboot and shutdown still > end up with Mr. Green at resume. That kind of suggests that the ACPI platform code is hitting the hardware directly - we've seen similar issues with PATA controllers. The right thing to do here is almost certainly just to avoid explicitly powering down hardware on hibernation. -- Matthew Garrett | [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 2:53 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > So, next I'll try "shutdown" to see if it work. I was using "platform". > Ok, that would be good to try. "shutdown" does power down properly. But still green on resume. > Looks like the AR registers are hosed, which is what I thought I fixed... Can > you attach your i915_drv.c file just so I can sanity check it? Attached. Thanks, Jeff. /* i915_drv.c -- i830,i845,i855,i865,i915 driver -*- linux-c -*- */ /* * * Copyright 2003 Tungsten Graphics, Inc., Cedar Park, Texas. * All Rights Reserved. * * Permission is hereby granted, free of charge, to any person obtaining a * copy of this software and associated documentation files (the * "Software"), to deal in the Software without restriction, including * without limitation the rights to use, copy, modify, merge, publish, * distribute, sub license, and/or sell copies of the Software, and to * permit persons to whom the Software is furnished to do so, subject to * the following conditions: * * The above copyright notice and this permission notice (including the * next paragraph) shall be included in all copies or substantial portions * of the Software. * * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. * IN NO EVENT SHALL TUNGSTEN GRAPHICS AND/OR ITS SUPPLIERS BE LIABLE FOR * ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. * */ #include "drmP.h" #include "drm.h" #include "i915_drm.h" #include "i915_drv.h" #include "drm_pciids.h" static struct pci_device_id pciidlist[] = { i915_PCI_IDS }; enum pipe { PIPE_A = 0, PIPE_B, }; static bool i915_pipe_enabled(struct drm_device *dev, enum pipe pipe) { struct drm_i915_private *dev_priv = dev->dev_private; if (pipe == PIPE_A) return (I915_READ(DPLL_A) & DPLL_VCO_ENABLE); else return (I915_READ(DPLL_B) & DPLL_VCO_ENABLE); } static void i915_save_palette(struct drm_device *dev, enum pipe pipe) { struct drm_i915_private *dev_priv = dev->dev_private; unsigned long reg = (pipe == PIPE_A ? PALETTE_A : PALETTE_B); u32 *array; int i; if (!i915_pipe_enabled(dev, pipe)) return; if (pipe == PIPE_A) array = dev_priv->save_palette_a; else array = dev_priv->save_palette_b; for(i = 0; i < 256; i++) array[i] = I915_READ(reg + (i << 2)); } static void i915_restore_palette(struct drm_device *dev, enum pipe pipe) { struct drm_i915_private *dev_priv = dev->dev_private; unsigned long reg = (pipe == PIPE_A ? PALETTE_A : PALETTE_B); u32 *array; int i; if (!i915_pipe_enabled(dev, pipe)) return; if (pipe == PIPE_A) array = dev_priv->save_palette_a; else array = dev_priv->save_palette_b; for(i = 0; i < 256; i++) I915_WRITE(reg + (i << 2), array[i]); } static u8 i915_read_indexed(u16 index_port, u16 data_port, u8 reg) { outb(reg, index_port); return inb(data_port); } static u8 i915_read_ar(u16 st01, u8 reg, u16 palette_enable) { inb(st01); outb(palette_enable | reg, VGA_AR_INDEX); return inb(VGA_AR_DATA_READ); } static void i915_write_ar(u8 st01, u8 reg, u8 val, u16 palette_enable) { inb(st01); outb(palette_enable | reg, VGA_AR_INDEX); outb(val, VGA_AR_DATA_WRITE); } static void i915_write_indexed(u16 index_port, u16 data_port, u8 reg, u8 val) { outb(reg, index_port); outb(val, data_port); } static void i915_save_vga(struct drm_device *dev) { struct drm_i915_private *dev_priv = dev->dev_private; int i; u16 cr_index, cr_data, st01; /* VGA color palette registers */ dev_priv->saveDACMASK = inb(VGA_DACMASK); /* DACCRX automatically increments during read */ outb(0, VGA_DACRX); /* Read 3 bytes of color data from each index */ for (i = 0; i < 256 * 3; i++) dev_priv->saveDACDATA[i] = inb(VGA_DACDATA); /* MSR bits */ dev_priv->saveMSR = inb(VGA_MSR_READ); if (dev_priv->saveMSR & VGA_MSR_CGA_MODE) { cr_index = VGA_CR_INDEX_CGA; cr_data = VGA_CR_DATA_CGA; st01 = VGA_ST01_CGA; } else { cr_index = VGA_CR_INDEX_MDA; cr_data = VGA_CR_DATA_MDA; st01 = VGA_ST01_MDA; } /* CRT controller regs */ i915_write_indexed(cr_index, cr_data, 0x11, i915_read_indexed(cr_index, cr_data, 0x11)
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 10:29 am Jeff Chua wrote: > > I know I fixed that problem in at least one configuration... Can you > > try: # echo test > /sys/power/disk > > # echo disk > /sys/power/state > > and see if that also turns your screen green? > > Yes, still green. But I got it to actual reboot with ... > > echo reboot > /sys/power/disk > > So, next I'll try "shutdown" to see if it work. I was using "platform". Ok, that would be good to try. > > Also, getting a GPU register dump would be helpful. The intel_reg_dumper > > tool > > Attached are the two dumps from console. One prior to suspend, and one > after resume. Looks like the AR registers are hosed, which is what I thought I fixed... Can you attach your i915_drv.c file just so I can sanity check it? Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 10:37 am Linus Torvalds wrote: > This *sounds* like some part of the suspend-to-disk sequence is doing > something stupid like trying to access the screen after it has been turned > off, which doesn't surprise me at all. My oft-stated opinion has been that > suspend-to-disk isn't a suspend at all, and should never have been > confused with "suspending" anything. > > It's "snapshot-and-restore", and my opinion is that: > > - it should *never* call "suspend()"/"resume()" at all (that should be >reserved purely for suspend-to-RAM and has real power management >issues!) > > - it should have a totally separate "halt/unhalt/restore" thing >that has nothing what-so-ever to do with power management, and is >purely about stopping the hardware for things like USB and network >cards (which otherwise do things like scan their command lists >asynchronously) and making sure that the driver state is consistent >with that stopped hw state. > > - the people who confuse snapshot/restore with suspend/resume are >horrible people that cause problems exactly because driver people then >get those things mixed up, and something like the video suspend/resume >should probably never have impacted suspend-to-disk in the first place! Totally agreed. I remember when I started getting hibernation bug reports against this new code and boggling at how hibernate was actually done. The driver actually gets its ->suspend routine called twice with two different pm_message_t values. We tried to do different stuff depending on the pm_message_t (like only putting the device in D3hot if PM_EVENT_SUSPEND), but it appears we're not doing enough... > So there seems to be two (probably largely independent) problems: > > - the hang at shutdown that requires you to press-and-hold the power >button to actually cut power. > >At a guess: putting the VGA device into D3hot makes the ACPI code that >actually does the shutoff unhappy. Probably because it wants to access >the device, and ends up not ever getting the replies it wants, since >the hardware has been turned off. Sounds like a good theory... now if we could just use set_power_state in the suspend case only. That's what the latest code *tries* to do... JEsse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Jeff Chua wrote: On Feb 20, 2008 2:19 PM, Jeff Chua I'll try the "idle=poll" to see if that works and will try some printk I don't know what exactly the i915_suspend() and i915_resume() are supposed to do because it works better without them. After inserting "return 0;" right at the top of those two functions, suspend (and power-off properly), and resume (without green screen) works just fine. .. Does this machine have more than one CPU core? If so.. Does your kernel have CONFIG_HOTPLUG_CPU=y (if not, enable it). ?? - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, 21 Feb 2008, Jeff Chua wrote: > > > That said, before you do anything else, try if suspend-to-RAM works. > > Linus, guess I missed this part ... so before touch anything, I did > tried suspend-to-ram, and it works on console and in X. Ok, so this is with clean current -git, and nothing disabled? > And suspend-to-disk hangs, but I can still press and hold the power > button to power it off. The "press and hold for five seconds" is actually a hardware feature of the southbridge (well, I guess there is "software" in there too, but it's the embedded kind). So the fact that it powers off at that point means nothing, it just means that ok, your kernel is hung, but the hardware still works ;) This *sounds* like some part of the suspend-to-disk sequence is doing something stupid like trying to access the screen after it has been turned off, which doesn't surprise me at all. My oft-stated opinion has been that suspend-to-disk isn't a suspend at all, and should never have been confused with "suspending" anything. It's "snapshot-and-restore", and my opinion is that: - it should *never* call "suspend()"/"resume()" at all (that should be reserved purely for suspend-to-RAM and has real power management issues!) - it should have a totally separate "halt/unhalt/restore" thing that has nothing what-so-ever to do with power management, and is purely about stopping the hardware for things like USB and network cards (which otherwise do things like scan their command lists asynchronously) and making sure that the driver state is consistent with that stopped hw state. - the people who confuse snapshot/restore with suspend/resume are horrible people that cause problems exactly because driver people then get those things mixed up, and something like the video suspend/resume should probably never have impacted suspend-to-disk in the first place! HOWEVER, that's a separate fight I've had, and in the meantime: > Then upon powering on and resume, I get the ugly green "console" screen. > I can still type and move around. Starting X runs fine. Ctrl-Alt-Del or > switching back to console will get back to the green screen. .. so this implies that while the laptop apparently hung at the end of the snapshotting, the snapshotting did actually work, and it must have hung at the very end, presumably when it tried to actually turn the power off. So there seems to be two (probably largely independent) problems: - the hang at shutdown that requires you to press-and-hold the power button to actually cut power. At a guess: putting the VGA device into D3hot makes the ACPI code that actually does the shutoff unhappy. Probably because it wants to access the device, and ends up not ever getting the replies it wants, since the hardware has been turned off. - the fact that we restore something wrong for you and the screen is green. At a guess: the restore_vga ends up restoring some state that wasn't correctly and fully saved. IOW, I think your patch that disables the two lines actually ends up pretty much matching the two *different* problems. Can you confirm that doing those two parts of that patch individually actually does individually fix the two issues? (Ie disabling D3hot makes it shut down nicely but resume with green text, while disabling just restore_vga() ends up with shutdown problems, but once you press-and-hold the power button, the thing will then restore nicely)+ Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 1:50 AM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > I would like to know what they're for. > They're for saving and restoring GPU state across suspend/resume. They're > particularly useful if your machine doesn't re-POST at resume time. In that > case your GPU may be totally uninitialized, so either the kernel or X has to > set it up for you (X only does that partially). Ok. A lot to digest. > Interesting, which chipset do you have? AFAIK that shouldn't cause a hang. (II) intel(0): Integrated Graphics Chipset: Intel(R) 945GM > I know I fixed that problem in at least one configuration... Can you try: > # echo test > /sys/power/disk > # echo disk > /sys/power/state > and see if that also turns your screen green? Yes, still green. But I got it to actual reboot with ... echo reboot > /sys/power/disk So, next I'll try "shutdown" to see if it work. I was using "platform". > Also, getting a GPU register dump would be helpful. The intel_reg_dumper tool Attached are the two dumps from console. One prior to suspend, and one after resume. Thanks, Jeff. (II): DumpRegsBegin (II):VCLK_DIVISOR_VGA0: 0x00031108 (n = 3, m1 = 17, m2 = 8) (II):VCLK_DIVISOR_VGA1: 0x00031406 (n = 3, m1 = 20, m2 = 6) (II):VCLK_POST_DIV: 0x00020002 (vga0 p1 = 4, p2 = 2, vga1 p1 = 2, p2 = 2) (II):DPLL_TEST: 0x00010001 () (II): CACHE_MODE_0: 0x6820 (II): D_STATE: 0x (II):DSPCLK_GATE_D: 0x1000 (clock gates disabled: DPLUNIT) (II): RENCLK_GATE_D1: 0x (II): RENCLK_GATE_D2: 0x (II):SDVOB: 0x0048 (disabled, pipe A, stall disabled, not detected) (II):SDVOC: 0x0048 (disabled, pipe A, stall disabled, not detected) (II): SDVOUDI: 0x0077 (II): DSPARB: 0x1d9c (II): DSPFW1: 0x (II): DSPFW2: 0x (II): DSPFW3: 0x (II): ADPA: 0x40008c18 (disabled, pipe B, +hsync, +vsync) (II): LVDS: 0xc300 (enabled, pipe B, 18 bit, 1 channel) (II): DVOA: 0x (disabled, pipe A, no stall, -hsync, -vsync) (II): DVOB: 0x0048 (disabled, pipe A, no stall, -hsync, -vsync) (II): DVOC: 0x0048 (disabled, pipe A, no stall, -hsync, -vsync) (II): DVOA_SRCDIM: 0x (II): DVOB_SRCDIM: 0x (II): DVOC_SRCDIM: 0x (II): PP_CONTROL: 0x0001 (power target: on) (II):PP_STATUS: 0xc008 (on, ready, sequencing idle) (II): PFIT_CONTROL: 0x80002668 (II): PFIT_PGM_RATIOS: 0x (II): PORT_HOTPLUG_EN: 0x0020 (II):PORT_HOTPLUG_STAT: 0x (II): DSPACNTR: 0x (disabled, pipe A) (II): DSPASTRIDE: 0x (0 bytes) (II): DSPAPOS: 0x (0, 0) (II): DSPASIZE: 0x (1, 1) (II): DSPABASE: 0x (II): DSPASURF: 0x (II): DSPATILEOFF: 0x (II):PIPEACONF: 0x (disabled, single-wide) (II): PIPEASRC: 0x027f01df (640, 480) (II):PIPEASTAT: 0x8203 (status: FIFO_UNDERRUN VSYNC_INT_STATUS VBLANK_INT_STATUS OREG_UPDATE_STATUS) (II): FBC_CFB_BASE: 0x (II): FBC_LL_BASE: 0x (II): FBC_CONTROL: 0x (II): FBC_COMMAND: 0x (II): FBC_STATUS: 0x2000 (II): FBC_CONTROL2: 0x (II):FBC_FENCE_OFF: 0x (II): FBC_MOD_NUM: 0x (II): FPA0: 0x00031108 (n = 3, m1 = 17, m2 = 8) (II): FPA1: 0x00031108 (n = 3, m1 = 17, m2 = 8) (II): DPLL_A: 0x0483 (disabled, non-dvo, VGA, default clock, DAC/serial mode, p1 = 8, p2 = 10, SDVO mult 1) (II):DPLL_A_MD: 0x (II): HTOTAL_A: 0x031f027f (640 active, 800 total) (II): HBLANK_A: 0x03170287 (648 start, 792 end) (II): HSYNC_A: 0x02ef028f (656 start, 752 end) (II): VTOTAL_A: 0x020c01df (480 active, 525 total) (II): VBLANK_A: 0x020401e7 (488 start, 517 end) (II): VSYNC_A: 0x01eb01e9 (490 start, 492 end) (II):BCLRPAT_A: 0x (II): VSYNCSHIFT_A: 0x (II): DSPBCNTR: 0x4900 (disabled, pipe B) (II): DSPBSTRIDE: 0x0280 (640 bytes) (II): DSPBPOS: 0x (0, 0) (II): DSPBSIZE: 0x018f02cf (720, 400) (II): DSPBBASE: 0x (II): DSPBSURF: 0x (II): DSPBTILEOFF: 0x (II):PIPEBCONF: 0x8000 (enabled, single-wide) (II): PIPEBSRC: 0x027f018f (640, 400) (II):PIPEBSTAT: 0x8202 (status: FIFO_UNDERRUN VSYNC_INT_STATUS VBLANK_INT_STATUS) (II): FPB0: 0x00020e09 (n = 2, m1 = 14, m2 = 9) (II): FPB1: 0x0
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 1:52 AM, Linus Torvalds <[EMAIL PROTECTED]> wrote: > Ahh. You're using the BIOS to re-initialize your video, aren't you? I don't know. Just pure simple "s2ram" without any options. > Let's try to narrow it down to what the interaction is. Are you using > something like acpi_sleep=s3_bios or similar? No. Not additional command line option except for resume=/dev/sda3 reboot=bios > That's what the kernel support is supposed to make unnecessary in the long > run, Ok, understand now. Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 1:28 AM, Linus Torvalds <[EMAIL PROTECTED]> wrote: > That said, before you do anything else, try if suspend-to-RAM works. Linus, guess I missed this part ... so before touch anything, I did tried suspend-to-ram, and it works on console and in X. And suspend-to-disk hangs, but I can still press and hold the power button to power it off. Then upon powering on and resume, I get the ugly green "console" screen. I can still type and move around. Starting X runs fine. Ctrl-Alt-Del or switching back to console will get back to the green screen. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, 21 Feb 2008, Jeff Chua wrote: > > Works without those two functions. Ahh. You're using the BIOS to re-initialize your video, aren't you? If STR works without X, then you have something else resuming graphics, and that may be what then interacts badly with the fact that the kernel also does so. > Ok, what's next? Let's try to narrow it down to what the interaction is. Are you using something like acpi_sleep=s3_bios or similar? That's what the kernel support is supposed to make unnecessary in the long run, along with all the video mode flickering (ie we should be able to resume to the video mode we want, not flicker through unnecessary modes). Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, February 20, 2008 9:17 am Jeff Chua wrote: > On Feb 20, 2008 2:19 PM, Jeff Chua > > > I'll try the "idle=poll" to see if that works and will try some printk > > I don't know what exactly the i915_suspend() and i915_resume() are > supposed to do because it works better without them. > > After inserting "return 0;" right at the top of those two functions, > suspend (and power-off properly), and resume (without green screen) works > just fine. > > I would like to know what they're for. They're for saving and restoring GPU state across suspend/resume. They're particularly useful if your machine doesn't re-POST at resume time. In that case your GPU may be totally uninitialized, so either the kernel or X has to set it up for you (X only does that partially). > Tested suspend-to-ram, and suspend-to-disk, both console and X on notebook > internal LCD display, all works without these two functions. > > But, anyway, got down to just one line in i915_drv.c causing the hang > during suspend. "pci_set_power_state(dev->pdev, PCI_D3hot);". Interesting, which chipset do you have? AFAIK that shouldn't cause a hang. > And green screen problem during resume is caused by i915_restore_vga(dev); I know I fixed that problem in at least one configuration... Can you try: # echo test > /sys/power/disk # echo disk > /sys/power/state and see if that also turns your screen green? Also, getting a GPU register dump would be helpful. The intel_reg_dumper tool is built as part of the xf86-video-driver build (git://anongit.freedesktop.org/git/xorg/driver/xf86-video-intel), can you pull that down and try it out? Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 1:28 AM, Linus Torvalds <[EMAIL PROTECTED]> wrote: > Try suspend-and-resume without X. Works without those two functions. > Also, try it on one of the more modern laptops - even *with* X. Again, still works. Tested on Lenovo X60s. > Basically, the kernel wants to be able to do what X does, because it means > that when it works, it works _so_ much better than doing it in X. So > getting it working is definitely worth it. > That said, before you do anything else, try if suspend-to-RAM works. Yes, still works. > That's the primary goal for this code anyway, and if it works that gives a > good hint. Ok, what's next? Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Thu, 21 Feb 2008, Jeff Chua wrote: > > After inserting "return 0;" right at the top of those two functions, suspend > (and power-off properly), and resume (without green screen) works just fine. > > I would like to know what they're for. Try suspend-and-resume without X. Also, try it on one of the more modern laptops - even *with* X. Basically, the kernel wants to be able to do what X does, because it means that when it works, it works _so_ much better than doing it in X. So getting it working is definitely worth it. That said, before you do anything else, try if suspend-to-RAM works. That's the primary goal for this code anyway, and if it works that gives a good hint. Suspend-to-disk is fundamentally different, and it's entirely possible that for the suspend-to-disk case we should just say "screw trying to suspend/resume graphics", since you'll have the BIOS resuming text-mode anyway, and there are no performance or debugging advantages. Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 21, 2008 1:17 AM, Jeff Chua <[EMAIL PROTECTED]> wrote: > > > On Feb 20, 2008 2:19 PM, Jeff Chua > > I'll try the "idle=poll" to see if that works and will try some printk Tried "idle=poll" but it has not effect. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 20, 2008 2:19 PM, Jeff Chua I'll try the "idle=poll" to see if that works and will try some printk I don't know what exactly the i915_suspend() and i915_resume() are supposed to do because it works better without them. After inserting "return 0;" right at the top of those two functions, suspend (and power-off properly), and resume (without green screen) works just fine. I would like to know what they're for. Tested suspend-to-ram, and suspend-to-disk, both console and X on notebook internal LCD display, all works without these two functions. But, anyway, got down to just one line in i915_drv.c causing the hang during suspend. "pci_set_power_state(dev->pdev, PCI_D3hot);". And green screen problem during resume is caused by i915_restore_vga(dev); So, let me where to go from here. Thanks, Jeff. --- linux/drivers/char/drm/i915_drv.c.bad 2008-02-20 11:29:14 +0800 +++ linux/drivers/char/drm/i915_drv.c 2008-02-21 00:58:37 +0800 @@ -369,7 +369,7 @@ if (state.event == PM_EVENT_SUSPEND) { /* Shut down the device */ pci_disable_device(dev->pdev); - pci_set_power_state(dev->pdev, PCI_D3hot); + //pci_set_power_state(dev->pdev, PCI_D3hot); } return 0; @@ -521,7 +521,7 @@ for (i = 0; i < 3; i++) I915_WRITE(SWF30 + (i << 2), dev_priv->saveSWF2[i]); - i915_restore_vga(dev); + //i915_restore_vga(dev); return 0; } - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 20, 2008 12:32 PM, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > On Tuesday, February 19, 2008 6:28 pm Linus Torvalds wrote: > > On Tue, 19 Feb 2008, Jesse Barnes wrote: > > > I found the same poweroff issue on my T61. It turned out to be related > > > to the C state code disabling interrupts when it shouldn't iirc. Booting > > > with 'idle=poll' seems to work around the problem. > > > > > > The "green screen" problem should be fixed (see the DRM git tree for > > > details). > Jeff, can you retest with Linus' tree? If you're still seeing problems, it > might help to add some printks to the i915 driver's suspend routine. Just > reading the regs really shouldn't cause a hang, but maybe the VGA bits are > subtly wrong again... The funny thing is the screen is now normal during suspend, but the green came back after suspend! And the suspend still does NOT power off with lastest Linus's tree. I'll try the "idle=poll" to see if that works and will try some printk as well. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Tuesday, February 19, 2008 6:28 pm Linus Torvalds wrote: > On Tue, 19 Feb 2008, Jesse Barnes wrote: > > I found the same poweroff issue on my T61. It turned out to be related > > to the C state code disabling interrupts when it shouldn't iirc. Booting > > with 'idle=poll' seems to work around the problem. > > > > The "green screen" problem should be fixed (see the DRM git tree for > > details). > > ..and the latter is hopefully now merged in my tree too (at least some of > the drm updates are). Cool, thanks. Jeff, can you retest with Linus' tree? If you're still seeing problems, it might help to add some printks to the i915 driver's suspend routine. Just reading the regs really shouldn't cause a hang, but maybe the VGA bits are subtly wrong again... Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Tue, 19 Feb 2008, Jesse Barnes wrote: > > I found the same poweroff issue on my T61. It turned out to be related to > the > C state code disabling interrupts when it shouldn't iirc. Booting > with 'idle=poll' seems to work around the problem. > > The "green screen" problem should be fixed (see the DRM git tree for details). ..and the latter is hopefully now merged in my tree too (at least some of the drm updates are). Linus - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Wednesday, 20 of February 2008, Jesse Barnes wrote: > I found the same poweroff issue on my T61. It turned out to be related to > the > C state code disabling interrupts when it shouldn't iirc. Booting > with 'idle=poll' seems to work around the problem. However, this issue is supposed to be fixed in 2.6.25-rc2, so most probably there is another problem in there. Thanks, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
I found the same poweroff issue on my T61. It turned out to be related to the C state code disabling interrupts when it shouldn't iirc. Booting with 'idle=poll' seems to work around the problem. The "green screen" problem should be fixed (see the DRM git tree for details). Jesse On Tuesday, February 19, 2008 4:53 pm Jeff Chua wrote: > On Feb 16, 2008 5:00 AM, Greg KH <[EMAIL PROTECTED]> wrote: > > > Also, I've tried CONFIG_DETECT_SOFTLOCKUP=n, but this doesn't fix it > > > either. > > > > Ok, this looks to be something else. > > > > > Here's the last dmesg after suspend-to-disk and hang there... > > > > > > CPU 1 is now offline > > > SMP alternatives: switching to UP code > > > PM: Syncing filesystems ... done. > > > Freezing user space processes ... (elapsed 0.00 seconds) done. > > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > > > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > > > PM: Freed 0 kbytes in 0.10 seconds (0.00 MB/s) > > > ACPI: Preparing to enter system sleep state S4 > > > Suspending console(s) > > > > > > [ ... it just hangs here ... press power-switch does the job, and > > > system is able to resume upon powering on ] > > > > Wait, this is a suspend-to-disk issue. Totally different than the "will > > not power off" issue. > > > > Can you start a new thread on this, and add the suspend people to it? > > I bisected down this one commit that causes the problem with > suspend-to-disk on Lenovo X60s (i945 chipset). > > commit ba8bbcf6ff4650712f64c0ef61139c73898e2165 > Author: Jesse Barnes <[EMAIL PROTECTED]> > Date: Thu Nov 22 14:14:14 2007 +1000 > > i915: add suspend/resume support > > Add suspend/resume support to the i915 driver. Moves some of the > initialization into the driver load routine, and fixes up places where > we assumed no dev_private existed in some of the cleanup paths. This > allows us to suspend/resume properly even if X isn't running. > > Signed-off-by: Dave Airlie <[EMAIL PROTECTED]> > > > There where problem reverting the some i915 files with the latest > linux git pull, so I copied those i915*.{h,c} prior to this commit, > and problem went away. > > > Suspend-to-ram, suspend-to-disk all working now. > > > Thanks, > Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
On Feb 16, 2008 5:00 AM, Greg KH <[EMAIL PROTECTED]> wrote: > > Also, I've tried CONFIG_DETECT_SOFTLOCKUP=n, but this doesn't fix it either. > > Ok, this looks to be something else. > > > Here's the last dmesg after suspend-to-disk and hang there... > > > > CPU 1 is now offline > > SMP alternatives: switching to UP code > > PM: Syncing filesystems ... done. > > Freezing user space processes ... (elapsed 0.00 seconds) done. > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. > > PM: Shrinking memory... ^H-^Hdone (0 pages freed) > > PM: Freed 0 kbytes in 0.10 seconds (0.00 MB/s) > > ACPI: Preparing to enter system sleep state S4 > > Suspending console(s) > > > > [ ... it just hangs here ... press power-switch does the job, and > > system is able to resume upon powering on ] > > Wait, this is a suspend-to-disk issue. Totally different than the "will > not power off" issue. > > Can you start a new thread on this, and add the suspend people to it? I bisected down this one commit that causes the problem with suspend-to-disk on Lenovo X60s (i945 chipset). commit ba8bbcf6ff4650712f64c0ef61139c73898e2165 Author: Jesse Barnes <[EMAIL PROTECTED]> Date: Thu Nov 22 14:14:14 2007 +1000 i915: add suspend/resume support Add suspend/resume support to the i915 driver. Moves some of the initialization into the driver load routine, and fixes up places where we assumed no dev_private existed in some of the cleanup paths. This allows us to suspend/resume properly even if X isn't running. Signed-off-by: Dave Airlie <[EMAIL PROTECTED]> There where problem reverting the some i915 files with the latest linux git pull, so I copied those i915*.{h,c} prior to this commit, and problem went away. Suspend-to-ram, suspend-to-disk all working now. Thanks, Jeff. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html