[RFC V3 PATCH 10/13] tests/qtest/bios-table-tests: Update ACPI table binaries for microvm

Annie Li posted 13 patches 8 months, 1 week ago
[RFC V3 PATCH 10/13] tests/qtest/bios-table-tests: Update ACPI table binaries for microvm
Posted by Annie Li 8 months, 1 week ago
Following is the diff generated by the step 5 and 6 in
tests/qtest/bios-tables-test.c

Diff from iasl to show changes in DSDT table:

--- /tmp/asl-2F0N42.dsl 2025-04-11 12:50:02.892883467 +0000
+++ /tmp/asl-C81N42.dsl 2025-04-11 12:50:02.890883378 +0000
@@ -1,30 +1,30 @@
 /*
  * Intel ACPI Component Architecture
  * AML/ASL+ Disassembler version 20210604 (64-bit version)
  * Copyright (c) 2000 - 2021 Intel Corporation
  *
  * Disassembling to symbolic ASL+ operators
  *
- * Disassembly of tests/data/acpi/x86/microvm/DSDT, Fri Apr 11 12:50:02 2025
+ * Disassembly of /tmp/aml-481N42, Fri Apr 11 12:50:02 2025
  *
  * Original Table Header:
  *     Signature        "DSDT"
- *     Length           0x0000016D (365)
+ *     Length           0x000001BA (442)
  *     Revision         0x02
- *     Checksum         0x11
+ *     Checksum         0xD5
  *     OEM ID           "BOCHS "
  *     OEM Table ID     "BXPC    "
  *     OEM Revision     0x00000001 (1)
  *     Compiler ID      "BXPC"
  *     Compiler Version 0x00000001 (1)
  */
 DefinitionBlock ("", "DSDT", 2, "BOCHS ", "BXPC    ", 0x00000001)
 {
     Scope (_SB)
     {
         Device (FWCF)
         {
             Name (_HID, "QEMU0002")  // _HID: Hardware ID
             Name (_STA, 0x0B)  // _STA: Status
             Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
             {
@@ -66,56 +66,82 @@
                     0x00000009,
                 }
             })
             OperationRegion (EREG, SystemMemory, 0xFEA00000, 0x04)
             Field (EREG, DWordAcc, NoLock, WriteAsZeros)
             {
                 ESEL,   32
             }

             Method (_EVT, 1, Serialized)  // _EVT: Event
             {
                 Local0 = ESEL /* \_SB_.GED_.ESEL */
                 If (((Local0 & 0x02) == 0x02))
                 {
                     Notify (PWRB, 0x80) // Status Change
                 }
+
+                If (((Local0 & 0x10) == 0x10))
+                {
+                    Notify (SLPB, 0x80) // Status Change
+                }
             }
         }

         Device (PWRB)
         {
             Name (_HID, "PNP0C0C" /* Power Button Device */)  // _HID: Hardware ID
             Name (_UID, Zero)  // _UID: Unique ID
         }

+        Device (SLPB)
+        {
+            Name (_HID, EisaId ("PNP0C0E") /* Sleep Button Device */)  // _HID: Hardware ID
+            OperationRegion (\SLP, SystemIO, 0x0201, One)
+            Field (\SLP, ByteAcc, NoLock, WriteAsZeros)
+            {
+                SBP,    1
+            }
+        }
+
         Device (VR07)
         {
             Name (_HID, "LNRO0005")  // _HID: Hardware ID
             Name (_UID, 0x07)  // _UID: Unique ID
             Name (_CCA, One)  // _CCA: Cache Coherency Attribute
             Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
             {
                 Memory32Fixed (ReadWrite,
                     0xFEB00E00,         // Address Base
                     0x00000200,         // Address Length
                     )
                 Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive, ,, )
                 {
                     0x00000017,
                 }
             })
         }
     }

     Scope (\)
     {
+        Name (_S3, Package (0x04)  // _S3_: S3 System State
+        {
+            0x03,
+            Zero,
+            Zero,
+            Zero
+        })
+    }
+
+    Scope (\)
+    {
         Name (_S5, Package (0x04)  // _S5_: S5 System State
         {
             0x05,
             Zero,
             Zero,
             Zero
         })
     }
 }

