Re: [PATCH 0/2] Freezer fixes for 2.6.24

[Franck Bui-Huu <vagabon.xyz@gmail.com>]

Rafael J. Wysocki wrote:
> On Wednesday, 21 of November 2007, Franck Bui-Huu wrote:
>> Rafael J. Wysocki wrote:
>>> The second one is intended to fix the APM emulation breakage caused before
>>> 2.6.23, but unfortunately I can't test it.
>> Why not ?
>>
>> Just hack Kconfig to be able to select this driver on x86 architecture
>> and it should work fine, no ?
> 
> Well, I'm afraid I have no suitable userland here ...
> 

I don't have either.

Anyway I just verified that the current code is broken. I put 2
processes waiting for a suspend evetn by reading the /dev/apm_bios
device and a third process that does the suspend event by doing an
ioctl() on the apm device. The system doesn't suspend (the kernel
dumped a list of the current tasks, instead of printing "Freezing of
tasks failed after..." BTW) and eventually the third process exits
with an incorrect value: -EINTR.

Applying your patch makes the previous test case work.

I also discovered that s2ram doesn't work on my laptop and had to use
standby suspend. I'll try to find out why.

		Franck