On Thu, 14 Jun 2018 14:41:24 +1000 David Gibson <da...@gibson.dropbear.id.au> wrote:
> In pnv_core_realize() we call two functions with an Error * parameter in > succession, which will go badly if they both cause errors. In fact, a > failure in either of them indicates a qemu internal error, so we can just > use &error_abort in both cases. > > Signed-off-by: David Gibson <da...@gibson.dropbear.id.au> > --- Reviewed-by: Greg Kurz <gr...@kaod.org> > hw/ppc/pnv_core.c | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/hw/ppc/pnv_core.c b/hw/ppc/pnv_core.c > index 13ad7d9e04..01f47c8037 100644 > --- a/hw/ppc/pnv_core.c > +++ b/hw/ppc/pnv_core.c > @@ -172,12 +172,9 @@ static void pnv_core_realize(DeviceState *dev, Error > **errp) > object_initialize(obj, size, typename); > > snprintf(name, sizeof(name), "thread[%d]", i); > - object_property_add_child(OBJECT(pc), name, obj, &local_err); > + object_property_add_child(OBJECT(pc), name, obj, &error_abort); > object_property_add_alias(obj, "core-pir", OBJECT(pc), > - "pir", &local_err); > - if (local_err) { > - goto err; > - } > + "pir", &error_abort); > object_unref(obj); > } >