Signed-off-by: Annie Li <annie.li@oracle.com>
---
 tests/data/acpi/x86/microvm/DSDT            | Bin 365 -> 442 bytes
 tests/data/acpi/x86/microvm/DSDT.ioapic2    | Bin 365 -> 442 bytes
 tests/data/acpi/x86/microvm/DSDT.pcie       | Bin 3023 -> 3100 bytes
 tests/data/acpi/x86/microvm/DSDT.rtc        | Bin 404 -> 481 bytes
 tests/data/acpi/x86/microvm/DSDT.usb        | Bin 414 -> 491 bytes
 tests/qtest/bios-tables-test-allowed-diff.h |   5 -----
 6 files changed, 5 deletions(-)

diff --git a/tests/data/acpi/x86/microvm/DSDT b/tests/data/acpi/x86/microvm/DSDT
index f477668f2ee80241e47b340ad4a30f5480df2049..ee199a4fb65ab1a2d811f37d9b5bc67bccd34939 100644
GIT binary patch
delta 127
zcmaFMw2PU`CD<io7b61$)76Pw(oD|c6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmHREo~ex#LmLS
V%)r1PAP~dA5g%*{7GMPm005{09v1)r

delta 54
zcmdnR{FaH!CD<h-myv;iNpK>UG?TmVMD;8tAGV2;eHmpZK9rfv$0$2FjM0<JG+v0E
Kg^QJefdK$+3k%2q

diff --git a/tests/data/acpi/x86/microvm/DSDT.ioapic2 b/tests/data/acpi/x86/microvm/DSDT.ioapic2
index e5924a49962e0cff4228bcfc77ebcf48cb4a2219..7512d1527cf222d6f75a92f051f9eda402e28376 100644
GIT binary patch
delta 127
zcmaFMw2PU`CD<io7b61$Q~yLRX(ng!iRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFOqO8umNt$TVrSuE
UW?*0t5Qt&mhz~Xe3$OwO0ErwO(EtDd

delta 54
zcmdnR{FaH!CD<h-myv;i>GVV{X(o5!iRxKQK5P>w`!dQ-d?+)Sk5P7V7^5ebX}l0S
K3l}Q`0|Nky8w`Q~

diff --git a/tests/data/acpi/x86/microvm/DSDT.pcie b/tests/data/acpi/x86/microvm/DSDT.pcie
index 8eacd21d6ecdf9a3cd3e4f03cf1b40748dcbf53e..5e3f80cbf82cab52fe310160c88d0a11ff069bf5 100644
GIT binary patch
delta 128
zcmX>vK1YJfCD<iIhKGTHiDM#{G?VMziRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFY?feL%O!0bFT~Em
W#mvCKARrLKz!4v83Kn1m3IG6dDjnGX

delta 55
zcmbOuabBFuCD<k8JU0Ualj}q-X(rE|6V<breAp&V_GOfv_)umtAEWH%FvitfT&D3t
L>?~ZY3=9kauQ3gZ

diff --git a/tests/data/acpi/x86/microvm/DSDT.rtc b/tests/data/acpi/x86/microvm/DSDT.rtc
index e375473b5f033d374dd2a64a9f814fd16c3e262f..11258908703b64a835bd14307490c62c5ef7301f 100644
GIT binary patch
delta 129
zcmbQj{E(T;CD<k8AtM6=)8dI-(o9~G6V)d&`La*k<;$oynS)V=Z-Kz%>I5zU1}=fN
zV4r}=ii`@Hnm~pVN4$rp3y<Rk9=_;?7@$BrBR3-xV{{`AnBy4i1QeNU&*&{}952Mq
X!o|$Mz#t$H!@v<AYzh`&1quKFJNg{0

delta 79
zcmaFJJcXIdCD<io3L^sp)BTBD(oBA$6V)d&`LIpg<;y5LnS)VAI=V?RAUw#4Bi_T)
jg*U)2z`)tSnSmod)YFAway6qjmub8ZI|~;p0|Ns9En5;N

