On Fri, 3 Feb 2023 19:09:00 +0100 Philippe Mathieu-Daudé <phi...@linaro.org> wrote:
> No need to use the low-level QOM API when an object > inherits from QDev. Directly use the QDev API to set > its properties. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > hw/core/numa.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/core/numa.c b/hw/core/numa.c > index d8d36b16d8..9b6f9848e3 100644 > --- a/hw/core/numa.c > +++ b/hw/core/numa.c > @@ -34,6 +34,7 @@ > #include "qapi/opts-visitor.h" > #include "qapi/qapi-visit-machine.h" > #include "sysemu/qtest.h" > +#include "hw/qdev-properties.h" > #include "hw/core/cpu.h" > #include "hw/mem/pc-dimm.h" > #include "migration/vmstate.h" > @@ -740,8 +741,7 @@ void numa_cpu_pre_plug(const CPUArchId *slot, DeviceState > *dev, Error **errp) > /* due to bug in libvirt, it doesn't pass node-id from props on > * device_add as expected, so we have to fix it up here */ > if (slot->props.has_node_id) { > - object_property_set_int(OBJECT(dev), "node-id", > - slot->props.node_id, errp); > + qdev_prop_set_int32(dev, "node-id", slot->props.node_id); broken, see cover letter reply > } > } else if (node_id != slot->props.node_id) { > error_setg(errp, "invalid node-id, must be %"PRId64,