qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	qemu-devel@nongnu.org,
	Marcel Apfelbaum <marcel.apfelbaum@gmail.com>,
	Richard Henderson <rth@twiddle.net>
Cc: Igor Mammedov <imammedo@redhat.com>
Subject: [Qemu-devel] [PULL v4 43/43] tests: use -numa memdev option in tests instead of legacy 'mem' option
Date: Wed,  3 Jul 2019 15:07:26 -0300	[thread overview]
Message-ID: <20190703180726.31267-44-ehabkost@redhat.com> (raw)
In-Reply-To: <20190703180726.31267-1-ehabkost@redhat.com>

From: Igor Mammedov <imammedo@redhat.com>

it will test preferred memdev option more extensively and remove
undesired deprecation warnings during 'make check'

Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Message-Id: <20190702140745.27767-3-imammedo@redhat.com>
[ehabkost: remove numa-test.c changes]
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---
 tests/bios-tables-test.c | 40 ++++++++++++++++++++++++++++------------
 1 file changed, 28 insertions(+), 12 deletions(-)

diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
index 0ce55182f2..d863233fe9 100644
--- a/tests/bios-tables-test.c
+++ b/tests/bios-tables-test.c
@@ -688,6 +688,8 @@ static void test_acpi_q35_tcg_mmio64(void)
     };
 
     test_acpi_one("-m 128M,slots=1,maxmem=2G "
+                  "-object memory-backend-ram,id=ram0,size=128M "
+                  "-numa node,memdev=ram0 "
                   "-device pci-testdev,membar=2G",
                   &data);
     free_test_data(&data);
@@ -701,7 +703,9 @@ static void test_acpi_piix4_tcg_cphp(void)
     data.machine = MACHINE_PC;
     data.variant = ".cphp";
     test_acpi_one("-smp 2,cores=3,sockets=2,maxcpus=6"
-                  " -numa node -numa node"
+                  " -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);
@@ -715,7 +719,9 @@ static void test_acpi_q35_tcg_cphp(void)
     data.machine = MACHINE_Q35;
     data.variant = ".cphp";
     test_acpi_one(" -smp 2,cores=3,sockets=2,maxcpus=6"
-                  " -numa node -numa node"
+                  " -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);
@@ -766,7 +772,9 @@ static void test_acpi_q35_tcg_memhp(void)
     data.machine = MACHINE_Q35;
     data.variant = ".memhp";
     test_acpi_one(" -m 128,slots=3,maxmem=1G"
-                  " -numa node -numa node"
+                  " -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);
@@ -780,7 +788,9 @@ static void test_acpi_piix4_tcg_memhp(void)
     data.machine = MACHINE_PC;
     data.variant = ".memhp";
     test_acpi_one(" -m 128,slots=3,maxmem=1G"
-                  " -numa node -numa node"
+                  " -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);
@@ -793,7 +803,8 @@ static void test_acpi_q35_tcg_numamem(void)
     memset(&data, 0, sizeof(data));
     data.machine = MACHINE_Q35;
     data.variant = ".numamem";
-    test_acpi_one(" -numa node -numa node,mem=128", &data);
+    test_acpi_one(" -object memory-backend-ram,id=ram0,size=128M"
+                  " -numa node -numa node,memdev=ram0", &data);
     free_test_data(&data);
 }
 
@@ -804,7 +815,8 @@ static void test_acpi_piix4_tcg_numamem(void)
     memset(&data, 0, sizeof(data));
     data.machine = MACHINE_PC;
     data.variant = ".numamem";
-    test_acpi_one(" -numa node -numa node,mem=128", &data);
+    test_acpi_one(" -object memory-backend-ram,id=ram0,size=128M"
+                  " -numa node -numa node,memdev=ram0", &data);
     free_test_data(&data);
 }
 
@@ -818,17 +830,21 @@ static void test_acpi_tcg_dimm_pxm(const char *machine)
     test_acpi_one(" -machine nvdimm=on,nvdimm-persistence=cpu"
                   " -smp 4,sockets=4"
                   " -m 128M,slots=3,maxmem=1G"
-                  " -numa node,mem=32M,nodeid=0"
-                  " -numa node,mem=32M,nodeid=1"
-                  " -numa node,mem=32M,nodeid=2"
-                  " -numa node,mem=32M,nodeid=3"
+                  " -object memory-backend-ram,id=ram0,size=32M"
+                  " -object memory-backend-ram,id=ram1,size=32M"
+                  " -object memory-backend-ram,id=ram2,size=32M"
+                  " -object memory-backend-ram,id=ram3,size=32M"
+                  " -numa node,memdev=ram0,nodeid=0"
+                  " -numa node,memdev=ram1,nodeid=1"
+                  " -numa node,memdev=ram2,nodeid=2"
+                  " -numa node,memdev=ram3,nodeid=3"
                   " -numa cpu,node-id=0,socket-id=0"
                   " -numa cpu,node-id=1,socket-id=1"
                   " -numa cpu,node-id=2,socket-id=2"
                   " -numa cpu,node-id=3,socket-id=3"
-                  " -object memory-backend-ram,id=ram0,size=128M"
+                  " -object memory-backend-ram,id=ram4,size=128M"
                   " -object memory-backend-ram,id=nvm0,size=128M"
-                  " -device pc-dimm,id=dimm0,memdev=ram0,node=1"
+                  " -device pc-dimm,id=dimm0,memdev=ram4,node=1"
                   " -device nvdimm,id=dimm1,memdev=nvm0,node=2",
                   &data);
     free_test_data(&data);
-- 
2.18.0.rc1.1.g3f1ff2140



  parent reply	other threads:[~2019-07-03 18:36 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-03 18:06 [Qemu-devel] [PULL v4 00/43] Machine and x86 queue, 2019-07-03 Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 01/43] hw/boards: Add struct CpuTopology to MachineState Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 02/43] machine: Refactor smp-related call chains to pass MachineState Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 03/43] general: Replace global smp variables with smp machine properties Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 04/43] hw/ppc: Replace global smp variables with machine smp properties Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 05/43] hw/riscv: " Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 06/43] hw/s390x: " Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 07/43] hw/i386: " Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 08/43] hw/arm: " Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 09/43] hw: Replace global smp variables with MachineState for all remaining archs Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 10/43] vl.c: Replace smp global variables with smp machine properties Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 11/43] i386: Add die-level cpu topology to x86CPU on PCMachine Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 12/43] hw/i386: Adjust nr_dies with configured smp_dies for PCMachine Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 13/43] i386/cpu: Consolidate die-id validity in smp context Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 14/43] i386: Update new x86_apicid parsing rules with die_offset support Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 15/43] pc: fix possible NULL pointer dereference in pc_machine_get_device_memory_region_size() Eduardo Habkost
