]> git.ipfire.org Git - thirdparty/qemu.git/commitdiff
qtest/qom-test: Shallow testing of qom-list / qom-get
authorMarkus Armbruster <armbru@redhat.com>
Fri, 25 Jul 2025 13:50:31 +0000 (15:50 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Mon, 1 Sep 2025 11:43:43 +0000 (13:43 +0200)
This test traverses the QOM sub-tree rooted at /machine with a
combination of qom-list and qom-get.  In my x86_64 testing, it runs
almost 12000 QMP commands in 34 seconds.  With -m slow, we test more
machines, and it takes almost 84000 commands in almost four minutes.

Since commit 3dd93992ffb (tests/qtest/qom-test: unit test for
qom-list-get), the test traverses this tree a second time, with
qom-list-get.  In my x86_64 testing, this takes some 200 QMP commands
and around two seconds, and some 1100 in just under 12s with -m slow.

Traversing the entire tree is useful, because it exercise the QOM
property getters.  Traversing it twice not so much.

Make the qom-list / qom-get test shallow unless -m slow is given:
don't recurse.  Cuts the number of commands to around 600, and run
time to under 5s for me.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20250725135034.2280477-3-armbru@redhat.com>
Reviewed-by: Steve Sistare <steven.sistare@oracle.com>
tests/qtest/qom-test.c

index 4ade1c728c014ed614634b43661f380c4d34ad37..7dea0d802ddb5fe011d814efd7202174a1d3090e 100644 (file)
@@ -180,7 +180,7 @@ static void test_properties(QTestState *qts, const char *path, bool recurse)
         links = g_slist_delete_link(links, links);
     }
     while (children) {
-        test_properties(qts, children->data, true);
+        test_properties(qts, children->data, g_test_slow());
         g_free(children->data);
         children = g_slist_delete_link(children, children);
     }