Re: system_powerdown not working for qemu-kvm 0.12.4?

[Ruben Kerkhof <ruben@xxxxxxxxxxxxxxxx>]

On Fri, Oct 15, 2010 at 09:01, Gleb Natapov <gleb@xxxxxxxxxx> wrote:
> On Fri, Oct 15, 2010 at 03:45:06AM +0200, Ruben Kerkhof wrote:
>> Is there anything I can do to help resolve this?
>> Would rolling back this commit have any impact on other operating systems?
>>
> Patch below should fix the problem.
>
> diff --git a/src/acpi-dsdt.dsl b/src/acpi-dsdt.dsl
> index bb0a176..dafcf45 100644
> --- a/src/acpi-dsdt.dsl
> +++ b/src/acpi-dsdt.dsl
> @@ -73,7 +73,7 @@ DefinitionBlock (
> Â#define prt_slot3(nr) prt_slot(nr, LNKC, LNKD, LNKA, LNKB)
> Â Â Â Â Â Â Â Âprt_slot0(0x0000),
> Â Â Â Â Â Â Â Â/* Device 1 is power mgmt device, and can only use irq 9 */
> - Â Â Â Â Â Â Â Package() { 0x0001ffff, 0, 0, 9 },
> + Â Â Â Â Â Â Â Package() { 0x0001ffff, 0, LNKS, 0 },
> Â Â Â Â Â Â Â ÂPackage() { 0x0001ffff, 1, LNKB, 0 },
> Â Â Â Â Â Â Â ÂPackage() { 0x0001ffff, 2, LNKC, 0 },
> Â Â Â Â Â Â Â ÂPackage() { 0x0001ffff, 3, LNKD, 0 },
> @@ -634,6 +634,46 @@ DefinitionBlock (
> Â Â Â Â Â Â Â Â Â Â Store (TMP, PRQ3)
> Â Â Â Â Â Â Â Â }
> Â Â Â Â }
> + Â Â Â ÂDevice(LNKS){
> + Â Â Â Â Â Â Â ÂName(_HID, EISAID("PNP0C0F")) Â Â // PCI interrupt link
> + Â Â Â Â Â Â Â ÂName(_UID, 5)
> + Â Â Â Â Â Â Â ÂName(_PRS, ResourceTemplate(){
> + Â Â Â Â Â Â Â Â Â ÂInterrupt (, Level, ActiveHigh, Shared)
> + Â Â Â Â Â Â Â Â Â Â Â Â{ 9 }
> + Â Â Â Â Â Â Â Â})
> + Â Â Â Â Â Â Â ÂMethod (_STA, 0, NotSerialized)
> + Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â ÂStore (0x0B, Local0)
> + Â Â Â Â Â Â Â Â Â ÂIf (And (0x80, PRQ0, Local1))
> + Â Â Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â Â Â Â Store (0x09, Local0)
> + Â Â Â Â Â Â Â Â Â Â}
> + Â Â Â Â Â Â Â Â Â ÂReturn (Local0)
> + Â Â Â Â Â Â Â Â}
> + Â Â Â Â Â Â Â ÂMethod (_DIS, 0, NotSerialized)
> + Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â ÂOr (PRQ0, 0x80, PRQ0)
> + Â Â Â Â Â Â Â Â}
> + Â Â Â Â Â Â Â ÂMethod (_CRS, 0, NotSerialized)
> + Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â ÂName (PRR0, ResourceTemplate ()
> + Â Â Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â Â Â ÂInterrupt (, Level, ActiveHigh, Shared)
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â{9}
> + Â Â Â Â Â Â Â Â Â Â})
> + Â Â Â Â Â Â Â Â Â ÂCreateDWordField (PRR0, 0x05, TMP)
> + Â Â Â Â Â Â Â Â Â ÂStore (PRQ0, Local0)
> + Â Â Â Â Â Â Â Â Â ÂIf (LLess (Local0, 0x80))
> + Â Â Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â Â Â ÂStore (Local0, TMP)
> + Â Â Â Â Â Â Â Â Â Â}
> + Â Â Â Â Â Â Â Â Â ÂElse
> + Â Â Â Â Â Â Â Â Â Â{
> + Â Â Â Â Â Â Â Â Â Â Â ÂStore (Zero, TMP)
> + Â Â Â Â Â Â Â Â Â Â}
> + Â Â Â Â Â Â Â Â Â ÂReturn (PRR0)
> + Â Â Â Â Â Â Â Â}
> + Â Â Â Â}
> Â Â }
>
> Â Â /*
> --
> Â Â Â Â Â Â Â Â Â Â Â ÂGleb.
>

It does indeed.

Thanks a lot!

Ruben
ÿô.nlj·Ÿ®‰­†+%ŠË±é¥Šwÿº{.nlj·¤¾oâ^n‡r¡öë¨è&£ûz¹Þúzf£¢·hšˆ§~†­†Ûÿÿïÿ‘ê_èæ+v‰¨þ)ßø