Re: [PATCH] Don't quit IO thread before all lvcpus stopped

Avi Kivity
2008-04-24T16:14:02Z

Dor Laor wrote:
> On Thu, 2008-04-24 at 15:47 +0300, Avi Kivity wrote:
>
>> Dor Laor wrote:
>>
>>> while investigating the revert of "fix sci irq set when acpi timer" I
>>> discovered the reason. Please also re-revert the original patch.
>>>
>>>
>> Applied, but system_powerdown still doesn't work with the sci acpi timer
>> fix.
>>
>>
>>
>
> With the above patch and revert the reversion of "fix sci irq set when
> acpi timer" it works for me. Can you give some details or try again?
>
> My host is 2.6.24.3-50-fc8, your kvm head (user
> da00aa5090cf2de4259960ea659ac783bd65a041 + re-revert, kernel
> 1736e3f0a3e173a7f452b32ad19bfff085a1a255).
> Guest is winXp with acpi hal. After it boots I go to the monitor and
> issue a system_powerdown.
> Everything works and there is no segfaults (tested multiple time)
>
>

With current master, Windows and Linux (FC6, x86_64) both shut down
correctly after system_powerdown. With the sci timer thing re-applied,
Windows continues to work but Linux no longer responds to ACPI powerdown.

--
error compiling committee.c: too many arguments to function