[PATCH] partial fix for Thinkpads and suspend

From: Stephen Rothwell (sfr@linuxcare.com)
Date: Tue Aug 08 2000 - 21:32:02 EST


Hi Linus,

One of the recent changes to the power management code was to disable
interrupts from just before suspending until after the drivers have
been notified of the resume. This broke the Thinkpads. They do not
completely resume anymore. It turns out that I have just been presented
with a Thinkpad (for work), so I need to fix this :-)

Please apply the enclosed patch. It is not quite completely correct,
but at least the Thinkpads (which are the only laptops that require
the CONFIG_APM_ALLOW_INTS option) will resume. The window we open
is small, but it may be that we will still get interrupts for devices
that need to be reinitialised first.

A more complete solution is for each device to disable its interrupt
before the suspend (I think).

Cheers,
Stephen

-- 
Stephen Rothwell, Open Source Project Engineer, Linuxcare, Inc.
+61-2-62628990 tel, +61-2-62628991 fax 
sfr@linuxcare.com, http://www.linuxcare.com/ 
Linuxcare. Support for the revolution.

--- 2.4.0-test6pre9/arch/i386/kernel/apm.c Wed Aug 2 15:20:17 2000 +++ fixed/arch/i386/kernel/apm.c Wed Aug 9 12:00:47 2000 @@ -410,7 +410,7 @@ #ifndef CONFIG_APM_ALLOW_INTS # define APM_DO_CLI __cli() #else -# define APM_DO_CLI +# define APM_DO_CLI __sti() #endif #ifdef APM_ZERO_SEGS # define APM_DECL_SEGS \

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Aug 15 2000 - 21:00:17 EST