qdev: Convert to qdev_unrealize() with Coccinelle

For readability, and consistency with qbus_realize().

Coccinelle script:

    @ depends on !(file in "hw/core/qdev.c")@
    typedef DeviceState;
    DeviceState *dev;
    symbol false, error_abort;
    @@
    -    object_property_set_bool(OBJECT(dev), false, "realized", &error_abort);
    +    qdev_unrealize(dev);

    @ depends on !(file in "hw/core/qdev.c") && !(file in "hw/core/bus.c")@
    expression dev;
    symbol false, error_abort;
    @@
    -    object_property_set_bool(OBJECT(dev), false, "realized", &error_abort);
    +    qdev_unrealize(DEVICE(dev));

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20200610053247.1583243-8-armbru@redhat.com>
This commit is contained in:
Markus Armbruster 2020-06-10 07:31:56 +02:00
parent f1483b466d
commit 981c3dcd94
11 changed files with 16 additions and 18 deletions

View File

@ -266,7 +266,7 @@ void acpi_pcihp_device_unplug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
{ {
trace_acpi_pci_unplug(PCI_SLOT(PCI_DEVICE(dev)->devfn), trace_acpi_pci_unplug(PCI_SLOT(PCI_DEVICE(dev)->devfn),
acpi_pcihp_get_bsel(pci_get_bus(PCI_DEVICE(dev)))); acpi_pcihp_get_bsel(pci_get_bus(PCI_DEVICE(dev))));
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev, void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,

View File

@ -56,7 +56,7 @@ static void multi_serial_pci_exit(PCIDevice *dev)
for (i = 0; i < pci->ports; i++) { for (i = 0; i < pci->ports; i++) {
s = pci->state + i; s = pci->state + i;
object_property_set_bool(OBJECT(s), false, "realized", &error_abort); qdev_unrealize(DEVICE(s));
memory_region_del_subregion(&pci->iobar, &s->io); memory_region_del_subregion(&pci->iobar, &s->io);
g_free(pci->name[i]); g_free(pci->name[i]);
} }

View File

@ -68,7 +68,7 @@ static void serial_pci_exit(PCIDevice *dev)
PCISerialState *pci = DO_UPCAST(PCISerialState, dev, dev); PCISerialState *pci = DO_UPCAST(PCISerialState, dev, dev);
SerialState *s = &pci->state; SerialState *s = &pci->state;
object_property_set_bool(OBJECT(s), false, "realized", &error_abort); qdev_unrealize(DEVICE(s));
qemu_free_irq(s->irq); qemu_free_irq(s->irq);
} }

View File

