[Adeos-main] [Resent][PATCH] allow hibernation of I-pipe kernels

[Adeos-main] [Resent][PATCH] allow hibernation of I-pipe kernels

[Jan Kiszka]

[-- Attachment #1.1: Type: text/plain, Size: 106 bytes --]

I'm using this patch since ages on my notebook. Would be nice to have
this available by default.

Jan

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: hard-irq-disable-on-suspend-resume.patch --]
[-- Type: text/x-patch; name="hard-irq-disable-on-suspend-resume.patch", Size: 1230 bytes --]

---
 kernel/power/swsusp.c |    4 ++++
 1 file changed, 4 insertions(+)

Index: linux-2.6.22-ipipe/kernel/power/swsusp.c
===================================================================
--- linux-2.6.22-ipipe.orig/kernel/power/swsusp.c
+++ linux-2.6.22-ipipe/kernel/power/swsusp.c
@@ -278,6 +278,7 @@ int swsusp_suspend(void)
 		return error;
 
 	local_irq_disable();
+	local_irq_disable_hw_cond();
 	/* At this point, device_suspend() has been called, but *not*
 	 * device_power_down(). We *must* device_power_down() now.
 	 * Otherwise, drivers for some devices (e.g. interrupt controllers)
@@ -299,6 +300,7 @@ int swsusp_suspend(void)
 	 */
 	device_power_up();
  Enable_irqs:
+	local_irq_enable_hw_cond();
 	local_irq_enable();
 	return error;
 }
@@ -308,6 +310,7 @@ int swsusp_resume(void)
 	int error;
 
 	local_irq_disable();
+	local_irq_disable_hw_cond();
 	/* NOTE:  device_power_down() is just a suspend() with irqs off;
 	 * it has no special "power things down" semantics
 	 */
@@ -334,6 +337,7 @@ int swsusp_resume(void)
 	restore_processor_state();
 	touch_softlockup_watchdog();
 	device_power_up();
+	local_irq_enable_hw_cond();
 	local_irq_enable();
 	return error;
 }

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

Re: [Adeos-main] [Resent][PATCH] allow hibernation of I-pipe kernels

[Philippe Gerum]

On Sat, 2007-09-15 at 09:55 +0200, Jan Kiszka wrote:
> I'm using this patch since ages on my notebook. Would be nice to have
> this available by default.
> 

Merged, thanks.

> Jan
-- 
Philippe.