On Tue, Oct 05, 2021 at 01:57:35PM +0200, Eric Auger wrote:
> > diff --git a/hw/virtio/virtio-iommu-pci.c b/hw/virtio/virtio-iommu-pci.c
> > index 770c286be7..f30eb16cbf 100644
> > --- a/hw/virtio/virtio-iommu-pci.c
> > +++ b/hw/virtio/virtio-iommu-pci.c
> > @@ -48,16 +48,9 @@ static void virtio_iommu_pci_realize(VirtIOPCIProxy
> > *vpci_dev, Error **errp)
> > VirtIOIOMMU *s = VIRTIO_IOMMU(vdev);
> >
> > if (!qdev_get_machine_hotplug_handler(DEVICE(vpci_dev))) {
> > - MachineClass *mc = MACHINE_GET_CLASS(qdev_get_machine());
> > -
> > - error_setg(errp,
> > - "%s machine fails to create iommu-map device tree
> > bindings",
> > - mc->name);
> actually this does not work. To add a hint you need the *errp to be set.
> Otherwise when running through this path you will get
>
> emu-system-x86_64: ../util/error.c:158: error_append_hint: Assertion
> `err && errp != &error_abort && errp != &error_fatal' failed.
>
> replace the error_append_hint with an error_setg (without the \n)
Woops sorry, will fix
Thanks,
Jean
