On Tue, 28 Apr 2026 16:45:16 -0700, Pierrick Bouvier 
<[email protected]> wrote:
> diff --git a/system/vl.c b/system/vl.c
> index 0e1fc217b4f..b4a5ea6f857 100644
> --- a/system/vl.c
> +++ b/system/vl.c
> @@ -2889,6 +2889,9 @@ void qemu_init(int argc, char **argv)
>      os_setup_limits();
>  
>      module_init_info(qemu_modinfo);
> +    /* We need to initialize QOM first to detect target */
> +    module_call_init(MODULE_INIT_QOM);
> +
>      module_allow_arch(target_name());

This change is not trivial. By moving the call here, we lose the
initializations from qemu_init_subsystems(). Verifying that all
type_init() code is safe to run without it is a tedious task. It might
be better to introduce a new MODULE_INIT_TARGET_INFO instead

-- 
Marc-AndrĂ© Lureau <[email protected]>


Reply via email to