[PATCH 0/6] Add bios-tables-test for LoongArch64 system

[PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Bibo Mao]

This patchset add bios-tables-test for LoongArch64 virt machine
system. It works with UEFI bios, with uefi-test-tools LoongArch64
support is added to build bios-tables-test.loongarch64.iso.

Also with test case bios-tables-test, LoongArch64 support is added
and some basic ACPI tables is added here. It passes with command
make check-qtest-loongarch64 with the result:
 qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed

Bibo Mao (6):
  uefi-test-tools:: Add LoongArch64 support
  tests/data/uefi-boot-images: Add ISO image for LoongArch system
  tests/qtest/bios-tables-test: Add basic testing for LoongArch64
  tests/acpi: Add empty ACPI data files for LoongArch64
  tests/acpi: Fill acpi table data for LoongArch
  tests/qtest: Enable bios-tables-test for LoongArch

 tests/data/acpi/loongarch64/virt/APIC         | Bin 0 -> 108 bytes
 .../data/acpi/loongarch64/virt/APIC.topology  | Bin 0 -> 213 bytes
 tests/data/acpi/loongarch64/virt/DSDT         | Bin 0 -> 3614 bytes
 tests/data/acpi/loongarch64/virt/DSDT.memhp   | Bin 0 -> 4951 bytes
 tests/data/acpi/loongarch64/virt/FACP         | Bin 0 -> 268 bytes
 tests/data/acpi/loongarch64/virt/MCFG         | Bin 0 -> 60 bytes
 tests/data/acpi/loongarch64/virt/PPTT         | Bin 0 -> 76 bytes
 .../data/acpi/loongarch64/virt/PPTT.topology  | Bin 0 -> 296 bytes
 tests/data/acpi/loongarch64/virt/SLIT         |   0
 tests/data/acpi/loongarch64/virt/SLIT.memhp   | Bin 0 -> 48 bytes
 tests/data/acpi/loongarch64/virt/SPCR         | Bin 0 -> 80 bytes
 tests/data/acpi/loongarch64/virt/SRAT         | Bin 0 -> 104 bytes
 tests/data/acpi/loongarch64/virt/SRAT.memhp   | Bin 0 -> 184 bytes
 .../data/acpi/loongarch64/virt/SRAT.topology  | Bin 0 -> 216 bytes
 .../bios-tables-test.loongarch64.iso.qcow2    | Bin 0 -> 12800 bytes
 tests/qtest/bios-tables-test.c                |  62 ++++++++++++++++++
 tests/qtest/meson.build                       |   3 +-
 tests/uefi-test-tools/Makefile                |   5 +-
 .../UefiTestToolsPkg/UefiTestToolsPkg.dsc     |   6 +-
 tests/uefi-test-tools/uefi-test-build.config  |  10 +++
 20 files changed, 82 insertions(+), 4 deletions(-)
 create mode 100644 tests/data/acpi/loongarch64/virt/APIC
 create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
 create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
 create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/FACP
 create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
 create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
 create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
 create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
 create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology
 create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2


base-commit: b69801dd6b1eb4d107f7c2f643adf0a4e3ec9124
-- 
2.39.3

[PATCH 1/6] uefi-test-tools:: Add LoongArch64 support

[Bibo Mao]

Add support to build bios-tables-test iso image for LoongArch system.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 tests/uefi-test-tools/Makefile                         |  5 +++--
 .../UefiTestToolsPkg/UefiTestToolsPkg.dsc              |  6 +++++-
 tests/uefi-test-tools/uefi-test-build.config           | 10 ++++++++++
 3 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/tests/uefi-test-tools/Makefile b/tests/uefi-test-tools/Makefile
index f4eaebd8ff..8ee6fb3571 100644
--- a/tests/uefi-test-tools/Makefile
+++ b/tests/uefi-test-tools/Makefile
@@ -12,7 +12,7 @@
 
 edk2_dir              := ../../roms/edk2
 images_dir            := ../data/uefi-boot-images
-emulation_targets     := arm aarch64 i386 x86_64 riscv64
+emulation_targets     := arm aarch64 i386 x86_64 riscv64 loongarch64
 uefi_binaries         := bios-tables-test
 intermediate_suffixes := .efi .fat .iso.raw
 
@@ -56,7 +56,8 @@ Build/%.iso.raw: Build/%.fat
 # stripped from, the argument.
 map_arm_to_uefi     = $(subst arm,ARM,$(1))
 map_aarch64_to_uefi = $(subst aarch64,AA64,$(call map_arm_to_uefi,$(1)))
-map_riscv64_to_uefi = $(subst riscv64,RISCV64,$(call map_aarch64_to_uefi,$(1)))
+map_loongarch64_to_uefi = $(subst loongarch64,LOONGARCH64,$(call map_aarch64_to_uefi,$(1)))
+map_riscv64_to_uefi = $(subst riscv64,RISCV64,$(call map_loongarch64_to_uefi,$(1)))
 map_i386_to_uefi    = $(subst i386,IA32,$(call map_riscv64_to_uefi,$(1)))
 map_x86_64_to_uefi  = $(subst x86_64,X64,$(call map_i386_to_uefi,$(1)))
 map_to_uefi         = $(subst .,,$(call map_x86_64_to_uefi,$(1)))
diff --git a/tests/uefi-test-tools/UefiTestToolsPkg/UefiTestToolsPkg.dsc b/tests/uefi-test-tools/UefiTestToolsPkg/UefiTestToolsPkg.dsc
index 0902fd3c73..facf8df1fa 100644
--- a/tests/uefi-test-tools/UefiTestToolsPkg/UefiTestToolsPkg.dsc
+++ b/tests/uefi-test-tools/UefiTestToolsPkg/UefiTestToolsPkg.dsc
@@ -19,7 +19,7 @@
   PLATFORM_VERSION        = 0.1
   PLATFORM_NAME           = UefiTestTools
   SKUID_IDENTIFIER        = DEFAULT
-  SUPPORTED_ARCHITECTURES = ARM|AARCH64|IA32|X64|RISCV64
+  SUPPORTED_ARCHITECTURES = ARM|AARCH64|IA32|X64|RISCV64|LOONGARCH64
   BUILD_TARGETS           = DEBUG
 
 [BuildOptions.IA32]
@@ -65,6 +65,10 @@
 [LibraryClasses.RISCV64]
   BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
 
+[LibraryClasses.LOONGARCH64]
+  BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
+  StackCheckLib|MdePkg/Library/StackCheckLibNull/StackCheckLibNull.inf
+
 [PcdsFixedAtBuild]
   gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8040004F
   gEfiMdePkgTokenSpaceGuid.PcdDebugPropertyMask|0x2F
diff --git a/tests/uefi-test-tools/uefi-test-build.config b/tests/uefi-test-tools/uefi-test-build.config
index a4c61fc97a..8bf4826634 100644
--- a/tests/uefi-test-tools/uefi-test-build.config
+++ b/tests/uefi-test-tools/uefi-test-build.config
@@ -21,6 +21,16 @@ dest = ./Build
 arch = AARCH64
 cpy1 = AARCH64/BiosTablesTest.efi  bios-tables-test.aarch64.efi
 
+####################################################################################
+# loongarch64
+
+[build.loongarch64]
+conf = UefiTestToolsPkg/UefiTestToolsPkg.dsc
+plat = UefiTestTools
+dest = ./Build
+arch = LOONGARCH64
+cpy1 = LOONGARCH64/BiosTablesTest.efi  bios-tables-test.loongarch64.efi
+
 ####################################################################################
 # riscv64
 
-- 
2.39.3

[PATCH 2/6] tests/data/uefi-boot-images: Add ISO image for LoongArch system

[Bibo Mao]

To test ACPI tables, edk2 needs to be booted with a disk image having
EFI partition. This image is created using UefiTestToolsPkg.

The image is generated with the following command:
  make -f tests/uefi-test-tools/Makefile

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 .../bios-tables-test.loongarch64.iso.qcow2      | Bin 0 -> 12800 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2

