mirror of https://github.com/xqemu/xqemu.git
spapr: do proper error propagation in spapr_cpu_core_realize_child()
This patch changes spapr_cpu_core_realize_child() to have a local error pointer and use error_propagate() as it is supposed to be done. Signed-off-by: Greg Kurz <groug@kaod.org> Reviewed-by: Bharata B Rao <bharata@linux.vnet.ibm.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
8e758dee66
commit
f11235b920
|
@ -262,18 +262,20 @@ out:
|
||||||
|
|
||||||
static int spapr_cpu_core_realize_child(Object *child, void *opaque)
|
static int spapr_cpu_core_realize_child(Object *child, void *opaque)
|
||||||
{
|
{
|
||||||
Error **errp = opaque;
|
Error **errp = opaque, *local_err = NULL;
|
||||||
sPAPRMachineState *spapr = SPAPR_MACHINE(qdev_get_machine());
|
sPAPRMachineState *spapr = SPAPR_MACHINE(qdev_get_machine());
|
||||||
CPUState *cs = CPU(child);
|
CPUState *cs = CPU(child);
|
||||||
PowerPCCPU *cpu = POWERPC_CPU(cs);
|
PowerPCCPU *cpu = POWERPC_CPU(cs);
|
||||||
|
|
||||||
object_property_set_bool(child, true, "realized", errp);
|
object_property_set_bool(child, true, "realized", &local_err);
|
||||||
if (*errp) {
|
if (local_err) {
|
||||||
|
error_propagate(errp, local_err);
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
spapr_cpu_init(spapr, cpu, errp);
|
spapr_cpu_init(spapr, cpu, &local_err);
|
||||||
if (*errp) {
|
if (local_err) {
|
||||||
|
error_propagate(errp, local_err);
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Loading…
Reference in New Issue