Re: [3/5] 2.6.21-rc4: known regressions (v2)

Thomas Meyer
2007-03-25T14:16:32Z

Rafael J. Wysocki schrieb:
> On Sunday, 25 March 2007 14:03, Eric W. Biederman wrote:
>
>> Thomas Meyer <thomas@m3y3r.de> writes:
>>
>>
>>> Eric W. Biederman schrieb:
>>>
>>>> Thomas could you verify the patch below makes the problem go away
>>>> for you.
>>>>
>>>>
>>> The patch solves the problem. I'm writing this after the third suspend
>>> and resume cycle.
>>> msi irq stays enabled for libata device:
>>> cat /sys/devices/pci0000\:00/0000\:00\:1f.2/irq
>>> 218
>>>
>>> The first suspend to disk is ok. The second suspend to disk has a
>>> strange behaviour:
>>> 1.) write pm image
>>> 2.) the system disable the non-boot cpus again (i guess this happens in
>>> power_down())
>>>
>
> Yes, in kernel/power/disk.c:power_down() .
>
> Please comment out the disable_nonboot_cpus() in there and retest (but please
> test the latest Linus' tree).
>
>
Without disable_nonboot_cpus in power_down the computer powers down
without the mysterious "wait for the next interrupt" hang.
>>> 3.) the system doesn't power down.
>>> 4.) pressing any key and the system powers down.
>>>
>>> The same is true for the third suspend cycle. Maybe an acpi problem?
>>>
>> Sounds possible. You could probably verify it isn't my patch but running
>> an unpatched kernel without msi support. As I think the crash you saw should
>> only be reproducible when using devices that support msi.
>>
>> Unless I hear different I'm going to assume that this second case is a
>> completely different problem.
>>
>
> I think it is different too.
>
Yes, it's a different problem