xemu-project/xemu
1
0
Fork 0
mirror of https://github.com/xemu-project/xemu.git synced 2024-11-23 19:49:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

spapr: Rename machine init functions for clarity

Browse Source 
Machine objects have two init functions - the generic QOM level
instance_init which should only do static object initialization, and
the Machine specific MachineClass::init which does the actual
construction of the machine.

In spapr the functions implementing these two have names -
ppc_machine_initfn() and ppc_spapr_init() - which don't correspond closely
to either of those.  To prevent people (read, me) from confusing which is
which, rename them spapr_instance_init() and spapr_machine_init() to
make it clearer which is which.

While we're there rename ppc_spapr_reset() to spapr_machine_reset() to
match.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Reviewed-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
This commit is contained in:
David Gibson 2017-12-08 12:47:34 +11:00
parent 2a83f9976e
commit bcb5ce08cf
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
hw/ppc/spapr.c
View File

@ -1454,7 +1454,7 @@ static int spapr_reset_drcs(Object *child, void *opaque)
return 0; return 0;
} }
static void ppc_spapr_reset(void) static void spapr_machine_reset(void)
{ {
MachineState *machine = MACHINE(qdev_get_machine()); MachineState *machine = MACHINE(qdev_get_machine());
sPAPRMachineState *spapr = SPAPR_MACHINE(machine); sPAPRMachineState *spapr = SPAPR_MACHINE(machine);
@ -2292,7 +2292,7 @@ out:
} }
/* pSeries LPAR / sPAPR hardware init */ /* pSeries LPAR / sPAPR hardware init */
static void ppc_spapr_init(MachineState *machine) static void spapr_machine_init(MachineState *machine)
{ {
sPAPRMachineState *spapr = SPAPR_MACHINE(machine); sPAPRMachineState *spapr = SPAPR_MACHINE(machine);
sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(machine); sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(machine);
@ -2820,7 +2820,7 @@ static void spapr_set_vsmt(Object *obj, Visitor *v, const char *name,
visit_type_uint32(v, name, (uint32_t *)opaque, errp); visit_type_uint32(v, name, (uint32_t *)opaque, errp);
} }
static void spapr_machine_initfn(Object *obj) static void spapr_instance_init(Object *obj)
{ {
sPAPRMachineState *spapr = SPAPR_MACHINE(obj); sPAPRMachineState *spapr = SPAPR_MACHINE(obj);
@ -3777,8 +3777,8 @@ static void spapr_machine_class_init(ObjectClass *oc, void *data)
* functions for the specific versioned machine types can override * functions for the specific versioned machine types can override
* these details for backwards compatibility * these details for backwards compatibility
*/ */
mc->init = ppc_spapr_init; mc->init = spapr_machine_init;
mc->reset = ppc_spapr_reset; mc->reset = spapr_machine_reset;
mc->block_default_type = IF_SCSI; mc->block_default_type = IF_SCSI;
mc->max_cpus = 1024; mc->max_cpus = 1024;
mc->no_parallel = 1; mc->no_parallel = 1;
@ -3825,7 +3825,7 @@ static const TypeInfo spapr_machine_info = {
.parent = TYPE_MACHINE, .parent = TYPE_MACHINE,
.abstract = true, .abstract = true,
.instance_size = sizeof(sPAPRMachineState), .instance_size = sizeof(sPAPRMachineState),
.instance_init = spapr_machine_initfn, .instance_init = spapr_instance_init,
.instance_finalize = spapr_machine_finalizefn, .instance_finalize = spapr_machine_finalizefn,
.class_size = sizeof(sPAPRMachineClass), .class_size = sizeof(sPAPRMachineClass),
.class_init = spapr_machine_class_init, .class_init = spapr_machine_class_init,