qdev: remove cannot_destroy_with_object_finalize_yet
As all users have been removed, we can remove cannot_destroy_with_object_finalize_yet field from the DeviceClass structure. Signed-off-by: Laurent Vivier <lvivier@redhat.com> Message-Id: <20170414083717.13641-5-lvivier@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com> Acked-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Markus Armbruster <armbru@redhat.com>
This commit is contained in:
parent
d28fca153b
commit
08f00df4f4
|
@ -113,19 +113,6 @@ typedef struct DeviceClass {
|
||||||
* TODO remove once we're there
|
* TODO remove once we're there
|
||||||
*/
|
*/
|
||||||
bool cannot_instantiate_with_device_add_yet;
|
bool cannot_instantiate_with_device_add_yet;
|
||||||
/*
|
|
||||||
* Does this device model survive object_unref(object_new(TNAME))?
|
|
||||||
* All device models should, and this flag shouldn't exist. Some
|
|
||||||
* devices crash in object_new(), some crash or hang in
|
|
||||||
* object_unref(). Makes introspecting properties with
|
|
||||||
* qmp_device_list_properties() dangerous. Bad, because it's used
|
|
||||||
* by -device FOO,help. This flag serves to protect that code.
|
|
||||||
* It should never be set without a comment explaining why it is
|
|
||||||
* set.
|
|
||||||
* TODO remove once we're there
|
|
||||||
*/
|
|
||||||
bool cannot_destroy_with_object_finalize_yet;
|
|
||||||
|
|
||||||
bool hotpluggable;
|
bool hotpluggable;
|
||||||
|
|
||||||
/* callbacks */
|
/* callbacks */
|
||||||
|
|
5
qmp.c
5
qmp.c
|
@ -548,11 +548,6 @@ DevicePropertyInfoList *qmp_device_list_properties(const char *typename,
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (DEVICE_CLASS(klass)->cannot_destroy_with_object_finalize_yet) {
|
|
||||||
error_setg(errp, "Can't list properties of device '%s'", typename);
|
|
||||||
return NULL;
|
|
||||||
}
|
|
||||||
|
|
||||||
obj = object_new(typename);
|
obj = object_new(typename);
|
||||||
|
|
||||||
object_property_iter_init(&iter, obj);
|
object_property_iter_init(&iter, obj);
|
||||||
|
|
Loading…
Reference in New Issue