diff --git a/tests/data/acpi/x86/microvm/DSDT.usb b/tests/data/acpi/x86/microvm/DSDT.usb
index d63fd84620a2689120b32262f964e6a098d71632..7250d9f8ad45fca91afeda1089090eee11cd6594 100644
GIT binary patch
delta 127
zcmbQo{F<4|CD<k8H6sH9)6<Du(o8N=6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmFrBW)Zn#LmLS
V%)r1PAP~dA5g%*{7GMPm006!m9&i8v

delta 54
zcmaFOJdc^nCD<io9wP$-lj1}!X(kV`iRxKQK5P>w`!dQ-d?+)Sk5P7V7-JQeX}l0S
K3l}Q`0|NkddJG`|

diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index bb028db137..dfb8523c8b 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,6 +1 @@
 /* List of comma-separated changed AML files to ignore */
-"tests/data/acpi/x86/microvm/DSDT",
-"tests/data/acpi/x86/microvm/DSDT.ioapic2",
-"tests/data/acpi/x86/microvm/DSDT.pcie",
-"tests/data/acpi/x86/microvm/DSDT.rtc",
-"tests/data/acpi/x86/microvm/DSDT.usb",
-- 
2.43.5
Re: [RFC V3 PATCH 10/13] tests/qtest/bios-table-tests: Update ACPI table binaries for microvm
Posted by Gustavo Romero 8 months ago
Hi Annie,

