On Fri, 1 Oct 2021 18:33:53 +0100 Jean-Philippe Brucker <jean-phili...@linaro.org> wrote:
> To propagate errors to the caller of the pre_plug callback, use the > object_poperty_set*() functions directly instead of the qdev_prop_set*() > helpers. > > Suggested-by: Igor Mammedov <imamm...@redhat.com> > Signed-off-by: Jean-Philippe Brucker <jean-phili...@linaro.org> Reviewed-by: Igor Mammedov <imamm...@redhat.com> > --- > hw/arm/virt.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/hw/arm/virt.c b/hw/arm/virt.c > index 36f0261ef4..ac307b6030 100644 > --- a/hw/arm/virt.c > +++ b/hw/arm/virt.c > @@ -2465,8 +2465,9 @@ static void > virt_machine_device_pre_plug_cb(HotplugHandler *hotplug_dev, > db_start, db_end, > VIRTIO_IOMMU_RESV_MEM_T_MSI); > > - qdev_prop_set_uint32(dev, "len-reserved-regions", 1); > - qdev_prop_set_string(dev, "reserved-regions[0]", resv_prop_str); > + object_property_set_uint(OBJECT(dev), "len-reserved-regions", 1, > errp); > + object_property_set_str(OBJECT(dev), "reserved-regions[0]", > + resv_prop_str, errp); > g_free(resv_prop_str); > } > }
