summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2012-05-14 17:11:47 -0600
committerEric Blake <eblake@redhat.com>2012-05-16 10:23:06 -0600
commit5a8262a0aebe64f7cf5dff1a784833788c966279 (patch)
tree2e6dd714134023085d437bdf57468c324c70d962 /tests/nodeinfotest.c
parentAdd a virLogMessage alternative taking va_list args (diff)
downloadlibvirt-5a8262a0aebe64f7cf5dff1a784833788c966279.tar.gz
libvirt-5a8262a0aebe64f7cf5dff1a784833788c966279.tar.bz2
libvirt-5a8262a0aebe64f7cf5dff1a784833788c966279.zip
nodeinfo: test more details
Make it obvious why we need Osier's patch in commit 10d9038b to fix NUMA parsing of an AMD machine with two cores sharing a socket id. * tests/nodeinfotest.c (linuxTestCompareFiles): Enhance the test. * tests/nodeinfodata/linux-nodeinfo-sysfs-test-*-output.txt: Update.
Diffstat (limited to 'tests/nodeinfotest.c')
-rw-r--r--tests/nodeinfotest.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/tests/nodeinfotest.c b/tests/nodeinfotest.c
index 2b4c3a0b6..a2b89dd94 100644
--- a/tests/nodeinfotest.c
+++ b/tests/nodeinfotest.c
@@ -60,9 +60,12 @@ linuxTestCompareFiles(const char *cpuinfofile,
}
VIR_FORCE_FCLOSE(cpuinfo);
- if (virAsprintf(&actualData, "CPUs: %u, MHz: %u, Nodes: %u, Cores: %u\n",
- nodeinfo.cpus, nodeinfo.mhz, nodeinfo.nodes,
- nodeinfo.cores) < 0)
+ if (virAsprintf(&actualData,
+ "CPUs: %u/%u, MHz: %u, Nodes: %u, Sockets: %u, "
+ "Cores: %u, Threads: %u\n",
+ nodeinfo.cpus, VIR_NODEINFO_MAXCPUS(nodeinfo),
+ nodeinfo.mhz, nodeinfo.nodes, nodeinfo.sockets,
+ nodeinfo.cores, nodeinfo.threads) < 0)
goto fail;
if (STRNEQ(actualData, expectData)) {