Commit d7741743 authored by Eduardo Habkost's avatar Eduardo Habkost Committed by Michael S. Tsirkin
Browse files

globals: Allow global properties to be optional 



Making some global properties optional will let us simplify
compat code when a given property works on most (but not all)
subclasses of a given type.

Device types will be able to opt out from optional compat
properties by simply not registering those properties.

Signed-off-by: default avatarEduardo Habkost <ehabkost@redhat.com>
Reviewed-by: default avatarCornelia Huck <cohuck@redhat.com>
Reviewed-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: default avatarMichael S. Tsirkin <mst@redhat.com>
Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
parent ecd3b89b

include/hw/qdev-core.h

+3 −0
Original line number Diff line number Diff line
@@ -250,6 +250,8 @@ struct PropertyInfo { 
/**

 * GlobalProperty:

 * @used: Set to true if property was used when initializing a device.

 * @optional: If set to true, GlobalProperty will be skipped without errors

 *            if the property doesn't exist.

 *

 * An error is fatal for non-hotplugged devices, when the global is applied.

 */
@@ -258,6 +260,7 @@ typedef struct GlobalProperty { 
    const char *property;

    const char *value;

    bool used;

    bool optional;

} GlobalProperty;



static inline void

qom/object.c

+3 −0
Original line number Diff line number Diff line
@@ -385,6 +385,9 @@ void object_apply_global_props(Object *obj, const GPtrArray *props, Error **errp 
        if (object_dynamic_cast(obj, p->driver) == NULL) {

            continue;

        }

        if (p->optional && !object_property_find(obj, p->property, NULL)) {

            continue;

        }

        p->used = true;

        object_property_parse(obj, p->value, p->property, &err);

        if (err != NULL) {