* [PATCH 1/6] uefi-test-tools:: Add LoongArch64 support
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-02-28 9:27 ` [PATCH 2/6] tests/data/uefi-boot-images: Add ISO image for LoongArch system Bibo Mao
` (7 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply related [flat|nested] 19+ messages in thread* [PATCH 2/6] tests/data/uefi-boot-images: Add ISO image for LoongArch system
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
2025-02-28 9:27 ` [PATCH 1/6] uefi-test-tools:: Add LoongArch64 support Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-02-28 9:27 ` [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64 Bibo Mao
` (6 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
2025-02-28 9:27 ` [PATCH 1/6] uefi-test-tools:: Add LoongArch64 support Bibo Mao
2025-02-28 9:27 ` [PATCH 2/6] tests/data/uefi-boot-images: Add ISO image for LoongArch system Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-04-01 13:41 ` Igor Mammedov
2025-02-28 9:27 ` [PATCH 4/6] tests/acpi: Add empty ACPI data files " Bibo Mao
` (5 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply related [flat|nested] 19+ messages in thread* Re: [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64
2025-02-28 9:27 ` [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64 Bibo Mao
@ 2025-04-01 13:41 ` Igor Mammedov
2025-05-19 2:02 ` Bibo Mao
0 siblings, 1 reply; 19+ messages in thread
From: Igor Mammedov @ 2025-04-01 13:41 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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);
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64
2025-04-01 13:41 ` Igor Mammedov
@ 2025-05-19 2:02 ` Bibo Mao
0 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-05-19 2:02 UTC (permalink / raw)
To: Igor Mammedov
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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);
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (2 preceding siblings ...)
2025-02-28 9:27 ` [PATCH 3/6] tests/qtest/bios-tables-test: Add basic testing for LoongArch64 Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-04-01 13:48 ` Igor Mammedov
2025-02-28 9:27 ` [PATCH 5/6] tests/acpi: Fill acpi table data for LoongArch Bibo Mao
` (4 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64
2025-02-28 9:27 ` [PATCH 4/6] tests/acpi: Add empty ACPI data files " Bibo Mao
@ 2025-04-01 13:48 ` Igor Mammedov
2025-05-19 2:46 ` Bibo Mao
2025-05-20 10:07 ` Bibo Mao
0 siblings, 2 replies; 19+ messages in thread
From: Igor Mammedov @ 2025-04-01 13:48 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64
2025-04-01 13:48 ` Igor Mammedov
@ 2025-05-19 2:46 ` Bibo Mao
2025-05-20 10:07 ` Bibo Mao
1 sibling, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-05-19 2:46 UTC (permalink / raw)
To: Igor Mammedov
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/6] tests/acpi: Add empty ACPI data files for LoongArch64
2025-04-01 13:48 ` Igor Mammedov
2025-05-19 2:46 ` Bibo Mao
@ 2025-05-20 10:07 ` Bibo Mao
1 sibling, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-05-20 10:07 UTC (permalink / raw)
To: Igor Mammedov
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 5/6] tests/acpi: Fill acpi table data for LoongArch
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (3 preceding siblings ...)
2025-02-28 9:27 ` [PATCH 4/6] tests/acpi: Add empty ACPI data files " Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-02-28 9:27 ` [PATCH 6/6] tests/qtest: Enable bios-tables-test " Bibo Mao
` (3 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (4 preceding siblings ...)
2025-02-28 9:27 ` [PATCH 5/6] tests/acpi: Fill acpi table data for LoongArch Bibo Mao
@ 2025-02-28 9:27 ` Bibo Mao
2025-03-19 12:06 ` Fabiano Rosas
2025-04-01 13:42 ` Igor Mammedov
2025-02-28 12:56 ` [PATCH 0/6] Add bios-tables-test for LoongArch64 system Gerd Hoffmann
` (2 subsequent siblings)
8 siblings, 2 replies; 19+ messages in thread
From: Bibo Mao @ 2025-02-28 9:27 UTC (permalink / raw)
To: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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
^ permalink raw reply related [flat|nested] 19+ messages in thread* Re: [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch
2025-02-28 9:27 ` [PATCH 6/6] tests/qtest: Enable bios-tables-test " Bibo Mao
@ 2025-03-19 12:06 ` Fabiano Rosas
2025-04-01 13:42 ` Igor Mammedov
1 sibling, 0 replies; 19+ messages in thread
From: Fabiano Rosas @ 2025-03-19 12:06 UTC (permalink / raw)
To: Bibo Mao, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé,
Gerd Hoffmann
Cc: qemu-devel
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>
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch
2025-02-28 9:27 ` [PATCH 6/6] tests/qtest: Enable bios-tables-test " Bibo Mao
2025-03-19 12:06 ` Fabiano Rosas
@ 2025-04-01 13:42 ` Igor Mammedov
1 sibling, 0 replies; 19+ messages in thread
From: Igor Mammedov @ 2025-04-01 13:42 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (5 preceding siblings ...)
2025-02-28 9:27 ` [PATCH 6/6] tests/qtest: Enable bios-tables-test " Bibo Mao
@ 2025-02-28 12:56 ` Gerd Hoffmann
2025-04-01 13:50 ` Igor Mammedov
2025-05-11 13:17 ` Michael S. Tsirkin
8 siblings, 0 replies; 19+ messages in thread
From: Gerd Hoffmann @ 2025-02-28 12:56 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Igor Mammedov,
Ani Sinha, Paolo Bonzini, Philippe Mathieu-Daudé, qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (6 preceding siblings ...)
2025-02-28 12:56 ` [PATCH 0/6] Add bios-tables-test for LoongArch64 system Gerd Hoffmann
@ 2025-04-01 13:50 ` Igor Mammedov
2025-05-19 2:14 ` Bibo Mao
2025-05-11 13:17 ` Michael S. Tsirkin
8 siblings, 1 reply; 19+ messages in thread
From: Igor Mammedov @ 2025-04-01 13:50 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system
2025-04-01 13:50 ` Igor Mammedov
@ 2025-05-19 2:14 ` Bibo Mao
0 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-05-19 2:14 UTC (permalink / raw)
To: Igor Mammedov
Cc: Fabiano Rosas, Laurent Vivier, Michael S . Tsirkin, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system
2025-02-28 9:27 [PATCH 0/6] Add bios-tables-test for LoongArch64 system Bibo Mao
` (7 preceding siblings ...)
2025-04-01 13:50 ` Igor Mammedov
@ 2025-05-11 13:17 ` Michael S. Tsirkin
2025-05-19 1:02 ` Bibo Mao
8 siblings, 1 reply; 19+ messages in thread
From: Michael S. Tsirkin @ 2025-05-11 13:17 UTC (permalink / raw)
To: Bibo Mao
Cc: Fabiano Rosas, Laurent Vivier, Igor Mammedov, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH 0/6] Add bios-tables-test for LoongArch64 system
2025-05-11 13:17 ` Michael S. Tsirkin
@ 2025-05-19 1:02 ` Bibo Mao
0 siblings, 0 replies; 19+ messages in thread
From: Bibo Mao @ 2025-05-19 1:02 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Fabiano Rosas, Laurent Vivier, Igor Mammedov, Ani Sinha,
Paolo Bonzini, Philippe Mathieu-Daudé, Gerd Hoffmann,
qemu-devel
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
^ permalink raw reply [flat|nested] 19+ messages in thread