Hi, During "[PATCH v2 05/10] qom/globals: generalize object_property_set_globals()" review, Eduardo suggested to rework the GlobalProperty handling, so that -global is limited to QDev only and we avoid mixing the machine compats and the user-provided -global properties (instead of generalizing -global to various object kinds, like I proposed in v2).
"qdev: do not mix compat props with global props" patch decouples a bit user-provided -global from machine compat properties. This allows to get rid of "user_provided" and "errp" fields in following patches. Instead of explcitely calling object_apply_global_props() in the various object post_init, I opted for creating a new TYPE_COMPAT_PROPS interface. The interface approach gives a lot more flexibility on which objects can have compat props. This requires some interface improvments in "qom: teach interfaces to implement post-init". A new compat property "x-use-canonical-path-for-ramblock-id" is added to hostmem for legacy canonical path names, set to true for -file and -memfd with qemu < 3.2. (this series was initially titled "[PATCH v2 00/10] hostmem: use object "id" for memory region name with >= 3.1", but its focus is more in refactoring the global and compatilibity properties handling now) v4: - fix accel/machine compat props apply order inversion - split "qdev: do not mix compat props with global props": replace using current_machine by new compat_props global, introduce SET_COMPAT() macro, use GPtrArray. - add a comment about "qdev-props: call object_apply_global_props()" error reporting - add "arm: replace instance_post_init()", to solve the post-init ordering issue - add "hw/arm/virt: add virt-3.2 machine type" for completeness - commit message improvements - rebased, add r-b tags v3: - GlobalProperties improvements/cleanups - drop generalizing the -global idea - "replace" the set_globals flag with a TYPE_COMPAT_PROPS interface - update hw/i386 machine version to 3.2 - add "qom: make interface types abstract" interface cleanup v2: - replace "qom/user-creatable: add a few helper macros" patch for a more optimized "qom: make user_creatable_complete() specific to UserCreatable" - rename register_global_list() to register_global_properties() - call object_property_set_globals() after post-init - add and use a ObjectClass.set_globals flag, instead of dynamically check object class in object_property_set_globals() - use object "id" in >= 3.1 instead of canonical path, add compat property "x-use-canonical-path-for-ramblock-id" in base hostmem class. Edgar E. Iglesias (1): net: cadence_gem: Remove incorrect assert() Eric Auger (1): MAINTAINERS: Add an ARM SMMU section Greg Kurz (1): 9p: fix QEMU crash when renaming files Marc-André Lureau (20): 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 hw: apply accel compat properties without touching globals hw: apply machine compat properties without touching globals hw: remove SET_MACHINE_COMPAT 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: teach interfaces to implement post-init qom: add object_class_get_class_data() arm: replace instance_post_init() machine: add compat-props interface hw/i386: add pc-i440fx-3.2 & pc-q35-3.2 hw/arm/virt: add virt-3.2 machine type hostmem: use object id for memory region name with >= 3.1 Max Filippov (4): target/xtensa: gdbstub fix register counting target/xtensa: drop num_[core_]regs from dc232b/dc233c configs target/xtensa: xtfpga: provide default memory sizes MAINTAINERS: add missing xtensa patterns Thomas Huth (1): MAINTAINERS: Assign some more files in the hw/arm/ directory -- 2.20.0.rc1
