Markus, all patches in this series got at least one Reviewed-by line. I would like to have your Acked-by to merge it through my tree, so I can apply Igor's -cpu + global property series that depend on it.
On Mon, Jun 20, 2016 at 12:52:53PM -0300, Eduardo Habkost wrote: > This series includes multiple changes to the way errors are > handled by the global property system. > > Git tree: > https://github.com/ehabkost/qemu-hacks.git work/global-error-handling > > Changes v1 -> v2: > * Fix test-qdev-global-props unit test > * Remove subprocess code and "nouser" test case > * Reported-by: Igor Mammedov <imamm...@redhat.com> > * Reword doc comments for GlobalProperty::errp > * Suggested-by: Markus Armbruster <arm...@redhat.com> > * Squashed "Use error_prepend() for errors applying globals" > and "qdev: GlobalProperty.errp field" > * Suggested-by: Markus Armbruster <arm...@redhat.com> > * Simplify object_class_dynamic_cast(oc, TYPE_DEVICE) checks > * Suggested-by: Markus Armbruster <arm...@redhat.com> > * New patch: "qdev: Remove qdev_prop_register_global_list()" > > Eduardo Habkost (10): > qdev: Don't stop applying globals on first error > qdev: Eliminate qemu_add_globals() function > vl: Reject invalid class names on -global > qdev: GlobalProperty.errp field > machine: Add machine_register_compat_props() function > vl: Set errp to &error_abort on machine compat_props > qdev: Eliminate "global not used" warning > qdev: Eliminate GlobalProperty 'used' and 'user_provided' fields > qdev: Remove qdev_prop_register_global_list() > machine: Skip global registration for non-existing classes > > hw/core/machine.c | 26 +++++++++ > hw/core/qdev-properties-system.c | 21 +------ > hw/core/qdev-properties.c | 55 +++---------------- > include/hw/boards.h | 1 + > include/hw/qdev-core.h | 9 ++- > include/hw/qdev-properties.h | 2 - > include/qemu/config-file.h | 1 - > tests/test-qdev-global-props.c | 115 > ++++----------------------------------- > vl.c | 37 ++++++++++--- > 9 files changed, 80 insertions(+), 187 deletions(-) > > -- > 2.5.5 > -- Eduardo