This patch series provides infrastructure and documentation for marking QOM struct fields as private to the class implementation.
Patch 1 is the implementation (which is a trivial five lines!) and documentation of the code pattern that it's intended to be used with. Patches 2, 3 and 4 are examples of its use: patch 2 deals with the ARM GIC classes, as an example of a largish class with some subclasses. Patches 3 and 4 are conversions of much simpler and smaller devices, so might be easier to look at first. A .h file which uses this pattern ends up with half a dozen extra lines of boilerplate, which is slightly sad but not too awful. It would be pretty easy to autogenerate (along with the type macros themselves) if we decided to do that in future, though. Example of the compiler message if you try to touch a field which is private: arm_gic_kvm.c:559:5: error: ‘iomem’ is deprecated (declared at /root/qemu/include/hw/intc/arm_gic_common.h:105): this field is private [-Werror=deprecated-declarations] Changes since the RFC back in July last year: * name the macro 'qom_private' rather than '__private' * redid patch 2 as the GIC code has been rearranged somewhat * patches 3 and 4 are new Peter Maydell (4): Provide infrastructure for marking private QOM struct fields arm_gic: Use new qom_private macro to mark private fields a9scu: Use qom_private to mark private fields arm11scu: Use qom_private to mark private fields hw/intc/arm_gic.c | 3 ++ hw/intc/arm_gic_common.c | 2 ++ hw/intc/arm_gic_kvm.c | 2 ++ hw/intc/armv7m_nvic.c | 2 ++ hw/misc/a9scu.c | 2 ++ hw/misc/arm11scu.c | 2 ++ include/hw/intc/arm_gic.h | 12 ++++++-- include/hw/intc/arm_gic_common.h | 62 +++++++++++++++++++++++----------------- include/hw/misc/a9scu.h | 16 +++++++---- include/hw/misc/arm11scu.h | 14 ++++++--- include/qemu/compiler.h | 10 +++++++ include/qom/object.h | 47 ++++++++++++++++++++++++++++++ 12 files changed, 136 insertions(+), 38 deletions(-) -- 1.9.2
