Re: [Qemu-devel] [PATCH v2 4/5] vl: Document dependencies hiding in global and compat props

[Markus Armbruster <armbru@redhat.com>]

"Dr. David Alan Gilbert" <dgilbert@redhat.com> writes:

> * Markus Armbruster (armbru@redhat.com) wrote:
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> ---
>>  vl.c | 24 ++++++++++++++++++++++--
>>  1 file changed, 22 insertions(+), 2 deletions(-)
>> 
>> diff --git a/vl.c b/vl.c
>> index e4d7ad6b85..8c6d06b01d 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -4285,16 +4285,36 @@ int main(int argc, char **argv, char **envp)
>>      current_machine->maxram_size = maxram_size;
>>      current_machine->ram_slots = ram_slots;
>>  
>> +    /*
>> +     * Note: uses machine properties such as kernel-irqchip, must run
>> +     * after machine_set_property().
>> +     */
>>      configure_accelerator(current_machine, argv[0]);
>>  
>> +    /*
>> +     * Beware, QOM objects created before this point miss global and
>> +     * compat properties.
>> +     *
>> +     * Global properties get set up by qdev_prop_register_global(),
>> +     * called from user_register_global_props(), and certain option
>> +     * desugaring.
>
> Not to stop this patch series; but could we add a way to actually
> detect when this has happened?  Like setting a flag on QOM objects
> created before this point for example?

I started to work on something like this, but had to switch to more
urgent work.