[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] ACPI 5.1: 7.3.2 System \_Sx states
From: |
Doran, Mark |
Subject: |
Re: [Qemu-devel] ACPI 5.1: 7.3.2 System \_Sx states |
Date: |
Sun, 8 Mar 2015 19:29:37 +0000 |
Thanks for the report. You might want to bring this to the attention of
Dean Nelson and/or Al Stone -- as regular attendees of the ACPI Work Group
they can file an ECR or show you how to do that if that's appropriate.
--
Cheers,
Mark.
> -----Original Message-----
> From: Michael S. Tsirkin [mailto:address@hidden
> Sent: Sunday, March 8, 2015 4:42 AM
> To: Doran, Mark
> Cc: address@hidden; address@hidden
> Subject: ACPI 5.1: 7.3.2 System \_Sx states
>
> Hello!
> I have noticed the following inconsistency in ACPI spec 5.1.
> 7.3.2 System \_Sx states
>
> States:
>
> Return Value:
> A Package containing an Integer containing register values for sleeping
>
> It goes on to specify Table 7-221 System State Package which describes a
> byte array in terms of byte length/byte offsets.
>
> In practice, I see that implementations define \_Sx as a package
> containing 4 integers.
>
> Here's an example package generated by IASL:
>
> 52: Name(_S4, Package(0x04) {
> 53: 0x2,
> 54: 0x2,
> 55: Zero,
> 56: Zero
> 57: })
>
> 00000091: 08 5F 53 34 5F ......... "._S4_"
> 00000096: 12 08 04 0A 02 0A 02 00 "........"
> 0000009E: 00 ..................... "."
>
> In other words, there are 4 integer values, and these are not at offsets
> specified by the spec.
>
> Suggested corrected wording:
>
> Return Value:
> A Package containing a list of 4 Integers containing register values for
> sleeping
>
> and update Table 7-221 System State Package
>
> as follows:
>
>
> Type Description
>
> Integer Value for PM1a_CNT.SLP_TYP register to enter this system
> state. On
> HW-reduced
> platforms, this is the HW-reduced Sleep Type value for
> SLEEP_CONTROL_REG.SLP_TYP.
>
> Integer Value for PM1b_CNT.SLP_TYP register to enter this
system
> state. To enter any
> given state, OSPM must write the PM1a_CNT.SLP_TYP register
> before the
> PM1b_CNT.SLP_TYP register. On HW-reduced platforms, this
value
> is
> ignored.
>
> Integer Reserved. This value is ignored by the OSPM.
> Integer Reserved. This value is ignored by the OSPM.
>
> --
> MST
smime.p7s
Description: S/MIME cryptographic signature