On 4/11/25 17:44, Annie Li wrote:
> Following is the diff generated by the step 5 and 6 in
> tests/qtest/bios-tables-test.c
> 
> Diff from iasl to show changes in DSDT table:
> 
> --- /tmp/asl-2F0N42.dsl 2025-04-11 12:50:02.892883467 +0000
> +++ /tmp/asl-C81N42.dsl 2025-04-11 12:50:02.890883378 +0000
> @@ -1,30 +1,30 @@
>   /*
>    * Intel ACPI Component Architecture
>    * AML/ASL+ Disassembler version 20210604 (64-bit version)
>    * Copyright (c) 2000 - 2021 Intel Corporation
>    *
>    * Disassembling to symbolic ASL+ operators
>    *
> - * Disassembly of tests/data/acpi/x86/microvm/DSDT, Fri Apr 11 12:50:02 2025
> + * Disassembly of /tmp/aml-481N42, Fri Apr 11 12:50:02 2025

Same comment from 4/13 regarding diff confusing git am and b4 and about the filename.

Otherwise:

Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>


Cheers,
Gustavo

>    *
>    * Original Table Header:
>    *     Signature        "DSDT"
> - *     Length           0x0000016D (365)
> + *     Length           0x000001BA (442)
>    *     Revision         0x02
> - *     Checksum         0x11
> + *     Checksum         0xD5
>    *     OEM ID           "BOCHS "
>    *     OEM Table ID     "BXPC    "
>    *     OEM Revision     0x00000001 (1)
>    *     Compiler ID      "BXPC"
>    *     Compiler Version 0x00000001 (1)
>    */
>   DefinitionBlock ("", "DSDT", 2, "BOCHS ", "BXPC    ", 0x00000001)
>   {
>       Scope (_SB)
>       {
>           Device (FWCF)
>           {
>               Name (_HID, "QEMU0002")  // _HID: Hardware ID
>               Name (_STA, 0x0B)  // _STA: Status
>               Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
>               {
> @@ -66,56 +66,82 @@
>                       0x00000009,
>                   }
>               })
>               OperationRegion (EREG, SystemMemory, 0xFEA00000, 0x04)
>               Field (EREG, DWordAcc, NoLock, WriteAsZeros)
>               {
>                   ESEL,   32
>               }
> 
>               Method (_EVT, 1, Serialized)  // _EVT: Event
>               {
>                   Local0 = ESEL /* \_SB_.GED_.ESEL */
>                   If (((Local0 & 0x02) == 0x02))
>                   {
>                       Notify (PWRB, 0x80) // Status Change
>                   }
> +
> +                If (((Local0 & 0x10) == 0x10))
> +                {
> +                    Notify (SLPB, 0x80) // Status Change
> +                }
>               }
>           }
> 
>           Device (PWRB)
>           {
>               Name (_HID, "PNP0C0C" /* Power Button Device */)  // _HID: Hardware ID
>               Name (_UID, Zero)  // _UID: Unique ID
>           }
> 
> +        Device (SLPB)
> +        {
> +            Name (_HID, EisaId ("PNP0C0E") /* Sleep Button Device */)  // _HID: Hardware ID
> +            OperationRegion (\SLP, SystemIO, 0x0201, One)
> +            Field (\SLP, ByteAcc, NoLock, WriteAsZeros)
> +            {
> +                SBP,    1
> +            }
> +        }
> +
>           Device (VR07)
>           {
>               Name (_HID, "LNRO0005")  // _HID: Hardware ID
>               Name (_UID, 0x07)  // _UID: Unique ID
>               Name (_CCA, One)  // _CCA: Cache Coherency Attribute
>               Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
>               {
>                   Memory32Fixed (ReadWrite,
>                       0xFEB00E00,         // Address Base
>                       0x00000200,         // Address Length
>                       )
>                   Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive, ,, )
>                   {
>                       0x00000017,
>                   }
>               })
>           }
>       }
> 
>       Scope (\)
>       {
> +        Name (_S3, Package (0x04)  // _S3_: S3 System State
> +        {
> +            0x03,
> +            Zero,
> +            Zero,
> +            Zero
> +        })
> +    }
> +
> +    Scope (\)
> +    {
>           Name (_S5, Package (0x04)  // _S5_: S5 System State
>           {
>               0x05,
>               Zero,
>               Zero,
>               Zero
>           })
>       }
>   }
> 
> Signed-off-by: Annie Li <annie.li@oracle.com>
> ---
>   tests/data/acpi/x86/microvm/DSDT            | Bin 365 -> 442 bytes
>   tests/data/acpi/x86/microvm/DSDT.ioapic2    | Bin 365 -> 442 bytes
>   tests/data/acpi/x86/microvm/DSDT.pcie       | Bin 3023 -> 3100 bytes
>   tests/data/acpi/x86/microvm/DSDT.rtc        | Bin 404 -> 481 bytes
>   tests/data/acpi/x86/microvm/DSDT.usb        | Bin 414 -> 491 bytes
>   tests/qtest/bios-tables-test-allowed-diff.h |   5 -----
>   6 files changed, 5 deletions(-)
> 
> diff --git a/tests/data/acpi/x86/microvm/DSDT b/tests/data/acpi/x86/microvm/DSDT
> index f477668f2ee80241e47b340ad4a30f5480df2049..ee199a4fb65ab1a2d811f37d9b5bc67bccd34939 100644
> GIT binary patch
> delta 127
> zcmaFMw2PU`CD<io7b61$)76Pw(oD|c6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
> zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmHREo~ex#LmLS
> V%)r1PAP~dA5g%*{7GMPm005{09v1)r
> 
> delta 54
> zcmdnR{FaH!CD<h-myv;iNpK>UG?TmVMD;8tAGV2;eHmpZK9rfv$0$2FjM0<JG+v0E
> Kg^QJefdK$+3k%2q
> 
> diff --git a/tests/data/acpi/x86/microvm/DSDT.ioapic2 b/tests/data/acpi/x86/microvm/DSDT.ioapic2
> index e5924a49962e0cff4228bcfc77ebcf48cb4a2219..7512d1527cf222d6f75a92f051f9eda402e28376 100644
> GIT binary patch
> delta 127
> zcmaFMw2PU`CD<io7b61$Q~yLRX(ng!iRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
> zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFOqO8umNt$TVrSuE
> UW?*0t5Qt&mhz~Xe3$OwO0ErwO(EtDd
> 
> delta 54
> zcmdnR{FaH!CD<h-myv;i>GVV{X(o5!iRxKQK5P>w`!dQ-d?+)Sk5P7V7^5ebX}l0S
> K3l}Q`0|Nky8w`Q~
> 
> diff --git a/tests/data/acpi/x86/microvm/DSDT.pcie b/tests/data/acpi/x86/microvm/DSDT.pcie
> index 8eacd21d6ecdf9a3cd3e4f03cf1b40748dcbf53e..5e3f80cbf82cab52fe310160c88d0a11ff069bf5 100644
> GIT binary patch
> delta 128
> zcmX>vK1YJfCD<iIhKGTHiDM#{G?VMziRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
> zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFY?feL%O!0bFT~Em
> W#mvCKARrLKz!4v83Kn1m3IG6dDjnGX
> 
> delta 55
> zcmbOuabBFuCD<k8JU0Ualj}q-X(rE|6V<breAp&V_GOfv_)umtAEWH%FvitfT&D3t
> L>?~ZY3=9kauQ3gZ
> 
> diff --git a/tests/data/acpi/x86/microvm/DSDT.rtc b/tests/data/acpi/x86/microvm/DSDT.rtc
> index e375473b5f033d374dd2a64a9f814fd16c3e262f..11258908703b64a835bd14307490c62c5ef7301f 100644
> GIT binary patch
> delta 129
> zcmbQj{E(T;CD<k8AtM6=)8dI-(o9~G6V)d&`La*k<;$oynS)V=Z-Kz%>I5zU1}=fN
> zV4r}=ii`@Hnm~pVN4$rp3y<Rk9=_;?7@$BrBR3-xV{{`AnBy4i1QeNU&*&{}952Mq
> X!o|$Mz#t$H!@v<AYzh`&1quKFJNg{0
> 
> delta 79
> zcmaFJJcXIdCD<io3L^sp)BTBD(oBA$6V)d&`LIpg<;y5LnS)VAI=V?RAUw#4Bi_T)
> jg*U)2z`)tSnSmod)YFAway6qjmub8ZI|~;p0|Ns9En5;N
> 
> diff --git a/tests/data/acpi/x86/microvm/DSDT.usb b/tests/data/acpi/x86/microvm/DSDT.usb
> index d63fd84620a2689120b32262f964e6a098d71632..7250d9f8ad45fca91afeda1089090eee11cd6594 100644
> GIT binary patch
> delta 127
> zcmbQo{F<4|CD<k8H6sH9)6<Du(o8N=6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
> zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmFrBW)Zn#LmLS
> V%)r1PAP~dA5g%*{7GMPm006!m9&i8v
> 
> delta 54
> zcmaFOJdc^nCD<io9wP$-lj1}!X(kV`iRxKQK5P>w`!dQ-d?+)Sk5P7V7-JQeX}l0S
> K3l}Q`0|NkddJG`|
> 
> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> index bb028db137..dfb8523c8b 100644
> --- a/tests/qtest/bios-tables-test-allowed-diff.h
> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> @@ -1,6 +1 @@
>   /* List of comma-separated changed AML files to ignore */
> -"tests/data/acpi/x86/microvm/DSDT",
> -"tests/data/acpi/x86/microvm/DSDT.ioapic2",
> -"tests/data/acpi/x86/microvm/DSDT.pcie",
> -"tests/data/acpi/x86/microvm/DSDT.rtc",
> -"tests/data/acpi/x86/microvm/DSDT.usb",
Re: [RFC V3 PATCH 10/13] tests/qtest/bios-table-tests: Update ACPI table binaries for microvm
Posted by Annie Li 8 months ago
Hi Gustavo,