@ -200,8 +200,7 @@ static void bus_set_realized(Object *obj, bool value, Error **errp)
} else if (!value && bus->realized) { } else if (!value && bus->realized) {
QTAILQ_FOREACH(kid, &bus->children, sibling) { QTAILQ_FOREACH(kid, &bus->children, sibling) {
DeviceState *dev = kid->child; DeviceState *dev = kid->child;
object_property_set_bool(OBJECT(dev), false, "realized", qdev_unrealize(dev);
&error_abort);
} }
if (bc->unrealize) { if (bc->unrealize) {
bc->unrealize(bus); bc->unrealize(bus);

View File

@ -1386,7 +1386,7 @@ static void pc_memory_unplug(HotplugHandler *hotplug_dev,
} }
pc_dimm_unplug(PC_DIMM(dev), MACHINE(pcms)); pc_dimm_unplug(PC_DIMM(dev), MACHINE(pcms));
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
out: out:
error_propagate(errp, local_err); error_propagate(errp, local_err);
} }
@ -1494,7 +1494,7 @@ static void pc_cpu_unplug_cb(HotplugHandler *hotplug_dev,
found_cpu = pc_find_cpu_slot(MACHINE(pcms), cpu->apic_id, NULL); found_cpu = pc_find_cpu_slot(MACHINE(pcms), cpu->apic_id, NULL);
found_cpu->cpu = NULL; found_cpu->cpu = NULL;
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
/* decrement the number of CPUs */ /* decrement the number of CPUs */
x86ms->boot_cpus--; x86ms->boot_cpus--;

View File

@ -460,7 +460,7 @@ void pcie_cap_slot_plug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
void pcie_cap_slot_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev, void pcie_cap_slot_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
Error **errp) Error **errp)
{ {
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
static void pcie_unplug_device(PCIBus *bus, PCIDevice *dev, void *opaque) static void pcie_unplug_device(PCIBus *bus, PCIDevice *dev, void *opaque)

View File

@ -547,7 +547,7 @@ void shpc_device_plug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
void shpc_device_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev, void shpc_device_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
Error **errp) Error **errp)
{ {
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
void shpc_device_unplug_request_cb(HotplugHandler *hotplug_dev, void shpc_device_unplug_request_cb(HotplugHandler *hotplug_dev,

View File

@ -3671,7 +3671,7 @@ static void spapr_memory_unplug(HotplugHandler *hotplug_dev, DeviceState *dev)
SpaprDimmState *ds = spapr_pending_dimm_unplugs_find(spapr, PC_DIMM(dev)); SpaprDimmState *ds = spapr_pending_dimm_unplugs_find(spapr, PC_DIMM(dev));
pc_dimm_unplug(PC_DIMM(dev), MACHINE(hotplug_dev)); pc_dimm_unplug(PC_DIMM(dev), MACHINE(hotplug_dev));
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
spapr_pending_dimm_unplugs_remove(spapr, ds); spapr_pending_dimm_unplugs_remove(spapr, ds);
} }
@ -3764,7 +3764,7 @@ static void spapr_core_unplug(HotplugHandler *hotplug_dev, DeviceState *dev)
assert(core_slot); assert(core_slot);
core_slot->cpu = NULL; core_slot->cpu = NULL;
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
static static
@ -4037,7 +4037,7 @@ void spapr_phb_release(DeviceState *dev)
static void spapr_phb_unplug(HotplugHandler *hotplug_dev, DeviceState *dev) static void spapr_phb_unplug(HotplugHandler *hotplug_dev, DeviceState *dev)
{ {
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
static void spapr_phb_unplug_request(HotplugHandler *hotplug_dev, static void spapr_phb_unplug_request(HotplugHandler *hotplug_dev,
@ -4073,7 +4073,7 @@ static void spapr_tpm_proxy_unplug(HotplugHandler *hotplug_dev, DeviceState *dev
{ {
SpaprMachineState *spapr = SPAPR_MACHINE(OBJECT(hotplug_dev)); SpaprMachineState *spapr = SPAPR_MACHINE(OBJECT(hotplug_dev));
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
object_unparent(OBJECT(dev)); object_unparent(OBJECT(dev));
spapr->tpm_proxy = NULL; spapr->tpm_proxy = NULL;
} }

View File

@ -1587,8 +1587,7 @@ static void spapr_pci_unplug(HotplugHandler *plug_handler,
return; return;
} }
object_property_set_bool(OBJECT(plugged_dev), false, "realized", qdev_unrealize(plugged_dev);
&error_abort);
} }
static void spapr_pci_unplug_request(HotplugHandler *plug_handler, static void spapr_pci_unplug_request(HotplugHandler *plug_handler,

View File

@ -54,7 +54,7 @@ static void ccw_device_unplug(HotplugHandler *hotplug_dev,
css_generate_sch_crws(sch->cssid, sch->ssid, sch->schid, 1, 0); css_generate_sch_crws(sch->cssid, sch->ssid, sch->schid, 1, 0);
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
static void virtual_css_bus_reset(BusState *qbus) static void virtual_css_bus_reset(BusState *qbus)

View File

@ -1003,7 +1003,7 @@ static void s390_pcihost_unplug(HotplugHandler *hotplug_dev, DeviceState *dev,
pbdev->fh, pbdev->fid); pbdev->fh, pbdev->fid);
bus = pci_get_bus(pci_dev); bus = pci_get_bus(pci_dev);
devfn = pci_dev->devfn; devfn = pci_dev->devfn;
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
s390_pci_msix_free(pbdev); s390_pci_msix_free(pbdev);
s390_pci_iommu_free(s, bus, devfn); s390_pci_iommu_free(s, bus, devfn);
@ -1014,7 +1014,7 @@ static void s390_pcihost_unplug(HotplugHandler *hotplug_dev, DeviceState *dev,
pbdev->fid = 0; pbdev->fid = 0;
QTAILQ_REMOVE(&s->zpci_devs, pbdev, link); QTAILQ_REMOVE(&s->zpci_devs, pbdev, link);
g_hash_table_remove(s->zpci_table, &pbdev->idx); g_hash_table_remove(s->zpci_table, &pbdev->idx);
object_property_set_bool(OBJECT(dev), false, "realized", &error_abort); qdev_unrealize(dev);
} }
} }