Message ID | 20181204142023.15982-1-marcandre.lureau@redhat.com |
---|---|
Headers | show |
Series | Generalize machine compatibility properties | expand |
Sorry, I forgot to change the subject prefix, it should read for-4.0 :)
On Tue, Dec 04, 2018 at 06:20:04PM +0400, Marc-André Lureau wrote: [...] > Marc-André Lureau (19): > tests: qdev_prop_check_globals() doesn't return "all_used" > qom: make interface types abstract > qom: make user_creatable_complete() specific to UserCreatable > accel: register global_props like machine globals > qdev: move qdev_prop_register_global_list() to tests > qom: remove unimplemented class_finalize Patches 1-6 (above) queued on machine-next. Thanks! > hw: apply accel compat properties without touching globals > hw: apply machine compat properties without touching globals > hw: remove SET_MACHINE_COMPAT > qdev: make a separate helper function to apply compat properties > qdev: all globals are now user-provided > qdev-props: convert global_props to GPtrArray > qdev-props: remove errp from GlobalProperty > qdev-props: call object_apply_global_props() > qom: add object_class_get_class_data() > RFC: arm: replace instance_post_init() > hw/i386: add pc-i440fx-4.0 & pc-q35-4.0 > hw/arm/virt: add virt-4.0 machine type > hostmem: use object id for memory region name with >= 4.0 >
On Mon, 10 Dec 2018 15:07:18 -0200 Eduardo Habkost <ehabkost@redhat.com> wrote: > On Tue, Dec 04, 2018 at 06:20:04PM +0400, Marc-André Lureau wrote: > [...] > > Marc-André Lureau (19): > > tests: qdev_prop_check_globals() doesn't return "all_used" > > qom: make interface types abstract > > qom: make user_creatable_complete() specific to UserCreatable > > accel: register global_props like machine globals > > qdev: move qdev_prop_register_global_list() to tests > > qom: remove unimplemented class_finalize > > Patches 1-6 (above) queued on machine-next. Thanks! let's drop patch 6 for now until we decide what to do with GPtrArray leaks > > > hw: apply accel compat properties without touching globals > > hw: apply machine compat properties without touching globals > > hw: remove SET_MACHINE_COMPAT > > qdev: make a separate helper function to apply compat properties > > qdev: all globals are now user-provided > > qdev-props: convert global_props to GPtrArray > > qdev-props: remove errp from GlobalProperty > > qdev-props: call object_apply_global_props() > > qom: add object_class_get_class_data() > > RFC: arm: replace instance_post_init() > > hw/i386: add pc-i440fx-4.0 & pc-q35-4.0 > > hw/arm/virt: add virt-4.0 machine type > > hostmem: use object id for memory region name with >= 4.0 > > >
On Mon, Dec 10, 2018 at 06:31:59PM +0100, Igor Mammedov wrote: > On Mon, 10 Dec 2018 15:07:18 -0200 > Eduardo Habkost <ehabkost@redhat.com> wrote: > > > On Tue, Dec 04, 2018 at 06:20:04PM +0400, Marc-André Lureau wrote: > > [...] > > > Marc-André Lureau (19): > > > tests: qdev_prop_check_globals() doesn't return "all_used" > > > qom: make interface types abstract > > > qom: make user_creatable_complete() specific to UserCreatable > > > accel: register global_props like machine globals > > > qdev: move qdev_prop_register_global_list() to tests > > > qom: remove unimplemented class_finalize > > > > Patches 1-6 (above) queued on machine-next. Thanks! > let's drop patch 6 for now until we decide what to do with GPtrArray leaks I appreciate your effort to make sure there are no memory allocations that could make Valgrind complain, but I don't see the point of keeping dead code that was never called in the tree. Are the new GPtrArray allocations different from the g_malloc0() calls in type_new() and type_initialize()? Doesn't Valgrind complain about them today?
On Mon, 10 Dec 2018 15:39:32 -0200 Eduardo Habkost <ehabkost@redhat.com> wrote: > On Mon, Dec 10, 2018 at 06:31:59PM +0100, Igor Mammedov wrote: > > On Mon, 10 Dec 2018 15:07:18 -0200 > > Eduardo Habkost <ehabkost@redhat.com> wrote: > > > > > On Tue, Dec 04, 2018 at 06:20:04PM +0400, Marc-André Lureau wrote: > > > [...] > > > > Marc-André Lureau (19): > > > > tests: qdev_prop_check_globals() doesn't return "all_used" > > > > qom: make interface types abstract > > > > qom: make user_creatable_complete() specific to UserCreatable > > > > accel: register global_props like machine globals > > > > qdev: move qdev_prop_register_global_list() to tests > > > > qom: remove unimplemented class_finalize > > > > > > Patches 1-6 (above) queued on machine-next. Thanks! > > let's drop patch 6 for now until we decide what to do with GPtrArray leaks > > I appreciate your effort to make sure there are no memory > allocations that could make Valgrind complain, but I don't see > the point of keeping dead code that was never called in the tree. ps: it looks like machine_finalize() isn't called either, but it at least documents what should be cleaned up. Maybe we should keep code removed by 6, for the same reasons. > Are the new GPtrArray allocations different from the g_malloc0() > calls in type_new() and type_initialize()? Doesn't Valgrind > complain about them today? Currently we don't clean a bunch of onetime allocations, so I guess we can continue to ignore them in this case as well.