On 4/17/2025 1:36 PM, Gustavo Romero wrote:
> Hi Annie,
>
> On 4/11/25 17:44, Annie Li wrote:
>> Following is the diff generated by the step 5 and 6 in
>> tests/qtest/bios-tables-test.c
>>
>> Diff from iasl to show changes in DSDT table:
>>
>> --- /tmp/asl-2F0N42.dsl 2025-04-11 12:50:02.892883467 +0000
>> +++ /tmp/asl-C81N42.dsl 2025-04-11 12:50:02.890883378 +0000
>> @@ -1,30 +1,30 @@
>>   /*
>>    * Intel ACPI Component Architecture
>>    * AML/ASL+ Disassembler version 20210604 (64-bit version)
>>    * Copyright (c) 2000 - 2021 Intel Corporation
>>    *
>>    * Disassembling to symbolic ASL+ operators
>>    *
>> - * Disassembly of tests/data/acpi/x86/microvm/DSDT, Fri Apr 11 
>> 12:50:02 2025
>> + * Disassembly of /tmp/aml-481N42, Fri Apr 11 12:50:02 2025
>
> Same comment from 4/13 regarding diff confusing git am and b4 and 
> about the filename.

Nod, will fix it.

Thanks

Annie

>
> Otherwise:
>
> Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
>
>
> Cheers,
> Gustavo
>
>>    *
>>    * Original Table Header:
>>    *     Signature        "DSDT"
>> - *     Length           0x0000016D (365)
>> + *     Length           0x000001BA (442)
>>    *     Revision         0x02
>> - *     Checksum         0x11
>> + *     Checksum         0xD5
>>    *     OEM ID           "BOCHS "
>>    *     OEM Table ID     "BXPC    "
>>    *     OEM Revision     0x00000001 (1)
>>    *     Compiler ID      "BXPC"
>>    *     Compiler Version 0x00000001 (1)
>>    */
>>   DefinitionBlock ("", "DSDT", 2, "BOCHS ", "BXPC    ", 0x00000001)
>>   {
>>       Scope (_SB)
>>       {
>>           Device (FWCF)
>>           {
>>               Name (_HID, "QEMU0002")  // _HID: Hardware ID
>>               Name (_STA, 0x0B)  // _STA: Status
>>               Name (_CRS, ResourceTemplate ()  // _CRS: Current 
>> Resource Settings
>>               {
>> @@ -66,56 +66,82 @@
>>                       0x00000009,
>>                   }
>>               })
>>               OperationRegion (EREG, SystemMemory, 0xFEA00000, 0x04)
>>               Field (EREG, DWordAcc, NoLock, WriteAsZeros)
>>               {
>>                   ESEL,   32
>>               }
>>
>>               Method (_EVT, 1, Serialized)  // _EVT: Event
>>               {
>>                   Local0 = ESEL /* \_SB_.GED_.ESEL */
>>                   If (((Local0 & 0x02) == 0x02))
>>                   {
>>                       Notify (PWRB, 0x80) // Status Change
>>                   }
>> +
>> +                If (((Local0 & 0x10) == 0x10))
>> +                {
>> +                    Notify (SLPB, 0x80) // Status Change
>> +                }
>>               }
>>           }
>>
>>           Device (PWRB)
>>           {
>>               Name (_HID, "PNP0C0C" /* Power Button Device */) // 
>> _HID: Hardware ID
>>               Name (_UID, Zero)  // _UID: Unique ID
>>           }
>>
>> +        Device (SLPB)
>> +        {
>> +            Name (_HID, EisaId ("PNP0C0E") /* Sleep Button Device 
>> */)  // _HID: Hardware ID
>> +            OperationRegion (\SLP, SystemIO, 0x0201, One)
>> +            Field (\SLP, ByteAcc, NoLock, WriteAsZeros)
>> +            {
>> +                SBP,    1
>> +            }
>> +        }
>> +
>>           Device (VR07)
>>           {
>>               Name (_HID, "LNRO0005")  // _HID: Hardware ID
>>               Name (_UID, 0x07)  // _UID: Unique ID
>>               Name (_CCA, One)  // _CCA: Cache Coherency Attribute
>>               Name (_CRS, ResourceTemplate ()  // _CRS: Current 
>> Resource Settings
>>               {
>>                   Memory32Fixed (ReadWrite,
>>                       0xFEB00E00,         // Address Base
>>                       0x00000200,         // Address Length
>>                       )
>>                   Interrupt (ResourceConsumer, Level, ActiveHigh, 
>> Exclusive, ,, )
>>                   {
>>                       0x00000017,
>>                   }
>>               })
>>           }
>>       }
>>
>>       Scope (\)
>>       {
>> +        Name (_S3, Package (0x04)  // _S3_: S3 System State
>> +        {
>> +            0x03,
>> +            Zero,
>> +            Zero,
>> +            Zero
>> +        })
>> +    }
>> +
>> +    Scope (\)
>> +    {
>>           Name (_S5, Package (0x04)  // _S5_: S5 System State
>>           {
>>               0x05,
>>               Zero,
>>               Zero,
>>               Zero
>>           })
>>       }
>>   }
>>
>> Signed-off-by: Annie Li <annie.li@oracle.com>
>> ---
>>   tests/data/acpi/x86/microvm/DSDT            | Bin 365 -> 442 bytes
>>   tests/data/acpi/x86/microvm/DSDT.ioapic2    | Bin 365 -> 442 bytes
>>   tests/data/acpi/x86/microvm/DSDT.pcie       | Bin 3023 -> 3100 bytes
>>   tests/data/acpi/x86/microvm/DSDT.rtc        | Bin 404 -> 481 bytes
>>   tests/data/acpi/x86/microvm/DSDT.usb        | Bin 414 -> 491 bytes
>>   tests/qtest/bios-tables-test-allowed-diff.h |   5 -----
>>   6 files changed, 5 deletions(-)
>>
>> diff --git a/tests/data/acpi/x86/microvm/DSDT 
>> b/tests/data/acpi/x86/microvm/DSDT
>> index 
>> f477668f2ee80241e47b340ad4a30f5480df2049..ee199a4fb65ab1a2d811f37d9b5bc67bccd34939 
>> 100644
>> GIT binary patch
>> delta 127
>> zcmaFMw2PU`CD<io7b61$)76Pw(oD|c6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
>> zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmHREo~ex#LmLS
>> V%)r1PAP~dA5g%*{7GMPm005{09v1)r
>>
>> delta 54
>> zcmdnR{FaH!CD<h-myv;iNpK>UG?TmVMD;8tAGV2;eHmpZK9rfv$0$2FjM0<JG+v0E
>> Kg^QJefdK$+3k%2q
>>
>> diff --git a/tests/data/acpi/x86/microvm/DSDT.ioapic2 
>> b/tests/data/acpi/x86/microvm/DSDT.ioapic2
>> index 
>> e5924a49962e0cff4228bcfc77ebcf48cb4a2219..7512d1527cf222d6f75a92f051f9eda402e28376 
>> 100644
>> GIT binary patch
>> delta 127
>> zcmaFMw2PU`CD<io7b61$Q~yLRX(ng!iRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
>> zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFOqO8umNt$TVrSuE
>> UW?*0t5Qt&mhz~Xe3$OwO0ErwO(EtDd
>>
>> delta 54
>> zcmdnR{FaH!CD<h-myv;i>GVV{X(o5!iRxKQK5P>w`!dQ-d?+)Sk5P7V7^5ebX}l0S
>> K3l}Q`0|Nky8w`Q~
>>
>> diff --git a/tests/data/acpi/x86/microvm/DSDT.pcie 
>> b/tests/data/acpi/x86/microvm/DSDT.pcie
>> index 
>> 8eacd21d6ecdf9a3cd3e4f03cf1b40748dcbf53e..5e3f80cbf82cab52fe310160c88d0a11ff069bf5 
>> 100644
>> GIT binary patch
>> delta 128
>> zcmX>vK1YJfCD<iIhKGTHiDM#{G?VMziRxKQzU&hx`!ecHd?>@WKwxrp0+#>-mq1&v
>> zPr$@q3YwZg+KD6H!_$Sw@d6KDbVCeKAfAz%k%=+7kq69i40ZyFY?feL%O!0bFT~Em
>> W#mvCKARrLKz!4v83Kn1m3IG6dDjnGX
>>
>> delta 55
>> zcmbOuabBFuCD<k8JU0Ualj}q-X(rE|6V<breAp&V_GOfv_)umtAEWH%FvitfT&D3t
>> L>?~ZY3=9kauQ3gZ
>>
>> diff --git a/tests/data/acpi/x86/microvm/DSDT.rtc 
>> b/tests/data/acpi/x86/microvm/DSDT.rtc
>> index 
>> e375473b5f033d374dd2a64a9f814fd16c3e262f..11258908703b64a835bd14307490c62c5ef7301f 
>> 100644
>> GIT binary patch
>> delta 129
>> zcmbQj{E(T;CD<k8AtM6=)8dI-(o9~G6V)d&`La*k<;$oynS)V=Z-Kz%>I5zU1}=fN
>> zV4r}=ii`@Hnm~pVN4$rp3y<Rk9=_;?7@$BrBR3-xV{{`AnBy4i1QeNU&*&{}952Mq
>> X!o|$Mz#t$H!@v<AYzh`&1quKFJNg{0
>>
>> delta 79
>> zcmaFJJcXIdCD<io3L^sp)BTBD(oBA$6V)d&`LIpg<;y5LnS)VAI=V?RAUw#4Bi_T)
>> jg*U)2z`)tSnSmod)YFAway6qjmub8ZI|~;p0|Ns9En5;N
>>
>> diff --git a/tests/data/acpi/x86/microvm/DSDT.usb 
>> b/tests/data/acpi/x86/microvm/DSDT.usb
>> index 
>> d63fd84620a2689120b32262f964e6a098d71632..7250d9f8ad45fca91afeda1089090eee11cd6594 
>> 100644
>> GIT binary patch
>> delta 127
>> zcmbQo{F<4|CD<k8H6sH9)6<Du(o8N=6V<breAy>X_GQ$Y_)vy#fxzVI1TFyvE`hdS
>> zpMZ(K6f`w~v=c|Xho=jV;{_hR=!O`eKs+NiBNJnEBM+G480-WTnJmFrBW)Zn#LmLS
>> V%)r1PAP~dA5g%*{7GMPm006!m9&i8v
>>
>> delta 54
>> zcmaFOJdc^nCD<io9wP$-lj1}!X(kV`iRxKQK5P>w`!dQ-d?+)Sk5P7V7-JQeX}l0S
>> K3l}Q`0|NkddJG`|
>>
>> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h 
>> b/tests/qtest/bios-tables-test-allowed-diff.h
>> index bb028db137..dfb8523c8b 100644
>> --- a/tests/qtest/bios-tables-test-allowed-diff.h
>> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
>> @@ -1,6 +1 @@
>>   /* List of comma-separated changed AML files to ignore */
>> -"tests/data/acpi/x86/microvm/DSDT",
>> -"tests/data/acpi/x86/microvm/DSDT.ioapic2",
>> -"tests/data/acpi/x86/microvm/DSDT.pcie",
>> -"tests/data/acpi/x86/microvm/DSDT.rtc",
>> -"tests/data/acpi/x86/microvm/DSDT.usb",
>