* [PATCH 3/3] tests: acpi: update expected blobs
2024-12-10 16:39 [PATCH 0/3] cpuhp: ensure that cpu hotremove works the 1st time Igor Mammedov
@ 2024-12-10 16:39 ` Igor Mammedov
0 siblings, 0 replies; 9+ messages in thread
From: Igor Mammedov @ 2024-12-10 16:39 UTC (permalink / raw)
To: qemu-devel; +Cc: mst, imammedo, eric.mackay, anisinha
previous patch has changed cpu hotplug AML, expected diff:
@@ -2942,6 +2942,7 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001)
{
Acquire (\_SB.PCI0.PRES.CPLK, 0xFFFF)
Name (CNEW, Package (0xFF) {})
+ Name (CEJL, Package (0xFF) {})
Local3 = Zero
Local4 = One
While ((Local4 == One))
@@ -2949,6 +2950,7 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001)
Local4 = Zero
Local0 = One
Local1 = Zero
+ Local5 = Zero
While (((Local0 == One) && (Local3 < One)))
{
Local0 = Zero
@@ -2959,7 +2961,7 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001)
Break
}
- If ((Local1 == 0xFF))
+ If (((Local1 == 0xFF) || (Local5 == 0xFF)))
{
Local4 = One
Break
@@ -2972,10 +2974,11 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001)
Local1++
Local0 = One
}
- ElseIf ((\_SB.PCI0.PRES.CRMV == One))
+
+ If ((\_SB.PCI0.PRES.CRMV == One))
{
- CTFY (Local3, 0x03)
- \_SB.PCI0.PRES.CRMV = One
+ CEJL [Local5] = Local3
+ Local5++
Local0 = One
}
@@ -2992,6 +2995,16 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001)
\_SB.PCI0.PRES.CINS = One
Local2++
}
+
+ Local2 = Zero
+ While ((Local2 < Local5))
+ {
+ Local3 = DerefOf (CEJL [Local2])
+ CTFY (Local3, 0x03)
+ \_SB.PCI0.PRES.CSEL = Local3
+ \_SB.PCI0.PRES.CRMV = One
+ Local2++
+ }
}
Release (\_SB.PCI0.PRES.CPLK)
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
tests/qtest/bios-tables-test-allowed-diff.h | 42 ------------------
tests/data/acpi/x86/pc/DSDT | Bin 8526 -> 8593 bytes
tests/data/acpi/x86/pc/DSDT.acpierst | Bin 8437 -> 8504 bytes
tests/data/acpi/x86/pc/DSDT.acpihmat | Bin 9851 -> 9918 bytes
tests/data/acpi/x86/pc/DSDT.bridge | Bin 15397 -> 15464 bytes
tests/data/acpi/x86/pc/DSDT.cphp | Bin 8990 -> 9057 bytes
tests/data/acpi/x86/pc/DSDT.dimmpxm | Bin 10180 -> 10247 bytes
tests/data/acpi/x86/pc/DSDT.hpbridge | Bin 8477 -> 8544 bytes
tests/data/acpi/x86/pc/DSDT.hpbrroot | Bin 5033 -> 5100 bytes
tests/data/acpi/x86/pc/DSDT.ipmikcs | Bin 8598 -> 8665 bytes
tests/data/acpi/x86/pc/DSDT.memhp | Bin 9885 -> 9952 bytes
tests/data/acpi/x86/pc/DSDT.nohpet | Bin 8384 -> 8451 bytes
tests/data/acpi/x86/pc/DSDT.numamem | Bin 8532 -> 8599 bytes
tests/data/acpi/x86/pc/DSDT.roothp | Bin 12319 -> 12386 bytes
tests/data/acpi/x86/q35/DSDT | Bin 8355 -> 8422 bytes
tests/data/acpi/x86/q35/DSDT.acpierst | Bin 8372 -> 8439 bytes
tests/data/acpi/x86/q35/DSDT.acpihmat | Bin 9680 -> 9747 bytes
.../data/acpi/x86/q35/DSDT.acpihmat-generic-x | Bin 12565 -> 12632 bytes
.../acpi/x86/q35/DSDT.acpihmat-noinitiator | Bin 8634 -> 8701 bytes
tests/data/acpi/x86/q35/DSDT.applesmc | Bin 8401 -> 8468 bytes
tests/data/acpi/x86/q35/DSDT.bridge | Bin 11968 -> 12035 bytes
tests/data/acpi/x86/q35/DSDT.core-count | Bin 12913 -> 12980 bytes
tests/data/acpi/x86/q35/DSDT.core-count2 | Bin 33770 -> 33837 bytes
tests/data/acpi/x86/q35/DSDT.cphp | Bin 8819 -> 8886 bytes
tests/data/acpi/x86/q35/DSDT.cxl | Bin 13146 -> 13213 bytes
tests/data/acpi/x86/q35/DSDT.dimmpxm | Bin 10009 -> 10076 bytes
tests/data/acpi/x86/q35/DSDT.ipmibt | Bin 8430 -> 8497 bytes
tests/data/acpi/x86/q35/DSDT.ipmismbus | Bin 8443 -> 8510 bytes
tests/data/acpi/x86/q35/DSDT.ivrs | Bin 8372 -> 8439 bytes
tests/data/acpi/x86/q35/DSDT.memhp | Bin 9714 -> 9781 bytes
tests/data/acpi/x86/q35/DSDT.mmio64 | Bin 9485 -> 9552 bytes
tests/data/acpi/x86/q35/DSDT.multi-bridge | Bin 13208 -> 13275 bytes
tests/data/acpi/x86/q35/DSDT.noacpihp | Bin 8235 -> 8302 bytes
tests/data/acpi/x86/q35/DSDT.nohpet | Bin 8213 -> 8280 bytes
tests/data/acpi/x86/q35/DSDT.numamem | Bin 8361 -> 8428 bytes
tests/data/acpi/x86/q35/DSDT.pvpanic-isa | Bin 8456 -> 8523 bytes
tests/data/acpi/x86/q35/DSDT.thread-count | Bin 12913 -> 12980 bytes
tests/data/acpi/x86/q35/DSDT.thread-count2 | Bin 33770 -> 33837 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm12 | Bin 8961 -> 9028 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm2 | Bin 8987 -> 9054 bytes
tests/data/acpi/x86/q35/DSDT.type4-count | Bin 18589 -> 18656 bytes
tests/data/acpi/x86/q35/DSDT.viot | Bin 14612 -> 14679 bytes
tests/data/acpi/x86/q35/DSDT.xapic | Bin 35718 -> 35785 bytes
43 files changed, 42 deletions(-)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index a1047913af..dfb8523c8b 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,43 +1 @@
/* List of comma-separated changed AML files to ignore */
-"tests/data/acpi/x86/pc/DSDT",
-"tests/data/acpi/x86/pc/DSDT.acpierst",
-"tests/data/acpi/x86/pc/DSDT.acpihmat",
-"tests/data/acpi/x86/pc/DSDT.bridge",
-"tests/data/acpi/x86/pc/DSDT.cphp",
-"tests/data/acpi/x86/pc/DSDT.dimmpxm",
-"tests/data/acpi/x86/pc/DSDT.hpbridge",
-"tests/data/acpi/x86/pc/DSDT.hpbrroot",
-"tests/data/acpi/x86/pc/DSDT.ipmikcs",
-"tests/data/acpi/x86/pc/DSDT.memhp",
-"tests/data/acpi/x86/pc/DSDT.nohpet",
-"tests/data/acpi/x86/pc/DSDT.numamem",
-"tests/data/acpi/x86/pc/DSDT.roothp",
-"tests/data/acpi/x86/q35/DSDT",
-"tests/data/acpi/x86/q35/DSDT.acpierst",
-"tests/data/acpi/x86/q35/DSDT.acpihmat",
-"tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x",
-"tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator",
-"tests/data/acpi/x86/q35/DSDT.applesmc",
-"tests/data/acpi/x86/q35/DSDT.bridge",
-"tests/data/acpi/x86/q35/DSDT.core-count",
-"tests/data/acpi/x86/q35/DSDT.core-count2",
-"tests/data/acpi/x86/q35/DSDT.cphp",
-"tests/data/acpi/x86/q35/DSDT.cxl",
-"tests/data/acpi/x86/q35/DSDT.dimmpxm",
-"tests/data/acpi/x86/q35/DSDT.ipmibt",
-"tests/data/acpi/x86/q35/DSDT.ipmismbus",
-"tests/data/acpi/x86/q35/DSDT.ivrs",
-"tests/data/acpi/x86/q35/DSDT.memhp",
-"tests/data/acpi/x86/q35/DSDT.mmio64",
-"tests/data/acpi/x86/q35/DSDT.multi-bridge",
-"tests/data/acpi/x86/q35/DSDT.noacpihp",
-"tests/data/acpi/x86/q35/DSDT.nohpet",
-"tests/data/acpi/x86/q35/DSDT.numamem",
-"tests/data/acpi/x86/q35/DSDT.pvpanic-isa",
-"tests/data/acpi/x86/q35/DSDT.thread-count",
-"tests/data/acpi/x86/q35/DSDT.thread-count2",
-"tests/data/acpi/x86/q35/DSDT.tis.tpm12",
-"tests/data/acpi/x86/q35/DSDT.tis.tpm2",
-"tests/data/acpi/x86/q35/DSDT.type4-count",
-"tests/data/acpi/x86/q35/DSDT.viot",
-"tests/data/acpi/x86/q35/DSDT.xapic",
diff --git a/tests/data/acpi/x86/pc/DSDT b/tests/data/acpi/x86/pc/DSDT
index 8b8235fe79e2fa08a6f840c8479edb75f5a047b9..60d50b088a362556fd54395cb15364d6c0936be5 100644
GIT binary patch
delta 191
zcmX@-G|`#MCD<ioq9OwWqy0v%ojgoV&XbSwY+!P*m|V{Lm)%p!IoR2cW3nutxHgBg
zpKG`f(|-<US1%tRUBHlBz?ibgUvzQ`V*x`-0b@b|Lt+6#>LNex$(#6mIrt_{PUQMO
zIdw7<zk|{OmB}&sEb+lk@d3`B1_42?!OlUxVT=XI9U#+E8A?+pm-G8c6)+?%a$=d9
mlv>aXmPleqb`EiiOy**qJd<CY6{w{&X>uH&*yh{(#ex8Vk2-Y#
delta 140
zcmV;70CWG5L(W19L{mgmP9XpQ0co)cxeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5OWbzU?~#-
diff --git a/tests/data/acpi/x86/pc/DSDT.acpierst b/tests/data/acpi/x86/pc/DSDT.acpierst
index 06829b9c6c6d726d955dc7c99bc9f42448e22aeb..4c434c25c0b1602f22128e352781df498fa69ddf 100644
GIT binary patch
delta 191
zcmezBxWkFdCD<jzLXm-iv3(=gP97#F=gCKTHZVC@OfKjB%kC-V9PI4JF<F*RT${t$
z&ox|#=|6|FtCtUuE?`J5U`$!$FFHAev4A0^fH9$fA+dlVb&((U<V}3O9DEZeCvyFt
zoI06_-$7}C%H$Y*miS<&_yA{5gMc8{VCNv;Fvf!94v=Z745g`)%lZAJ3K)_WIk8Ml
mN-byxOC&KQJBPSMCUY@Qp2@Gy3e-}XG&zn>Z1Zh?DM0|M$vRg6
delta 140
zcmV;70CWGiLiIrkL{mgm^&kKM0kW|QxeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5E>Em)hYD=
diff --git a/tests/data/acpi/x86/pc/DSDT.acpihmat b/tests/data/acpi/x86/pc/DSDT.acpihmat
index 2fe355ebdbb858fa9247d09112e21712e3eddc45..61b7d5caa55c44dbf69d649110c6b14bb4c3fdf5 100644
GIT binary patch
delta 175
zcmezEv(J~yCD<iopBe)L<DZROJ9(HKJtrUK*}&v)ySbc~gOSNgYH}l=iZX|@tCx=u
z)Bgg7<O0T&MGj(<Qy2>vQVJLo3K$X#7*ZGcb5GvO=f}Y}adINp|H-M7`S=}_7N|^)
z(PxPdc8U*h_B03xat(G4@(p7wNbUfcmda3?I=PwOPo{t&X^|7l)TGpcX0SvOL$Y&-
YTVyg9b3yWCM?O_%pr*~|`PYa60N6n`+5i9m
delta 117
zcmV-*0E+*<P5Vp=L{mgmdnNz?0Vc5uxeNkMM3cn~umVguvv&;$0Rl%BlYtH-7;pe%
zZ~<hZMG})_0dN3hZ~<U&0AZp{43o$XP6G*(VUr#YKm?&Tpf!_V4;2YRR7P223IlKf
XlVJ}NlYkFS0$V4O!3`6$DG-$w=m8<V
diff --git a/tests/data/acpi/x86/pc/DSDT.bridge b/tests/data/acpi/x86/pc/DSDT.bridge
index 4d4067c182a6625db1e877408eb7436113884b50..d43e148bed19160f39d88ccf3364544150a3f87f 100644
GIT binary patch
delta 191
zcmZ2l@uGsuCD<h-!-j!@(RU-)P97#F=gCKTHZVC@OfKjB%kC-V9PI4JF<F*RT${t$
z&ox|#=|6|FtCtUuE?`J5U`$!$FFHAev4A0^fH9$fA+dlVb&((U<V}3O9DEZeCvyFt
zoI06_-$7}C%H$Y*miS<&_yA{5gMc8{VCNv;Fvf!94v=Z745g`)%lZAJ3K)_WIk8Ml
mN-byxOC&KQJBPSMCUY@Qp2@Gy3e-}XG&zn>Z1Zh?FC75!n>zpi
delta 140
zcmV;70CWH7c%^s>L{mgmB|HEC0e7(qxeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5K1hi?<t-D
diff --git a/tests/data/acpi/x86/pc/DSDT.cphp b/tests/data/acpi/x86/pc/DSDT.cphp
index 045a52e75b7fcd4e5f840a758c548231498b96e4..9fda0b56638e02097e58dd4536c9c5955986e88e 100644
GIT binary patch
delta 167
zcmbQ|_Rx*XCD<h-QJH~(annYwojgoV36qcVY+!N--CWLV$;j>{<s9tn$1(W>ulVG1
zd>qOg&aPfQLQMY)7?KMZQx-XhO-^AfU`Q!oOekPTEMQ1o<j+0%9iJZu-^9s@T>mGh
zPPXTFU|ygydGbPjVIIbU<PMPjREE;j$y@mSqzV|47CEs@O-e0j21_I{Bs+(=MJ97G
RPrk;l&K%?$wn;!v7yuUwG#~%~
delta 134
zcmV;10D1r6MxI6rL{mgm9wPt%0nM=rxeNkNTa(2MumVj`vv&<S0S8AGLsLUe2$S#)
z7n9Bo2pDhxV{idvqD2yuWC3shWN-msZ~$SVO$?Ly4o(9JlVOuj4?qN=H=s3>kq;Fd
oLsUjtV+sRs0bDNxUsFO~P(w*DP*O!xLsCsvld%s@vqcab6Skcv^Z)<=
diff --git a/tests/data/acpi/x86/pc/DSDT.dimmpxm b/tests/data/acpi/x86/pc/DSDT.dimmpxm
index 205219b99d903555125c4b07fc047c42993eb338..5b6471c8db9003b39bf5e20af34061f3e71cdbd5 100644
GIT binary patch
delta 173
zcmX@&-yXo_66_MfuED^-7{8J0EDw`+$mE+m8<;#jH+S<YFfw^bO<v2VqRip!>g6NE
z^uK^1xqvZck%QRe6vhIElmf<t0*1r_hSWv=+>>AO`El?~oSew@e{$+%D}D#11uBzc
z^jYGAo#F$WJq-eaT!Wp1e8U(Ek~=`Ar81PJPF~ONCsn|Zw8)8NYEo)JGgu;tA=x>^
WEi##ldGbYmb!MQJ&A<6iiUR=B8#YG(
delta 141
zcmZn<IO5Oc66_LkM4f?wapp#@vph^50h4d?Y+!P8-Q3Nqz{un-HhC?dibMfJasgw?
zB3GfwDU1aSDFuuP1q_Lce0e5+;qznWoSZn>i{HU?q4@&S$uasY@xe~<0nVNV0YR?8
e&OyFmjLso$k;z=l1&mlEC$Hi6+w354TpR%WuPVd<
diff --git a/tests/data/acpi/x86/pc/DSDT.hpbridge b/tests/data/acpi/x86/pc/DSDT.hpbridge
index 8fa8b519ec65bd5099c45f4e1c85b11b47a23845..67fe28699fbb261cfc7a52b2291f9965ab93c6a8 100644
GIT binary patch
delta 191
zcmbR1^uUSBCD<h-L6L!hv3MibP97#F=gCKTHZVC@OfKjB%kC-V9PI4JF<F*RT${t$
z&ox|#=|6|FtCtUuE?`J5U`$!$FFHAev4A0^fH9$fA+dlVb&((U<V}3O9DEZeCvyFt
zoI06_-$7}C%H$Y*miS<&_yA{5gMc8{VCNv;Fvf!94v=Z745g`)%lZAJ3K)_WIk8Ml
mN-byxOC&KQJBPSMCUY@Qp2@Gy3e-}XG&zn>Z1Zh?CqV$0Q#xb-
delta 140
zcmV;70CWG~LY+biL{mgm9U%Y!0iCf5xeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5JC}S!YKj(
diff --git a/tests/data/acpi/x86/pc/DSDT.hpbrroot b/tests/data/acpi/x86/pc/DSDT.hpbrroot
index 01719462a72fd6d40ce433dac601e4b94eae574c..077a4cc988dc417a1bc9317dddd2dbd96ff1ff50 100644
GIT binary patch
delta 195
zcmZ3f{zje4CD<k8jW7cPWAa9>9Bw8j=gAe^8<-p{CTsEhW%rbF4tDnAn0$dpT${t$
z&ox|#=|6|FtCtUuE?`J5U`$!$FFHAev4A0^fH9$fA+dlVb&((U<Wyc?4!((#6S@9R
zPMy4!*FkB4%H$Y*miS<&_yA{5gMc8{VCNv;Fvf!94v=Z745g`)wfOv`3K)_WIk8Ml
qN-byxOC&KQJBPSMCUY@Q&gT`I{Fg_N6{xK=X|f=%*ycVyX;uK4GdnQ=
delta 157
zcmaE(zEYjbCD<ior7!~nWA8?;9Bw9myU7*Y8<>2JCu{NiWp@*E4tDnAn0$dpT!F*c
z&ox|#>3;!3asgw?A{U{_DU1aSDFuuP1q_Lce0U}o@cJ@yPEMSBl-I#@q4@&S$uasY
u@xe~<0nVNV0YR?8&OyFmjLso$k;z=l1&mlEC#&=MO%~)8+kBBvniT*|<t>~5
diff --git a/tests/data/acpi/x86/pc/DSDT.ipmikcs b/tests/data/acpi/x86/pc/DSDT.ipmikcs
index 0ca664688b16baa3a06b8440181de4f17511c6b0..9b2e81a7bcefb5c0e2dfbd2bbc5b6ea501f86306 100644
GIT binary patch
delta 191
zcmbQ{eAAiBCD<k8rXm9a<L-@IJ9(I#oF^aU*}&vrF}a-gFT1CdbFi}?$7ESPacvG~
zKi6;}rvDtyu3kPsx_}|MfH7r}zv$!?#sY?v0>*>_hQtDf)J1;WlQ;4Ca_~)@oXGWm
za_VFzeg~xmDwAXMS>l78;scyL4FZB(gPntX!x#&aJ3ywTGL)uHF6Z}?Dqu)j<is*H
nDYc*(ERn>J>>T12nassJc_zO)D^N>m(&RWkvCX&ncMAdl7dShT
delta 140
zcmV;70CWG@LzY7dL{mgmmLUKD0p+m@xeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5W5k-i7GGv
diff --git a/tests/data/acpi/x86/pc/DSDT.memhp b/tests/data/acpi/x86/pc/DSDT.memhp
index 03ff464ba4e72082fce0921815cfc09ca20b561a..9c66ccf150af1622d1b788a1ae04a6e5136cff9e 100644
GIT binary patch
delta 191
zcmbR1`@omWCD<k8ff@q?qwq$qojgn)&XbSwY+!P;m|V{Lm)%p!IoR2cW3nutxHgBg
zpKG`f(|-<US1%tRUBHlBz?ibgUvzQ`V*x`-0b@b|Lt+6#>LNex$(#6mIrt_{PUQMO
zIdw7<zk|{OmB}&sEb+lk@d3`B1_42?!OlUxVT=XI9U#+E8A?+pm-G8c6)+?%a$=d9
mlv>aXmPleqb`EiiOy**qJd<CY6{w{&X>uH&*yh{(hr|HPnL985
delta 140
zcmV;70CWG~O`S~&L{mgmohASP0b8*OxeNkDKa<4_umVFglXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5W*IV{wdA?
diff --git a/tests/data/acpi/x86/pc/DSDT.nohpet b/tests/data/acpi/x86/pc/DSDT.nohpet
index b081030f0ed171e52b13e28cfdc8770a04c2806e..28dbd8d8949d1421da9312cf0440d7ae3b64916e 100644
GIT binary patch
delta 195
zcmX@$*zCmR66_MftjNH?$gz=2n}^BCd9pdr1||oK$uD{SvU^H72Rr+5Oz!6u*XD5c
za}5__`p@C)>g5BZ3mB3M7*iJci%w2qEMQ0}U`!}rNGxDTUF63-S%uG+gKy&GM6Umn
zQzw`3IVdeqnH;0f5+CdoAK>h15D?@V>>T79##oTt0WvL>p)_^!OFloT0*0hTPApTC
qQVW{F5=jin&LM7*$z05n_4vdl@8K0>1!^lzntXv*Y_l(au^<4GLppx|
delta 157
zcmZp6I^f9V66_LkK!JgQ(Rd@5HV>1(-DGo~4NShqlV9@uWp@*E4tDnAnB31RuE62!
z=Nc}=^uK^1xqvZck&Dpe6vhIElmf<t0*1szK0K54`Fxo<CnrvB;d3xuXuiO7a*RGp
ue6Uk|fU~DTK#*&&bC7QsqjQK`WHJ|X0V5X4$<O%wCSTwc+dP54SP%dOB`#_J
diff --git a/tests/data/acpi/x86/pc/DSDT.numamem b/tests/data/acpi/x86/pc/DSDT.numamem
index 2c98cafbff5db04410b35a1151eaf18723a4dad7..e256bbce790152f045247db631d9f1da81f90499 100644
GIT binary patch
delta 191
zcmccOG~JoYCD<iox*`Ju<LQlDJ9(HqoF^aU*}&vxF}a-gFT1CdbFi}?$7ESPacvG~
zKi6;}rvDtyu3kPsx_}|MfH7r}zv$!?#sY?v0>*>_hQtDf)J1;WlQ;4Ca_~)@oXGWm
za_VFzeg~xmDwAXMS>l78;scyL4FZB(gPntX!x#&aJ3ywTGL)uHF6Z}?Dqu)j<is*H
mDYc*(ERn>J>>T12nassJc_zO)D^N>m(&RWkvCX&nD+K}fZ963Z
delta 140
zcmV;70CWGBL)1bFL{mgmR3QKW0TZzbxeNkDKa<4_umVFglXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5P1<Qdnm&I
diff --git a/tests/data/acpi/x86/pc/DSDT.roothp b/tests/data/acpi/x86/pc/DSDT.roothp
index da018dca9e3102e811107994248719ab5278c505..0557810ddc18dc280d039163c72b25428a2486c1 100644
GIT binary patch
delta 191
zcmbQA@F;=HCD<h-$$)`@@$^QnojgoV&XbSwY+!P*m|V{Lm)%p!IoR2cW3nutxHgBg
zpKG`f(|-<US1%tRUBHlBz?ibgUvzQ`V*x`-0b@b|Lt+6#>LNex$(#6mIrt_{PUQMO
zIdw7<zk|{OmB}&sEb+lk@d3`B1_42?!OlUxVT=XI9U#+E8A?+pm-G8c6)+?%a$=d9
mlv>aXmPleqb`EiiOy**qJd<CY6{w{&X>uH&*yh{(E(!n-m^&T-
delta 140
zcmV;70CWH1V4q+LL{mgmA20v_0rjy8xeNkNK9j`^umVjplXngM2Sye{Q$tP&lN}Bh
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43oVMO#=y&VUrRMKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Imhz4;7Pi4^ER_4i>W>5JVulO)0zp
diff --git a/tests/data/acpi/x86/q35/DSDT b/tests/data/acpi/x86/q35/DSDT
index fb89ae0ac6d4346e33156e9e4d3718698a0a1a8e..51ad37a351bffae8fbc9ba17f72c25ef61822f59 100644
GIT binary patch
delta 180
zcmZ4N_{@>ZCD<k8nF0d?Bkx8oc1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(8Dl0RR%vHv<3w
delta 109
zcmV-z0FwXaL8CzmL{mgmqaXkP0XMM<2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRveH7+b|#D
diff --git a/tests/data/acpi/x86/q35/DSDT.acpierst b/tests/data/acpi/x86/q35/DSDT.acpierst
index 46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b..dbd4f858354df0f4c050fd0b914581154f340ee8 100644
GIT binary patch
delta 180
zcmdnu_}!7qCD<k8y8;6P<Jyf}?2=4Q&Xa{DH!wL^Og<|4m)%p!IoR2cW3qva_~Zvt
z9LgNdu3kPuO#cfQk_#A97Ws=#PGKxyNGV`UC}2n|U`So$$302fmxFKO<V3FjlT#-L
zNINJkP?;Q~&k`T(6d&O1X%G<P8tfe88^&0W+yOEym7z3s@=<9&sRD+iMNTYJlTr(s
d!4gRf$<85ek;z=llb=hgGXu437L-}c0sx)UH|GEV
delta 109
zcmV-z0FwXrL9{^%L{mgmv>*Tg0oSn#2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRve)OE4?7a
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat b/tests/data/acpi/x86/q35/DSDT.acpihmat
index 61c5bd52a42242e85090934e8e45bf01642609d6..952752e30e9dfc9e2085e8fceaa0740dda6db89c 100644
GIT binary patch
delta 159
zcmccMJ=urLCD<iISdD>!@#ID>c1b2j&&k4)8<_lUHy@ScU}W-=ntWbLMVZ6d)yqeS
z>3;!3asgw?A_uX_DU1aSDFuuP1q_J=45^F!xhHc=`*HA1oSew@e{$;N7-<Kl1uByl
zNQ-h8BzJ&xr!thLPQEDZCsV+Xw8)8NYEo)JGgu;tA=x>^Ei##lxgdFRzmzI-kZ;&#
IS((k80E*WzCIA2c
delta 141
zcmbR2bHSU-CD<k8f+_<8<Cl$G?2=4=E|Y~NH!%5FZayl>!N}w;Hu=1iibMfJasgw?
zB3GfwDU1aSDFuuP1q_Lce0e5|N&7K#PEMShBkf?i(0qaE<QRRH_+Y2_0B28wfFRdk
e=OEuOM&}T>$Yd_&0!A#7lg~-}ZBCNe$O!;@8!4;+
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x b/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x
index 497706c9742a9ea5396d6c9c4cc1cc2a4a530339..e95258cbd8681103a642f8973bd1ac9ef229cff7 100644
GIT binary patch
delta 173
zcmbQ5bR&t&CD<h-!jOT1F>fOmyCjpp-(+FQ4NShyn~zG0Ffw^bO};IqqRip!>g6NE
z^uK^1xqvZck%QRe6vhIElmf<t0*1r_hSWv=+>^zn{W$n0PEO?dKRI=BhO~py0+q=z
z`YiFmPVoWGo(2IyuEEYhzF~|7$sHimQW;89C*PI!lPX|HTI9qsH7T{A87z^+kn9}d
V7MaY&Jo&%0Ix|qqW=)wlA^>)AH8}tP
delta 141
zcmcbSG&PCKCD<iI)R2LJars6rc1b25ugSua8<@Q8Hy@Q0VPtX_n|xbJMWTQqxqvZc
zk*m<;6vhIElmf<t0*1szzC4qar2UvVCnrv>kajR#XuiO7a*RGpe6Uk|fU~DTK#*&&
ebC7QsqjQK`WHJ|X0V5X4$+x8aHs{H_5&-}%Gbwuj
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator b/tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator
index 3aaa2bbdf54a0d0cade14421e84c6ec5a42f96fa..ba2a7d0004be7cd7220716dc7e8594be87197b98 100644
GIT binary patch
delta 200
zcmdnx{MVVwCD<k8uOb5jW6MS^c1b4BkjcW58<^ZZHy@Q$VC3|Yat?O(<A_$CTqq^1
z`~N?Ov!83Y5EGDg_3{DI1q{grj46v8#3rXO7BHj~FeVf*Bo;8FF7oG|tSs%v!8dVo
zBG>=Psgp~j9h4TROpeiKi4S&)4{-K02nccwb`J6lV=PGS0GXD`P?|dVrL><^0YlOv
uCzh#6sRhkoi6n+(=McBZWG?2(deUN(cS{Mf0=1PUO+GIrw%J!EmlXghfj&V1
delta 162
zcmezCyvv!(CD<iomm&iL<KB&2?2=4w0h5I#H!!)lZaylhz{u$?<{a$o#}TbOxll@2
z>HmKYXFu0)A*TNY49NwIDT`c%CZ{kKFr*YPCKNCvF7oA>tS9Zq%sDx6a*MQs=|b}b
zrjujzS>l78;scyL4FZB(gPntX!x)`I+#-{?m<t%ONKSqx?Kk<nl-TA8GP$e(dgCz@
diff --git a/tests/data/acpi/x86/q35/DSDT.applesmc b/tests/data/acpi/x86/q35/DSDT.applesmc
index 944209adeaa5bbb722431161c404cb51b8209993..b6cb840953ea539092f601e08b7122fc999b3e1b 100644
GIT binary patch
delta 180
zcmccUIK_#}CD<iIM3I4k@y|vsc1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(6c|0RRe*H!J`E
delta 109
zcmV-z0FwWdLeW7AL{mgm(I5Z-0VlBv2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvf+r*iaw}
diff --git a/tests/data/acpi/x86/q35/DSDT.bridge b/tests/data/acpi/x86/q35/DSDT.bridge
index d9938dba8fa5d405f7696c0dbdc24f3ae42ec934..1939fda2507cde6fcb6f7a093897f9bd2cb987ef 100644
GIT binary patch
delta 180
zcmX>Q+Z@N`66_Mftk1x}sIifYU6RSkd9twN1||oK$wwvsvU^H72Rr+5Og4}apZq|I
zLz%<b)yqeS>3;!3asgw?B7f1zDU1aSDFuuP1q_J=45^F!xF<>ba_~)@oXGWma_Zy&
zX$PeRDwAXMS>l78;scyL4FZB(gPntX!x#&aJ3xk|GL)uHJ}T`eRltz6$cbfYQffgn
cSR#ob**U~5GMS5c@^fi*W}ueMf->uc0FDnfXaE2J
delta 109
zcmV-z0FwWMUcg-nL{mgmz%Bp)0aLLG2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvfJo@Ma)J
diff --git a/tests/data/acpi/x86/q35/DSDT.core-count b/tests/data/acpi/x86/q35/DSDT.core-count
index a24b04cbdbf09383b933a42a2a15182545543a87..41c0832ab5041ff5361598813ec28fe7442b191b 100644
GIT binary patch
delta 168
zcmeyEvL%(vCD<ioixC3@W9vpPc1fm&gvr8^8<~1THXoC$VrKV}at?O(<CxqaBR)A!
zmP47t+11NOi0OX;LvjIQ$|47`$tjEl3@HVS2?Y#^1q`W+{JAIB$@+2dO`M#_^?!2e
z<nyu)%nMW|Pd1bj=3y*I?f~geWhhOZ>>}qURltz6$cbfYQffgnSR#ob**U~5GMS5c
Ra+;hvbC7S?<_U5Sg#j0XG*JKm
delta 134
zcmV;10D1qkW$|PRL{mgmaWVh^0k5$N2N?p5TayzRu>y)vv&I>E0|!SILsLUe2$O&u
z7n5Ec2pDhxV{idvqD2yuWC3shWN-msZ~$SVO$?KX9ZmxYlVOwJ9Y6%3H=s3>As!VR
oLsUjtV+sRs0bDNxUsFO~P(w*DP*O!xLsCsvlR+L%v(+Bo6Qh?Xg#Z8m
diff --git a/tests/data/acpi/x86/q35/DSDT.core-count2 b/tests/data/acpi/x86/q35/DSDT.core-count2
index 3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8..153b45f0f7443d25cecc2a752fb6dbd921160e78 100644
GIT binary patch
delta 160
zcmaFW&a}3JiOVI}B}BJ{fr0VbMlN<qrq1%o!jc=A+Hy7@lT_nn@|K$XPEAFL!`ao#
zM~LZv0Yh>DW6B~&vB@cn1q>+#j0pt{i3JR)iyS6vst0iJO`M#_^?!2e<QjDcrUfdK
z52%ZB7bJIpG^a9@rcVB-?k7{gkhI8&WolAtK{Hq)i6PlJ#4R$Ji@6|q@;WtD<{;m&
J&9)kAqXG4WGspk{
delta 142
zcmZ46!St$~iOVI}CFE5z0|O)DMlN<qrnchA!jc=Anlm;ZlT_nn@(`Q+PEAF;fFZen
zF=dgP(Bu@x0)~_V#)JZf#6^CSP1FOJIVUGh?ooFzU1+|*baIS7OMI|Xe1Nm3K|qje
fuyc@a7^8EDTVyg9a{(h3$;t23{Wdpgtd0f%C_O9p
diff --git a/tests/data/acpi/x86/q35/DSDT.cphp b/tests/data/acpi/x86/q35/DSDT.cphp
index 20955d0aa30120553da35d5a6640055d26255cf9..231bc23d932e832ffa12dd253bcf54245b5ef88f 100644
GIT binary patch
delta 158
zcmezDvdxvtCD<ion-T*9W5Y%+c1b3ugvr8^8<-qIHy@R>WMuM^nk22F%;D_n<s-!O
zzkngRfH7r}gV^L0#sY?v0>*>_hQtDf)J6W>lWnB^IQS+`PUQMOIdyWcv;)%umB|;S
zMY#)-J3zWq8A?+pGs*bL6fh($a$=d9lv>aXmPleqb`EiiOy*)PNS=H^N|iatH*B+?
Hj4B%dq#Q5J
delta 124
zcmV-?0E7RwMe{@oL{mgmb0Po$0h_T32N?oSTayzRumVj`v&9)X0Rl%Bk{cu#Z~$X)
z0c4^@5|d;BZ~$a*0bp<dVWLe8lSdm)0|}F1lbstt1fe&eHIv^P6&yoUMp<JD18@Oc
eF9csxLSIlrNia}SMN>mkO;(ft8&0#693uw7dL)Se
diff --git a/tests/data/acpi/x86/q35/DSDT.cxl b/tests/data/acpi/x86/q35/DSDT.cxl
index 3c34d4dcab16783abe65f6fa5e64eb69d40795fb..0f1ccdfcc3ffbf151c172015cc4bf18bc4ead218 100644
GIT binary patch
delta 180
zcmcbWHaDHiCD<iot}z1xqvJ*{c1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(9zk0{}PkHv|9x
delta 109
zcmV-z0FwWmXWC{8L{mgmS~CCu0c)`e2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvbqc@Dv~7
diff --git a/tests/data/acpi/x86/q35/DSDT.dimmpxm b/tests/data/acpi/x86/q35/DSDT.dimmpxm
index 228374b55bd544116e359f659e546fc66cf8a895..eb5b6e9f52107d9c95e38e94a67a6b5001beafc1 100644
GIT binary patch
delta 173
zcmbQ~cgK&*CD<h-MxB9yQFkMkv?P;v$YgcN4NRV%n{P@gFfw^bP5v#VqRip!>g6NE
z^uK^1xqvZck%QRe6vhIElmf<t0*1r_hSWv=+>_0v{W$n0PEO?dKRI=BhqQyz0+q=z
z`YiFmPVoWGo(2IyuEEYhzF~|7$sHimQW;89C;yf9lPX|HTI9qsH7T{A87z^+kn9}d
V7MaY&JXu~wof)WQv!~1jE&ziMG`Ro(
delta 141
zcmccPH`9;HCD<iIQk{W;F>NE4v?P;9z+`pF4NPvXn{P@gFfzG|P5v#VB2mDQT)>#J
z$W>@^3S$97N&#a+0Yl;<U!KWM(tgaGlM^S;kajR#XuiO7a*RGpe6Uk|fU~DTK#*&&
ebC7QsqjQK`WHJ|X0V5X4$-kuiHuuS#;{pIXrYYJ0
diff --git a/tests/data/acpi/x86/q35/DSDT.ipmibt b/tests/data/acpi/x86/q35/DSDT.ipmibt
index 45f911ada5645f158f3d6c0c430ec1d52cadc5d8..524fc9f4ee09fd7a5bec62818fd87b6ec300dee8 100644
GIT binary patch
delta 180
zcmaFoxY3ErCD<jzP?3Rw(P1MOyCjp7^JHPk4NMLelaEUNW%rbF4tDnAm~0>;KKX$Z
zhcbt=tCx=u)Bgg7<O0T&MgF3bQy2>vQVJLo3K$X#7*ZGcaZi%=<=~q*Ig#uC<kZOl
z(hf=sR3^vhv&08G#RoWh8UzHn20I7&hA|c-cYq8_WhhOZd{o*`s(>MBkrT_*q|}0D
dutX9=vU7-AWHJ}?<mb}r%s?%h1!eBC007LtHsk;R
delta 109
zcmV-z0FwW)LheBdL{mgm?jQgF0c)`e2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvg;|6`>%)
diff --git a/tests/data/acpi/x86/q35/DSDT.ipmismbus b/tests/data/acpi/x86/q35/DSDT.ipmismbus
index e5d6811bee1233d74236453c49060390d74d4416..d04d215a1d0fbc77739084d100a35af47a1c1a62 100644
GIT binary patch
delta 180
zcmezExX+2pCD<jzPLY9uapOiVc1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(CA=0RS-zH;DiM
delta 109
zcmV-z0FwW{Li<4qL{mgm`yc=S0o<_)2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvhRAPqrav
diff --git a/tests/data/acpi/x86/q35/DSDT.ivrs b/tests/data/acpi/x86/q35/DSDT.ivrs
index 46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b..dbd4f858354df0f4c050fd0b914581154f340ee8 100644
GIT binary patch
delta 180
zcmdnu_}!7qCD<k8y8;6P<Jyf}?2=4Q&Xa{DH!wL^Og<|4m)%p!IoR2cW3qva_~Zvt
z9LgNdu3kPuO#cfQk_#A97Ws=#PGKxyNGV`UC}2n|U`So$$302fmxFKO<V3FjlT#-L
zNINJkP?;Q~&k`T(6d&O1X%G<P8tfe88^&0W+yOEym7z3s@=<9&sRD+iMNTYJlTr(s
d!4gRf$<85ek;z=llb=hgGXu437L-}c0sx)UH|GEV
delta 109
zcmV-z0FwXrL9{^%L{mgmv>*Tg0oSn#2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRve)OE4?7a
diff --git a/tests/data/acpi/x86/q35/DSDT.memhp b/tests/data/acpi/x86/q35/DSDT.memhp
index 5ce081187a578ba7145a9ba20d30be36c13b7663..f73ade9bf6e4545f9912ed654a282884a54cec79 100644
GIT binary patch
delta 180
zcmez5z14@yCD<jzRE>dw@xewec1b1==gGp78<^ZICLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(7p?1OQcUH@*M>
delta 109
zcmV-z0FwW;O!7+#L{mgm@+ANO0V%Nx2N?oIKa&#~umVFglf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvh0624*0f
diff --git a/tests/data/acpi/x86/q35/DSDT.mmio64 b/tests/data/acpi/x86/q35/DSDT.mmio64
index bdf36c4d575bfc4eb2eac3f00c9b7b4270f88677..f0ddb4c83cdc9afdf4f289a66ed6bf0d630fd623 100644
GIT binary patch
delta 180
zcmeD6y5Pm-66_KZpvu6&xNsvEyCjo`^JHPk4NPtplaEUNW%rbF4tDnAm~0>;KKX$Z
zhcbt=tCx=u)Bgg7<O0T&MgF3bQy2>vQVJLo3K$X#7*ZGcaZi%=<=~q*Ig#uC<kZOl
z(hf=sR3^vhv&08G#RoWh8UzHn20I7&hA|c-cYq8_WhhOZd{o*`s(>MBkrT_*q|}0D
dutX9=vU7-AWHJ}?<mb}r%s?%h1!ewl000NpH)a3;
delta 109
zcmV-z0FwXEN{vbiL{mgm4J7~o0qL;{2N?oIKa&#~umVFglf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvh{W)LtM&
diff --git a/tests/data/acpi/x86/q35/DSDT.multi-bridge b/tests/data/acpi/x86/q35/DSDT.multi-bridge
index 1db43a69e4c2affd8bd678bbef4d3c228380288e..3ad19e3f5e480db1c449b838c83833f7665186cd 100644
GIT binary patch
delta 180
zcmbP{emkAZCD<k8wlM<(qtZq$c1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(B;M0{}}VH(me$
delta 109
zcmV-z0FwXPXP9RSL{mgmm@@zX0Zg$92N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvd*F{DB}m
diff --git a/tests/data/acpi/x86/q35/DSDT.noacpihp b/tests/data/acpi/x86/q35/DSDT.noacpihp
index 8bc16887e1c963c61aaecf71712a09c0554f6d67..9f7261d1b06bbf5d8a3e5a7a46b247a2a21eb544 100644
GIT binary patch
delta 206
zcmZ4O@XmqDCD<h-Pl17faluBeUE)km&XbRcZ(wq;m|P+8m(x?qIoR2cBU*VfqolCz
z|Nk7$ey-s{OhDSz%LhmoFeDc+rY!Ooot(m0z>reFm{7owSiq3F$d7ySW=UUuzKN3)
zx&BX1<@yg)wzOb!ptSI0St&K81uBzc^jYGAo#F$WJq-eaT!Wp1e8U(Ek~=^Kr!thL
zPOgyhlPX|HTI9qsH7T{A87z^+kn9}d7MaY&Jb9LsIxA31Y0~6)NwLj$r2JU{in&3W
delta 161
zcmaFou-bvkCD<iITY-Ur@zh4HUE)msc9V~ZZ(#B@o?IdEm(xwmIoR2cBU*VfqolCX
z|Nk7$ey-s{O#cfQk_#A97P$ybPGKxyNGV`UC}2oj<ij(0pQJA{=j6o6!cq>V3(Xgp
yPL9!Mi4S&)4{-K02nccwb`J6lV{{I2i%jNXE?~qWIk{BIZ*sh(*k*ZYe^vm<H!xlR
diff --git a/tests/data/acpi/x86/q35/DSDT.nohpet b/tests/data/acpi/x86/q35/DSDT.nohpet
index c13e45e3612646cc2e30f00b3b7e53335da816ea..c089b5877a0f4d808abd4d8d9396ee7d2a9a78e5 100644
GIT binary patch
delta 191
zcmbR0aKnMiCD<h-LV<yS@!CeNN(m+>=gF-S8<-p{CYwwCW%rbF4tDnAn0!}KT${t$
z&ox|#=|6|FtCtUuE?`J5U`$!$FFHAev4A0^fH9$fA+dlVb&((U<RU3w4!((#6S@9R
zPMv&6%0X#?%H$Y*miS<&_yA{5gMc8{VCNv;Fvf!94v=Z745g`)&87XM3K)_WIk8Ml
mN-byxOC&KQJBPSMCUY@Qj+9ns1!^fxnk*+Jwt2R66$=3Otvbs9
delta 140
zcmV;70CWG?K$SoWL{mgm6(9fr0RXWIc^CpuK9hwQumVjplQ$Xt2Sye{Q$tP&lie8?
zAP7TFMOP96|8M|fZ~<hZL=ux^0dN3hZ~<U&0AZp`43m8tO#=y&VUyAtKp>$vpf!_R
uF9csxLSIlrNia}SMN>mkO;!OzR7P223Img|8x@l@8%~oR8WywO8+rsM6)9){
diff --git a/tests/data/acpi/x86/q35/DSDT.numamem b/tests/data/acpi/x86/q35/DSDT.numamem
index ba6669437e65952f24516ded954b33fe54bdedfb..2867f5b44498d788fc0effd0bf616317821be88e 100644
GIT binary patch
delta 180
zcmZ4K_{NdTCD<k8jRFG$<IIg*?2=3#&Xa{DH!!(bOg<|4m)%p!IoR2cW3qva_~Zvt
z9LgNdu3kPuO#cfQk_#A97Ws=#PGKxyNGV`UC}2n|U`So$$302fmxFKO<V3FjlT#-L
zNINJkP?;Q~&k`T(6d&O1X%G<P8tfe88^&0W+yOEym7z3s@=<9&sRD+iMNTYJlTr(s
d!4gRf$<85ek;z=llb=hgGXu437L=L60sw(vH?sf$
delta 109
zcmV-z0FwXgL8(CsL{mgmsUQFV0pYO<2N?oIKa&#~umVFglf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRveZD8^j=y
diff --git a/tests/data/acpi/x86/q35/DSDT.pvpanic-isa b/tests/data/acpi/x86/q35/DSDT.pvpanic-isa
index 6ad42873e91c80cef5a42224cb4d31936dad59b4..02cc07f010f880684216ba8925c8f3f55cfd80aa 100644
GIT binary patch
delta 180
zcmeBhI_<>e66_M<t;oQ@xPK!TyCjp7^JHPk4NMLelaEUNW%rbF4tDnAm~0>;KKX$Z
zhcbt=tCx=u)Bgg7<O0T&MgF3bQy2>vQVJLo3K$X#7*ZGcaZi%=<=~q*Ig#uC<kZOl
z(hf=sR3^vhv&08G#RoWh8UzHn20I7&hA|c-cYq8_WhhOZd{o*`s(>MBkrT_*q|}0D
dutX9=vU7-AWHJ}?<mb}r%s?%h1!cao008^CH%tHk
delta 109
zcmV-z0FwX9LWn{NL{mgm2q6Ff0qL;{2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvh&N%k3a4
diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count b/tests/data/acpi/x86/q35/DSDT.thread-count
index a24b04cbdbf09383b933a42a2a15182545543a87..41c0832ab5041ff5361598813ec28fe7442b191b 100644
GIT binary patch
delta 168
zcmeyEvL%(vCD<ioixC3@W9vpPc1fm&gvr8^8<~1THXoC$VrKV}at?O(<CxqaBR)A!
zmP47t+11NOi0OX;LvjIQ$|47`$tjEl3@HVS2?Y#^1q`W+{JAIB$@+2dO`M#_^?!2e
z<nyu)%nMW|Pd1bj=3y*I?f~geWhhOZ>>}qURltz6$cbfYQffgnSR#ob**U~5GMS5c
Ra+;hvbC7S?<_U5Sg#j0XG*JKm
delta 134
zcmV;10D1qkW$|PRL{mgmaWVh^0k5$N2N?p5TayzRu>y)vv&I>E0|!SILsLUe2$O&u
z7n5Ec2pDhxV{idvqD2yuWC3shWN-msZ~$SVO$?KX9ZmxYlVOwJ9Y6%3H=s3>As!VR
oLsUjtV+sRs0bDNxUsFO~P(w*DP*O!xLsCsvlR+L%v(+Bo6Qh?Xg#Z8m
diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count2 b/tests/data/acpi/x86/q35/DSDT.thread-count2
index 3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8..153b45f0f7443d25cecc2a752fb6dbd921160e78 100644
GIT binary patch
delta 160
zcmaFW&a}3JiOVI}B}BJ{fr0VbMlN<qrq1%o!jc=A+Hy7@lT_nn@|K$XPEAFL!`ao#
zM~LZv0Yh>DW6B~&vB@cn1q>+#j0pt{i3JR)iyS6vst0iJO`M#_^?!2e<QjDcrUfdK
z52%ZB7bJIpG^a9@rcVB-?k7{gkhI8&WolAtK{Hq)i6PlJ#4R$Ji@6|q@;WtD<{;m&
J&9)kAqXG4WGspk{
delta 142
zcmZ46!St$~iOVI}CFE5z0|O)DMlN<qrnchA!jc=Anlm;ZlT_nn@(`Q+PEAF;fFZen
zF=dgP(Bu@x0)~_V#)JZf#6^CSP1FOJIVUGh?ooFzU1+|*baIS7OMI|Xe1Nm3K|qje
fuyc@a7^8EDTVyg9a{(h3$;t23{Wdpgtd0f%C_O9p
diff --git a/tests/data/acpi/x86/q35/DSDT.tis.tpm12 b/tests/data/acpi/x86/q35/DSDT.tis.tpm12
index e381ce4cbf2b11f56a2d0537db4d21acc97450c9..d0330d26a54b89c02a17b06ef5f55c72e28e406e 100644
GIT binary patch
delta 180
zcmZp4JL1OW66_M<qRha+XuFY%U6RSkd9twN1||oK$wwvsvU^H72Rr+5Og4}apZq|I
zLz%<b)yqeS>3;!3asgw?B7f1zDU1aSDFuuP1q_J=45^F!xF<>ba_~)@oXGWma_Zy&
zX$PeRDwAXMS>l78;scyL4FZB(gPntX!x#&aJ3xk|GL)uHJ}T`eRltz6$cbfYQffgn
cSR#ob**U~5GMS5c@^fi*W}ueMf-)c20H4n`x&QzG
delta 109
zcmV-z0FwX2MuA2OL{mgm0V4na0cWua2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvhjImmMFN
diff --git a/tests/data/acpi/x86/q35/DSDT.tis.tpm2 b/tests/data/acpi/x86/q35/DSDT.tis.tpm2
index a09253042ce4a715922027245de8a2ab7449c5b7..b05563deedc65df50f35b2399862d9ee8d4d1e0e 100644
GIT binary patch
delta 180
zcmbR3cF&E=CD<h-PMLv$@%u(Dc1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(9b@0{~2EHr)UK
delta 109
zcmV-z0FwXSMw><oL{mgm8zTS!0U@yp2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvZWin{Xbx
diff --git a/tests/data/acpi/x86/q35/DSDT.type4-count b/tests/data/acpi/x86/q35/DSDT.type4-count
index edc23198cdb47a981bcbc82bc8e392b815abb554..00807e7fd4d758bc2ab9c69ac8869cf6864399f7 100644
GIT binary patch
delta 200
zcmbO`k@3MqMlP3Nmyib@3=E8mHgd5`GW8sqEG)T^sdM}0W0I5D*uA8jgPr|2CNnCC
zYjZgJxrPfd{pWCY_3{DI1q{grj46v8#3rXO7BHj~FeVf*Bo;8FF7oG|JWaumgKy&G
zM6UmnQzt)Ha8O#HGC4+{B|g|GKET=2ARx#!*g42IjIkiO17un%Luu;d7)3v+0*0hT
wPApTCQVW{F5=jin&LM7*$z05n7b=KN)>9B<2Wl%#idLVTs35#~zv4YL0HE7HJpcdz
delta 163
zcmaDbk#X)sMlP3Nmyo$03=E97H*&E{GIj2oEG)T^scrM-W0I5D*xkjPgPr|2CNnCC
zD{wgbxrPfd{V!lhE?`Vq<SH~dg|UDkrGPP^fFW^_FVEzK3VzI-lM^R@S8y<0XuiO7
za*RGpe6Uk|fU~DTK#*&&bC7QsqjQK`WHJ|X0V5X4$&reFOwsC-6BUFvGb!Cu0{{}w
BF&_W`
diff --git a/tests/data/acpi/x86/q35/DSDT.viot b/tests/data/acpi/x86/q35/DSDT.viot
index 4c93dfd5c4b362714d3f9aa606a838d4625b3369..c3d83e67660ee3fd59f6fae6242270bed4a567f1 100644
GIT binary patch
delta 180
zcmbPIbiIhnCD<h-+>(KT@%~0Gc1b2D=gGp78<-p{CLfjj%kC-V9PI4JG1)*yeDVV+
z4rLB!S1%tSrvC*D$pwrli~L0=r!W>Uq!chF6fh(fFr+T><DMk#%fUBsaw6CN$*GeA
zq#cwNs7#L0XNeDXiVtx1GzbWC4R#Ll4Pz`w?f@B<%21j*`KYv?Q~^WMA}5xqNvQ?R
dV2LD#WakjK$Yd_&$<L+LnSok13(7Fd0swk8H!T1F
delta 109
zcmV-z0FwXLa+GokL{mgm6gdC@0SU1R2N?oSK9ds}umVjplf@bR0!9{-#2O?RZ~$X)
z0c4^?5|d;BZ~$a*0bp<dVWLb7lM5S70|}F1lVBS_1)(>fHItJ;91{ouLsUjtV+sS4
PKO7a4!W&MrRvZBxw>BRO
diff --git a/tests/data/acpi/x86/q35/DSDT.xapic b/tests/data/acpi/x86/q35/DSDT.xapic
index d4acd851c62c956436a436f9fa6d08fc5f370fa7..227d421f16ed1824a87e8a91da734828f8b48cbf 100644
GIT binary patch
delta 195
zcmZph&UA7*6PHV{OUTJ?1_s7U8@bpenOZ+i7M9$|)bwogG091M?A}t&!OnghlNmL{
zwK$ypT*HN!{&P6Hdien90*2%Q#*{^lVv|!C3m8%g7!wK@5(^ko7dcFxsu94!H*s<z
z*Z;|>lV4~!C@oN#9HY+?AM6w#;OuD-5ab%{9ON6uSdiQSGAxy$G<9;Urk_*+L((EA
rmZ?dp1<hcIB!*<?5Vy!=F6PPgn(FL8Eu~4(>XQ>Sgf}11JeCLm_4q#(
delta 162
zcmX>(ovCd)6PHV{OGsNc0|VpRja=-KOigbm3rlWf>VLTTnB*irb`LS<U}ry$$&4D}
z@*K{7uHiyV{|gwB3m8)txd}~9VJu)sDPT+}U`Sl#H+g|Z05j+0#K}K2984FQFEE`P
zqt6l_>=Ym1>}e1X<QnW8<QvB59O4$4%*9;5h(&U8l%^k3wEE;k4dKnqTE`LrM+q_U
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 0/3] workaround Windows always reading _DSM(func=7)
@ 2025-01-15 12:53 Igor Mammedov
2025-01-15 12:53 ` [PATCH 1/3] tests: acpi: whitelist expected blobs Igor Mammedov
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Igor Mammedov @ 2025-01-15 12:53 UTC (permalink / raw)
To: qemu-devel; +Cc: mst, anisinha
subj causes PCI devices that do not have acpi-index assigned,
having bogus 'PCI Label Id = 0'.
The later triggers another Windows bug that leads to guest hang
in case of localized Windows versions [1].
Workaround is for QEMU to return invalid Package on _DSM(func=7) request
if acpi-index is not configured, which makes Windows ignore returned value
and not assign 'PCI Label Id'.
It should help [1] users to boot localized Windows variants as long as
no PCI devices have acpi-index explicitly assigned.
1) https://gitlab.com/qemu-project/qemu/-/issues/774
Igor Mammedov (3):
tests: acpi: whitelist expected blobs
pci: acpi: Windows 'PCI Label Id' bug workaround
tests: acpi: update expected blobs
tests/qtest/bios-tables-test-allowed-diff.h | 40 ++++++++++++++++++
hw/i386/acpi-build.c | 33 ++++++++++-----
tests/data/acpi/x86/pc/DSDT | Bin 8526 -> 8544 bytes
tests/data/acpi/x86/pc/DSDT.acpierst | Bin 8437 -> 8455 bytes
tests/data/acpi/x86/pc/DSDT.acpihmat | Bin 9851 -> 9869 bytes
tests/data/acpi/x86/pc/DSDT.bridge | Bin 15397 -> 15415 bytes
tests/data/acpi/x86/pc/DSDT.cphp | Bin 8990 -> 9008 bytes
tests/data/acpi/x86/pc/DSDT.dimmpxm | Bin 10180 -> 10198 bytes
tests/data/acpi/x86/pc/DSDT.hpbridge | Bin 8477 -> 8495 bytes
tests/data/acpi/x86/pc/DSDT.ipmikcs | Bin 8598 -> 8616 bytes
tests/data/acpi/x86/pc/DSDT.memhp | Bin 9885 -> 9903 bytes
tests/data/acpi/x86/pc/DSDT.nohpet | Bin 8384 -> 8402 bytes
tests/data/acpi/x86/pc/DSDT.numamem | Bin 8532 -> 8550 bytes
tests/data/acpi/x86/pc/DSDT.roothp | Bin 12319 -> 12337 bytes
tests/data/acpi/x86/q35/DSDT | Bin 8355 -> 8373 bytes
tests/data/acpi/x86/q35/DSDT.acpierst | Bin 8372 -> 8390 bytes
tests/data/acpi/x86/q35/DSDT.acpihmat | Bin 9680 -> 9698 bytes
.../data/acpi/x86/q35/DSDT.acpihmat-generic-x | Bin 12565 -> 12583 bytes
.../acpi/x86/q35/DSDT.acpihmat-noinitiator | Bin 8634 -> 8652 bytes
tests/data/acpi/x86/q35/DSDT.applesmc | Bin 8401 -> 8419 bytes
tests/data/acpi/x86/q35/DSDT.bridge | Bin 11968 -> 11986 bytes
tests/data/acpi/x86/q35/DSDT.core-count | Bin 12913 -> 12931 bytes
tests/data/acpi/x86/q35/DSDT.core-count2 | Bin 33770 -> 33788 bytes
tests/data/acpi/x86/q35/DSDT.cphp | Bin 8819 -> 8837 bytes
tests/data/acpi/x86/q35/DSDT.cxl | Bin 13146 -> 13164 bytes
tests/data/acpi/x86/q35/DSDT.dimmpxm | Bin 10009 -> 10027 bytes
tests/data/acpi/x86/q35/DSDT.ipmibt | Bin 8430 -> 8448 bytes
tests/data/acpi/x86/q35/DSDT.ipmismbus | Bin 8443 -> 8461 bytes
tests/data/acpi/x86/q35/DSDT.ivrs | Bin 8372 -> 8390 bytes
tests/data/acpi/x86/q35/DSDT.memhp | Bin 9714 -> 9732 bytes
tests/data/acpi/x86/q35/DSDT.mmio64 | Bin 9485 -> 9503 bytes
tests/data/acpi/x86/q35/DSDT.multi-bridge | Bin 13208 -> 13226 bytes
tests/data/acpi/x86/q35/DSDT.nohpet | Bin 8213 -> 8231 bytes
tests/data/acpi/x86/q35/DSDT.numamem | Bin 8361 -> 8379 bytes
tests/data/acpi/x86/q35/DSDT.pvpanic-isa | Bin 8456 -> 8474 bytes
tests/data/acpi/x86/q35/DSDT.thread-count | Bin 12913 -> 12931 bytes
tests/data/acpi/x86/q35/DSDT.thread-count2 | Bin 33770 -> 33788 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm12 | Bin 8961 -> 8979 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm2 | Bin 8987 -> 9005 bytes
tests/data/acpi/x86/q35/DSDT.type4-count | Bin 18589 -> 18607 bytes
tests/data/acpi/x86/q35/DSDT.viot | Bin 14612 -> 14630 bytes
tests/data/acpi/x86/q35/DSDT.xapic | Bin 35718 -> 35736 bytes
42 files changed, 63 insertions(+), 10 deletions(-)
--
2.43.0
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/3] tests: acpi: whitelist expected blobs
2025-01-15 12:53 [PATCH 0/3] workaround Windows always reading _DSM(func=7) Igor Mammedov
@ 2025-01-15 12:53 ` Igor Mammedov
2025-01-15 14:09 ` Ani Sinha
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
2025-01-15 12:53 ` [PATCH 3/3] tests: acpi: update expected blobs Igor Mammedov
2 siblings, 1 reply; 9+ messages in thread
From: Igor Mammedov @ 2025-01-15 12:53 UTC (permalink / raw)
To: qemu-devel; +Cc: mst, anisinha
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
tests/qtest/bios-tables-test-allowed-diff.h | 40 +++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..085dfa9ff4 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,41 @@
/* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/x86/pc/DSDT",
+"tests/data/acpi/x86/pc/DSDT.acpierst",
+"tests/data/acpi/x86/pc/DSDT.acpihmat",
+"tests/data/acpi/x86/pc/DSDT.bridge",
+"tests/data/acpi/x86/pc/DSDT.cphp",
+"tests/data/acpi/x86/pc/DSDT.dimmpxm",
+"tests/data/acpi/x86/pc/DSDT.hpbridge",
+"tests/data/acpi/x86/pc/DSDT.ipmikcs",
+"tests/data/acpi/x86/pc/DSDT.memhp",
+"tests/data/acpi/x86/pc/DSDT.nohpet",
+"tests/data/acpi/x86/pc/DSDT.numamem",
+"tests/data/acpi/x86/pc/DSDT.roothp",
+"tests/data/acpi/x86/q35/DSDT",
+"tests/data/acpi/x86/q35/DSDT.acpierst",
+"tests/data/acpi/x86/q35/DSDT.acpihmat",
+"tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x",
+"tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator",
+"tests/data/acpi/x86/q35/DSDT.applesmc",
+"tests/data/acpi/x86/q35/DSDT.bridge",
+"tests/data/acpi/x86/q35/DSDT.core-count",
+"tests/data/acpi/x86/q35/DSDT.core-count2",
+"tests/data/acpi/x86/q35/DSDT.cphp",
+"tests/data/acpi/x86/q35/DSDT.cxl",
+"tests/data/acpi/x86/q35/DSDT.dimmpxm",
+"tests/data/acpi/x86/q35/DSDT.ipmibt",
+"tests/data/acpi/x86/q35/DSDT.ipmismbus",
+"tests/data/acpi/x86/q35/DSDT.ivrs",
+"tests/data/acpi/x86/q35/DSDT.memhp",
+"tests/data/acpi/x86/q35/DSDT.mmio64",
+"tests/data/acpi/x86/q35/DSDT.multi-bridge",
+"tests/data/acpi/x86/q35/DSDT.nohpet",
+"tests/data/acpi/x86/q35/DSDT.numamem",
+"tests/data/acpi/x86/q35/DSDT.pvpanic-isa",
+"tests/data/acpi/x86/q35/DSDT.thread-count",
+"tests/data/acpi/x86/q35/DSDT.thread-count2",
+"tests/data/acpi/x86/q35/DSDT.tis.tpm12",
+"tests/data/acpi/x86/q35/DSDT.tis.tpm2",
+"tests/data/acpi/x86/q35/DSDT.type4-count",
+"tests/data/acpi/x86/q35/DSDT.viot",
+"tests/data/acpi/x86/q35/DSDT.xapic",
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround
2025-01-15 12:53 [PATCH 0/3] workaround Windows always reading _DSM(func=7) Igor Mammedov
2025-01-15 12:53 ` [PATCH 1/3] tests: acpi: whitelist expected blobs Igor Mammedov
@ 2025-01-15 12:53 ` Igor Mammedov
2025-01-16 16:27 ` Fiona Ebner
` (2 more replies)
2025-01-15 12:53 ` [PATCH 3/3] tests: acpi: update expected blobs Igor Mammedov
2 siblings, 3 replies; 9+ messages in thread
From: Igor Mammedov @ 2025-01-15 12:53 UTC (permalink / raw)
To: qemu-devel; +Cc: mst, anisinha
Current versions of Windows call _DSM(func=7) regardless
of whether it is supported or not. It leads to NICs having bogus
'PCI Label Id = 0', where none should be set at all.
Also presence of 'PCI Label Id' triggers another Windows bug
on localized versions that leads to hangs. The later bug is fixed
in latest updates for 'Windows Server' but not in consumer
versions of Windows (and there is no plans to fix it
as far as I'm aware).
Given it's easy, implement Microsoft suggested workaround
(return invalid Package) so that affected Windows versions
could boot on QEMU.
This would effectvely remove bogus 'PCI Label Id's on NICs,
but MS teem confirmed that flipping 'PCI Label Id' should not
change 'Network Connection' ennumeration, so it should be safe
for QEMU to change _DSM without any compat code.
Smoke tested with WinXP and WS2022
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/774
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
hw/i386/acpi-build.c | 33 +++++++++++++++++++++++----------
1 file changed, 23 insertions(+), 10 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 733b8f0851..1311a0d4f3 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -654,6 +654,7 @@ static Aml *aml_pci_pdsm(void)
Aml *acpi_index = aml_local(2);
Aml *zero = aml_int(0);
Aml *one = aml_int(1);
+ Aml *not_supp = aml_int(0xFFFFFFFF);
Aml *func = aml_arg(2);
Aml *params = aml_arg(4);
Aml *bnum = aml_derefof(aml_index(params, aml_int(0)));
@@ -678,7 +679,7 @@ static Aml *aml_pci_pdsm(void)
*/
ifctx1 = aml_if(aml_lnot(
aml_or(aml_equal(acpi_index, zero),
- aml_equal(acpi_index, aml_int(0xFFFFFFFF)), NULL)
+ aml_equal(acpi_index, not_supp), NULL)
));
{
/* have supported functions */
@@ -704,18 +705,30 @@ static Aml *aml_pci_pdsm(void)
{
Aml *pkg = aml_package(2);
- aml_append(pkg, zero);
- /*
- * optional, if not impl. should return null string
- */
- aml_append(pkg, aml_string("%s", ""));
- aml_append(ifctx, aml_store(pkg, ret));
-
aml_append(ifctx, aml_store(aml_call2("AIDX", bnum, sunum), acpi_index));
+ aml_append(ifctx, aml_store(pkg, ret));
/*
- * update acpi-index to actual value
+ * Windows calls func=7 without checking if it's available,
+ * as workaround Microsoft has suggested to return invalid for func7
+ * Package, so return 2 elements package but only initialize elements
+ * when acpi_index is supported and leave them uninitialized, which
+ * leads elements to being Uninitialized ObjectType and should trip
+ * Windows into discarding result as an unexpected and prevent setting
+ * bogus 'PCI Label' on the device.
*/
- aml_append(ifctx, aml_store(acpi_index, aml_index(ret, zero)));
+ ifctx1 = aml_if(aml_lnot(aml_lor(
+ aml_equal(acpi_index, zero), aml_equal(acpi_index, not_supp)
+ )));
+ {
+ aml_append(ifctx1, aml_store(acpi_index, aml_index(ret, zero)));
+ /*
+ * optional, if not impl. should return null string
+ */
+ aml_append(ifctx1, aml_store(aml_string("%s", ""),
+ aml_index(ret, one)));
+ }
+ aml_append(ifctx, ifctx1);
+
aml_append(ifctx, aml_return(ret));
}
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 3/3] tests: acpi: update expected blobs
2025-01-15 12:53 [PATCH 0/3] workaround Windows always reading _DSM(func=7) Igor Mammedov
2025-01-15 12:53 ` [PATCH 1/3] tests: acpi: whitelist expected blobs Igor Mammedov
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
@ 2025-01-15 12:53 ` Igor Mammedov
2 siblings, 0 replies; 9+ messages in thread
From: Igor Mammedov @ 2025-01-15 12:53 UTC (permalink / raw)
To: qemu-devel; +Cc: mst, anisinha
_DSM function 7 AML should have followig change:
If ((Arg2 == 0x07))
{
- Local0 = Package (0x02)
- {
- Zero,
- ""
- }
Local2 = AIDX (DerefOf (Arg4 [Zero]), DerefOf (Arg4 [One]
))
- Local0 [Zero] = Local2
+ Local0 = Package (0x02) {}
+ If (!((Local2 == Zero) || (Local2 == 0xFFFFFFFF)))
+ {
+ Local0 [Zero] = Local2
+ Local0 [One] = ""
+ }
+
Return (Local0)
}
}
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
tests/data/acpi/x86/pc/DSDT | Bin 8526 -> 8544 bytes
tests/data/acpi/x86/pc/DSDT.acpierst | Bin 8437 -> 8455 bytes
tests/data/acpi/x86/pc/DSDT.acpihmat | Bin 9851 -> 9869 bytes
tests/data/acpi/x86/pc/DSDT.bridge | Bin 15397 -> 15415 bytes
tests/data/acpi/x86/pc/DSDT.cphp | Bin 8990 -> 9008 bytes
tests/data/acpi/x86/pc/DSDT.dimmpxm | Bin 10180 -> 10198 bytes
tests/data/acpi/x86/pc/DSDT.hpbridge | Bin 8477 -> 8495 bytes
tests/data/acpi/x86/pc/DSDT.ipmikcs | Bin 8598 -> 8616 bytes
tests/data/acpi/x86/pc/DSDT.memhp | Bin 9885 -> 9903 bytes
tests/data/acpi/x86/pc/DSDT.nohpet | Bin 8384 -> 8402 bytes
tests/data/acpi/x86/pc/DSDT.numamem | Bin 8532 -> 8550 bytes
tests/data/acpi/x86/pc/DSDT.roothp | Bin 12319 -> 12337 bytes
tests/data/acpi/x86/q35/DSDT | Bin 8355 -> 8373 bytes
tests/data/acpi/x86/q35/DSDT.acpierst | Bin 8372 -> 8390 bytes
tests/data/acpi/x86/q35/DSDT.acpihmat | Bin 9680 -> 9698 bytes
.../data/acpi/x86/q35/DSDT.acpihmat-generic-x | Bin 12565 -> 12583 bytes
.../acpi/x86/q35/DSDT.acpihmat-noinitiator | Bin 8634 -> 8652 bytes
tests/data/acpi/x86/q35/DSDT.applesmc | Bin 8401 -> 8419 bytes
tests/data/acpi/x86/q35/DSDT.bridge | Bin 11968 -> 11986 bytes
tests/data/acpi/x86/q35/DSDT.core-count | Bin 12913 -> 12931 bytes
tests/data/acpi/x86/q35/DSDT.core-count2 | Bin 33770 -> 33788 bytes
tests/data/acpi/x86/q35/DSDT.cphp | Bin 8819 -> 8837 bytes
tests/data/acpi/x86/q35/DSDT.cxl | Bin 13146 -> 13164 bytes
tests/data/acpi/x86/q35/DSDT.dimmpxm | Bin 10009 -> 10027 bytes
tests/data/acpi/x86/q35/DSDT.ipmibt | Bin 8430 -> 8448 bytes
tests/data/acpi/x86/q35/DSDT.ipmismbus | Bin 8443 -> 8461 bytes
tests/data/acpi/x86/q35/DSDT.ivrs | Bin 8372 -> 8390 bytes
tests/data/acpi/x86/q35/DSDT.memhp | Bin 9714 -> 9732 bytes
tests/data/acpi/x86/q35/DSDT.mmio64 | Bin 9485 -> 9503 bytes
tests/data/acpi/x86/q35/DSDT.multi-bridge | Bin 13208 -> 13226 bytes
tests/data/acpi/x86/q35/DSDT.nohpet | Bin 8213 -> 8231 bytes
tests/data/acpi/x86/q35/DSDT.numamem | Bin 8361 -> 8379 bytes
tests/data/acpi/x86/q35/DSDT.pvpanic-isa | Bin 8456 -> 8474 bytes
tests/data/acpi/x86/q35/DSDT.thread-count | Bin 12913 -> 12931 bytes
tests/data/acpi/x86/q35/DSDT.thread-count2 | Bin 33770 -> 33788 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm12 | Bin 8961 -> 8979 bytes
tests/data/acpi/x86/q35/DSDT.tis.tpm2 | Bin 8987 -> 9005 bytes
tests/data/acpi/x86/q35/DSDT.type4-count | Bin 18589 -> 18607 bytes
tests/data/acpi/x86/q35/DSDT.viot | Bin 14612 -> 14630 bytes
tests/data/acpi/x86/q35/DSDT.xapic | Bin 35718 -> 35736 bytes
40 files changed, 0 insertions(+), 0 deletions(-)
diff --git a/tests/data/acpi/x86/pc/DSDT b/tests/data/acpi/x86/pc/DSDT
index 8b8235fe79e2fa08a6f840c8479edb75f5a047b9..7fcbecc7991c9230cf2ab2ad7e33e2285655e2a1 100644
GIT binary patch
delta 89
zcmX@-^uUSBCD<h-L6L!hQEDRB113MwiEnl=xo}Nh#+YqlJ~@kvy};4aC8D__hk*e|
rFft?+2r)4wERdQsadHyF<RqT|Kv0m>k-)%Ez{}8)z_{6%Nm?EN8897b
delta 71
zcmaFhbk2#(CD<jzPmzIvF>@l<112AliEnl=IdM*2#+a?7IysAry+DYSiGi0Pp}^78
aC8D__hk*e|Fft?+By}V(Y<6XmmInX<(-dg{
diff --git a/tests/data/acpi/x86/pc/DSDT.acpierst b/tests/data/acpi/x86/pc/DSDT.acpierst
index 06829b9c6c6d726d955dc7c99bc9f42448e22aeb..5fb2b6376fdef3e9a8ce8861f8b333de01c154a8 100644
GIT binary patch
delta 89
zcmezB*zUyT66_MfuE@Z^m^6{=0h6ET#5X&bT(~ALW6U-&pPa?TUf}5I64Bg|!@vL}
r7#WfZgqWBT7D!E+I5~-7auUyfASg)cNMK+n;AQAYVBBoXbVv>W0=*r}
delta 71
zcmZp7`s&E#66_N4Re^zlannSu2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC%5NDcrkCKby7
diff --git a/tests/data/acpi/x86/pc/DSDT.acpihmat b/tests/data/acpi/x86/pc/DSDT.acpihmat
index 2fe355ebdbb858fa9247d09112e21712e3eddc45..51c70746c6c8ae33c48bf46e49a6f4ec2278feb4 100644
GIT binary patch
delta 89
zcmezE)9cIS66_MvtH!{<czq(*113MwiEnl=xo}Nh#+YqlJ~@kvy};4aC8D__hk*e|
rFft?+2r)4wERdQsadHyF<RqT|Kv0m>k-)%Ez{}8)z_{6%$wLJITN@sE
delta 71
zcmeD6{q4i$66_LEt;WE>s5X)70h5oX&boH!>hW6V}kot(wRULeHE#K6mtP~hn4
a64Bg|!@vL}7#WfZk~$I?HoG!;r~m-^6%=^@
diff --git a/tests/data/acpi/x86/pc/DSDT.bridge b/tests/data/acpi/x86/pc/DSDT.bridge
index 4d4067c182a6625db1e877408eb7436113884b50..ec920896faf05c5415ef4e485696a26a10d06d80 100644
GIT binary patch
delta 89
zcmZ2lvAu%JCD<jz+=hXHQDY+4113MwiEnl=xo}Nh#+YqlJ~@kvy};4aC8D__hk*e|
rFft?+2r)4wERdQsadHyF<RqT|Kv0m>k-)%Ez{}8)z_{6%>60Y@BO4xc
delta 71
zcmdm9v9yBACD<iI)rNtAv3w%e112AliEnl=IdM*2#+a?7IysAry+DYSiGi0Pp}^78
aC8D__hk*e|Fft?+By}V(Y<6Y(WC;KX>J@YV
diff --git a/tests/data/acpi/x86/pc/DSDT.cphp b/tests/data/acpi/x86/pc/DSDT.cphp
index 045a52e75b7fcd4e5f840a758c548231498b96e4..b4d0f151ce4c21d7dd93d02aad29643d3cf3aea5 100644
GIT binary patch
delta 89
zcmbQ|w!w|dCD<jzK$(Gov2G&Q113MwiEnl=xo}Nh#+YqlJ~@kvy};4aC8D__hk*e|
rFft?+2r)4wERdQsadHyF<RqT|Kv0m>k-)%Ez{}8)z_{6%>9qm?3v(VA
delta 71
zcmdnsHqVXACD<iIPMLv$@$5vd2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC&mS^)s|YZVv(
diff --git a/tests/data/acpi/x86/pc/DSDT.dimmpxm b/tests/data/acpi/x86/pc/DSDT.dimmpxm
index 205219b99d903555125c4b07fc047c42993eb338..4a0a7a14e262ac85b402e3719608cb93c408c35d 100644
GIT binary patch
delta 89
zcmX@&f6brECD<k8nmPjmqt`^P2TXpV6W{D$a^af1j4|89d~y~Sdx4{;OGI-=4g&*_
rU}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpN1j(?nGOP~sk!
delta 71
zcmccSf5e~5CD<k8h&lrU<IIU%514#JCcfFh<it678DqAR>f|gg_5vYRCI()HgaSuT
amx$($90mp;!N`zQkkpaDu-TPqqACC_0Tq`3
diff --git a/tests/data/acpi/x86/pc/DSDT.hpbridge b/tests/data/acpi/x86/pc/DSDT.hpbridge
index 8fa8b519ec65bd5099c45f4e1c85b11b47a23845..8a410533d48ec34b174dcfe487b1fcb33cbc27b3 100644
GIT binary patch
delta 89
zcmbR1wBCu!CD<iIUy*@<(QhKx113MwiEnl=xo}Nh#+YqlJ~@kvy};4aC8D__hk*e|
rFft?+2r)4wERdQsadHyF<RqT|Kv0m>k-)%Ez{}8)z_{6%>6IJ+`1&2#
delta 71
zcmZ4QG}npCCD<iIR*`{$aqdK}2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC&mN)7<%OBC1u
diff --git a/tests/data/acpi/x86/pc/DSDT.ipmikcs b/tests/data/acpi/x86/pc/DSDT.ipmikcs
index 0ca664688b16baa3a06b8440181de4f17511c6b0..20b1263496affb0e514a9d16ed75a59c0238ccd8 100644
GIT binary patch
delta 89
zcmbQ{yuz8wCD<iog(3q3<Ftuf519N!C%)Oi<ia(18DqAI`Q$7v_5w#wmx$($90mp;
r!N`zQAjHI!us~|k#K}nvlaqM<13^JjM*;&w0WU*G0^??5ret{lF2WuM
delta 71
zcmZ4CJk6QQCD<ionj!-O<I{;;514#JCcfFh<it678DqAR>f|gg_5vYRCI()HgaSuT
amx$($90mp;!N`zQkkpaDu-TO<SsnlpSrrEW
diff --git a/tests/data/acpi/x86/pc/DSDT.memhp b/tests/data/acpi/x86/pc/DSDT.memhp
index 03ff464ba4e72082fce0921815cfc09ca20b561a..a9ccae14e3a7111384a73bd92e49c2da9b52a96d 100644
GIT binary patch
delta 89
zcmbR1yWW?}CD<ioy&3}pBhN&x2TXpV6W{D$a^af1j4|89d~y~Sdx4{;OGI-=4g&*_
rU}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpN1jQ<e$<6}}y}
delta 71
zcmZ4QJJ*-XCD<iot{MXaWAsF>2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC%br2+u{z7)3r
diff --git a/tests/data/acpi/x86/pc/DSDT.nohpet b/tests/data/acpi/x86/pc/DSDT.nohpet
index b081030f0ed171e52b13e28cfdc8770a04c2806e..c1aa156a877f171b072aba51e82d38d11ad20f2f 100644
GIT binary patch
delta 89
zcmX@$c*&8=CD<k8k^%z*<CBS8519N!C%)Oi<ia(18DqAI`Q$7v_5w#wmx$($90mp;
r!N`zQAjHI!us~|k#K}nvlaqM<13^JjM*;&w0WU*G0^??5rd~M!ZbTmU
delta 71
zcmccQc)*d%CD<k8fC2*pqwz$p2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC&GD+d4x-4yo#
diff --git a/tests/data/acpi/x86/pc/DSDT.numamem b/tests/data/acpi/x86/pc/DSDT.numamem
index 2c98cafbff5db04410b35a1151eaf18723a4dad7..d207b18c7620965313a5aaf2398e745254619ab2 100644
GIT binary patch
delta 89
zcmccO^vsFNCD<h-O_70t@yJB32TXpV6W{D$a^af1j4|89d~y~Sdx4{;OGI-=4g&*_
rU}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpN1jlcGETQ3@U_
delta 71
zcmaFnbj69wCD<h-M3I4kQFtQP112AliEnl=IdM*2#+a?7IysAry+DYSiGi0Pp}^78
aC8D__hk*e|Fft?+By}V(Y<6W*lm`IwW)v#`
diff --git a/tests/data/acpi/x86/pc/DSDT.roothp b/tests/data/acpi/x86/pc/DSDT.roothp
index da018dca9e3102e811107994248719ab5278c505..947a9613303ffb5445a02aea586bd4a13b78ff50 100644
GIT binary patch
delta 89
zcmbQAurYzlCD<jz(13w~aoI$!2TXpV6W{D$a^af1j4|89d~y~Sdx4{;OGI-=4g&*_
rU}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpN1j(;Hm?E$JST
delta 71
zcmdm(Fh7CICD<iI-hhFD@#{pc2TVR96W{D$a^jr4j4@kDb#fLLdw~!u69X?pLV=^F
aOGI-=4g&*_U}Q)tNa{#n*zC&mMi&4PSQU~0
diff --git a/tests/data/acpi/x86/q35/DSDT b/tests/data/acpi/x86/q35/DSDT
index fb89ae0ac6d4346e33156e9e4d3718698a0a1a8e..3223604f7d9aa1e8df84826aa614b55a2ab91941 100644
GIT binary patch
delta 91
zcmZ4NxYd!%CD<ios{#W9<BN@43mKXGL?^Fh+`;6+H94Ir+r)fw78iSgqo+$mb4LyX
t1CU^3NGcFwVoF#bHEH7HB!<aJJpX~9AgLpPfuVqxp(BBD^B1P|vH(309_|1D
delta 73
zcmdn$xY&`)CD<iou>u1FqxnXzg^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dV_q){0OOhy?*IS*
diff --git a/tests/data/acpi/x86/q35/DSDT.acpierst b/tests/data/acpi/x86/q35/DSDT.acpierst
index 46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b..88dd156d9532100d2f9003c06077cc399a84d5f0 100644
GIT binary patch
delta 91
zcmdnuc+8Q@CD<k8m;wU>W5-6Wg^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3utlSpYG(9?}2+
delta 73
zcmX@+xW$pnCD<ioivj}!<MoYP3mKVwL?*9g+`;6;IXRsvTS;|t78iSg5GxY{FGE6s
cqo+$mb4LyX1CU^3NGeF`NMP8^#(Y2)02X@`(*OVf
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat b/tests/data/acpi/x86/q35/DSDT.acpihmat
index 61c5bd52a42242e85090934e8e45bf01642609d6..fe708b4403e9a0600f7562c9ce6a98ceb0168aae 100644
GIT binary patch
delta 91
zcmccM{m7fkCD<k8ktzcN<I;^>3mKXGL?^Fh+`;6+H94Ir+r)fw78iSgqo+$mb4LyX
t1CU^3NGcFwVoF#bHEH7HB!<aJJpX~9AgLpPfuVqxp(BBD^B1N&N&sKqAAJA-
delta 73
zcmaFleZiZ{CD<k8f+_<8<Cl$G3mKVwL?*9g+`;6;IXRsvTS;|t78iSg5GxY{FGE6s
cqo+$mb4LyX1CU^3NGeF`NMP8^#(YN!06Oj!egFUf
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x b/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x
index 497706c9742a9ea5396d6c9c4cc1cc2a4a530339..cc6e0a338540a90b6d2e4398a73db07483f79836 100644
GIT binary patch
delta 91
zcmbQ5v^<H+CD<iI-H?HSF=8XvLPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{DnzT4*>P09f|+|
delta 73
zcmZ3UG&PCKCD<iI)R2LJars8Bg^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dV^-7y0Ox!YivR!s
diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator b/tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator
index 3aaa2bbdf54a0d0cade14421e84c6ec5a42f96fa..a9a7cec8d7e2830275bfc1b69fb4fa116ecc884b 100644
GIT binary patch
delta 91
zcmdnxe8!o}CD<k8j3NUAW8Oxtg^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3uuAIRG)t9@YQ=
delta 73
zcmX@(yvv!(CD<iomm&iL<KB&23mKVwL?*9g+`;6;IXRsvTS;|t78iSg5GxY{FGE6s
cqo+$mb4LyX1CU^3NGeF`NMP8^#(Yc;02aj**8l(j
diff --git a/tests/data/acpi/x86/q35/DSDT.applesmc b/tests/data/acpi/x86/q35/DSDT.applesmc
index 944209adeaa5bbb722431161c404cb51b8209993..bebf7d716c85d677c34ad0d4a4cd5377d52e4052 100644
GIT binary patch
delta 91
zcmccU_}G!lCD<k8u>u1F<Bg463mKXGL?^Fh+`;6+H94Ir+r)fw78iSgqo+$mb4LyX
t1CU^3NGcFwVoF#bHEH7HB!<aJJpX~9AgLpPfuVqxp(BBD^B1POvH)kWAC>?B
delta 73
zcmaFtc+ru|CD<k8q5=a0qxwd!g^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dW4<d300p%amjD0&
diff --git a/tests/data/acpi/x86/q35/DSDT.bridge b/tests/data/acpi/x86/q35/DSDT.bridge
index d9938dba8fa5d405f7696c0dbdc24f3ae42ec934..74b8553af453b157ebc041bf9a1cf77d9751ca2e 100644
GIT binary patch
delta 91
zcmX>QdnuO7CD<k8k{$yCBg;mvg^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3ut-Z2&G<9?}2+
delta 73
zcmcZ<dmxs}CD<k8fF1(_WAH|<g^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dV?Lt|02J>O(*OVf
diff --git a/tests/data/acpi/x86/q35/DSDT.core-count b/tests/data/acpi/x86/q35/DSDT.core-count
index a24b04cbdbf09383b933a42a2a15182545543a87..d893781416b83305c3b0becc553f54fb04672fda 100644
GIT binary patch
delta 91
zcmeyE(wxfW66_MvY{bC8n6Qy+AtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=!tN4*)ER9*F<|
delta 73
zcmZop{g}e#66_LEXvDz4xPBwoLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKG1uw?01^KbiU0rr
diff --git a/tests/data/acpi/x86/q35/DSDT.core-count2 b/tests/data/acpi/x86/q35/DSDT.core-count2
index 3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8..ac87bc5db07e46285f4e42ca1d884905c8050cf6 100644
GIT binary patch
delta 93
zcmaFW&h)39iOVI}CFD;t0|Voxja&;Enfyd2uVvi9<ia&MohjSId~y~Sdx4{;OGI-=
v4g&*_U}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpPN}rf&@Z%sC+{
delta 75
zcmey<&h)CCiOVI}CFE5z0|O)DMy`d7Og<u$*D~&4a^jqv&XldBIysAry+DYSiGi0P
ep}^78C8D__hk*e|Fft?+By}V(Y-VHr)&KxjpcX3t
diff --git a/tests/data/acpi/x86/q35/DSDT.cphp b/tests/data/acpi/x86/q35/DSDT.cphp
index 20955d0aa30120553da35d5a6640055d26255cf9..6b8eca2586b874bb514fa400776006ae9a74fc25 100644
GIT binary patch
delta 91
zcmezD((20P66_Mvs>Hy+=(CY)AtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=!r*4*(4|9vuJx
delta 73
zcmZp5{p`Z!66_LEti-^;IC~@4LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKG1toj0Qoo+9smFU
diff --git a/tests/data/acpi/x86/q35/DSDT.cxl b/tests/data/acpi/x86/q35/DSDT.cxl
index 3c34d4dcab16783abe65f6fa5e64eb69d40795fb..6d71a26fab6cc0b55675b65835232a562c5ee32d 100644
GIT binary patch
delta 91
zcmcbW_9l(XCD<h-$C!bEQD!68LPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{Dmpa001sL9&P{t
delta 73
zcmaEpb}Nm`CD<h-%9w$HF?%D|LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF{c>-02K)oZvX%Q
diff --git a/tests/data/acpi/x86/q35/DSDT.dimmpxm b/tests/data/acpi/x86/q35/DSDT.dimmpxm
index 228374b55bd544116e359f659e546fc66cf8a895..5a0ff97b9e1a680ab0bcde0cce4272b15cb6c214 100644
GIT binary patch
delta 91
zcmbQ~x7v@(CD<iITb+S{QG6rULPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{DnzX1pw6?9UTAw
delta 73
zcmZ4OH`9;HCD<iIQk{W;F>NE)LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF{`Qo0M0QJ9smFU
diff --git a/tests/data/acpi/x86/q35/DSDT.ipmibt b/tests/data/acpi/x86/q35/DSDT.ipmibt
index 45f911ada5645f158f3d6c0c430ec1d52cadc5d8..a7e79e498335f3ffa1eb63c56b4a3bd9bcd5eed4 100644
GIT binary patch
delta 91
zcmaFo*x<zF66_Mfpvb_$D7}$uAtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=)Q2768uK9nSy&
delta 73
zcmZp0dgsXH66_N4PJw}eF?%D|LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKG5?YU00{FG&;S4c
diff --git a/tests/data/acpi/x86/q35/DSDT.ipmismbus b/tests/data/acpi/x86/q35/DSDT.ipmismbus
index e5d6811bee1233d74236453c49060390d74d4416..075ac613d2ba1348775c2ac6c38c5e85221b577b 100644
GIT binary patch
delta 91
zcmezE*z3gQ66_MftH{8>*t3yqAtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=&pA2LSv(9d!Tz
delta 73
zcmeBm`t8W&66_N4TY-Ur@y<rBg^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dV`i5F04u2!b^rhX
diff --git a/tests/data/acpi/x86/q35/DSDT.ivrs b/tests/data/acpi/x86/q35/DSDT.ivrs
index 46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b..88dd156d9532100d2f9003c06077cc399a84d5f0 100644
GIT binary patch
delta 91
zcmdnuc+8Q@CD<k8m;wU>W5-6Wg^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3utlSpYG(9?}2+
delta 73
zcmX@+xW$pnCD<ioivj}!<MoYP3mKVwL?*9g+`;6;IXRsvTS;|t78iSg5GxY{FGE6s
cqo+$mb4LyX1CU^3NGeF`NMP8^#(Y2)02X@`(*OVf
diff --git a/tests/data/acpi/x86/q35/DSDT.memhp b/tests/data/acpi/x86/q35/DSDT.memhp
index 5ce081187a578ba7145a9ba20d30be36c13b7663..812ed0fd18be86004ee4cb76222e8601909ecd46 100644
GIT binary patch
delta 91
zcmez5-QvUL66_MfqQ=0$cylAyLPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{DtYC5&#su9=iYl
delta 73
zcmZqi`Q**z66_N4NtJ<tQF9~LLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKG5=En00Yq!yZ`_I
diff --git a/tests/data/acpi/x86/q35/DSDT.mmio64 b/tests/data/acpi/x86/q35/DSDT.mmio64
index bdf36c4d575bfc4eb2eac3f00c9b7b4270f88677..3d5131761ced4fdb018671a363fd805a10251e8e 100644
GIT binary patch
delta 91
zcmeD6n(xKs66_Kpugbu{IBO%<LPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{DnzU2>|T39b^Ci
delta 73
zcmbR5)$7IO66_MftIEK@_;MrHLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF-s}|0N>IRWdHyG
diff --git a/tests/data/acpi/x86/q35/DSDT.multi-bridge b/tests/data/acpi/x86/q35/DSDT.multi-bridge
index 1db43a69e4c2affd8bd678bbef4d3c228380288e..a98567b923e88bdd400ac1acd32fdb169e7e77ca 100644
GIT binary patch
delta 91
zcmbP{zABx|CD<iol`#VY<KK;33mKXGL?^Fh+`;6+H94Ir+r)fw78iSgqo+$mb4LyX
t1CU^3NGcFwVoF#bHEH7HB!<aJJpX~9AgLpPfuVqxp(BBD^B1Ni1^`%#A4vcJ
delta 73
zcmZ3LJ|msWCD<iohA{&Jqt8aJg^Wx-B9qrL?qG7_oSe>-t)x0Ri;KNLh?R+fmm#6R
c(bFZOxg&>x0Z1@1Bo!oeBrt4dV_sqa0Qo-@N&o-=
diff --git a/tests/data/acpi/x86/q35/DSDT.nohpet b/tests/data/acpi/x86/q35/DSDT.nohpet
index c13e45e3612646cc2e30f00b3b7e53335da816ea..a550cf23c0de681b6afdd7998872d9165b1c5dee 100644
GIT binary patch
delta 91
zcmbR0u-t*mCD<iIU4emtapOj=g^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3sYxECBOV9en@*
delta 73
zcmZ4PFx7#}CD<iIRDpqkkzpg(LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF)PXf0IA~=egFUf
diff --git a/tests/data/acpi/x86/q35/DSDT.numamem b/tests/data/acpi/x86/q35/DSDT.numamem
index ba6669437e65952f24516ded954b33fe54bdedfb..22b2a21705762f7afab457e7dd2d5fd06f8d7c84 100644
GIT binary patch
delta 91
zcmZ4KxZ9D-CD<iow*mtL<D`vT3mKXGL?^Fh+`;6+H94Ir+r)fw78iSgqo+$mb4LyX
t1CU^3NGcFwVoF#bHEH7HB!<aJJpX~9AgLpPfuVqxp(BBD^B1PAvH&S!9<Tra
delta 73
zcmdn(xYCi!CD<ior2+#3<HL<y3mKVwL?*9g+`;6;IXRsvTS;|t78iSg5GxY{FGE6s
cqo+$mb4LyX1CU^3NGeF`NMP8^#=KP)01v|yu>b%7
diff --git a/tests/data/acpi/x86/q35/DSDT.pvpanic-isa b/tests/data/acpi/x86/q35/DSDT.pvpanic-isa
index 6ad42873e91c80cef5a42224cb4d31936dad59b4..c26f9b1ba6f0c949e509c0028c7888ca6e4a768a 100644
GIT binary patch
delta 91
zcmeBhn&rgh66_KprO3d*IBO%<LPjP((aCEWcQCndO-^UZHZh-^#l>FW=;;#C+>yh;
t03;Y0k_v>Fm=YF9O`144iD7aQ&wn5&Na{#nU?|{a=tyAP{DnzG4glb{9XJ30
delta 73
zcmbQ`)ZxVC66_Mfp~%3%_;MrHLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF^k9n0M<woIRF3v
diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count b/tests/data/acpi/x86/q35/DSDT.thread-count
index a24b04cbdbf09383b933a42a2a15182545543a87..d893781416b83305c3b0becc553f54fb04672fda 100644
GIT binary patch
delta 91
zcmeyE(wxfW66_MvY{bC8n6Qy+AtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=!tN4*)ER9*F<|
delta 73
zcmZop{g}e#66_LEXvDz4xPBwoLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKG1uw?01^KbiU0rr
diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count2 b/tests/data/acpi/x86/q35/DSDT.thread-count2
index 3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8..ac87bc5db07e46285f4e42ca1d884905c8050cf6 100644
GIT binary patch
delta 93
zcmaFW&h)39iOVI}CFD;t0|Voxja&;Enfyd2uVvi9<ia&MohjSId~y~Sdx4{;OGI-=
v4g&*_U}Q)t5Mp9VSRgfN;^ZWT$w@r_fuJC%BY}aTfR~{ofpPN}rf&@Z%sC+{
delta 75
zcmey<&h)CCiOVI}CFE5z0|O)DMy`d7Og<u$*D~&4a^jqv&XldBIysAry+DYSiGi0P
ep}^78C8D__hk*e|Fft?+By}V(Y-VHr)&KxjpcX3t
diff --git a/tests/data/acpi/x86/q35/DSDT.tis.tpm12 b/tests/data/acpi/x86/q35/DSDT.tis.tpm12
index e381ce4cbf2b11f56a2d0537db4d21acc97450c9..0a91b7ba857d508113c7df8e9c8072ff73c3d6c3 100644
GIT binary patch
delta 91
zcmZp4o9xEr66_Kptjxf`D6x@iAtRHY=;XDGJD6O!CZ{uHo0w0|;$kmw^mK`6?#N+a
t01}K0Nd-boObH95CQY22#4tID=RXh>By}V(Fck1IbR;lt{=&qo006bo9HsyO
delta 73
zcmbR2*67CN66_MfsLa5?n7)x~AtRHI$mF$*JD8j}C#N%IE2&P-;$kllVr63BWk@J+
b^mK`6?#N+a01}K0Nd-wA2@IRrn0XZdwM-JH
diff --git a/tests/data/acpi/x86/q35/DSDT.tis.tpm2 b/tests/data/acpi/x86/q35/DSDT.tis.tpm2
index a09253042ce4a715922027245de8a2ab7449c5b7..b9c83bea1fb891eea31ab63f175a8898408ef596 100644
GIT binary patch
delta 91
zcmbR3w$_cyCD<iISDAr<@zO@Fg^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3sY}0ssi29moIx
delta 73
zcmZ4MHrtKMCD<iITA6`?QE?;JLPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF{>*80J?q>$p8QV
diff --git a/tests/data/acpi/x86/q35/DSDT.type4-count b/tests/data/acpi/x86/q35/DSDT.type4-count
index edc23198cdb47a981bcbc82bc8e392b815abb554..16b46775879d9a28090220e2ecd93fcba20a9c1b 100644
GIT binary patch
delta 93
zcmbO`k#YS*MlP3Nmyq=y3=E7t8@U!TGWm&4Udy<H$%SijI#afZ`Q$7v_5w#wmx$($
v90mp;!N`zQAjHI!us~|k#K}nvlaqM<13^JjM*;&w0WU*G0^{Z{Oe<XhWAz_w
delta 75
zcmZ2Kk#X)sMlP3Nmyo$03=E97H*zgxWbzT2yq0kXlN0CUbf#=2)yY|0>;*!sObomX
e2?dUxE)mTgISdRyf{`JqAgLpPVKW=^N>>0sC>3o0
diff --git a/tests/data/acpi/x86/q35/DSDT.viot b/tests/data/acpi/x86/q35/DSDT.viot
index 4c93dfd5c4b362714d3f9aa606a838d4625b3369..7cdde13b45d322855cfbb3894165d5c60dd20fa7 100644
GIT binary patch
delta 91
zcmbPIw5*8BCD<iI&60tEao0w!g^WyoqLbG$?qG7^nw-v*ZDKw-i;KO$(bFZOxg&>x
t0Z1@1BozoTF(oXJnly2862s&qp8r5lkkpaDz)--;(2>Bn`3sYR82}&F9vA=s
delta 73
zcmZ2hG^L2kCD<iI#FBx5k#i&0LPjPZk;!WrcQ83|PEKdaR#Kgu#l>DA#LC3L%aBmu
c=;;#C+>yh;03;Y0k_wVK5*RkKF)Nq>0L>s182|tP
diff --git a/tests/data/acpi/x86/q35/DSDT.xapic b/tests/data/acpi/x86/q35/DSDT.xapic
index d4acd851c62c956436a436f9fa6d08fc5f370fa7..5a6310f45371c734fb03ac27bb864457eaeee7ad 100644
GIT binary patch
delta 93
zcmZph&NO2>6PHV{OUR6F1_s7?8@U!TGWm&4Udy<H$%SijI#afZ`Q$7v_5w#wmx$($
v90mp;!N`zQAjHI!us~|k#K}nvlaqM<13^JjM*;&w0WU*G0^{Z{Op`hQYsnwB
delta 75
zcmbO+ovCd)6PHV{OGsNc0|VpRja&;EnS4YhuVvi9<it5Sohe&Mb#fLLdw~!u69X?p
eLV=^FOGI-=4g&*_U}Q)tNa{#n*v!T}sRIB+;1#z3
--
2.43.0
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] tests: acpi: whitelist expected blobs
2025-01-15 12:53 ` [PATCH 1/3] tests: acpi: whitelist expected blobs Igor Mammedov
@ 2025-01-15 14:09 ` Ani Sinha
0 siblings, 0 replies; 9+ messages in thread
From: Ani Sinha @ 2025-01-15 14:09 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-devel, mst
On Wed, Jan 15, 2025 at 6:23 PM Igor Mammedov <imammedo@redhat.com> wrote:
>
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Acked-by: Ani Sinha <anisinha@redhat.com>
> ---
> tests/qtest/bios-tables-test-allowed-diff.h | 40 +++++++++++++++++++++
> 1 file changed, 40 insertions(+)
>
> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> index dfb8523c8b..085dfa9ff4 100644
> --- a/tests/qtest/bios-tables-test-allowed-diff.h
> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> @@ -1 +1,41 @@
> /* List of comma-separated changed AML files to ignore */
> +"tests/data/acpi/x86/pc/DSDT",
> +"tests/data/acpi/x86/pc/DSDT.acpierst",
> +"tests/data/acpi/x86/pc/DSDT.acpihmat",
> +"tests/data/acpi/x86/pc/DSDT.bridge",
> +"tests/data/acpi/x86/pc/DSDT.cphp",
> +"tests/data/acpi/x86/pc/DSDT.dimmpxm",
> +"tests/data/acpi/x86/pc/DSDT.hpbridge",
> +"tests/data/acpi/x86/pc/DSDT.ipmikcs",
> +"tests/data/acpi/x86/pc/DSDT.memhp",
> +"tests/data/acpi/x86/pc/DSDT.nohpet",
> +"tests/data/acpi/x86/pc/DSDT.numamem",
> +"tests/data/acpi/x86/pc/DSDT.roothp",
> +"tests/data/acpi/x86/q35/DSDT",
> +"tests/data/acpi/x86/q35/DSDT.acpierst",
> +"tests/data/acpi/x86/q35/DSDT.acpihmat",
> +"tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x",
> +"tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator",
> +"tests/data/acpi/x86/q35/DSDT.applesmc",
> +"tests/data/acpi/x86/q35/DSDT.bridge",
> +"tests/data/acpi/x86/q35/DSDT.core-count",
> +"tests/data/acpi/x86/q35/DSDT.core-count2",
> +"tests/data/acpi/x86/q35/DSDT.cphp",
> +"tests/data/acpi/x86/q35/DSDT.cxl",
> +"tests/data/acpi/x86/q35/DSDT.dimmpxm",
> +"tests/data/acpi/x86/q35/DSDT.ipmibt",
> +"tests/data/acpi/x86/q35/DSDT.ipmismbus",
> +"tests/data/acpi/x86/q35/DSDT.ivrs",
> +"tests/data/acpi/x86/q35/DSDT.memhp",
> +"tests/data/acpi/x86/q35/DSDT.mmio64",
> +"tests/data/acpi/x86/q35/DSDT.multi-bridge",
> +"tests/data/acpi/x86/q35/DSDT.nohpet",
> +"tests/data/acpi/x86/q35/DSDT.numamem",
> +"tests/data/acpi/x86/q35/DSDT.pvpanic-isa",
> +"tests/data/acpi/x86/q35/DSDT.thread-count",
> +"tests/data/acpi/x86/q35/DSDT.thread-count2",
> +"tests/data/acpi/x86/q35/DSDT.tis.tpm12",
> +"tests/data/acpi/x86/q35/DSDT.tis.tpm2",
> +"tests/data/acpi/x86/q35/DSDT.type4-count",
> +"tests/data/acpi/x86/q35/DSDT.viot",
> +"tests/data/acpi/x86/q35/DSDT.xapic",
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
@ 2025-01-16 16:27 ` Fiona Ebner
2025-01-17 6:29 ` Michael Tokarev
2025-01-17 7:05 ` Ani Sinha
2 siblings, 0 replies; 9+ messages in thread
From: Fiona Ebner @ 2025-01-16 16:27 UTC (permalink / raw)
To: Igor Mammedov, qemu-devel; +Cc: mst, anisinha
Am 15.01.25 um 13:53 schrieb Igor Mammedov:
> Current versions of Windows call _DSM(func=7) regardless
> of whether it is supported or not. It leads to NICs having bogus
> 'PCI Label Id = 0', where none should be set at all.
>
> Also presence of 'PCI Label Id' triggers another Windows bug
> on localized versions that leads to hangs. The later bug is fixed
> in latest updates for 'Windows Server' but not in consumer
> versions of Windows (and there is no plans to fix it
> as far as I'm aware).
>
> Given it's easy, implement Microsoft suggested workaround
> (return invalid Package) so that affected Windows versions
> could boot on QEMU.
> This would effectvely remove bogus 'PCI Label Id's on NICs,
> but MS teem confirmed that flipping 'PCI Label Id' should not
> change 'Network Connection' ennumeration, so it should be safe
> for QEMU to change _DSM without any compat code.
>
> Smoke tested with WinXP and WS2022
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/774
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Tested-by: Fiona Ebner <f.ebner@proxmox.com>
Fixes the VirtIO NIC issue with a German Windows 10 guest for me.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
2025-01-16 16:27 ` Fiona Ebner
@ 2025-01-17 6:29 ` Michael Tokarev
2025-01-17 7:05 ` Ani Sinha
2 siblings, 0 replies; 9+ messages in thread
From: Michael Tokarev @ 2025-01-17 6:29 UTC (permalink / raw)
To: Igor Mammedov, qemu-devel; +Cc: mst, anisinha, qemu-stable
15.01.2025 15:53, Igor Mammedov wrote:
> Current versions of Windows call _DSM(func=7) regardless
> of whether it is supported or not. It leads to NICs having bogus
> 'PCI Label Id = 0', where none should be set at all.
>
> Also presence of 'PCI Label Id' triggers another Windows bug
> on localized versions that leads to hangs. The later bug is fixed
> in latest updates for 'Windows Server' but not in consumer
> versions of Windows (and there is no plans to fix it
> as far as I'm aware).
>
> Given it's easy, implement Microsoft suggested workaround
> (return invalid Package) so that affected Windows versions
> could boot on QEMU.
> This would effectvely remove bogus 'PCI Label Id's on NICs,
> but MS teem confirmed that flipping 'PCI Label Id' should not
> change 'Network Connection' ennumeration, so it should be safe
> for QEMU to change _DSM without any compat code.
While this is not a qemu bug fix, this change feels like a good
candidate for qemu-stable, - what do you think? I picked it up
for current stable series, which are 7.2, 8.2, 9.1 and 9.2.
Thanks,
/mjt
> Smoke tested with WinXP and WS2022
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/774
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> ---
> hw/i386/acpi-build.c | 33 +++++++++++++++++++++++----------
> 1 file changed, 23 insertions(+), 10 deletions(-)
>
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index 733b8f0851..1311a0d4f3 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -654,6 +654,7 @@ static Aml *aml_pci_pdsm(void)
> Aml *acpi_index = aml_local(2);
> Aml *zero = aml_int(0);
> Aml *one = aml_int(1);
> + Aml *not_supp = aml_int(0xFFFFFFFF);
> Aml *func = aml_arg(2);
> Aml *params = aml_arg(4);
> Aml *bnum = aml_derefof(aml_index(params, aml_int(0)));
> @@ -678,7 +679,7 @@ static Aml *aml_pci_pdsm(void)
> */
> ifctx1 = aml_if(aml_lnot(
> aml_or(aml_equal(acpi_index, zero),
> - aml_equal(acpi_index, aml_int(0xFFFFFFFF)), NULL)
> + aml_equal(acpi_index, not_supp), NULL)
> ));
> {
> /* have supported functions */
> @@ -704,18 +705,30 @@ static Aml *aml_pci_pdsm(void)
> {
> Aml *pkg = aml_package(2);
>
> - aml_append(pkg, zero);
> - /*
> - * optional, if not impl. should return null string
> - */
> - aml_append(pkg, aml_string("%s", ""));
> - aml_append(ifctx, aml_store(pkg, ret));
> -
> aml_append(ifctx, aml_store(aml_call2("AIDX", bnum, sunum), acpi_index));
> + aml_append(ifctx, aml_store(pkg, ret));
> /*
> - * update acpi-index to actual value
> + * Windows calls func=7 without checking if it's available,
> + * as workaround Microsoft has suggested to return invalid for func7
> + * Package, so return 2 elements package but only initialize elements
> + * when acpi_index is supported and leave them uninitialized, which
> + * leads elements to being Uninitialized ObjectType and should trip
> + * Windows into discarding result as an unexpected and prevent setting
> + * bogus 'PCI Label' on the device.
> */
> - aml_append(ifctx, aml_store(acpi_index, aml_index(ret, zero)));
> + ifctx1 = aml_if(aml_lnot(aml_lor(
> + aml_equal(acpi_index, zero), aml_equal(acpi_index, not_supp)
> + )));
> + {
> + aml_append(ifctx1, aml_store(acpi_index, aml_index(ret, zero)));
> + /*
> + * optional, if not impl. should return null string
> + */
> + aml_append(ifctx1, aml_store(aml_string("%s", ""),
> + aml_index(ret, one)));
> + }
> + aml_append(ifctx, ifctx1);
> +
> aml_append(ifctx, aml_return(ret));
> }
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
2025-01-16 16:27 ` Fiona Ebner
2025-01-17 6:29 ` Michael Tokarev
@ 2025-01-17 7:05 ` Ani Sinha
2 siblings, 0 replies; 9+ messages in thread
From: Ani Sinha @ 2025-01-17 7:05 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-devel, mst
On Wed, Jan 15, 2025 at 6:23 PM Igor Mammedov <imammedo@redhat.com> wrote:
>
> Current versions of Windows call _DSM(func=7) regardless
> of whether it is supported or not. It leads to NICs having bogus
> 'PCI Label Id = 0', where none should be set at all.
>
> Also presence of 'PCI Label Id' triggers another Windows bug
> on localized versions that leads to hangs. The later bug is fixed
> in latest updates for 'Windows Server' but not in consumer
> versions of Windows (and there is no plans to fix it
> as far as I'm aware).
>
> Given it's easy, implement Microsoft suggested workaround
> (return invalid Package) so that affected Windows versions
> could boot on QEMU.
> This would effectvely remove bogus 'PCI Label Id's on NICs,
> but MS teem confirmed that flipping 'PCI Label Id' should not
> change 'Network Connection' ennumeration, so it should be safe
> for QEMU to change _DSM without any compat code.
>
> Smoke tested with WinXP and WS2022
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/774
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> ---
> hw/i386/acpi-build.c | 33 +++++++++++++++++++++++----------
> 1 file changed, 23 insertions(+), 10 deletions(-)
>
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index 733b8f0851..1311a0d4f3 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -654,6 +654,7 @@ static Aml *aml_pci_pdsm(void)
> Aml *acpi_index = aml_local(2);
> Aml *zero = aml_int(0);
> Aml *one = aml_int(1);
> + Aml *not_supp = aml_int(0xFFFFFFFF);
> Aml *func = aml_arg(2);
> Aml *params = aml_arg(4);
> Aml *bnum = aml_derefof(aml_index(params, aml_int(0)));
> @@ -678,7 +679,7 @@ static Aml *aml_pci_pdsm(void)
> */
> ifctx1 = aml_if(aml_lnot(
> aml_or(aml_equal(acpi_index, zero),
> - aml_equal(acpi_index, aml_int(0xFFFFFFFF)), NULL)
> + aml_equal(acpi_index, not_supp), NULL)
> ));
> {
> /* have supported functions */
> @@ -704,18 +705,30 @@ static Aml *aml_pci_pdsm(void)
> {
> Aml *pkg = aml_package(2);
>
> - aml_append(pkg, zero);
> - /*
> - * optional, if not impl. should return null string
> - */
> - aml_append(pkg, aml_string("%s", ""));
> - aml_append(ifctx, aml_store(pkg, ret));
> -
> aml_append(ifctx, aml_store(aml_call2("AIDX", bnum, sunum), acpi_index));
> + aml_append(ifctx, aml_store(pkg, ret));
> /*
> - * update acpi-index to actual value
> + * Windows calls func=7 without checking if it's available,
> + * as workaround Microsoft has suggested to return invalid for func7
> + * Package, so return 2 elements package but only initialize elements
> + * when acpi_index is supported and leave them uninitialized, which
> + * leads elements to being Uninitialized ObjectType and should trip
> + * Windows into discarding result as an unexpected and prevent setting
> + * bogus 'PCI Label' on the device.
This comment is very confusing!
> */
> - aml_append(ifctx, aml_store(acpi_index, aml_index(ret, zero)));
> + ifctx1 = aml_if(aml_lnot(aml_lor(
> + aml_equal(acpi_index, zero), aml_equal(acpi_index, not_supp)
> + )));
So this conditional checks if the acpi index is supported (because its
aml_lnot()).
> + {
> + aml_append(ifctx1, aml_store(acpi_index, aml_index(ret, zero)));
> + /*
> + * optional, if not impl. should return null string
> + */
I know this comes from the existing code but I am still confused. Why
is this appending "return null string" logic to "if acpi index is
supprted" conditional?
> + aml_append(ifctx1, aml_store(aml_string("%s", ""),
> + aml_index(ret, one)));
> + }
> + aml_append(ifctx, ifctx1);
> +
> aml_append(ifctx, aml_return(ret));
> }
>
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2025-01-17 7:06 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-15 12:53 [PATCH 0/3] workaround Windows always reading _DSM(func=7) Igor Mammedov
2025-01-15 12:53 ` [PATCH 1/3] tests: acpi: whitelist expected blobs Igor Mammedov
2025-01-15 14:09 ` Ani Sinha
2025-01-15 12:53 ` [PATCH 2/3] pci: acpi: Windows 'PCI Label Id' bug workaround Igor Mammedov
2025-01-16 16:27 ` Fiona Ebner
2025-01-17 6:29 ` Michael Tokarev
2025-01-17 7:05 ` Ani Sinha
2025-01-15 12:53 ` [PATCH 3/3] tests: acpi: update expected blobs Igor Mammedov
-- strict thread matches above, loose matches on Subject: below --
2024-12-10 16:39 [PATCH 0/3] cpuhp: ensure that cpu hotremove works the 1st time Igor Mammedov
2024-12-10 16:39 ` [PATCH 3/3] tests: acpi: update expected blobs Igor Mammedov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.