This reduces the type hierarchy of the vmapple-virtio-blk-pci-base
type and vmapple-virtio-root/vmapple-virtio-aux leaf types with a single
vmapple-virtio-blk-pci type which exposes a 'variant' enum property
which can be set to 'aux' or 'root'.
This change removes a bunch of device type boilerplate at the cost of
defining a new qapi enum and qdev property type.
Documentation for the vmapple machine type is also updated.
Signed-off-by: Phil Dennis-Jordan <phil@philjordan.eu>
---
docs/system/arm/vmapple.rst | 4 +--
hw/core/qdev-properties-system.c | 8 +++++
hw/vmapple/virtio-blk.c | 50 +++++++----------------------
include/hw/qdev-properties-system.h | 6 ++++
include/hw/vmapple/vmapple.h | 4 +--
qapi/virtio.json | 14 ++++++++
6 files changed, 43 insertions(+), 43 deletions(-)
diff --git a/docs/system/arm/vmapple.rst b/docs/system/arm/vmapple.rst
index 67942474b93..efe39b68dc4 100644
--- a/docs/system/arm/vmapple.rst
+++ b/docs/system/arm/vmapple.rst
@@ -56,8 +56,8 @@ to get better interactive access into the target system:
-drive file="$DISK",if=pflash,format=raw \
-drive file="$AUX",if=none,id=aux,format=raw \
-drive file="$DISK",if=none,id=root,format=raw \
- -device vmapple-virtio-aux,drive=aux \
- -device vmapple-virtio-root,drive=root \
+ -device vmapple-virtio-blk-pci,variant=aux,drive=aux \
+ -device vmapple-virtio-blk-pci,variant=root,drive=root \
-netdev user,id=net0,ipv6=off,hostfwd=tcp::2222-:22,hostfwd=tcp::5901-:5900 \
-device virtio-net-pci,netdev=net0
diff --git a/hw/core/qdev-properties-system.c b/hw/core/qdev-properties-system.c
index 35deef05f32..8bf8a3442d6 100644
--- a/hw/core/qdev-properties-system.c
+++ b/hw/core/qdev-properties-system.c
@@ -1239,3 +1239,11 @@ const PropertyInfo qdev_prop_iothread_vq_mapping_list = {
.set = set_iothread_vq_mapping_list,
.release = release_iothread_vq_mapping_list,
};
+
+const PropertyInfo qdev_prop_vmapple_virtio_blk_variant = {
+ .name = "VMAppleVirtioBlkVariant",
+ .enum_table = &VMAppleVirtioBlkVariant_lookup,
+ .get = qdev_propinfo_get_enum,
+ .set = qdev_propinfo_set_enum,
+ .set_default_value = qdev_propinfo_set_default_value_enum,
+};
diff --git a/hw/vmapple/virtio-blk.c b/hw/vmapple/virtio-blk.c
index 40b33bbdac5..f883e65a5de 100644
--- a/hw/vmapple/virtio-blk.c
+++ b/hw/vmapple/virtio-blk.c
@@ -24,6 +24,7 @@
#include "qemu/module.h"
#include "qapi/error.h"
+#define TYPE_VMAPPLE_VIRTIO_BLK "vmapple-virtio-blk"
OBJECT_DECLARE_TYPE(VMAppleVirtIOBlk, VMAppleVirtIOBlkClass, VMAPPLE_VIRTIO_BLK)
typedef struct VMAppleVirtIOBlkClass {
@@ -41,14 +42,10 @@ typedef struct VMAppleVirtIOBlk {
/*
* vmapple-virtio-blk-pci: This extends VirtioPCIProxy.
*/
-#define TYPE_VMAPPLE_VIRTIO_BLK_PCI "vmapple-virtio-blk-pci-base"
OBJECT_DECLARE_SIMPLE_TYPE(VMAppleVirtIOBlkPCI, VMAPPLE_VIRTIO_BLK_PCI)
#define VIRTIO_BLK_T_APPLE_BARRIER 0x10000
-#define VIRTIO_APPLE_TYPE_ROOT 1
-#define VIRTIO_APPLE_TYPE_AUX 2
-
static bool vmapple_virtio_blk_handle_unknown_request(VirtIOBlockReq *req,
MultiReqBuffer *mrb,
uint32_t type)
@@ -109,7 +106,7 @@ static const TypeInfo vmapple_virtio_blk_info = {
struct VMAppleVirtIOBlkPCI {
VirtIOPCIProxy parent_obj;
VMAppleVirtIOBlk vdev;
- uint32_t apple_type;
+ VMAppleVirtioBlkVariant variant;
};
@@ -119,6 +116,8 @@ static Property vmapple_virtio_blk_pci_properties[] = {
VIRTIO_PCI_FLAG_USE_IOEVENTFD_BIT, true),
DEFINE_PROP_UINT32("vectors", VirtIOPCIProxy, nvectors,
DEV_NVECTORS_UNSPECIFIED),
+ DEFINE_PROP_VMAPPLE_VIRTIO_BLK_VARIANT("variant", VMAppleVirtIOBlkPCI, variant,
+ VM_APPLE_VIRTIO_BLK_VARIANT_UNSPECIFIED),
DEFINE_PROP_END_OF_LIST(),
};
@@ -128,6 +127,12 @@ static void vmapple_virtio_blk_pci_realize(VirtIOPCIProxy *vpci_dev, Error **err
DeviceState *vdev = DEVICE(&dev->vdev);
VirtIOBlkConf *conf = &dev->vdev.parent_obj.conf;
+ if (dev->variant == VM_APPLE_VIRTIO_BLK_VARIANT_UNSPECIFIED) {
+ error_setg(errp, "Device " TYPE_VMAPPLE_VIRTIO_BLK_PCI ": must specify "
+ "a variant, 'aux' or 'root'");
+ return;
+ }
+
if (conf->num_queues == VIRTIO_BLK_AUTO_NUM_QUEUES) {
conf->num_queues = virtio_pci_optimal_num_queues(0);
}
@@ -143,7 +148,7 @@ static void vmapple_virtio_blk_pci_realize(VirtIOPCIProxy *vpci_dev, Error **err
*/
virtio_add_feature(&dev->vdev.parent_obj.host_features, VIRTIO_BLK_F_ZONED);
/* Propagate the apple type down to the virtio-blk device */
- dev->vdev.apple_type = dev->apple_type;
+ dev->vdev.apple_type = dev->variant;
/* and spawn the virtio-blk device */
qdev_realize(vdev, BUS(&vpci_dev->bus), errp);
@@ -181,47 +186,16 @@ static void vmapple_virtio_blk_pci_instance_init(Object *obj)
}
static const VirtioPCIDeviceTypeInfo vmapple_virtio_blk_pci_info = {
- .base_name = TYPE_VMAPPLE_VIRTIO_BLK_PCI,
- .generic_name = "vmapple-virtio-blk-pci",
+ .generic_name = TYPE_VMAPPLE_VIRTIO_BLK_PCI,
.instance_size = sizeof(VMAppleVirtIOBlkPCI),
.instance_init = vmapple_virtio_blk_pci_instance_init,
.class_init = vmapple_virtio_blk_pci_class_init,
};
-static void vmapple_virtio_root_instance_init(Object *obj)
-{
- VMAppleVirtIOBlkPCI *dev = VMAPPLE_VIRTIO_BLK_PCI(obj);
-
- dev->apple_type = VIRTIO_APPLE_TYPE_ROOT;
-}
-
-static const TypeInfo vmapple_virtio_root_info = {
- .name = TYPE_VMAPPLE_VIRTIO_ROOT,
- .parent = "vmapple-virtio-blk-pci",
- .instance_size = sizeof(VMAppleVirtIOBlkPCI),
- .instance_init = vmapple_virtio_root_instance_init,
-};
-
-static void vmapple_virtio_aux_instance_init(Object *obj)
-{
- VMAppleVirtIOBlkPCI *dev = VMAPPLE_VIRTIO_BLK_PCI(obj);
-
- dev->apple_type = VIRTIO_APPLE_TYPE_AUX;
-}
-
-static const TypeInfo vmapple_virtio_aux_info = {
- .name = TYPE_VMAPPLE_VIRTIO_AUX,
- .parent = "vmapple-virtio-blk-pci",
- .instance_size = sizeof(VMAppleVirtIOBlkPCI),
- .instance_init = vmapple_virtio_aux_instance_init,
-};
-
static void vmapple_virtio_blk_register_types(void)
{
type_register_static(&vmapple_virtio_blk_info);
virtio_pci_types_register(&vmapple_virtio_blk_pci_info);
- type_register_static(&vmapple_virtio_root_info);
- type_register_static(&vmapple_virtio_aux_info);
}
type_init(vmapple_virtio_blk_register_types)
diff --git a/include/hw/qdev-properties-system.h b/include/hw/qdev-properties-system.h
index cdcc63056e5..6e428f3fcad 100644
--- a/include/hw/qdev-properties-system.h
+++ b/include/hw/qdev-properties-system.h
@@ -27,6 +27,8 @@ extern const PropertyInfo qdev_prop_pcie_link_speed;
extern const PropertyInfo qdev_prop_pcie_link_width;
extern const PropertyInfo qdev_prop_cpus390entitlement;
extern const PropertyInfo qdev_prop_iothread_vq_mapping_list;
+extern const PropertyInfo qdev_prop_vmapple_virtio_blk_variant;
+
#define DEFINE_PROP_PCI_DEVFN(_n, _s, _f, _d) \
DEFINE_PROP_SIGNED(_n, _s, _f, _d, qdev_prop_pci_devfn, int32_t)
@@ -94,4 +96,8 @@ extern const PropertyInfo qdev_prop_iothread_vq_mapping_list;
DEFINE_PROP(_name, _state, _field, qdev_prop_iothread_vq_mapping_list, \
IOThreadVirtQueueMappingList *)
+#define DEFINE_PROP_VMAPPLE_VIRTIO_BLK_VARIANT(_n, _s, _f, _d) \
+ DEFINE_PROP_SIGNED(_n, _s, _f, _d, qdev_prop_vmapple_virtio_blk_variant, \
+ VMAppleVirtioBlkVariant)
+
#endif
diff --git a/include/hw/vmapple/vmapple.h b/include/hw/vmapple/vmapple.h
index b20956e1286..9c1ad1bd8c3 100644
--- a/include/hw/vmapple/vmapple.h
+++ b/include/hw/vmapple/vmapple.h
@@ -18,8 +18,6 @@
#define TYPE_VMAPPLE_CFG "vmapple-cfg"
-#define TYPE_VMAPPLE_VIRTIO_BLK "vmapple-virtio-blk"
-#define TYPE_VMAPPLE_VIRTIO_ROOT "vmapple-virtio-root"
-#define TYPE_VMAPPLE_VIRTIO_AUX "vmapple-virtio-aux"
+#define TYPE_VMAPPLE_VIRTIO_BLK_PCI "vmapple-virtio-blk-pci"
#endif /* HW_VMAPPLE_VMAPPLE_H */
diff --git a/qapi/virtio.json b/qapi/virtio.json
index 2529c2d8b20..d351d2166ef 100644
--- a/qapi/virtio.json
+++ b/qapi/virtio.json
@@ -992,3 +992,17 @@
##
{ 'enum': 'GranuleMode',
'data': [ '4k', '8k', '16k', '64k', 'host' ] }
+
+##
+# @VMAppleVirtioBlkVariant:
+#
+# @unspecified: The default, not a valid setting.
+#
+# @root: Block device holding the root volume
+#
+# @aux: Block device holding auxiliary data required for boot
+#
+# Since: 9.2
+##
+{ 'enum': 'VMAppleVirtioBlkVariant',
+ 'data': [ 'unspecified', 'root', 'aux' ] }
--
2.39.3 (Apple Git-145)
On 2024/11/11 6:55, Phil Dennis-Jordan wrote: > This reduces the type hierarchy of the vmapple-virtio-blk-pci-base > type and vmapple-virtio-root/vmapple-virtio-aux leaf types with a single > vmapple-virtio-blk-pci type which exposes a 'variant' enum property > which can be set to 'aux' or 'root'. > > This change removes a bunch of device type boilerplate at the cost of > defining a new qapi enum and qdev property type. > > Documentation for the vmapple machine type is also updated. Please merge this change into "[PATCH v9 13/16] hw/vmapple/virtio-blk: Add support for apple virtio-blk". > > Signed-off-by: Phil Dennis-Jordan <phil@philjordan.eu> > --- > docs/system/arm/vmapple.rst | 4 +-- > hw/core/qdev-properties-system.c | 8 +++++ > hw/vmapple/virtio-blk.c | 50 +++++++---------------------- > include/hw/qdev-properties-system.h | 6 ++++ > include/hw/vmapple/vmapple.h | 4 +-- > qapi/virtio.json | 14 ++++++++ > 6 files changed, 43 insertions(+), 43 deletions(-) > > diff --git a/docs/system/arm/vmapple.rst b/docs/system/arm/vmapple.rst > index 67942474b93..efe39b68dc4 100644 > --- a/docs/system/arm/vmapple.rst > +++ b/docs/system/arm/vmapple.rst > @@ -56,8 +56,8 @@ to get better interactive access into the target system: > -drive file="$DISK",if=pflash,format=raw \ > -drive file="$AUX",if=none,id=aux,format=raw \ > -drive file="$DISK",if=none,id=root,format=raw \ > - -device vmapple-virtio-aux,drive=aux \ > - -device vmapple-virtio-root,drive=root \ > + -device vmapple-virtio-blk-pci,variant=aux,drive=aux \ > + -device vmapple-virtio-blk-pci,variant=root,drive=root \ > -netdev user,id=net0,ipv6=off,hostfwd=tcp::2222-:22,hostfwd=tcp::5901-:5900 \ > -device virtio-net-pci,netdev=net0 > > diff --git a/hw/core/qdev-properties-system.c b/hw/core/qdev-properties-system.c > index 35deef05f32..8bf8a3442d6 100644 > --- a/hw/core/qdev-properties-system.c > +++ b/hw/core/qdev-properties-system.c > @@ -1239,3 +1239,11 @@ const PropertyInfo qdev_prop_iothread_vq_mapping_list = { > .set = set_iothread_vq_mapping_list, > .release = release_iothread_vq_mapping_list, > }; > + > +const PropertyInfo qdev_prop_vmapple_virtio_blk_variant = { > + .name = "VMAppleVirtioBlkVariant", > + .enum_table = &VMAppleVirtioBlkVariant_lookup, > + .get = qdev_propinfo_get_enum, > + .set = qdev_propinfo_set_enum, > + .set_default_value = qdev_propinfo_set_default_value_enum, > +}; > diff --git a/hw/vmapple/virtio-blk.c b/hw/vmapple/virtio-blk.c > index 40b33bbdac5..f883e65a5de 100644 > --- a/hw/vmapple/virtio-blk.c > +++ b/hw/vmapple/virtio-blk.c > @@ -24,6 +24,7 @@ > #include "qemu/module.h" > #include "qapi/error.h" > > +#define TYPE_VMAPPLE_VIRTIO_BLK "vmapple-virtio-blk" > OBJECT_DECLARE_TYPE(VMAppleVirtIOBlk, VMAppleVirtIOBlkClass, VMAPPLE_VIRTIO_BLK) > > typedef struct VMAppleVirtIOBlkClass { > @@ -41,14 +42,10 @@ typedef struct VMAppleVirtIOBlk { > /* > * vmapple-virtio-blk-pci: This extends VirtioPCIProxy. > */ > -#define TYPE_VMAPPLE_VIRTIO_BLK_PCI "vmapple-virtio-blk-pci-base" > OBJECT_DECLARE_SIMPLE_TYPE(VMAppleVirtIOBlkPCI, VMAPPLE_VIRTIO_BLK_PCI) > > #define VIRTIO_BLK_T_APPLE_BARRIER 0x10000 > > -#define VIRTIO_APPLE_TYPE_ROOT 1 > -#define VIRTIO_APPLE_TYPE_AUX 2 > - > static bool vmapple_virtio_blk_handle_unknown_request(VirtIOBlockReq *req, > MultiReqBuffer *mrb, > uint32_t type) > @@ -109,7 +106,7 @@ static const TypeInfo vmapple_virtio_blk_info = { > struct VMAppleVirtIOBlkPCI { > VirtIOPCIProxy parent_obj; > VMAppleVirtIOBlk vdev; > - uint32_t apple_type; > + VMAppleVirtioBlkVariant variant; > }; > > > @@ -119,6 +116,8 @@ static Property vmapple_virtio_blk_pci_properties[] = { > VIRTIO_PCI_FLAG_USE_IOEVENTFD_BIT, true), > DEFINE_PROP_UINT32("vectors", VirtIOPCIProxy, nvectors, > DEV_NVECTORS_UNSPECIFIED), > + DEFINE_PROP_VMAPPLE_VIRTIO_BLK_VARIANT("variant", VMAppleVirtIOBlkPCI, variant, > + VM_APPLE_VIRTIO_BLK_VARIANT_UNSPECIFIED), > DEFINE_PROP_END_OF_LIST(), > }; > > @@ -128,6 +127,12 @@ static void vmapple_virtio_blk_pci_realize(VirtIOPCIProxy *vpci_dev, Error **err > DeviceState *vdev = DEVICE(&dev->vdev); > VirtIOBlkConf *conf = &dev->vdev.parent_obj.conf; > > + if (dev->variant == VM_APPLE_VIRTIO_BLK_VARIANT_UNSPECIFIED) { > + error_setg(errp, "Device " TYPE_VMAPPLE_VIRTIO_BLK_PCI ": must specify " > + "a variant, 'aux' or 'root'"); > + return; > + } > + > if (conf->num_queues == VIRTIO_BLK_AUTO_NUM_QUEUES) { > conf->num_queues = virtio_pci_optimal_num_queues(0); > } > @@ -143,7 +148,7 @@ static void vmapple_virtio_blk_pci_realize(VirtIOPCIProxy *vpci_dev, Error **err > */ > virtio_add_feature(&dev->vdev.parent_obj.host_features, VIRTIO_BLK_F_ZONED); > /* Propagate the apple type down to the virtio-blk device */ > - dev->vdev.apple_type = dev->apple_type; > + dev->vdev.apple_type = dev->variant; > /* and spawn the virtio-blk device */ > qdev_realize(vdev, BUS(&vpci_dev->bus), errp); > > @@ -181,47 +186,16 @@ static void vmapple_virtio_blk_pci_instance_init(Object *obj) > } > > static const VirtioPCIDeviceTypeInfo vmapple_virtio_blk_pci_info = { > - .base_name = TYPE_VMAPPLE_VIRTIO_BLK_PCI, > - .generic_name = "vmapple-virtio-blk-pci", > + .generic_name = TYPE_VMAPPLE_VIRTIO_BLK_PCI, > .instance_size = sizeof(VMAppleVirtIOBlkPCI), > .instance_init = vmapple_virtio_blk_pci_instance_init, > .class_init = vmapple_virtio_blk_pci_class_init, > }; > > -static void vmapple_virtio_root_instance_init(Object *obj) > -{ > - VMAppleVirtIOBlkPCI *dev = VMAPPLE_VIRTIO_BLK_PCI(obj); > - > - dev->apple_type = VIRTIO_APPLE_TYPE_ROOT; > -} > - > -static const TypeInfo vmapple_virtio_root_info = { > - .name = TYPE_VMAPPLE_VIRTIO_ROOT, > - .parent = "vmapple-virtio-blk-pci", > - .instance_size = sizeof(VMAppleVirtIOBlkPCI), > - .instance_init = vmapple_virtio_root_instance_init, > -}; > - > -static void vmapple_virtio_aux_instance_init(Object *obj) > -{ > - VMAppleVirtIOBlkPCI *dev = VMAPPLE_VIRTIO_BLK_PCI(obj); > - > - dev->apple_type = VIRTIO_APPLE_TYPE_AUX; > -} > - > -static const TypeInfo vmapple_virtio_aux_info = { > - .name = TYPE_VMAPPLE_VIRTIO_AUX, > - .parent = "vmapple-virtio-blk-pci", > - .instance_size = sizeof(VMAppleVirtIOBlkPCI), > - .instance_init = vmapple_virtio_aux_instance_init, > -}; > - > static void vmapple_virtio_blk_register_types(void) > { > type_register_static(&vmapple_virtio_blk_info); > virtio_pci_types_register(&vmapple_virtio_blk_pci_info); > - type_register_static(&vmapple_virtio_root_info); > - type_register_static(&vmapple_virtio_aux_info); > } > > type_init(vmapple_virtio_blk_register_types) > diff --git a/include/hw/qdev-properties-system.h b/include/hw/qdev-properties-system.h > index cdcc63056e5..6e428f3fcad 100644 > --- a/include/hw/qdev-properties-system.h > +++ b/include/hw/qdev-properties-system.h > @@ -27,6 +27,8 @@ extern const PropertyInfo qdev_prop_pcie_link_speed; > extern const PropertyInfo qdev_prop_pcie_link_width; > extern const PropertyInfo qdev_prop_cpus390entitlement; > extern const PropertyInfo qdev_prop_iothread_vq_mapping_list; > +extern const PropertyInfo qdev_prop_vmapple_virtio_blk_variant; > + Here is an extra line. > > #define DEFINE_PROP_PCI_DEVFN(_n, _s, _f, _d) \ > DEFINE_PROP_SIGNED(_n, _s, _f, _d, qdev_prop_pci_devfn, int32_t) > @@ -94,4 +96,8 @@ extern const PropertyInfo qdev_prop_iothread_vq_mapping_list; > DEFINE_PROP(_name, _state, _field, qdev_prop_iothread_vq_mapping_list, \ > IOThreadVirtQueueMappingList *) > > +#define DEFINE_PROP_VMAPPLE_VIRTIO_BLK_VARIANT(_n, _s, _f, _d) \ > + DEFINE_PROP_SIGNED(_n, _s, _f, _d, qdev_prop_vmapple_virtio_blk_variant, \ > + VMAppleVirtioBlkVariant) > + > #endif > diff --git a/include/hw/vmapple/vmapple.h b/include/hw/vmapple/vmapple.h > index b20956e1286..9c1ad1bd8c3 100644 > --- a/include/hw/vmapple/vmapple.h > +++ b/include/hw/vmapple/vmapple.h > @@ -18,8 +18,6 @@ > > #define TYPE_VMAPPLE_CFG "vmapple-cfg" > > -#define TYPE_VMAPPLE_VIRTIO_BLK "vmapple-virtio-blk" > -#define TYPE_VMAPPLE_VIRTIO_ROOT "vmapple-virtio-root" > -#define TYPE_VMAPPLE_VIRTIO_AUX "vmapple-virtio-aux" > +#define TYPE_VMAPPLE_VIRTIO_BLK_PCI "vmapple-virtio-blk-pci" > > #endif /* HW_VMAPPLE_VMAPPLE_H */ > diff --git a/qapi/virtio.json b/qapi/virtio.json > index 2529c2d8b20..d351d2166ef 100644 > --- a/qapi/virtio.json > +++ b/qapi/virtio.json > @@ -992,3 +992,17 @@ > ## > { 'enum': 'GranuleMode', > 'data': [ '4k', '8k', '16k', '64k', 'host' ] } > + > +## > +# @VMAppleVirtioBlkVariant: > +# > +# @unspecified: The default, not a valid setting. > +# > +# @root: Block device holding the root volume > +# > +# @aux: Block device holding auxiliary data required for boot > +# > +# Since: 9.2 > +## > +{ 'enum': 'VMAppleVirtioBlkVariant', > + 'data': [ 'unspecified', 'root', 'aux' ] }
© 2016 - 2024 Red Hat, Inc.