diff --git a/tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2 b/tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2
new file mode 100644
index 0000000000000000000000000000000000000000..2fc19b10b36f0493121c526646965ac7ea377b4c
GIT binary patch
literal 12800
zcmeHtbx@p5x97mXV1pAFB)ID!3GVI?2o~JkU4y#^4K4{52oAyBoe)Uy;7)Lz&HIwO
zyY<!Hz2ELX``y}mPEG&%ch2dm)2E-OtLLd!m6E*!006+>zyDcK{wAQe|7Jsf>VW=m
z_`eYD5BZM(;dg-Je-!jz{{NR1_-l#&Oa8yC0Nn5Q_(@R#NWXjVzxM+8KkZZRe=r09
z!ax1i|9@G`|Gbla`*-TdfG5QG&pVO-%KZPR0wRA$qXPb*{2%=OxA}MThj0EH|3`fC
zqhbD|{h#H3#y|gO<Nq=4&v>SPwEwgG&v>4HFaD1|_TTdx{_Q|3dWr*U{XzRb81lFK
zU$`-Gb!Ue{O-)Zu&(UI4I?=QAg6{oJ2RTK$5b;pC4G}1@%vV<pSA&q9BFT&TL_%<a
z5kly?I^ha<Ad3*fKxRbRbbkR1B18!|KwDfJMi3M)^i=VvB-rsyUFZBed4|jII=9sa
z0||Dg#9mRHj@^wc2OfaW<c1<@zpUa3coHS>D-Sg!rns(10YjZSLp-FA4{rYpQHgH%
z(!RcqciLD66>9o$hN(iArdr0VWyK5qjx0C*Y>+yc1FbmiuDc!#G44Jpt1(7TM}x{9
z?LKcF625-i*l`L_Q-L75g22JQkFq9?e^1i&LKXQG^B4G36B)zN?b7wk)g63#eq}<A
zFN5>R84t_|0NfoP7#kW64nk3-Xn&70O)&f|rVvCb+`kwA9t9*7rV6Em1E?7%7y^JO
zAPlU&E|n4L9=p!+&fd;WWQiECT!=c92r470E+Z-dF=T|0`wG!1!&a}cr0E2?2e53!
zLC-<55#F*~xcPg{S@#d(=UH9W(r|Wg02AMdx40E@XnJ4a%wIrd)-+zChCu-UA3IJ8
zZvIbNY%J9wfsvt!fsuiwfon)q!SKOAAeakkxqR77p8GDtpyExEj&4ET>K4S<vM4XF
zldgm9i+SvHAkNWe7T=Zy!g18zwnxMBjf#2gp>HrT4vy$&Z|T`OBq&rV7=$~h*%|0w
zQ_$4%yipdVAg6m#&GVXqvx=u4SyG6Ft|Kq+6$Lw6Xj*w5eSH&TutI%M>61GBkS09?
z{BRMS`4+>@?k>ZLGkXlYoX8F%;O>+RK?@f$PYmH9fZlIg{4U||h4^1v<%dVv=2&D|
zSXEV4f0sKtIx;FUXcBC=y0=vdK!h*(r2sB8+yMe+)ZPN&K%j=E^pbRX<gs1=ytjAL
zM%Of0>G6?belvqms%sVLDT?V&6bL#t`V^MY`Af+hObhk~SAy-q_h1*nKuTCXpxfBh
z_ynmN3q+P65V1S3I=(uHg`mDWy4tn6aZI-U)4*cqcNU#*jfK#9;MBrYMN9yIvshzt
zVy=X$XBhChgpgNB>8ia#&>eP1PIf3W)YQ<?FeHK(DhdVDi?H+Y7)GfKNEQ(z$T(KP
zPu?SneD4kk6b(c03wf(AYb}WaoSjfWChD^Xg(0-P>rIjX$utQ-r6vbE;UHjd$x_~%
zA*{ykMR38?Ebl-2f9qdjw-Na}3vk81m&_mk^#9(E&h-?B1u*<O5F6u(<48Sm+@mLa
zCh&y#O;3IZ{}U(7dO~8*lbyuoi9c_DLNd!IqyRr5<<JvSM~MLb5?U<7zme|k6EaYV
z{LL99o;Z`(6KCf5gRmz%i`f%rm3>0CKl!oc1OCOt#4rzhea$X+4&^AJ=DxRJ^x(<U
z&UX~yWM6x0P6G_S1%p(>4n-EphgUzR28&8^6bB9^X$N}6(unt~(W-**%~e5Uw4g{S
zDG(VJEQUH5K?dBLi!zI(HbVuk3gprRT84urG0>mo@l0aWZ-fCAP(>`L`2mLxfbW<!
z#5-|}3H<mZlfFOu$=RP<&S8*Ox(23mq77=8nx9x98#y{6o6AfEMsY)+JfES^2>5CU
zrh)>9atl;<%s^a-Aj3KUN7)#Ytr~Qp%$bZzR~QnHC=v6l<OpSQiQK1p!L1)|rt$?w
zx_do`_`!O0=wNd-;Bej2h_8E?{<I(AYh@Pof&1pX@%K%WXlnw1TLQa$j|e1eMK}3V
zVInaJWqP{IGbYI7?gyoX+1EHY1Pl?u>e)&qeH1=jLqiP0>j+Et*cj6cg~$c3I`;cS
zqkdks58MgpR&92N_t_a%e~0U+Nl@5v#}BJ8#Aj=vb=g^Ba#1zgY7dRx#K&pIq_|Y6
z^H5~vrm!6O>f=^Rm!>dR82gcJR7=+O%M^FhPxK5y#ADlZ1sAR=0nV%Iw6Ecwyi%%t
zXB8ZeBbk%^Q{hHm-+UN3Qs+rv6i|NcNuln!Tk0s`kcDw79N7WNVd~ouc5vH<ldb$<
z3#c&9iQ2QP{RETs87F!5OF&}kc-)6}$x9hF=BcqCOIK|v-1Kp=nXo3s<D{34d#dS0
zTYJ6o0O*?+>yUg2z5Ht~p?c3Ttaq4<?~t%2=<qulumq;f`MF7!U;xF07G-r&y#5Y;
zAFCKF@|K|{F%Iexb8{d&H4(i-D})#o7wupk={gF9h8*<@=kphva&;jbV~Y1$ge3wq
z#4tEiE+R{JIQdQXoR%M2zC&3K?KjyjTkVu<cZu_#iyF5DGd$_-5vh~r_vU-lsa5S!
z@wO9s_$+WF?K)o3wqYeyN>bxNUl*dOs?ITYX7(MMrbdj;G@7PTaZC)G<{+7B;AOCe
z+J0wiMqJoh=`ua5rsHlpuEHH0`5JfJzf@-s(aWbE)|~kd`UQXflEi?_&E2!_45LQ>
zO?k412Ik2PBNHKUm$@09mr#{%P^&Y#a>JIuu7&ew!kldcQ6Xv;8)|b5y{6llP=`ys
zYqN>amMr4&yz7M8Mr|a@@NA!iT1A=ol@A_d^8sC%wr;H*+(TI!Ls_2^7mk7j0+eZ&
z*h;-?j`AnVui3(6>JY-TL#0Ua?!57>^k8i|))z=&ujSgL1mf!|VMf9e{*S`B`}WJb
zo9Hi`?wbi>S#hMrp%t$tB=A?(k!1Fjg%@Lf@aJCg?hENJJ|MXzR?Ca2{$3{!KQndl
z5KLCH6Pen(ZsEMspzqYwO$e&gRa2#swVN~=7>d{kFJ`B)t}*MD$SDg1BgD|GaENSv
zx!G5VCWiIgT#vVP9Cr>?5H5nXn@}kBW5ty|zB}ph_jjmYiMNK;zMnY1-ey~xjk(PI
z;-z_8;h3M5^fBvKm~%t1b9>L~vp2l_o%b)8vs4Jrk7K+A@OIRu54HR1=FU8sCl9I6
zUZ~X!Lo^hAMY2RZDO1FI#}wn232rG9a_`jgT8j`XwOc@kq@lKo6n<AAA@xhG+vlyE
zsFfDy8g&`E^j2{s8?LlSymyN05@n;df!tEwyb-VgD+RD~s($25S<RchzBM7Gf;1p5
zuc6JUrjKZc1a6eQvnNRp6;w027t7SYI>_%B&jln5P4LVcWh&!)?fm@yX9kC=#ruF3
zyfpMjpj#rpJWv#Cz~YLvB$Ph0^Sd5}x#rpPMN5%rwFJC|u$gPQkE?~>I~f=>CN<F1
zw2K>;_LXGlP&ITIf?`uNv@KkWEl$I|4o4l#Uj#C|fa9P=(JUl)GrR`j<;jvC#9@6v
zQOHdx{??y4jtV8HLS%?Hl>@$2;V$-gu-Yb$?mT$^DNKHW_yxkV7K`%A54ey0M|rP^
z#t&rtr+_I@J@x7&L+T_*r6u)h`7YBfy*G^|bhF&Ur}bwl&0c`i9}d$)(phe1nitOf
zg=>WCR{|wirc+FvOe`W8I<C)G+|%B6@eIPTMenXWKNrjWxw4cPN<}6u6z6+Ce`s5i
zoelatK0a@4>RneEBQo_p6Z^d#RYMH+4>qX-5lZ)9iCG}kL7QHmz%tA3)%6KV^eIL)
zo8UJ0<2^#w$^}VoW*pnE)!g_ClUUyFFh9u&WaMq>{%N91))W+e3YCMB5X^$Zc~{=G
zkAnP($x*ld(dIDorub%=Od)ZL)X!vg34hfdFLYv@s2nd+BE6&!pM5!$%bEf^@mbvn
z*=Np*ywJ8tA`-Gq_m{FC$u;nULg&Y#t^omWLk)Y$@u&LbR0jC%ZP*vlT(*3-M=OI1
zC=mu2Kd2C^UeuBGvD@*MoG1zINJA{w;GVVoAY`P{03LqRdV?fYhW0%-*WL7RW@AE2
z&5?b*se~R=bjS=QVD95Up++aC`|F_ned|S)GOzPG*<s?>W%_DQAYmKL_ak?O?c_M-
zhr$3Oy|Pad+o&8%isxC8JXB6n>oI*Dq>Q`Rdt}v7^Hue{9dnQ0+b_%Y52~0B9URj@
z9>mws8~l!HzoVN`Eps^0?+|Rk&CbO<&#}dDJ5@vb%@=D815g|71L;w6b1cpX!rsTf
z6c2Q2kcyLVN918zdsG3=RP^}#BmDfDWKNmWQotPN2rK9qj=4jZL-{D82W_xLsXO`_
zGgW{kZqZ)?DKpGVr@c+>1um;IY%UFZa~o{RAN|V(qqvPZv(P1?oqNf^E}>c;-U!m4
zdnYi|)fE<v8^p@U-$jv0G*O&NTx5eB^=jPKR-qBCe76DgkvupgsGyqWx~Qm_At;O-
z*q3)?Fj#dZfZiSE;FWXix3Y4^Lj&1Ewy;Nzbd(8vvwfR`a0Wr0rbr*>i%EX5QD{JU
zv_DP)CREvp^~&|L6PUE-T4@T|^VyPKeXcKxZZWLT9-3(lBBC5rAc-u#^d-7Gg#02>
zwy&2`UF<5GU^grQLo^Iq*{F`)!U#$E0|@yPx)!`T6RZ*S*McBXW^?U<M1JUXR>sfX
z>y30YesbEHXntKek-$2$v^*YKsW;uksvq*2>hNt2iA#FZU4yOkXWdk~LKImy8fw(K
zo|HX7d%7UsRF1?dwU<loJTrhZk4I6hmjt#1>0?vE4vh0_N}w};<rFya)EWmB*jT`A
z0P*;8$>rVgcF_ahk}Px1E2Ih5mX5GJ!N?{%$E^eyP%vIt8D12xcjd@wDhPi`Ek-lJ
zgf0Js%2w1XrMhkvzPf+_h9^(1agG{`Phf(F#`NO_2(5lnCBc#+REMww9^=61v}S>u
zjk6=tkIEx5sElKgE~?DD`;@J}9px^CpI66XJzTB&?B!`@4CP^Rx9}usN-!~weSudH
zw|BL7)~y-6G%SZGXE+4iv;P_H+?j$a1hI<<m-UFxmi)(v0+-a>PYC}OxlQz}XR>v%
zYZbSQS%h<X@quqzV~ayhVHIm+sRo(OVJlL5AFXm_*OjRU=L&r3U@drtlz(!`Hu>8v
zbxpigd{p9UO700_OT}ItT7ES9nvge1-ppFv3}k6sq3NSy(*xhB_i9I9t4U^@nfmX8
zgRo@W%KFf=$Ze$eoZkkRCKHeG^9;nuwAzHIow`d@ZnHg&MP1f>6CCB<Ijy3%MT4KK
zdTmk3JXho|$9FJD*jagl8AuG3bCie5eU|^NBf_zXf`9GgVcRbWaG}`<ml7Mu><2;F
zp)b2ZL9f5y>Ck4;`uFlNusME+iVa1@MxiuL92aAL$BSiqkNM#<wvsK-c&=I3wyrwe
zcsV0`J6oGGHOw3{6ev^+7v(f;gJ*-+1wKiI>r<2IF(g$_5Um!uWs(;!X_1}ZP#;N}
z^P15p*{x*dJ&JgNH!MfEsex(sOmz-K-`JJL-@xXP!ge*>4$i4_gHP6W$ph7@Sdszb
z=>)g*0LLrRoCF`ym)XfR4=K<Vkiyu?Udk;&KH-XE$@AE6z&Z|Qe;QnV#jdUrboauz
zzhN~d!0nj7LAg8I$NW*`Wr8VJ8x~z7OUk%m<Tbns7rY3CYgKZ%vcof4ZZr(u3-U(1
zKt*yA7>Jm9ZqUuYcgm{B|NY}*-lr2b`v6}8kzyRit;!&4kD<HIeXX3UU5Xom=VwCG
zwh3@w+0PPmhAW%!OT^*aCL}%xyY3Q)Cbs=m<i3?d8umc6c?8xJ&4^`GeD~Bh5cQ^U
zv5^quNU}V=g0F}_rLk2-s}-@;Rq&X5yNzGn5oq;M&eOZ=M3`sC#>Rf|9ObCJyX0;S
zOwEZQ57kY*PonHNNX*5Q^%R-JXV|e2tQ|Q6l?$P6I=0=;7A(i3xL6#^SIrhsvktVL
z&b`m6&|ye#Kjr$`FVhG={HVM&{_FW-S*WPXI1`H;#gKFAMkZzRQR3)VSr9j$-{^8<
z1=1W+xiJN@U`bkfO%K=X48`KkLYO64*Qf0Lx{R|o0mDj0>3#Y_KDSft;L$j<S|1tz
zpf=+YWI7aXWoof~2aN46+yg^GE)^P*KLPfRw>@jEJxkS4Q_l}RCy><WYc~A72zb~S
zPF5N-&QPdyQnckTO%(&$=!aweTH29TYj;jAFB+<R&kK%uIYWQWxPn~=AV)^$BDZAx
zm<_HfUf)EktQ;bN9yQd8AX}2)DoRXL6pdBi$nhmMyxT(Gy9o97_KIqDexYweK#16T
ztR}=K0h6H)>0NrMQ7T<;_1r0%RIt33XM>`9Lhqc$kA;{5TXw;>X61|Ajng4&b~av2
z^^5GOjkqI7CB}BPUi5Aiow>372cV#F_8^~Vjg$w*RKM9b(Oc6K4CSl_=nh%K%d5S8
z&eH3We<@uV>gh*Xj%V-}vddpK7SjYu_qwU&ImnQE!b~f}Q`v+0r|$wx{L90?th#HR
z;G-VJxGXpD3cuv7qCu>JPm#77=a(IC;oGJ&8uv(!S&6!$n_^QPD_{SzGjfGG&03cc
zye!~c|BZBSi-xjJ^4lcj`Qv_lQj7w!UwL?*Z|z4AnV!G_<xYoly<mn3QLO0J`ZQsH
zXt(r#SBY+!Zppt?Q7!9%%X4o%QRh-ks*YFUWV}c07jUXCNmWbaXAjzt-eKuY6|4K5
z{pRr0%{cKw#>+A@4${&=<un1myAxmcn4MYiV%T=u3Myw#vS978)x+XUasZC$J%LSv
znd6`pPYLz~OjvE!Tstf)B$!B$4H4oD!La^S%LP4U=d?WKh~U^b<CHk5;k2yK+v!Xw
zbW|zCC^^b1rSEBur3Akcxe*8z&qN|2)EiIFHlug2oVDf|+FZq{8a;vCb*$ENKFYKH
z{+3&6-aau&-o2VIH$vX^eR4>B^FbH}dkpA$JuY&4-m$v4Te4R+58Eay2bN_|4V?$@
zsVrRu+`tH6Xmi<@1~vF4po-MCK$If?m3mkbIuRJ7bhH4>EaIDx(Pa_IB_8(%Qw0{O
zM4RReLRqqpxS_Wu1&*D=2z=c^!M*lUpcDir5J8Ucn1YC~wH)vP2IH7~NQNGkx5V`b
zC9LGQay(}+Ptf`i*Vn!&(y<I9CRw?kcz|7WbfI)QD#`Gap!`~a^EcFmlFahd!VKCm
zfx$AECQO_ivcQ>oMM`k@F(PCftw#Dp*~*ZwBZ)o804VdLMq1Y-vy&MO96*uxhA<V1
zkmYbcF`L_$aGLWik-`EeSfYa^z7Lp3d~I{dnZ3f)O>Q_jc;1yvv6ZO?<OCv#0|0`F
z?3TEykb#c7_LnsIZ7sbB9Q0ujd8M*o9!S0?L*s&IQx_QZ86Q2$A-R&d=O^SKOT;PF
zidyml9=xJ&xJdb}`|D%e$QZ^LUz*Z={o|u%V5iG>{>OLzY<-#=uh93J(Q4ID?SeQl
zW(`+GCNcuu+Yfv4s=v)eznDBObF>e?vter{TVlDkiJf<;K_~HY9BF2~k^dFcZPG2*
zu6UKTiLPL;Heto2&_C0LmpkoEB^9w_t)tQ#&!ZrpbVb9IbVdC>$%ALMNo>jr`IWDX
zHe!%u(9L)$@(oeANyyELqPD>rw3dR0&Eh_|M|@#BmxT7!`)9@OYKa+=9)!6h;ekeT
zXB-mT@l7V>Ab}pEN2@j1y_W@?ZYeuy3{aD#sS3usL(02)HNA3)@Vwa3HTBNB675ZV
zzR=#Nehjmg5Z7X3^5pNroQ0eWuk?+m7`eS(Q<tX@ejnm)LvZC$c2ZD2c%HdVOCv92
zW>7(XXNj1GqGN8Z_p9_;%1AwI)rJ9d+>E)5{?_k~6MQdt3`mpV_6wRv-H3C*^k9Eh
z>Baw}M&p78eG13~tyF$w5Ep$cycS(^r0MT?ohWRH_tz%~AI=K$FN<FqjT0TEmDY2_
zsEe&gnxA1#lPripBlK}i$YZf(i(|G>(M~CY3=1`MbWp#UYQj%JG;ueW1<^!5LxWby
z;s{qHS(r&V9hUEwuYQh6W9JRArxm4yhFkST8L0V&WredK%#&iiE$6LoJ?vU392y&z
zVcnW-mFtnW!@PB7WjqWW=0{O<i~)p>?aP^(z{%a^zbvJdW1S6~!MZJ1&(?jr^h&L|
zQ3BZZjIrW4c=1>D+HCJeNc@irz}YDKkPEf_WKg^N^`Qpbm}-_+-ImeC4f%)Tjr?#d
z8p#sEqBX{|Z}gq4pN4LQKOS*jC0dZ0>x0Go63Wiy%68Md>V&K9^`wR})d^gQ9%o36
zX~Q!PTFCgnZ&cq0*ca>yT+M$B*s~n|a%UG>;||B2QOh}8?}YoNyX#mSWnsQok6_9v
z(CE#Q7g<BDoo}>HEw0YF8%sNX{)caxqr6PZJ*=5c`O{H_=_cb@d*hi4?H~HU4|;b8
zcdI$2_{W)LHq2i41PenPzYYZsk|O!*n7^1r#MNw9JB#UICjRgddwzNAjE}@&9kEfT
z;!wg0dORXqgb_=sOfX5P;yo^ViO2kch>J%}dLDnEK$Bcc)n??98(JSKsdl0x{y{es
zk7H(1e1oPnG3=B4g8$vvHIg}VZ-3KxZ+hQu%l3J;Py4a>s1`HsdM;3C7b%;OFk)HL
ze#{ttNEP>VNHkaOhQ7h9%d5ymZ3y0K>-?ox(Z#1oi~D0uuUpBFQL$3a2M*tK#4ETr
zR9C(Utr2lZWYe?lA7vY_nwqV)5~W+z&G@grOLO(0U!Q?76Y!vY)mHXvNI^S@A)B8A
zON;<|BkA&eSE@gdX9-63DE6wy-JwKCxfZNVQ*Am4V<ftx73uoZk-sy=i+PA~k}}jf
z_OJZR`iA|ut?OgMp3q$@)i;61ocg;Ti2^Eh?xLTUkWD32`iUtQMq8n2BmR7l3rm^{
zjgi?60S(m&lja=hm6V*?FZ0p}lnSr78pXImDZ;)j0QXCGtM-=HIxc1gqHg_uc#D5?
zSUia(q(1B*PM7-f^}63rGTCiIb$51DU_)AEoo|FvW4tl{!x~a(!qXuwmpGDW<4ER+
z+^{!~+{2hkCxV<rq6<^P2u)lBEGD6%(Ah7igaCs>L=%ntu8m-4i(8EqqNSxZuZ@Ba
zhuz!jl7$voG;KVey>QcZrL{+XLpgajl}+F({VI}Y-My=EI`c4GbC0#VxB}!&UiyMx
zOjbJlHDs_P;^PBUUYpovDoC5;MP6ncz5b3?y#AZ3<cRGL*dN4`7+r{5P+hFH<}#S(
zc<g%L3iTeR@o|ifDN&HDJycdb&v)=+vquFs%g!ZIC*PBr#zH4{_S71<H?;5Alu=KW
zmeDxG%hk;|;r--dADGm#<>R?ceca0zc7{^)Y58DD`mlX=`CromXDX6MDCw!hXNMhD
z^E}l?W021ccHl33NvO7C69EBx!!ON&y0R^XQ#RUj#}q<>2Hwmgy@6pri{+Rd@ez{J
z&%8epnB0)gGvJL9UuLb6u*6ZNzMo&Y5(-eCiN)wtTC_s`tdV^Kr#C4q^G1{e>BNB*
z#*FTTeYRTs8b>`T4JoKkGGthiY+fW>EDUc5t2$^kFfaH^cF9-KED{b`Jw6l-5)$2`
z$hy!}*PYq?Ns2zP&`Fub&I=1tL?@mmvW`1He*=4r96v&?)0m7wL<I@4q+K@dO<=kw
zB>6y+Oo9p^UD+M$Cfi_>H;t?&h^g6ndf;gpekdrqc^Ga#D|~5mD*(5Esaf2$eKfBs
zdlYr7GlpPXM@JfBqQU}h<tq66KAeQ|=fcHD+C4hiou3`WyUH898=IX~@Qckh`t`~F
z%XP=7M+;-ZD0n?lL-5zm64jyX>QR_@`}vP2SP}yxN!*TXEBr%zbKf)uoOjq<)zgHO
zcV@i-6J~5p-r%cxIHnjscV6Uu%h_WTXa6N4uEq<&sUg?>y+aLcb*lj@?w3ptlN}Cn
zYRR7|&slYo4_;pcu{g0+xs|udT<K@ZH*%+wTBey7n=|QdYH~7mfHus_7}@&W%A{&7
zT-&E^a`y+QnvQulDLK{Ox(Tjt7ZCEx^ZLJf$x*YhKv6bSR~7DRt-*MSX{XX<-7SKp
zSmod6iF|OWQqYD+5gO93fo!+1lqHuKn-NmX&a@D_^C@?^KC)n^<p6rzmf?5ivt69i
zL%%LyWSlWdFcMOY2?}{*#Sf&oW_j)f><__8jfiPuYmSiU-_0w;%MnL*SMH*z;VK>{
zaMS28KKo3x)hkC#aYhs?Vj?(^#yMYS2_7`Z6mCK)hw~W5c#r9;{>__)MXJ7xh*`)i
zG)#rQxU8CIMsFPB0UfGlOWReQNm|A!Xs|ubo=tMy?k~WwUddb?>VzC(ThF+fBen`}
z7JCrDPQ5pQWYB&OGQjD`<GhM%ZMOe5t$$|#JuXw<90*pAu8%&WeP7UTQ~x8pII6yZ
z<i{r+quk@gi9VM1ntk`?FqmiD4bUxN#JUHHC3(RxqnLn(j<MvbN-!NJlBXC;=K6`p
z#6l!hBq!wZ*yfGu&vtXFExGM+T~X$K@2<Ho@x^luuN3A4X6*GZzUW$8D@2DGfs(gC
z_^Jr9@D@Vi#cE#-kuae-`uK_(ov-0v<uD(v@dfb^UEtUZ$Ay*lkt{_r9-!9GK(;Wh
zm`KjmPbo^o>Q;K?)i|t-_qJg##_lde_6AQ3Y0R1z@PAm^>6M4sp86cvu*cHmRP0vt
z8&n2PG<(=SZo9lL4EPBg1O$z0^ZI`xi+78ygpE#7P5vxMn^zmJI#3KSHdyS3@)f$S
zu2@bWHHH6Zsa?@2{}8-Dm;L+k=z)UAnkNHP*r@E;#z9o!)Cu^*pyx3u)h6`SKT&az
zaC}0rZx7WDKfAZYD5izq=j2=<d`!R;2*O4-MrvFX9VHCrLL|*%0ykcWUO4`A%`t?=
zIVw&@(8+W(yr$vTDk}1HQ0+{@?g}cU^^YgR!I{R}YsTxtM`CcUZP1DiBF(`7zHAt}
z!*O!po3*B^=NJgMZM9~8L(#$8h2u?R&xFj4c_Z&SVTd@@2V+`34mVbUeH7Pw^b$9U
zOX$=a_1_w%Py0CWeyThE%Yi9Xy^&~vou2qHju{ts?)?^Le2yl*r2AK5+oEG(xMEd3
zwBOe}4nHoEhAbeJM#{_vKbsa8r5AIcv|k9#G9)TASdp?;Ve<nYEiEh@35M&f6PV?w
zPZ|IyzQHqqU7@aR-<WGE;-xjhJWZf!4(qnL)AQ0@C7LWY0yA1TEfR+M+t>qjdCV1Z
z8w;tY6@2yC5!B5^Vhpw4Jp*@T^20B8!YiQ&YOn#UQ&xucnOO^s?*&z{tpqP&1%#Cl
zo3JGs`I4PZn>v@obfrvFH2|UoqubSy=%G#PdNKW1-`Fh@-iCA?xDniqa}0TwX?b%1
zycdWTgd_zDgNfu}ZrpM5aPjb*a=wKkyUGMIfQT8VIP+YA?{WaeOnF?u%v&avTey@R
z3@hYpDoH{=Df|>Ra-<7Pj5G0#5)%jX%p1Cme6CqR29@?uX7>j{l?Q-X;*@<b6fAjz
z2Sc?B52>@aEZ(n8OJ{u*K}A<u=l_5#PF3eG(7-#snWEYgjVxcUv2Sk~8-{@>9mRDu
z&Psi(O}Xpl%x!l(4Nd4ATC@rb$g>EWiI^KdiKX(z+RN=$d%k0iCwR?fOeso7BKt71
ztcF1Pfek^afPVBO*^UV1k~zs&caC7bNV0W)P3F5HLzOxF9`TZ!pIUqE+J-BycDXF!
zs>t}e<QtL+)=4R@f~$VL=!<0qB&Vp?+S+)O_GcBJHewoLZ}%xKbPpfr1s|H)cXN%D
zlAirgEOd*Kjz1;HPEd5{hJwS+)O<;}L3iuO5Az8xQB*<O6;Av%R^3@4r7-`}1UYF3
zA55Xv_(L8xKg$ro8Y&x!Kr_Q%EC)l@yZrJIrtd}iZMDJg2oW2CxYenBT%s{<LQw6^
zqlf0Rl<72o@>;bX=)O;$En0n@$v1ebfb)Z7;pN79nELIR$v!%Bim@vOjt!$?F~Pc7
z<Bu3j7BTI9A@*s}8Va?ZmK%=CfzDd{_+0UX0;%`}`RPC>00w(O{z~xc5=PT`Z8S-$
z`XkXbGlgbh7{?SMKyqK;Wq@c}X8Z*|R$-K(H~;&UC1C*}meg|Uv0ta*j~@+83?Ya>
zxckf39JP8?Zp(&`?%|J@1vFt_Tns#R^zpN3`k|srk9k$mJ1fd+AND=RW_`ZgwRZnx
z2xa5o6B24>Zjz47Uu#m7E`-bU?ih?pn7R6yeVe6J0h{`&idg$m46%bB@F|g9p<4wj
zjF?VZT3QO<mqxTF;?sxcV6G3;w(zg)Ox_n%DMI);I5?7;p!Wgr#8kh<L|RHpN?z|B
zgy11r`0A)-q|L`G>X%2hX^uq$Q*Nb=*@cZ>lhNus$w`zJeQvy8hpMf7qRf=iQm=Z)
zlQsEB<W(WJuPm##uOv7N-9d+0ON-WuKf5L`<ek1*&c^QhEVY*TI@IXrgl>gSAG!KT
zlb$$bD)R;r2(G{`oDd~tYg9s_7_Fg3`Jpd$&)G$SD5M}JK@8?vhLO+a=E?i;`493>
z-dPA(1WJ}w)JP@czp<S$%LUSsIQO#CWjXgFrs+D3p{v<9|IAtFvu+PcYKQemKji&i
P-!c3byK?a_mCk<yI~3Lu

literal 0
HcmV?d00001

-- 
2.39.3

[PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64

[Bibo Mao]

Add basic ACPI table test case for LoongArch64.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 tests/qtest/bios-tables-test.c | 62 ++++++++++++++++++++++++++++++++++
 1 file changed, 62 insertions(+)

diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 0a333ec435..008d7a0497 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
@@ -2420,6 +2420,59 @@ static void test_acpi_aarch64_virt_oem_fields(void)
     g_free(args);
 }
 
+#define LOONGARCH64_INIT_TEST_DATA(data)                          \
+    test_data data = {                                            \
+        .machine = "virt",                                        \
+        .arch    = "loongarch64",                                 \
+        .tcg_only = true,                                         \
+        .uefi_fl1 = "pc-bios/edk2-loongarch64-code.fd",           \
+        .uefi_fl2 = "pc-bios/edk2-loongarch64-vars.fd",           \
+        .cd = "tests/data/uefi-boot-images/"                      \
+              "bios-tables-test.loongarch64.iso.qcow2",           \
+        .ram_start = 0,                                           \
+        .scan_len = 128ULL * 1024 * 1024,                         \
+    }
+
+static void test_acpi_loongarch64_virt(void)
+{
+    LOONGARCH64_INIT_TEST_DATA(data);
+
+    test_acpi_one("-cpu la464 ", &data);
+    free_test_data(&data);
+}
+
+static void test_acpi_loongarch64_virt_topology(void)
+{
+    LOONGARCH64_INIT_TEST_DATA(data);
+
+    data.variant = ".topology";
+    test_acpi_one("-cpu la464 -smp sockets=1,cores=4,threads=2", &data);
+    free_test_data(&data);
+}
+
+static void test_acpi_loongarch64_virt_numamem(void)
+{
+    LOONGARCH64_INIT_TEST_DATA(data);
+
+    data.variant = ".numamem";
+    test_acpi_one(" -cpu la464 -object memory-backend-ram,id=ram0,size=128M"
+                  " -numa node,memdev=ram0", &data);
+    free_test_data(&data);
+}
+
+static void test_acpi_loongarch64_virt_memhp(void)
+{
+    LOONGARCH64_INIT_TEST_DATA(data);
+
+    data.variant = ".memhp";
+    test_acpi_one(" -cpu la464 -m 128,slots=3,maxmem=1G"
+                  " -object memory-backend-ram,id=ram0,size=64M"
+                  " -object memory-backend-ram,id=ram1,size=64M"
+                  " -numa node,memdev=ram0 -numa node,memdev=ram1"
+                  " -numa dist,src=0,dst=1,val=21",
+                  &data);
+    free_test_data(&data);
+}
 
 int main(int argc, char *argv[])
 {
@@ -2593,6 +2646,15 @@ int main(int argc, char *argv[])
             qtest_add_func("acpi/virt/numamem",
                            test_acpi_riscv64_virt_tcg_numamem);
         }
+    } else if (strcmp(arch, "loongarch64") == 0) {
+        if (has_tcg && qtest_has_device("virtio-blk-pci")) {
+            qtest_add_func("acpi/virt", test_acpi_loongarch64_virt);
+            qtest_add_func("acpi/virt/topology",
+                           test_acpi_loongarch64_virt_topology);
+            qtest_add_func("acpi/virt/numamem",
+                           test_acpi_loongarch64_virt_numamem);
+            qtest_add_func("acpi/virt/memhp", test_acpi_loongarch64_virt_memhp);
+        }
     }
     ret = g_test_run();
     boot_sector_cleanup(disk);
-- 
2.39.3

[PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64

[Bibo Mao]

Add empty acpi table for LoongArch64 virt machine, it is only empty
file and there is no data in these files.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 tests/data/acpi/loongarch64/virt/APIC | 0
 tests/data/acpi/loongarch64/virt/DSDT | 0
 tests/data/acpi/loongarch64/virt/FACP | 0
 tests/data/acpi/loongarch64/virt/MCFG | 0
 tests/data/acpi/loongarch64/virt/PPTT | 0
 tests/data/acpi/loongarch64/virt/SLIT | 0
 tests/data/acpi/loongarch64/virt/SPCR | 0
 tests/data/acpi/loongarch64/virt/SRAT | 0
 8 files changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 tests/data/acpi/loongarch64/virt/APIC
 create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
 create mode 100644 tests/data/acpi/loongarch64/virt/FACP
 create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
 create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
 create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
 create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT

diff --git a/tests/data/acpi/loongarch64/virt/APIC b/tests/data/acpi/loongarch64/virt/APIC
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/DSDT b/tests/data/acpi/loongarch64/virt/DSDT
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/FACP b/tests/data/acpi/loongarch64/virt/FACP
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/MCFG b/tests/data/acpi/loongarch64/virt/MCFG
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/PPTT b/tests/data/acpi/loongarch64/virt/PPTT
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/SLIT b/tests/data/acpi/loongarch64/virt/SLIT
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/SPCR b/tests/data/acpi/loongarch64/virt/SPCR
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/loongarch64/virt/SRAT b/tests/data/acpi/loongarch64/virt/SRAT
new file mode 100644
index 0000000000..e69de29bb2
-- 
2.39.3

[PATCH 5/6] tests/acpi: Fill acpi table data for LoongArch

[Bibo Mao]

The acpi table data is filled for LoongArch virt machine with the
following command:
  TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=./qemu-system-loongarch64 tests/qtest/bios-tables-test

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 tests/data/acpi/loongarch64/virt/APIC          | Bin 0 -> 108 bytes
 tests/data/acpi/loongarch64/virt/APIC.topology | Bin 0 -> 213 bytes
 tests/data/acpi/loongarch64/virt/DSDT          | Bin 0 -> 3614 bytes
 tests/data/acpi/loongarch64/virt/DSDT.memhp    | Bin 0 -> 4951 bytes
 tests/data/acpi/loongarch64/virt/FACP          | Bin 0 -> 268 bytes
 tests/data/acpi/loongarch64/virt/MCFG          | Bin 0 -> 60 bytes
 tests/data/acpi/loongarch64/virt/PPTT          | Bin 0 -> 76 bytes
 tests/data/acpi/loongarch64/virt/PPTT.topology | Bin 0 -> 296 bytes
 tests/data/acpi/loongarch64/virt/SLIT.memhp    | Bin 0 -> 48 bytes
 tests/data/acpi/loongarch64/virt/SPCR          | Bin 0 -> 80 bytes
 tests/data/acpi/loongarch64/virt/SRAT          | Bin 0 -> 104 bytes
 tests/data/acpi/loongarch64/virt/SRAT.memhp    | Bin 0 -> 184 bytes
 tests/data/acpi/loongarch64/virt/SRAT.topology | Bin 0 -> 216 bytes
 13 files changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
 create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
 create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
 create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology

diff --git a/tests/data/acpi/loongarch64/virt/APIC b/tests/data/acpi/loongarch64/virt/APIC
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..3477789f422cad54f16734b3ec9ad1ff5135165d 100644
GIT binary patch
literal 108
zcmZ<^@N~{$U|?YU>g4b25v<@85#X!<1dKp25F12;0Eiakhw;E%5ne`ShX4P;(hQ=)
ZK)Db4AdUkN9{^%8L6C9*AO@2T3;;5a3;_TD

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/APIC.topology b/tests/data/acpi/loongarch64/virt/APIC.topology
new file mode 100644
index 0000000000000000000000000000000000000000..e14bbf758464b146665bb0e46d193fe2b57b5db6
GIT binary patch
literal 213
zcmXwvxeb6Y5CqS>5dw=RrUP04oInH;2^CN$P0$A&m|b)}X?!!br+sygh?r~M+;a5u
zxw{@?a7X9naN9CeNOl0P5t8WV7)mq~DA7!zL^FdD%^XUYO<`j39)@XaT)hKz1UKj`
Ke^uZ=NBIDAN(})3

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/DSDT b/tests/data/acpi/loongarch64/virt/DSDT
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..6a619874ff25a5eacd68d0dd51e8314df3a7aad3 100644
GIT binary patch
literal 3614
zcmb`K&u<$=6vt=nb!@MHWbHUVoH((2;(&_QwIV<QDfZe)T<m1=rUc<it`w!B1cmTZ
zDitE+P$1&cTH=C)jL2VskXnfoH*QpLNI7ufjySsCx3lZa1Iw4Pl4tfi``+94nVt2#
zt-JM|m9!Arx3z71(=F9rckB`lO$b5f)SI@^b8Efs$t!VtyKTjKo6UO4*>bE^t1M!@
zYt1@Z+pcP8J8tGB<Hgfwg-DXCcRbb}7wP0YBzrlE*0=a_O2oAq^likcyY*}(l$kNi
z+@s-dKOBD5KE3zjHxI6UuVo8bHXllgKmLxS_TGMH@8I&y_x2Cp{c!(q@AkiPERV^N
zvYU{jpWAtOLXP%R=2-XVu|@NEWlWAnXmXN%a_9PlB<ufYl9h2uh9@WKCwCs1kYwZO
zlI)%|^_FEROVPI0M@!McB1{_zEGtcO{2S52<YT$1p4)LXKa{d!Mxn4`)KIUE2_?ry
zp{`O^%7Yz+q6e$S>Wqmc$9ODd{x4!-{?%CSm{@Xb6sy*E?4~kXRqF+MW&PK<YE_`z
z#s=D2ZEG}-?oKCDGzyNeMN{mHlCbcr<Lo1GOYET+6vTU{oX*Zn>wV)mo6r-(p`d3Y
z<hh3PvdAWB+z8S11R@Mk5!sZU91ihMAnklbUNB6{Yn~rW14*8U@7abtPuTj3JOk5^
z@H}Dac|wvWvhi%go@bCjSdeF68kU|g^*kZTGq{WHc?KEio@ZdHOV1#ANR>0fI3u2C
zU>cF0LGq9)=M>|_(pEU9I47ja8D*SN#u??DkSb@4amE;DjB`S&oN>k(XPj})38``#
zjMHG82IquSITMUC!8j9~6H?_&GR`F9Oma?0l{3XSQ;ajkIU!X}?77N{N;A$h=Y&)_
zry1uo<DBN4kSeFiI8DZBa!yE<Gs8GDj5EVIAyv*S<IFP7Ea!w&IcFH>4C9>PoRBJK
zj&bG~XO441s+_ZobCz+=a!yE<bB=M&G0r*838`|<GtPO&InOyERn7&*xxhFVI47ja
znP;4N#+m1wkSgaQ<6LB%i<}cu<y>N%ON?`gb3&?|1;$xmoCVGasd6qe&Sl2A%sC-d
z&K1VF!Z=qrC#1@WT37xzd%H}LaTYlzq~y#cdv>iV&hSqpO4w9Ut!zb%H@-*-6ufkN
zir=yL9IGRVeM=t02k6(tf_NwdueiN&G{fPL9Mxlu1PX2Un&XI=z0+xl+<LsX?b`b3
zfw1el<-umO|0WgS-S+R5e~N{*uXjJFK79Gto2Lhc4udP`W9TbKBq$D@R4CLPYK$Va
z1M`0Wmi7s*ls)V6VD5fDu#>|kT|6RV8b3SuE-lCDCgrn(++}<WwDrLynawZH|NN)B
z`f>Z-{9o<-Pj-6{@4qF^GGR@lMs!cwdZ*zkMUIXAT25ufu>LbYC7K7!&ok632lhOg
zpYBO5di^>2?Ad74d$J0y+D;SY0~Jg7=1Jj#A72)hVQoXY9ydCurSRO+rlEH~OOUi1
zZlhJoE%zF)?Zl5r+!&bsJ2wofQSLgsotkmnJ;~F_MagYjw!9b8DX+HzxYkx@8+oky
u*Cb=cNZ-Xiiwy9#2#pBH#jk6*i~pf^;eywMTuk#iV!ErnY*aT;s)#2;qXaGh

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/DSDT.memhp b/tests/data/acpi/loongarch64/virt/DSDT.memhp
new file mode 100644
index 0000000000000000000000000000000000000000..d7e2ff7031f5fd853a3a49e52f865f8d4ce49e2b
GIT binary patch
literal 4951
zcmcJT%WoT16vpq^PGXN8$9AT9HEqm8v??mtaa%wtQtYu)r=3ZR$88F(WJ@6>X-aF-
zhX@rS&7wfWrnQQM*tp=o0wJ{$D|X$1V3D$5!yd82d}kicJ!zCp=}4Km-@WIYIlnt+
zuJ0so<ty`pLMV4~b9&KA=FXS&Bo0LgL2bImbhT#XYSoQ1A$`tF1#88be1B=SlscYD
zi(u{SOde-l&nh^Vt@ugxt<6`3h>)v&JW-xZP-DyIT%BH=U5hV!MM%j(uOUv}$|o|u
z_@Eloe(n7B!<Dbh&AUH-`|$MlN@83|jQS$tkH35R7cO62SUGxeb@9qaHx{oftnJ7#
zy-SXi-5xo*xo!FP$WedB9P{ox8Z?iSyX2_)_D<4GZY!`ylKKBL$;@s^`u9%KO>V1a
zk0cAvmSlBfCSaseQYq@G>24{OP=rx6jIvUcl6xa6Ox~yU)~vFnxS^zq>4rkZ$f4fZ
zC6v?G4RxBbk{)a~6g^luR(Y3LP9KjY&Hr{R%)cDV+9j6L*Nv5Hmh>5Ewz8==>6LX~
z<7_GeWfe~0oXgF2=h3Q^;}hz5N$6rmEQ+K^;V+NYO|dK%a1|89ey2+1%Aj#gT~CAq
z;Z8@;vk^{PK|3uH5$accG(Dd1Lu5pvKM?73uo5_St|BiOMx8O+52l_ZPsF!veYPiT
zT}7UrsZZFRFtt4)$rIVwwtm~w%fK(l(=+uuo-nmNA<5IbOThN@G6>k7o@v1G^pb~^
zIeQpqkL~H1_Bfti@{lrTFXKdMOPsx&6H?~vW1M}AvyXE^%A7&Q8DyM6&Iu`Vh8Smv
zafUc2q|B)@PL*-0oD)*!3^UF!;|z07NSQOjI3tWR!Z{&j&VI((&p7)zC#1|7Wt>sQ
z8ReXiGUou}9AKOSoD)*!j4{p_<BV}mNSQOvIOB{n&N(4v&IIF3FwO+$gp@h4D{*Sv
zep?3_=OE{VlsPrVsWDECb3)3T`xxgw#<`DkLdu*&jB|)_4slLMnRA$N4l~YS&Iu`V
zjxf#<#yP?{A!W`{#yQG3M>!{?%sIw5#~9}r=Y*6w_cPA@jB`Kdgp@hQ8Rt0T9Os;n
zGUoxtd4O>q;GB>$=RwAKkZ~U5oRBgn_PSEtPB6|1&I##oYLS|r%ZfHuB9X+8Dt0Tg
zJ?eX3^b2fw>G%wP>*8}%M+CoH&e3^@>y#J~kA&bmZo40+PNzeT@-amM8*S@ssU(7W
zrECaoI#ipp^uXqd(DU=@R<W<SL>us``FrM{iSeng=dWiUz5UC@%@s8OgGpS2xMmt8
zC=N9;6zUe8)CoGbV)vTM%BL7f+t%sU(7mQ-=XA$tutCPu-(GnTb^7Th>Gq0t6dwa-
zp!I^2&Eq$I{?j`CiFtSUFLU%K-E4)Lmqa`6R}?yl>V_F87c6O$qvQA5naYfsdTM@>
zG!K}cLv)o6{N~a8U|Sfx@EU#goGRpN&Mr8smu9eipv@A#dHONn#&<TBe&v+ox?U(_
zFNNn8jj4g^?J!BbU=@s{cC1!7R|z#p)LTQ%TXjV!7hd<%eavD>H!Xd(6+^d5TbDbf
z`EpKOuWrotIH|xSzr7*PrcSt-n2R4?U0q8qUA^A8eB*=U>f*JlH&z!Gues^P)#x4k
z1P720Rt$n$FP6-*h)|C@Qr(y`y7Beh;^${fGx`4F&1E~LnV&7I{_5=`+ll8Xp5H9z
zj3iD)oD6CiXqC>_lGWRXci3jdlxh~Nnqtn(T8ddV&naf%bWN$=PVTVun>2CeT;Uu}
zmY&oGYgQ#I@boDU(H3nrTvbd%KO?kkFHN8{))4jTOElK$e0TDGoje+K3fLNKI(M#2
zzgt%tb{Y?cT3XXdg?d##x{=swU8z?OQINKx9W~2VMUlcbzF87>z;M&%X{ZC9bzRGj
zn0S*zUJ9OV&@7)g7Way>`#MwM(eW>;k`71NMQZCcu}O|<6qy@NGl6E&X})3<yxt>d
zy1heY_PlF5hGwxrkB6M7VNf76j3PNu!=NCm)J+ZYC1DSfcT2lZeKfvGTCqFhcBgW^
z+e0mb#*9|JuSwkk$eBs{n7|4qwpTDhB~0S8@iLRDitsfsp&5m*+Z;kdwFcdph8q-|
z1{;(o4Y^N<+`7#Z$F!GGf2OkY0W_#8w;$7C6jmkhl+iL3UE>l}ofA@f5jC3fJ!NQ@
zu495#sMjpfhbgTPsgLnNVZTt3?BmF~Hb4uzXj4(pI5d@Vn`+9{$QBxBTgS01SNxP6
z$))8>@IVH1zm(nM-VPtH?(OLRG5GJy^SvEDaNXO{0{i@KryVx&BKlftb4lExSzV^4
zLe2DYE8JYd5{dTYa&=>rmdfp=OHb*@3`t@gPfr`(uYxgKp2Gru+#Pd%9Ng6rRfuhd
lP-y8V7xy0m+{OJ5*R~7x7vvI)T|i9hcrDQvQ1Oi?{sTM8gFOHM

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/FACP b/tests/data/acpi/loongarch64/virt/FACP
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..04d8d4c26fa2be24af40cd7a72959ec6b12790e5 100644
GIT binary patch
literal 268
zcmZ>BbPnKQWME+Ra`Jcf2v%^42yj*a0-z8Bhz+8t3Rnaf7&sUh<QVvX44@rUHJnJp
PfcDEk?FYF<7Rm+yEcXWi

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/MCFG b/tests/data/acpi/loongarch64/virt/MCFG
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..5f93b05abe1669fe4239edbba8aaf4fe666f5b95 100644
GIT binary patch
literal 60
rcmeZuc5}C3U|?XJ<K*w`5v<@85#X!<1dKp25F11z0R<?#9z+8GC8Y-e

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/PPTT b/tests/data/acpi/loongarch64/virt/PPTT
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..7a1258ecf123555b24462c98ccbb76b4ac1d0c2b 100644
GIT binary patch
literal 76
zcmWFt2nq3FU|?Wc;pFe^5v<@85#X!<1dKp25F11@h%hjKX%HI*fMQ%gwhD|7qyeJ>
B2LS*8

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/PPTT.topology b/tests/data/acpi/loongarch64/virt/PPTT.topology
new file mode 100644
index 0000000000000000000000000000000000000000..4017e2927c15c08c7e1a7e4d7eb10f416b63d69e
GIT binary patch
literal 296
zcmWFt2no?(WME**cJg=j2v%^42yj*a0-z8Bhz+6{L>L&rG>8oYKrs+dflv?<D8>h5
zS>O<Zn}MVkSq!AN1c;en`l055#F&x9KunOm=;}5A#aKX+AOO`15@Q7mK*T^ykX~f-
UK=!@?Vm2TK1E^k*7&}x50Csu}0RR91

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/SLIT.memhp b/tests/data/acpi/loongarch64/virt/SLIT.memhp
new file mode 100644
index 0000000000000000000000000000000000000000..67f00813af7b2356fe74eed943ab8dcf2291578b
GIT binary patch
literal 48
scmWIc@eDCwU|?W;;pFe^5v<@85#X!<1dKp25F11@0Wk=0iHdRo0OYg>0RR91

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/SPCR b/tests/data/acpi/loongarch64/virt/SPCR
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..3cc9bbcfb8051e632592d9db0fe3dba0af53ed8d 100644
GIT binary patch
literal 80
zcmWFza1IJ!U|?W6=;ZJ05v<@85#X!<1dKp25F12;fdT{L1I7pP5RMa&VrKx#G5-ew
Ih!}(f071qL0RR91

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/SRAT b/tests/data/acpi/loongarch64/virt/SRAT
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..ff234ce45cbdd32f57fc0668aba135992e5ca887 100644
GIT binary patch
literal 104
zcmWFzatz5}U|?We=j89~5v<@85#X!<1dKp25F12;FdPV=@)<SYGB5@QjDf5V0BFYt
A0RR91

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/SRAT.memhp b/tests/data/acpi/loongarch64/virt/SRAT.memhp
new file mode 100644
index 0000000000000000000000000000000000000000..60e8d2a136be43c6479740a28b85a1348f044322
GIT binary patch
literal 184
zcmWFzatztQz`($G$jRT?BUr&HBEVSz2pEB4AU22wVK@*#<uhu)Wnc^z7z0@!qXwJ@
WlZVkL<|C=&fSO|gGmja@0MY>X$p`@e

literal 0
HcmV?d00001

diff --git a/tests/data/acpi/loongarch64/virt/SRAT.topology b/tests/data/acpi/loongarch64/virt/SRAT.topology
new file mode 100644
index 0000000000000000000000000000000000000000..75e0703ae0d3348919e2b583d1b4f4b8bb3ef65d
GIT binary patch
literal 216
zcmWFzatyh_z`($`+R5M9BUr&HBEVSz2pEB4AU22wVK@-L;4`AhGokUB(fBNAd{#6*
V8ycS-na`*JcL|Ka0b?Mm0|55#2mt^9

literal 0
HcmV?d00001

-- 
2.39.3

[PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch

[Bibo Mao]

Enable bios-tables-test for LoongArch virt machine, it passes to run
with command "make check-qtest-loongarch64".

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 tests/qtest/meson.build | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index 8a6243382a..8628eebfbc 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -149,7 +149,8 @@ qtests_hppa = \
 
 qtests_loongarch64 = qtests_filter + \
   (config_all_devices.has_key('CONFIG_LOONGARCH_VIRT') ? ['numa-test'] : []) + \
-  ['boot-serial-test']
+  ['boot-serial-test'] + \
+  (unpack_edk2_blobs ? ['bios-tables-test'] : [])
 
 qtests_m68k = ['boot-serial-test'] + \
   qtests_filter
-- 
2.39.3

Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Gerd Hoffmann]

On Fri, Feb 28, 2025 at 05:27:23PM +0800, Bibo Mao wrote:
> This patchset add bios-tables-test for LoongArch64 virt machine
> system. It works with UEFI bios, with uefi-test-tools LoongArch64
> support is added to build bios-tables-test.loongarch64.iso.
> 
> Also with test case bios-tables-test, LoongArch64 support is added
> and some basic ACPI tables is added here. It passes with command
> make check-qtest-loongarch64 with the result:
>  qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed

UEFI bits look good to me.

Acked-by: Gerd Hoffmann <kraxel@redhat.com>

take care,
  Gerd

Re: [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch

[Fabiano Rosas]

Bibo Mao <maobibo@loongson.cn> writes:

> Enable bios-tables-test for LoongArch virt machine, it passes to run
> with command "make check-qtest-loongarch64".
>
> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
> ---
>  tests/qtest/meson.build | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
> index 8a6243382a..8628eebfbc 100644
> --- a/tests/qtest/meson.build
> +++ b/tests/qtest/meson.build
> @@ -149,7 +149,8 @@ qtests_hppa = \
>  
>  qtests_loongarch64 = qtests_filter + \
>    (config_all_devices.has_key('CONFIG_LOONGARCH_VIRT') ? ['numa-test'] : []) + \
> -  ['boot-serial-test']
> +  ['boot-serial-test'] + \
> +  (unpack_edk2_blobs ? ['bios-tables-test'] : [])
>  
>  qtests_m68k = ['boot-serial-test'] + \
>    qtests_filter

Reviewed-by: Fabiano Rosas <farosas@suse.de>

Re: [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64

[Igor Mammedov]

On Fri, 28 Feb 2025 17:27:26 +0800
Bibo Mao <maobibo@loongson.cn> wrote:

> Add basic ACPI table test case for LoongArch64.
> 
> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
> ---
>  tests/qtest/bios-tables-test.c | 62 ++++++++++++++++++++++++++++++++++
>  1 file changed, 62 insertions(+)
> 
> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
> index 0a333ec435..008d7a0497 100644
> --- a/tests/qtest/bios-tables-test.c
> +++ b/tests/qtest/bios-tables-test.c
> @@ -2420,6 +2420,59 @@ static void test_acpi_aarch64_virt_oem_fields(void)
>      g_free(args);
>  }
>  
> +#define LOONGARCH64_INIT_TEST_DATA(data)                          \
> +    test_data data = {                                            \
> +        .machine = "virt",                                        \
> +        .arch    = "loongarch64",                                 \
> +        .tcg_only = true,                                         \
> +        .uefi_fl1 = "pc-bios/edk2-loongarch64-code.fd",           \
> +        .uefi_fl2 = "pc-bios/edk2-loongarch64-vars.fd",           \
> +        .cd = "tests/data/uefi-boot-images/"                      \
> +              "bios-tables-test.loongarch64.iso.qcow2",           \
> +        .ram_start = 0,                                           \
> +        .scan_len = 128ULL * 1024 * 1024,                         \

while it consistent with other test, we should switch to using MiB macro
so I'd like to ask
 * 1st convert all similar places in bios-tables-test.c 'x * MiB' style
 * fix up this patch to use that as well

> +    }
> +
> +static void test_acpi_loongarch64_virt(void)
> +{
> +    LOONGARCH64_INIT_TEST_DATA(data);
> +
> +    test_acpi_one("-cpu la464 ", &data);
> +    free_test_data(&data);
> +}
> +
> +static void test_acpi_loongarch64_virt_topology(void)
> +{
> +    LOONGARCH64_INIT_TEST_DATA(data);
> +
> +    data.variant = ".topology";
> +    test_acpi_one("-cpu la464 -smp sockets=1,cores=4,threads=2", &data);
> +    free_test_data(&data);
> +}
> +
> +static void test_acpi_loongarch64_virt_numamem(void)
> +{
> +    LOONGARCH64_INIT_TEST_DATA(data);
> +
> +    data.variant = ".numamem";
> +    test_acpi_one(" -cpu la464 -object memory-backend-ram,id=ram0,size=128M"
> +                  " -numa node,memdev=ram0", &data);
> +    free_test_data(&data);
> +}
> +

> +static void test_acpi_loongarch64_virt_memhp(void)
> +{
> +    LOONGARCH64_INIT_TEST_DATA(data);
> +
> +    data.variant = ".memhp";
> +    test_acpi_one(" -cpu la464 -m 128,slots=3,maxmem=1G"

> +                  " -object memory-backend-ram,id=ram0,size=64M"
> +                  " -object memory-backend-ram,id=ram1,size=64M"
> +                  " -numa node,memdev=ram0 -numa node,memdev=ram1"
> +                  " -numa dist,src=0,dst=1,val=21",
I'd test numa specific stuff in test_acpi_loongarch64_virt_numamem()
and simplify this test case.

> +                  &data);
> +    free_test_data(&data);
> +}
>  
>  int main(int argc, char *argv[])
>  {
> @@ -2593,6 +2646,15 @@ int main(int argc, char *argv[])
>              qtest_add_func("acpi/virt/numamem",
>                             test_acpi_riscv64_virt_tcg_numamem);
>          }
> +    } else if (strcmp(arch, "loongarch64") == 0) {
> +        if (has_tcg && qtest_has_device("virtio-blk-pci")) {
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
is it necessary?
if yes, then add explanation to commit message, why it's need.

> +            qtest_add_func("acpi/virt", test_acpi_loongarch64_virt);
> +            qtest_add_func("acpi/virt/topology",
> +                           test_acpi_loongarch64_virt_topology);
> +            qtest_add_func("acpi/virt/numamem",
> +                           test_acpi_loongarch64_virt_numamem);
> +            qtest_add_func("acpi/virt/memhp", test_acpi_loongarch64_virt_memhp);
> +        }
>      }
>      ret = g_test_run();
>      boot_sector_cleanup(disk);

Re: [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch

[Igor Mammedov]

On Fri, 28 Feb 2025 17:27:29 +0800
Bibo Mao <maobibo@loongson.cn> wrote:

> Enable bios-tables-test for LoongArch virt machine, it passes to run
> with command "make check-qtest-loongarch64".
> 
> Signed-off-by: Bibo Mao <maobibo@loongson.cn>

I suggest merging it into of 3/6

> ---
>  tests/qtest/meson.build | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
> index 8a6243382a..8628eebfbc 100644
> --- a/tests/qtest/meson.build
> +++ b/tests/qtest/meson.build
> @@ -149,7 +149,8 @@ qtests_hppa = \
>  
>  qtests_loongarch64 = qtests_filter + \
>    (config_all_devices.has_key('CONFIG_LOONGARCH_VIRT') ? ['numa-test'] : []) + \
> -  ['boot-serial-test']
> +  ['boot-serial-test'] + \
> +  (unpack_edk2_blobs ? ['bios-tables-test'] : [])
>  
>  qtests_m68k = ['boot-serial-test'] + \
>    qtests_filter

Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64

[Igor Mammedov]

On Fri, 28 Feb 2025 17:27:27 +0800
Bibo Mao <maobibo@loongson.cn> wrote:

> Add empty acpi table for LoongArch64 virt machine, it is only empty
> file and there is no data in these files.

this patch after 3/6 doesn't make sense,
either drop.

Adding empty blobs, can be useful (as then you can get ACPI diff during test)
but only if you follow update acpi blos process as described in
bios-tables-test.c


as is, the next patch (5/6) should precede 3/6


> 
> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
> ---
>  tests/data/acpi/loongarch64/virt/APIC | 0
>  tests/data/acpi/loongarch64/virt/DSDT | 0
>  tests/data/acpi/loongarch64/virt/FACP | 0
>  tests/data/acpi/loongarch64/virt/MCFG | 0
>  tests/data/acpi/loongarch64/virt/PPTT | 0
>  tests/data/acpi/loongarch64/virt/SLIT | 0
>  tests/data/acpi/loongarch64/virt/SPCR | 0
>  tests/data/acpi/loongarch64/virt/SRAT | 0
>  8 files changed, 0 insertions(+), 0 deletions(-)
>  create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>  create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>  create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>  create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>  create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
> 
> diff --git a/tests/data/acpi/loongarch64/virt/APIC b/tests/data/acpi/loongarch64/virt/APIC
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/DSDT b/tests/data/acpi/loongarch64/virt/DSDT
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/FACP b/tests/data/acpi/loongarch64/virt/FACP
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/MCFG b/tests/data/acpi/loongarch64/virt/MCFG
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/PPTT b/tests/data/acpi/loongarch64/virt/PPTT
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/SLIT b/tests/data/acpi/loongarch64/virt/SLIT
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/SPCR b/tests/data/acpi/loongarch64/virt/SPCR
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/loongarch64/virt/SRAT b/tests/data/acpi/loongarch64/virt/SRAT
> new file mode 100644
> index 0000000000..e69de29bb2

Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Igor Mammedov]

On Fri, 28 Feb 2025 17:27:23 +0800
Bibo Mao <maobibo@loongson.cn> wrote:

> This patchset add bios-tables-test for LoongArch64 virt machine
> system. It works with UEFI bios, with uefi-test-tools LoongArch64
> support is added to build bios-tables-test.loongarch64.iso.
> 
> Also with test case bios-tables-test, LoongArch64 support is added
> and some basic ACPI tables is added here. It passes with command
> make check-qtest-loongarch64 with the result:
>  qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed
> 
> Bibo Mao (6):
>   uefi-test-tools:: Add LoongArch64 support
>   tests/data/uefi-boot-images: Add ISO image for LoongArch system
>   tests/qtest/bios-tables-test: Add basic testing for LoongArch64
>   tests/acpi: Add empty ACPI data files for LoongArch64
>   tests/acpi: Fill acpi table data for LoongArch

blobs no longer match (since cpu hotplug was merged after this series)
so blobs need to be updated and series needs to be re-spun

>   tests/qtest: Enable bios-tables-test for LoongArch
> 
>  tests/data/acpi/loongarch64/virt/APIC         | Bin 0 -> 108 bytes
>  .../data/acpi/loongarch64/virt/APIC.topology  | Bin 0 -> 213 bytes
>  tests/data/acpi/loongarch64/virt/DSDT         | Bin 0 -> 3614 bytes
>  tests/data/acpi/loongarch64/virt/DSDT.memhp   | Bin 0 -> 4951 bytes
>  tests/data/acpi/loongarch64/virt/FACP         | Bin 0 -> 268 bytes
>  tests/data/acpi/loongarch64/virt/MCFG         | Bin 0 -> 60 bytes
>  tests/data/acpi/loongarch64/virt/PPTT         | Bin 0 -> 76 bytes
>  .../data/acpi/loongarch64/virt/PPTT.topology  | Bin 0 -> 296 bytes
>  tests/data/acpi/loongarch64/virt/SLIT         |   0
>  tests/data/acpi/loongarch64/virt/SLIT.memhp   | Bin 0 -> 48 bytes
>  tests/data/acpi/loongarch64/virt/SPCR         | Bin 0 -> 80 bytes
>  tests/data/acpi/loongarch64/virt/SRAT         | Bin 0 -> 104 bytes
>  tests/data/acpi/loongarch64/virt/SRAT.memhp   | Bin 0 -> 184 bytes
>  .../data/acpi/loongarch64/virt/SRAT.topology  | Bin 0 -> 216 bytes
>  .../bios-tables-test.loongarch64.iso.qcow2    | Bin 0 -> 12800 bytes
>  tests/qtest/bios-tables-test.c                |  62 ++++++++++++++++++
>  tests/qtest/meson.build                       |   3 +-
>  tests/uefi-test-tools/Makefile                |   5 +-
>  .../UefiTestToolsPkg/UefiTestToolsPkg.dsc     |   6 +-
>  tests/uefi-test-tools/uefi-test-build.config  |  10 +++
>  20 files changed, 82 insertions(+), 4 deletions(-)
>  create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>  create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
>  create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>  create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>  create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>  create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>  create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
>  create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology
>  create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2
> 
> 
> base-commit: b69801dd6b1eb4d107f7c2f643adf0a4e3ec9124

Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Michael S. Tsirkin]

On Fri, Feb 28, 2025 at 05:27:23PM +0800, Bibo Mao wrote:
> This patchset add bios-tables-test for LoongArch64 virt machine
> system. It works with UEFI bios, with uefi-test-tools LoongArch64
> support is added to build bios-tables-test.loongarch64.iso.
> 
> Also with test case bios-tables-test, LoongArch64 support is added
> and some basic ACPI tables is added here. It passes with command
> make check-qtest-loongarch64 with the result:
>  qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed

Still waiting for Igor's comment on this to be addressed.


> Bibo Mao (6):
>   uefi-test-tools:: Add LoongArch64 support
>   tests/data/uefi-boot-images: Add ISO image for LoongArch system
>   tests/qtest/bios-tables-test: Add basic testing for LoongArch64
>   tests/acpi: Add empty ACPI data files for LoongArch64
>   tests/acpi: Fill acpi table data for LoongArch
>   tests/qtest: Enable bios-tables-test for LoongArch
> 
>  tests/data/acpi/loongarch64/virt/APIC         | Bin 0 -> 108 bytes
>  .../data/acpi/loongarch64/virt/APIC.topology  | Bin 0 -> 213 bytes
>  tests/data/acpi/loongarch64/virt/DSDT         | Bin 0 -> 3614 bytes
>  tests/data/acpi/loongarch64/virt/DSDT.memhp   | Bin 0 -> 4951 bytes
>  tests/data/acpi/loongarch64/virt/FACP         | Bin 0 -> 268 bytes
>  tests/data/acpi/loongarch64/virt/MCFG         | Bin 0 -> 60 bytes
>  tests/data/acpi/loongarch64/virt/PPTT         | Bin 0 -> 76 bytes
>  .../data/acpi/loongarch64/virt/PPTT.topology  | Bin 0 -> 296 bytes
>  tests/data/acpi/loongarch64/virt/SLIT         |   0
>  tests/data/acpi/loongarch64/virt/SLIT.memhp   | Bin 0 -> 48 bytes
>  tests/data/acpi/loongarch64/virt/SPCR         | Bin 0 -> 80 bytes
>  tests/data/acpi/loongarch64/virt/SRAT         | Bin 0 -> 104 bytes
>  tests/data/acpi/loongarch64/virt/SRAT.memhp   | Bin 0 -> 184 bytes
>  .../data/acpi/loongarch64/virt/SRAT.topology  | Bin 0 -> 216 bytes
>  .../bios-tables-test.loongarch64.iso.qcow2    | Bin 0 -> 12800 bytes
>  tests/qtest/bios-tables-test.c                |  62 ++++++++++++++++++
>  tests/qtest/meson.build                       |   3 +-
>  tests/uefi-test-tools/Makefile                |   5 +-
>  .../UefiTestToolsPkg/UefiTestToolsPkg.dsc     |   6 +-
>  tests/uefi-test-tools/uefi-test-build.config  |  10 +++
>  20 files changed, 82 insertions(+), 4 deletions(-)
>  create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>  create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
>  create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>  create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>  create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>  create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>  create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
>  create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
>  create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology
>  create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2
> 
> 
> base-commit: b69801dd6b1eb4d107f7c2f643adf0a4e3ec9124
> -- 
> 2.39.3

Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Bibo Mao]

On 2025/5/11 下午9:17, Michael S. Tsirkin wrote:
> On Fri, Feb 28, 2025 at 05:27:23PM +0800, Bibo Mao wrote:
>> This patchset add bios-tables-test for LoongArch64 virt machine
>> system. It works with UEFI bios, with uefi-test-tools LoongArch64
>> support is added to build bios-tables-test.loongarch64.iso.
>>
>> Also with test case bios-tables-test, LoongArch64 support is added
>> and some basic ACPI tables is added here. It passes with command
>> make check-qtest-loongarch64 with the result:
>>   qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed
> 
> Still waiting for Igor's comment on this to be addressed.
Sorry for the late reply because there is on vacation leave last week.

Thanks for your reminder and I do not notice Igor's comments until this 
email. And I will reply and try hard to address it ASAP.

Regards
Bibo Mao
> 
> 
>> Bibo Mao (6):
>>    uefi-test-tools:: Add LoongArch64 support
>>    tests/data/uefi-boot-images: Add ISO image for LoongArch system
>>    tests/qtest/bios-tables-test: Add basic testing for LoongArch64
>>    tests/acpi: Add empty ACPI data files for LoongArch64
>>    tests/acpi: Fill acpi table data for LoongArch
>>    tests/qtest: Enable bios-tables-test for LoongArch
>>
>>   tests/data/acpi/loongarch64/virt/APIC         | Bin 0 -> 108 bytes
>>   .../data/acpi/loongarch64/virt/APIC.topology  | Bin 0 -> 213 bytes
>>   tests/data/acpi/loongarch64/virt/DSDT         | Bin 0 -> 3614 bytes
>>   tests/data/acpi/loongarch64/virt/DSDT.memhp   | Bin 0 -> 4951 bytes
>>   tests/data/acpi/loongarch64/virt/FACP         | Bin 0 -> 268 bytes
>>   tests/data/acpi/loongarch64/virt/MCFG         | Bin 0 -> 60 bytes
>>   tests/data/acpi/loongarch64/virt/PPTT         | Bin 0 -> 76 bytes
>>   .../data/acpi/loongarch64/virt/PPTT.topology  | Bin 0 -> 296 bytes
>>   tests/data/acpi/loongarch64/virt/SLIT         |   0
>>   tests/data/acpi/loongarch64/virt/SLIT.memhp   | Bin 0 -> 48 bytes
>>   tests/data/acpi/loongarch64/virt/SPCR         | Bin 0 -> 80 bytes
>>   tests/data/acpi/loongarch64/virt/SRAT         | Bin 0 -> 104 bytes
>>   tests/data/acpi/loongarch64/virt/SRAT.memhp   | Bin 0 -> 184 bytes
>>   .../data/acpi/loongarch64/virt/SRAT.topology  | Bin 0 -> 216 bytes
>>   .../bios-tables-test.loongarch64.iso.qcow2    | Bin 0 -> 12800 bytes
>>   tests/qtest/bios-tables-test.c                |  62 ++++++++++++++++++
>>   tests/qtest/meson.build                       |   3 +-
>>   tests/uefi-test-tools/Makefile                |   5 +-
>>   .../UefiTestToolsPkg/UefiTestToolsPkg.dsc     |   6 +-
>>   tests/uefi-test-tools/uefi-test-build.config  |  10 +++
>>   20 files changed, 82 insertions(+), 4 deletions(-)
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>>   create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology
>>   create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2
>>
>>
>> base-commit: b69801dd6b1eb4d107f7c2f643adf0a4e3ec9124
>> -- 
>> 2.39.3

Re: [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64

[Bibo Mao]

On 2025/4/1 下午9:41, Igor Mammedov wrote:
> On Fri, 28 Feb 2025 17:27:26 +0800
> Bibo Mao <maobibo@loongson.cn> wrote:
> 
>> Add basic ACPI table test case for LoongArch64.
>>
>> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
>> ---
>>   tests/qtest/bios-tables-test.c | 62 ++++++++++++++++++++++++++++++++++
>>   1 file changed, 62 insertions(+)
>>
>> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
>> index 0a333ec435..008d7a0497 100644
>> --- a/tests/qtest/bios-tables-test.c
>> +++ b/tests/qtest/bios-tables-test.c
>> @@ -2420,6 +2420,59 @@ static void test_acpi_aarch64_virt_oem_fields(void)
>>       g_free(args);
>>   }
>>   
>> +#define LOONGARCH64_INIT_TEST_DATA(data)                          \
>> +    test_data data = {                                            \
>> +        .machine = "virt",                                        \
>> +        .arch    = "loongarch64",                                 \
>> +        .tcg_only = true,                                         \
>> +        .uefi_fl1 = "pc-bios/edk2-loongarch64-code.fd",           \
>> +        .uefi_fl2 = "pc-bios/edk2-loongarch64-vars.fd",           \
>> +        .cd = "tests/data/uefi-boot-images/"                      \
>> +              "bios-tables-test.loongarch64.iso.qcow2",           \
>> +        .ram_start = 0,                                           \
>> +        .scan_len = 128ULL * 1024 * 1024,                         \
> 
> while it consistent with other test, we should switch to using MiB macro
> so I'd like to ask
>   * 1st convert all similar places in bios-tables-test.c 'x * MiB' style
>   * fix up this patch to use that as well
yes, will use MiB macro.

> 
>> +    }
>> +
>> +static void test_acpi_loongarch64_virt(void)
>> +{
>> +    LOONGARCH64_INIT_TEST_DATA(data);
>> +
>> +    test_acpi_one("-cpu la464 ", &data);
>> +    free_test_data(&data);
>> +}
>> +
>> +static void test_acpi_loongarch64_virt_topology(void)
>> +{
>> +    LOONGARCH64_INIT_TEST_DATA(data);
>> +
>> +    data.variant = ".topology";
>> +    test_acpi_one("-cpu la464 -smp sockets=1,cores=4,threads=2", &data);
>> +    free_test_data(&data);
>> +}
>> +
>> +static void test_acpi_loongarch64_virt_numamem(void)
>> +{
>> +    LOONGARCH64_INIT_TEST_DATA(data);
>> +
>> +    data.variant = ".numamem";
>> +    test_acpi_one(" -cpu la464 -object memory-backend-ram,id=ram0,size=128M"
>> +                  " -numa node,memdev=ram0", &data);
>> +    free_test_data(&data);
>> +}
>> +
> 
>> +static void test_acpi_loongarch64_virt_memhp(void)
>> +{
>> +    LOONGARCH64_INIT_TEST_DATA(data);
>> +
>> +    data.variant = ".memhp";
>> +    test_acpi_one(" -cpu la464 -m 128,slots=3,maxmem=1G"
> 
>> +                  " -object memory-backend-ram,id=ram0,size=64M"
>> +                  " -object memory-backend-ram,id=ram1,size=64M"
>> +                  " -numa node,memdev=ram0 -numa node,memdev=ram1"
>> +                  " -numa dist,src=0,dst=1,val=21",
> I'd test numa specific stuff in test_acpi_loongarch64_virt_numamem()
> and simplify this test case.
Actually it is a little strange here. There is no SLIT information in 
numamem(), however it is in memhp() test case.

Will move it to numa test case and make memory hotplug case simpler.
> 
>> +                  &data);
>> +    free_test_data(&data);
>> +}
>>   
>>   int main(int argc, char *argv[])
>>   {
>> @@ -2593,6 +2646,15 @@ int main(int argc, char *argv[])
>>               qtest_add_func("acpi/virt/numamem",
>>                              test_acpi_riscv64_virt_tcg_numamem);
>>           }
>> +    } else if (strcmp(arch, "loongarch64") == 0) {
>> +        if (has_tcg && qtest_has_device("virtio-blk-pci")) {
>                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> is it necessary?
> if yes, then add explanation to commit message, why it's need.
 From the code it is not necessary if UEFI bios is used here. It passes 
to run if it is removed.

Will remove it in next patch.

Regards
Bibo Mao
> 
>> +            qtest_add_func("acpi/virt", test_acpi_loongarch64_virt);
>> +            qtest_add_func("acpi/virt/topology",
>> +                           test_acpi_loongarch64_virt_topology);
>> +            qtest_add_func("acpi/virt/numamem",
>> +                           test_acpi_loongarch64_virt_numamem);
>> +            qtest_add_func("acpi/virt/memhp", test_acpi_loongarch64_virt_memhp);
>> +        }
>>       }
>>       ret = g_test_run();
>>       boot_sector_cleanup(disk);

Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system

[Bibo Mao]

On 2025/4/1 下午9:50, Igor Mammedov wrote:
> On Fri, 28 Feb 2025 17:27:23 +0800
> Bibo Mao <maobibo@loongson.cn> wrote:
> 
>> This patchset add bios-tables-test for LoongArch64 virt machine
>> system. It works with UEFI bios, with uefi-test-tools LoongArch64
>> support is added to build bios-tables-test.loongarch64.iso.
>>
>> Also with test case bios-tables-test, LoongArch64 support is added
>> and some basic ACPI tables is added here. It passes with command
>> make check-qtest-loongarch64 with the result:
>>   qtest-loongarch64/bios-tables-test OK 29.52s 4 subtests passed
>>
>> Bibo Mao (6):
>>    uefi-test-tools:: Add LoongArch64 support
>>    tests/data/uefi-boot-images: Add ISO image for LoongArch system
>>    tests/qtest/bios-tables-test: Add basic testing for LoongArch64
>>    tests/acpi: Add empty ACPI data files for LoongArch64
>>    tests/acpi: Fill acpi table data for LoongArch
> 
> blobs no longer match (since cpu hotplug was merged after this series)
> so blobs need to be updated and series needs to be re-spun
yes, this patch need be re-spun since cpu hotplug is merged and RSDT 
table is replaced with XSDT table.

Will refresh the patch based on the latest version.

Regards
Bibo Mao
> 
>>    tests/qtest: Enable bios-tables-test for LoongArch
>>
>>   tests/data/acpi/loongarch64/virt/APIC         | Bin 0 -> 108 bytes
>>   .../data/acpi/loongarch64/virt/APIC.topology  | Bin 0 -> 213 bytes
>>   tests/data/acpi/loongarch64/virt/DSDT         | Bin 0 -> 3614 bytes
>>   tests/data/acpi/loongarch64/virt/DSDT.memhp   | Bin 0 -> 4951 bytes
>>   tests/data/acpi/loongarch64/virt/FACP         | Bin 0 -> 268 bytes
>>   tests/data/acpi/loongarch64/virt/MCFG         | Bin 0 -> 60 bytes
>>   tests/data/acpi/loongarch64/virt/PPTT         | Bin 0 -> 76 bytes
>>   .../data/acpi/loongarch64/virt/PPTT.topology  | Bin 0 -> 296 bytes
>>   tests/data/acpi/loongarch64/virt/SLIT         |   0
>>   tests/data/acpi/loongarch64/virt/SLIT.memhp   | Bin 0 -> 48 bytes
>>   tests/data/acpi/loongarch64/virt/SPCR         | Bin 0 -> 80 bytes
>>   tests/data/acpi/loongarch64/virt/SRAT         | Bin 0 -> 104 bytes
>>   tests/data/acpi/loongarch64/virt/SRAT.memhp   | Bin 0 -> 184 bytes
>>   .../data/acpi/loongarch64/virt/SRAT.topology  | Bin 0 -> 216 bytes
>>   .../bios-tables-test.loongarch64.iso.qcow2    | Bin 0 -> 12800 bytes
>>   tests/qtest/bios-tables-test.c                |  62 ++++++++++++++++++
>>   tests/qtest/meson.build                       |   3 +-
>>   tests/uefi-test-tools/Makefile                |   5 +-
>>   .../UefiTestToolsPkg/UefiTestToolsPkg.dsc     |   6 +-
>>   tests/uefi-test-tools/uefi-test-build.config  |  10 +++
>>   20 files changed, 82 insertions(+), 4 deletions(-)
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC.topology
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>>   create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT.topology
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.memhp
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT.topology
>>   create mode 100644 tests/data/uefi-boot-images/bios-tables-test.loongarch64.iso.qcow2
>>
>>
>> base-commit: b69801dd6b1eb4d107f7c2f643adf0a4e3ec9124

Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64

[Bibo Mao]

On 2025/4/1 下午9:48, Igor Mammedov wrote:
> On Fri, 28 Feb 2025 17:27:27 +0800
> Bibo Mao <maobibo@loongson.cn> wrote:
> 
>> Add empty acpi table for LoongArch64 virt machine, it is only empty
>> file and there is no data in these files.
> 
> this patch after 3/6 doesn't make sense,
> either drop.
Empty acpi table can be dropped.
> 
> Adding empty blobs, can be useful (as then you can get ACPI diff during test)
> but only if you follow update acpi blos process as described in
> bios-tables-test.c
> 
> 
> as is, the next patch (5/6) should precede 3/6
yes, it is strange about patch order. How about split patch 3/6 into 
smaller patches, one patch for one test case, and move specified acpi 
table into smaller patches.

Regards
Bibo Mao
> 
> 
>>
>> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
>> ---
>>   tests/data/acpi/loongarch64/virt/APIC | 0
>>   tests/data/acpi/loongarch64/virt/DSDT | 0
>>   tests/data/acpi/loongarch64/virt/FACP | 0
>>   tests/data/acpi/loongarch64/virt/MCFG | 0
>>   tests/data/acpi/loongarch64/virt/PPTT | 0
>>   tests/data/acpi/loongarch64/virt/SLIT | 0
>>   tests/data/acpi/loongarch64/virt/SPCR | 0
>>   tests/data/acpi/loongarch64/virt/SRAT | 0
>>   8 files changed, 0 insertions(+), 0 deletions(-)
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>>   create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>>
>> diff --git a/tests/data/acpi/loongarch64/virt/APIC b/tests/data/acpi/loongarch64/virt/APIC
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/DSDT b/tests/data/acpi/loongarch64/virt/DSDT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/FACP b/tests/data/acpi/loongarch64/virt/FACP
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/MCFG b/tests/data/acpi/loongarch64/virt/MCFG
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/PPTT b/tests/data/acpi/loongarch64/virt/PPTT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SLIT b/tests/data/acpi/loongarch64/virt/SLIT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SPCR b/tests/data/acpi/loongarch64/virt/SPCR
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SRAT b/tests/data/acpi/loongarch64/virt/SRAT
>> new file mode 100644
>> index 0000000000..e69de29bb2

Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64

[Bibo Mao]

On 2025/4/1 下午9:48, Igor Mammedov wrote:
> On Fri, 28 Feb 2025 17:27:27 +0800
> Bibo Mao <maobibo@loongson.cn> wrote:
> 
>> Add empty acpi table for LoongArch64 virt machine, it is only empty
>> file and there is no data in these files.
> 
> this patch after 3/6 doesn't make sense,
> either drop.
> 
> Adding empty blobs, can be useful (as then you can get ACPI diff during test)
> but only if you follow update acpi blos process as described in
> bios-tables-test.c
> 
> 
> as is, the next patch (5/6) should precede 3/6
Hi Ignor,

With file tests/qtest/bios-tables-test.c, there is commit steps.

  How to add or update the tests or commit changes that affect ACPI tables:
  Contributor:
  1. add empty files for new tables, if any, under tests/data/acpi
  2. list any changed files in tests/qtest/bios-tables-test-allowed-diff.h
  3. commit the above *before* making changes that affect the tables

I will follow steps of above in next round, how do you think of it?

Regards
Bibo Mao
> 
> 
>>
>> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
>> ---
>>   tests/data/acpi/loongarch64/virt/APIC | 0
>>   tests/data/acpi/loongarch64/virt/DSDT | 0
>>   tests/data/acpi/loongarch64/virt/FACP | 0
>>   tests/data/acpi/loongarch64/virt/MCFG | 0
>>   tests/data/acpi/loongarch64/virt/PPTT | 0
>>   tests/data/acpi/loongarch64/virt/SLIT | 0
>>   tests/data/acpi/loongarch64/virt/SPCR | 0
>>   tests/data/acpi/loongarch64/virt/SRAT | 0
>>   8 files changed, 0 insertions(+), 0 deletions(-)
>>   create mode 100644 tests/data/acpi/loongarch64/virt/APIC
>>   create mode 100644 tests/data/acpi/loongarch64/virt/DSDT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/FACP
>>   create mode 100644 tests/data/acpi/loongarch64/virt/MCFG
>>   create mode 100644 tests/data/acpi/loongarch64/virt/PPTT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SLIT
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SPCR
>>   create mode 100644 tests/data/acpi/loongarch64/virt/SRAT
>>
>> diff --git a/tests/data/acpi/loongarch64/virt/APIC b/tests/data/acpi/loongarch64/virt/APIC
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/DSDT b/tests/data/acpi/loongarch64/virt/DSDT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/FACP b/tests/data/acpi/loongarch64/virt/FACP
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/MCFG b/tests/data/acpi/loongarch64/virt/MCFG
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/PPTT b/tests/data/acpi/loongarch64/virt/PPTT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SLIT b/tests/data/acpi/loongarch64/virt/SLIT
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SPCR b/tests/data/acpi/loongarch64/virt/SPCR
>> new file mode 100644
>> index 0000000000..e69de29bb2
>> diff --git a/tests/data/acpi/loongarch64/virt/SRAT b/tests/data/acpi/loongarch64/virt/SRAT
>> new file mode 100644
>> index 0000000000..e69de29bb2