On Tue, Feb 20, 2024 at 09:24:42PM +0900, Akihiko Odaki wrote: > A device gets automatically unrealized when being unparented. > > Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com> > --- > hw/pci/pcie_sriov.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/hw/pci/pcie_sriov.c b/hw/pci/pcie_sriov.c > index e9b23221d713..8b1fd2a89ad7 100644 > --- a/hw/pci/pcie_sriov.c > +++ b/hw/pci/pcie_sriov.c > @@ -17,7 +17,6 @@ > #include "hw/qdev-properties.h" > #include "qemu/error-report.h" > #include "qemu/range.h" > -#include "qapi/error.h" > #include "trace.h" > > static PCIDevice *register_vf(PCIDevice *pf, int devfn,
error.h is used elsewhere in this file too, removing the include is not a good idea, doing so breaks bisect. > @@ -204,11 +203,7 @@ static void unregister_vfs(PCIDevice *dev) > trace_sriov_unregister_vfs(dev->name, PCI_SLOT(dev->devfn), > PCI_FUNC(dev->devfn), num_vfs); > for (i = 0; i < num_vfs; i++) { > - Error *err = NULL; > PCIDevice *vf = dev->exp.sriov_pf.vf[i]; > - if (!object_property_set_bool(OBJECT(vf), "realized", false, &err)) { > - error_reportf_err(err, "Failed to unplug: "); > - } > object_unparent(OBJECT(vf)); > object_unref(OBJECT(vf)); > } > > -- > 2.43.1