qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Igor Mammedov <imammedo@redhat.com>
To: qemu-devel@nongnu.org
Cc: mst@redhat.com, ehabkost@redhat.com, marcel@redhat.com,
	eblake@redhat.com, armbru@redhat.com, drjones@redhat.com
Subject: [Qemu-devel] [RFC 14/42] tests: acpi: check legacy CPU hotplug AML is present for 2.6 machine type
Date: Mon,  2 May 2016 14:33:23 +0200	[thread overview]
Message-ID: <1462192431-146342-15-git-send-email-imammedo@redhat.com> (raw)
In-Reply-To: <1462192431-146342-1-git-send-email-imammedo@redhat.com>

it should help to notice regression if legacy CPU hotplug
unintentionally disappears in old machine types.

Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
 tests/acpi-test-data/pc/DSDT.cphp_legacy | Bin 0 -> 5502 bytes
 tests/bios-tables-test.c                 |  13 +++++++++++++
 2 files changed, 13 insertions(+)
 create mode 100644 tests/acpi-test-data/pc/DSDT.cphp_legacy

diff --git a/tests/acpi-test-data/pc/DSDT.cphp_legacy b/tests/acpi-test-data/pc/DSDT.cphp_legacy
new file mode 100644
index 0000000000000000000000000000000000000000..2744030a49d16d93c9806334b84c658c8faa5435
GIT binary patch
literal 5502
zcmb7IUvCrH6+hQwC*x~787D&s1Qsj6(xnCb2Lh{BRqPojcJMfZ$Dve%I^!g$rrMTK
z7NiZ)R*6>%EN@#cRr_X;pW!34kA2*au>A^Kb>WHhoEcxo&F)mxlE>%I@1Aq+z2}^J
z$GW9A|9A|56-%pH#mbcKn3|3<1^`6UuTqWP1$$R7Ejug`i86aH;nVmi$?DJa(%Lfn
zOWXak>ppzqvX%$yFO}!(E$@q`?m7_Y(YnVu5z4${l^nY>XKK}~>{P0HviJ>`WT&V%
zKz7!uI#Ucez?9{RS#AmtLlCMljEdPbAPMlzy97k-J%#w`+vU2RF&ny*Q5PJmSp<(H
zu1BKQW0xy#FZos1g|3@8IOs!f;;U{8zHTMqpZ|{aq0c8}M{8K=G@say<BZWbL;<!z
zIe(d=p>XZa3G+^_;K!PjfMF<RuAoJLA1I5l5ttUOT16+o;ldK`YF3MImt(je{^l}S
zp^*TUD==O&%XJ1fK+=jtNLmSh6z$9vNjN5JcIQ%~siDze$r89&6z?oQw#+>6K#9x)
zTV`9Z1(C6Bushdpmh>T9lGM1rD#MBT16^=|;iyg6#NYV{&e!f-Fjm)w11G1C4wQLp
z9uxu=<t1GkZsPbLqB%ah%)XPplOVdx-bin7q2?FiV$`>tTdJ8wvDr&{bt<C(kMUKW
zW&Aad@bmbc;&c3ff5N}#lU$aaCB2+J&Q(go(N6>!yYtJM*&N0i1$8#qdRkJqH-_8Y
zhZoV|oE4s0W}iu)NwPe@>&H`$YO6+0ENYAe87R!qhJ=}UE4`IKo>zP`OJpX0#0-xN
z?ck1v+KIf4pdA_QC>8lIiFQrRYO(~p;xeW<F&gIJS_U6_#@@BcD_Dy1E6S(@F<L2G
zUO`2vZZudz2^7R=rTUtYv0166-30Z#sg+kE_F3W%4!kxgy=`_MXt;fW#C=feSVAR7
zxydQ*Rh>I#qY2(rWY_he^|BOy-r~F7;}(y=y0^2{S_dw{Mo;N(ASjMyQ>)+9VY7Fx
zyTN+tZi`2UVhc83q=sA(LFw)Wj|?THfkYZgG}!b$-(KemVBx~kO#pAFh5nvyGVnH^
zx-EFr>bv&dxhoPyOSMAP#>~d8tc;HwJ@WJ@$m(?U7OrY9zQcFhSoHBePpAvkTUsNI
zfJQbw!4qGOLEKNaZFoj99>Zs~mw4XBIk$=Kli2Rm#&MdM%t1jWv4K!~l5wr9^bDW=
zasuK~46~HuH-X=6P$z>Lq#!6fHSOqC3qB{B@A{HSs%q#{NIrkuu^JY8)()b;Zq}%h
zeBixw#5?Bidwb7~c+dIXa~R*K_ni9U-g9=xpt);Dyz|Ka;XXg&KJUBFA90@#ai1T!
z*N?i#1LD1BUl?&;@ZA@VxG#jbFW8;cD9+wG>OLB{7b!%i2yVG(y9ss7-)uDztTBa(
zBHCR=$F@~+)N7h)6d4|xnz>qUgX^UeTm~8K;HTxapHjj3KN5H@a#85F)RUTtDSbGq
z7jv=Uy^#8#a?PY-pXdKjSH{qv2ADvWa}UpF58RhavFGj+@--OjLaWDjTs{tu*V+1u
zB#Q6h$teL<`8vZ2pZ`<0wc%~}0lLuRSl{*L(yv|iLiT^T^vJb4H=~!S63s?=xJG=;
zua1;vu4O6@9(tcV`fX<G-lq>9ZEf896l(?#Y@P^Cv=ryd3|_-)mcc^?tHN)CuX)Da
zOZp5=VNtQaC}?Ij8!V2Rbc|vX`m}(644wbTSWQq<PI)EEkSVV~TTN4#5e&Qy`!xK%
zO!J?T7*l8NUUrtvvQy6H0ZBUZamAGSrNf@wDPA%vW{`I&vu<XK*#!(9%RXh?`~8Eh
z?M(Xzb5p1NXnOIkq1PLD9%0L|nc+9%;+SD+bvV@FuGnWucX06X7tdW9;kkxYuJxw7
zMzwA+!(VNW5%XfEr*@5oaXXN53B|~5xy-dMkdZ!9FB!CaBWAt7w!t@;=o)3qVIH3p
zXjd)H8d(2v5rV+VMq}Y>y1=u~ehPpWpCg}l2f;^(2kZg%Pa4Q4FxXl^P9a$d$Pa5}
zA-#y?KssX-?+mafk*s(|qg0_mMAXO-6155qAfiS_wn$6_jU~oc)99Z1oMfcmBA)re
zv@c{2hZ3S{q^tGx@h*u0r(DRV&+t=Ej*m{6rb^SKDf-hZ<T*$44$cY52^>2hXD}eg
zBhRGv$T%rIij!1>9im}Fa^b3a&M@y7?C1t~D0L3E=_kgj)r|fCbl+Gl-}ZioN-Z#{
z&KsEfc-RLSCAJgO1n&z0M5&HZ()+C}Rn*LCHj62(evDO+1N-dV!#ikjVHgpZTD@ss
z7!eRcJ%J+&aQ8}7++C?)D9e>{dQer^8iGnMjq^V|O+f^&^0cP(z`N+%<l7{ZQO<%?
zdxPq?$swQeci>;6`@(TQ=zyh91bmISEy@Q6e|r~b{{B7PS+yumH@LGNsbG@-&z|({
z>u+DPC>dFaniaDVEZTp!0qlG4*WX^jE2iR=@1wia{R+<1Sd?MVNMCiP<S1T)@q&xo
zHPX-ol%F3ipWV5<^eb&C2;f%PSi@b*6Z%LORbonvCDF^u_Fex%XUHS_<+n1wDUCK>
zj`x*WYP=PbE|S3cti5-J7>P<SLj{SC8c{W*Ff*|;pC&*N7P;S4Fy*mT`JIWbsrKHn
zU_3Um3?(T(EG?ssQ8$Yy#sfMn<PvIeZ4*{Di#K|D9ke?at=xdYB#3~DQ_ShuLy5=>
zqCkY*`N+!Mz=76UeegPoLnKhA{Qgh3UUEQcg67FALGjQ=R|v(IhY+Z+8WYf17_=a)
z;%OIH9SemDtK$MX9tOQCtm4a1U{wx<3afDejfX+639DQ{c_>s^#S=ORAQ1*#5LS}{
znhb>st0@6Zg+Z?is}lk`5egMn@j4#FIvEDNA*?C_s)Ry?)wF=7!=N7vt5X6x6$%wr
zrv-F63|bUcG4q3Isi9C|^_YMj3xk$~)foYu355!)#|89w7*rEhX9aXN6e_Hq5YQ80
zP+eF(DWE4qp~C8%fX;<M%fjj@0X-E86;@9R=;<)%lCb)LfPN4P6;{s(=$Rjbs+BqY
zcD)(tC3o8RPOwkC?!M0`$Y3`Mm<%yOvxp3Kq(HMv3~5q3?F9ZMwQpft8E_HehPo(n
z*c=8r@^x*fBWGCR108u=80yF!7V1Dp{;rd5a;J^m6_2)k3ky#0c(@^dsK+C2-@?)n
zdVJ|Y{!p(Fk3}H#8J|DYFA|UGEcClRf2j9ojEO1qc$n;4+K8TJjVU(N)4cT&J<S}G
oWvI^(KcZhm9=Cm{-z6UP>KVEx;s0~=M+<Eso^@+%K<Hfh4+O78OaK4?

literal 0
HcmV?d00001

diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
index 0352814..191e701 100644
--- a/tests/bios-tables-test.c
+++ b/tests/bios-tables-test.c
@@ -781,6 +781,17 @@ static void test_acpi_q35_tcg_bridge(void)
     free_test_data(&data);
 }
 
+static void test_acpi_piix4_tcg_cpuhp_legacy(void)
+{
+    test_data data;
+
+    memset(&data, 0, sizeof(data));
+    data.machine = MACHINE_PC;
+    data.variant = ".cphp_legacy";
+    test_acpi_one("-machine pc-i440fx-2.6,accel=tcg", &data);
+    free_test_data(&data);
+}
+
 int main(int argc, char *argv[])
 {
     const char *arch = qtest_get_arch();
@@ -797,6 +808,8 @@ int main(int argc, char *argv[])
         qtest_add_func("acpi/piix4/tcg/bridge", test_acpi_piix4_tcg_bridge);
         qtest_add_func("acpi/q35/tcg", test_acpi_q35_tcg);
         qtest_add_func("acpi/q35/tcg/bridge", test_acpi_q35_tcg_bridge);
+        qtest_add_func("acpi/piix4/tcg/cpuhp/legacy",
+                       test_acpi_piix4_tcg_cpuhp_legacy);
     }
     ret = g_test_run();
     boot_sector_cleanup(disk);
-- 
1.8.3.1

  parent reply	other threads:[~2016-05-02 12:34 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-02 12:33 [Qemu-devel] [RFC 00/42] ACPI CPU hotplug refactoring to support more than 255 CPUs and PXM/OST methods Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 01/42] acpi: add aml_debug() Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 02/42] acpi: add aml_refof() Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 03/42] pc: acpi: remove AML for empty/not used GPE handlers Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 04/42] pc: acpi: consolidate CPU hotplug AML Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 05/42] pc: acpi: consolidate \GPE._E02 with the rest of " Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 06/42] pc: acpi: cpu-hotplug: make AML CPU_foo defines local to cpu_hotplug_acpi_table.c Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 07/42] pc: acpi: mark current CPU hotplug functions as legacy Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 08/42] pc: acpi: consolidate legacy CPU hotplug in one file Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 09/42] pc: acpi: simplify build_legacy_cpu_hotplug_aml() signature Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 10/42] pc: piix4/ich9: add 'cpu-hotplug-legacy' property Igor Mammedov
2016-05-10 20:19   ` Eduardo Habkost
2016-05-11 13:33     ` Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 11/42] pc: add 2.7 machine Igor Mammedov
2016-05-10 20:20   ` Eduardo Habkost
2016-05-02 12:33 ` [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types Igor Mammedov
2016-05-10 20:24   ` Eduardo Habkost
2016-05-11 13:50     ` Igor Mammedov
2016-05-11 21:51       ` Michael S. Tsirkin
2016-05-11 23:28         ` Eduardo Habkost
2016-05-12  7:07           ` Michael S. Tsirkin
2016-05-12 10:29             ` Igor Mammedov
2016-05-12 10:52               ` Eduardo Habkost
2016-05-12 11:04                 ` [Qemu-devel] [libvirt] " Peter Krempa
2016-05-12 13:20                   ` Igor Mammedov
2016-05-12 13:27                     ` Peter Krempa
2016-05-12 14:30                       ` Igor Mammedov
2016-05-11 23:36       ` [Qemu-devel] " Eduardo Habkost
2016-05-12  7:05         ` Michael S. Tsirkin
2016-05-12 10:39           ` Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 13/42] tests: bios-tables-test: update tables with CPHP turned off by default Igor Mammedov
2016-05-02 12:33 ` Igor Mammedov [this message]
2016-05-02 12:33 ` [Qemu-devel] [RFC 15/42] acpi: extend ACPI interface to provide send_event hook Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 16/42] pc: use AcpiDeviceIfClass.send_event to issue GPE events Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 17/42] docs: update ACPI CPU hotplug spec with new protocol Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 18/42] acpi: hardware side of CPU hotplug Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 19/42] pc: add generic CPU unplug callbacks Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 20/42] machine: add cpu-hotplug machine option Igor Mammedov
2016-05-10 20:27   ` Eduardo Habkost
2016-05-11 14:00     ` Igor Mammedov
2016-05-11 15:00       ` Eduardo Habkost
2016-05-02 12:33 ` [Qemu-devel] [RFC 21/42] pc: q35: initialize new CPU hotplug hw Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 22/42] pc: piix4: " Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 23/42] tests: pc-cpu-test: turn on cpu-hotplug explicily Igor Mammedov
2016-05-10 20:28   ` Eduardo Habkost
2016-05-02 12:33 ` [Qemu-devel] [RFC 24/42] pc: acpi: cpuhp-legacy: switch ProcessorID to possible_cpus idx Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 25/42] tests: acpi: update cphp_legacy case with new ProcessorID values Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 26/42] pc: acpi: introduce AcpiDeviceIfClass.madt_cpu hook Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 27/42] acpi: add CPU devices AML to DSDT Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 28/42] acpi: add CPU hotplug methods " Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 29/42] tests: acpi: update expected files with new CPU AML description Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 30/42] qdev: hotplug: Introduce HotplugHandler.pre_plug() callback Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 31/42] pc: numa: replace node_cpu indexing by apic_id with possible_cpus index Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 32/42] pc: set X86CPU.node property if QEMU starts with numa enabled Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 33/42] target-i386: add X86CPU.node property Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 34/42] acpi: cpuhp: add command and data registers Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 35/42] acpi: cpuhp: provide cpu._PXM method if running in numa mode Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 36/42] acpi: cpuhp: add cpu._OST handling Igor Mammedov
2016-05-10 21:07   ` Eric Blake
2016-05-11 14:02     ` Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 37/42] tests: acpi: report names of expected files in verbose mode Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 38/42] tests: acpi: add CPU hotplug testcase Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 39/42] tests: acpi: add expected tables for CPU hotplug case Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 40/42] tests: acpi: extend CPU hotplug test with NUMA support Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 41/42] tests: acpi: cpuhp: add expected SRAT tables and update DSDT with _PXM support Igor Mammedov
2016-05-02 12:33 ` [Qemu-devel] [RFC 42/42] DO NOT APPLY: simulate CPU unplug when executin cpu-add on a present CPU Igor Mammedov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1462192431-146342-15-git-send-email-imammedo@redhat.com \
    --to=imammedo@redhat.com \
    --cc=armbru@redhat.com \
    --cc=drjones@redhat.com \
    --cc=eblake@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=marcel@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).