[Qemu-devel] Re: [Bochs-developers] [PATCH v2 2/6] Add S3 state to DSDT. Handleresume event in the BIOS.

[Gleb Natapov]

On Fri, Nov 07, 2008 at 12:03:02AM +0100, Sebastian Herbszt wrote:
>> +    Name (\_S3, Package (0x04)
>> +    {
>> +        0x01,  /* PM1a_CNT.SLP_TYP */
>> +        0x01,  /* PM1b_CNT.SLP_TYP */
>> +        Zero,  /* reserved */
>> +        Zero   /* reserved */
>> +    })
>> +    Name (\_S4, Package (0x04)
>> +    {
>> +        Zero,  /* PM1a_CNT.SLP_TYP */
>> +        Zero,  /* PM1b_CNT.SLP_TYP */
>> +        Zero,  /* reserved */
>> +        Zero   /* reserved */
>> +    })
>
> Do we need \_S4 for S3 support?
>
Strictly saying no we don't, but S4 is supported by the chipset
we emulate (it is the same as power-off), so why don't advertise
it explicitly?

Other comments are addressed in v3 of the patch series.

--
                        Gleb.