lkml.org 
[lkml]   [2000]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] partial fix for Thinkpads and suspend
Date
From
Hi Garst,

"Garst R. Reese" <reese@isn.net> writes:
>
> Stephen,
> I sent the attached patch to apm linuxcare that you seem to think has
> already been applied to the kernel. I don't think so. I never sent it to
> Linus or the list, and have not seen it in any test6prex.
> The problem I addressed was that there was a cli() instead of an
> APM_DO_CLI
> in int suspend(void);
>
> I've been running with the patch I submitted to fix this problem for
> some time.
> Your solution to do an sti() in APM_DO_CLI seems quite ugly. I grant
> that we do not want to do a cli before suspending, but I could not find
> anybody else doing that, so just not doing is probably sufficient.

I did NOT assume that your patch had been applied (I am the person behind
apm@linuxcare.com.au). What my patch does is to ensure that interrupts
are explicitly enabled whenever we do a BIOS call (including the BIOS call
to suspend the machine) rather than just assuming that interrupts were
enabled. This does work in my Thinkpad (admittedly a 600E).

I do not think that my patch is ugly and, in fact, implements the
semantics of the CONFIG option even better than before. :-)

Please try my patch ...

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

-
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/

\
 
 \ /
  Last update: 2005-03-22 13:58    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site