2019-07-03 18:06 ` [Qemu-devel] [PULL v4 16/43] machine: show if CLI option '-numa node, mem' is supported in QAPI schema Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 17/43] numa: deprecate 'mem' parameter of '-numa node' option Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 18/43] numa: deprecate implict memory distribution between nodes Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 19/43] hppa: Delete unused hppa_cpu_list() function Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 20/43] target/i386: fix feature check in hyperv-stub.c Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 21/43] deprecate -mem-path fallback to anonymous RAM Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 22/43] i386: Don't print warning if phys-bits was set automatically Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 23/43] i386: Fix signedness of hyperv_spinlock_attempts Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 24/43] i386: make 'hv-spinlocks' a regular uint32 property Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 25/43] x86/cpu: use FeatureWordArray to define filtered_features Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 26/43] i386: Remove unused host_cpudef variable Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 27/43] target/i386: Add CPUID.1F generation support for multi-dies PCMachine Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 28/43] machine: Refactor smp_parse() in vl.c as MachineClass::smp_parse() Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 29/43] vl.c: Add -smp, dies=* command line support and update doc Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 30/43] qmp: Add deprecation information to query-machines Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 31/43] i386: Introduce SnowRidge CPU model Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 32/43] qmp: Add "alias-of" field to query-cpu-definitions Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 33/43] i386: Add x-force-features option for testing Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 34/43] i386: Get model-id from CPU object on "-cpu help" Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 35/43] i386: Register versioned CPU models Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 36/43] i386: Define -IBRS, -noTSX, -IBRS versions of " Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 37/43] i386: Replace -noTSX, -IBRS, -IBPB CPU models with aliases Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 38/43] i386: Make unversioned CPU models be aliases Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 39/43] docs: Deprecate CPU model runnability guarantees Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 40/43] i386: Add Cascadelake-Server-v2 CPU model Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 41/43] numa: Make deprecation warnings conditional on !qtest_enabled() Eduardo Habkost
2019-07-03 18:07 ` [Qemu-devel] [PULL v4 42/43] numa: allow memory-less nodes when using memdev as backend Eduardo Habkost
2019-07-03 18:07 ` Eduardo Habkost [this message]
2019-07-03 18:49 ` [Qemu-devel] [PULL v4 00/43] Machine and x86 queue, 2019-07-03 Eduardo Habkost

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=20190703180726.31267-44-ehabkost@redhat.com \
    --to=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=marcel.apfelbaum@gmail.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /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).