[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v9 06/16] qtest/numa-test: Use detailed -smp CLIs in pc_dynamic_c
From: |
Yanan Wang |
Subject: |
[PATCH v9 06/16] qtest/numa-test: Use detailed -smp CLIs in pc_dynamic_cpu_cfg |
Date: |
Fri, 10 Sep 2021 15:30:15 +0800 |
Since commit 80d7835749 (qemu-options: rewrite help for -smp options),
the preference of sockets/cores in -smp parsing is considered liable
to change, and actually we are going to change it in a coming commit.
So it'll be more stable to use detailed -smp CLIs in testing if we
have strong dependency on the parsing results.
pc_dynamic_cpu_cfg currently assumes/needs that there will be 2 CPU
sockets with "-smp 2". To avoid breaking the test because of parsing
logic change, now explicitly use "-smp 2,sockets=2".
Cc: Thomas Huth <thuth@redhat.com>
Cc: Laurent Vivier <lvivier@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
---
tests/qtest/numa-test.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/qtest/numa-test.c b/tests/qtest/numa-test.c
index c677cd63c4..fd7a2e80a0 100644
--- a/tests/qtest/numa-test.c
+++ b/tests/qtest/numa-test.c
@@ -265,7 +265,8 @@ static void pc_dynamic_cpu_cfg(const void *data)
QTestState *qs;
g_autofree char *cli = NULL;
- cli = make_cli(data, "-nodefaults --preconfig -machine smp.cpus=2");
+ cli = make_cli(data, "-nodefaults --preconfig "
+ "-machine smp.cpus=2,smp.sockets=2");
qs = qtest_init(cli);
/* create 2 numa nodes */
--
2.23.0
- [PATCH v9 00/16] machine: smp parsing fixes and improvement, Yanan Wang, 2021/09/10
- [PATCH v9 03/16] machine: Uniformly use maxcpus to calculate the omitted parameters, Yanan Wang, 2021/09/10
- [PATCH v9 02/16] machine: Minor refactor/fix for the smp parsers, Yanan Wang, 2021/09/10
- [PATCH v9 04/16] machine: Set the value of cpus to match maxcpus if it's omitted, Yanan Wang, 2021/09/10
- [PATCH v9 08/16] machine: Prefer cores over sockets in smp parsing since 6.2, Yanan Wang, 2021/09/10
- [PATCH v9 10/16] machine: Tweak the order of topology members in struct CpuTopology, Yanan Wang, 2021/09/10
- [PATCH v9 05/16] machine: Improve the error reporting of smp parsing, Yanan Wang, 2021/09/10
- [PATCH v9 11/16] machine: Make smp_parse generic enough for all arches, Yanan Wang, 2021/09/10
- [PATCH v9 12/16] machine: Remove smp_parse callback from MachineClass, Yanan Wang, 2021/09/10
- [PATCH v9 06/16] qtest/numa-test: Use detailed -smp CLIs in pc_dynamic_cpu_cfg,
Yanan Wang <=
- [PATCH v9 01/16] machine: Deprecate "parameter=0" SMP configurations, Yanan Wang, 2021/09/10
- [PATCH v9 13/16] machine: Move smp_prefer_sockets to struct SMPCompatProps, Yanan Wang, 2021/09/10
- [PATCH v9 09/16] machine: Use ms instead of global current_machine in sanity-check, Yanan Wang, 2021/09/10
- [PATCH v9 14/16] machine: Put all sanity-check in the generic SMP parser, Yanan Wang, 2021/09/10
- [PATCH v9 07/16] qtest/numa-test: Use detailed -smp CLI in test_def_cpu_split, Yanan Wang, 2021/09/10
- [PATCH v9 16/16] tests/unit: Add a unit test for smp parsing, Yanan Wang, 2021/09/10
- [PATCH v9 15/16] machine: Split out the smp parsing code, Yanan Wang, 2021/09/10