motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-04 08:13:54 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

tests: Fix test-qdev-global-props on anonymous qdev realize()

Browse source 
test-qdev-global-props creates a few subprocesses and test things based on
qdev realize().  One thing was overlooked since the start, that anonymous
creations of qdev (then realize() the device) requires the machine object's
presence, as all these devices need to be attached to QOM tree, by default
to path "/machine/unattached".

The test didn't crash simply because container_get() has an implicit
semantic to silently create any missing container, hence what happened here
is container_get() (when running these tests) will try to create containers
at QOM path "/machine" on the fly.  That's probably unexpected by the test,
but worked like charm before.

We're going to fix device_set_realized() soon, but before that make the
test case prepared, by creating the machine object on its own.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
Message-ID: <20241121192202.4155849-4-peterx@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
This commit is contained in:
Peter Xu 2024-11-21 14:21:52 -05:00 • committed by Philippe Mathieu-Daudé
parent 6e1e04ef03
commit ff18687db3
1 changed files with 22 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

22
tests/unit/test-qdev-global-props.c
View file

@ -72,6 +72,26 @@ static const TypeInfo subclass_type = {
.parent = TYPE_STATIC_PROPS,
};
/*
* Initialize a fake machine, being prepared for future tests.
*
* All the tests later (even if to be run in subprocesses.. which will
* inherit the global states of the parent process) will try to create qdev
* and realize the device.
*
* Realization of such anonymous qdev (with no parent object) requires both
* the machine object and its "unattached" container to be at least present.
*/
static void test_init_machine(void)
{
/* This is a fake machine - it doesn't need to be a machine object */
Object *machine = object_property_add_new_container(
object_get_root(), "machine");
/* This container must exist for anonymous qdevs to realize() */
object_property_add_new_container(machine, "unattached");
}
/* Test simple static property setting to default value */
static void test_static_prop_subprocess(void)
{
@ -295,6 +315,8 @@ int main(int argc, char **argv)
type_register_static(&nohotplug_type);
type_register_static(&nondevice_type);
test_init_machine();
g_test_add_func("/qdev/properties/static/default/subprocess",
test_static_prop_subprocess);
g_test_add_func("/qdev/properties/static/default",