[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] hw/ppc/spapr.c: cleaning up qdev_get_machine() ca
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH] hw/ppc/spapr.c: cleaning up qdev_get_machine() calls |
Date: |
Sat, 9 Sep 2017 13:54:10 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Thu, Sep 07, 2017 at 05:21:23PM +0200, Greg Kurz wrote:
> On Wed, 6 Sep 2017 15:43:05 -0300
> Daniel Henrique Barboza <address@hidden> wrote:
>
> > This patch removes the qdev_get_machine() calls that are made in
> > spapr.c in situations where we can get an existing pointer for
> > the MachineState by either passing it as an argument to the function
> > or by using other already available pointers.
> >
> > The following changes were made:
> >
> > - spapr_node0_size: static function that is called two times:
> > at spapr_setup_hpt_and_vrma and ppc_spapr_init. In both cases we can
> > pass an existing MachineState pointer to it.
> >
> > - spapr_build_fdt: MachineState pointer can be retrieved from
> > the existing sPAPRMachineState pointer.
> >
> > - spapr_boot_set: the opaque in the first arg is a sPAPRMachineState
> > pointer as we can see inside ppc_spapr_init:
> >
> > qemu_register_boot_set(spapr_boot_set, spapr);
> >
> > We can get a MachineState pointer from it.
> >
> > - spapr_machine_device_plug and spapr_machine_device_unplug_request: the
> > MachineState, sPAPRMachineState, MachineClass and sPAPRMachineClass pointers
> > can all be retrieved from the HotplugHandler pointer.
> >
> > Signed-off-by: Daniel Henrique Barboza <address@hidden>
> > ---
>
> It makes sense for me.
>
> Reviewed-by: Greg Kurz <address@hidden>
Applied to ppc-for-2.11.
>
> > hw/ppc/spapr.c | 19 +++++++++----------
> > 1 file changed, 9 insertions(+), 10 deletions(-)
> >
> > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> > index cec441c..6ac3390 100644
> > --- a/hw/ppc/spapr.c
> > +++ b/hw/ppc/spapr.c
> > @@ -390,10 +390,8 @@ static int spapr_fixup_cpu_dt(void *fdt,
> > sPAPRMachineState *spapr)
> > return ret;
> > }
> >
> > -static hwaddr spapr_node0_size(void)
> > +static hwaddr spapr_node0_size(MachineState *machine)
> > {
> > - MachineState *machine = MACHINE(qdev_get_machine());
> > -
> > if (nb_numa_nodes) {
> > int i;
> > for (i = 0; i < nb_numa_nodes; ++i) {
> > @@ -1027,7 +1025,7 @@ static void *spapr_build_fdt(sPAPRMachineState *spapr,
> > hwaddr rtas_addr,
> > hwaddr rtas_size)
> > {
> > - MachineState *machine = MACHINE(qdev_get_machine());
> > + MachineState *machine = MACHINE(spapr);
> > MachineClass *mc = MACHINE_GET_CLASS(machine);
> > sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(machine);
> > int ret;
> > @@ -1347,7 +1345,7 @@ void spapr_setup_hpt_and_vrma(sPAPRMachineState
> > *spapr)
> > spapr_reallocate_hpt(spapr, hpt_shift, &error_fatal);
> >
> > if (spapr->vrma_adjust) {
> > - spapr->rma_size = kvmppc_rma_size(spapr_node0_size(),
> > + spapr->rma_size = kvmppc_rma_size(spapr_node0_size(MACHINE(spapr)),
> > spapr->htab_shift);
> > }
> > /* We're setting up a hash table, so that means we're not radix */
> > @@ -2007,7 +2005,7 @@ static SaveVMHandlers savevm_htab_handlers = {
> > static void spapr_boot_set(void *opaque, const char *boot_device,
> > Error **errp)
> > {
> > - MachineState *machine = MACHINE(qdev_get_machine());
> > + MachineState *machine = MACHINE(opaque);
> > machine->boot_order = g_strdup(boot_device);
> > }
> >
> > @@ -2154,7 +2152,7 @@ static void ppc_spapr_init(MachineState *machine)
> > MemoryRegion *rma_region;
> > void *rma = NULL;
> > hwaddr rma_alloc_size;
> > - hwaddr node0_size = spapr_node0_size();
> > + hwaddr node0_size = spapr_node0_size(machine);
> > long load_limit, fw_size;
> > char *filename;
> > Error *resize_hpt_err = NULL;
> > @@ -3198,7 +3196,8 @@ out:
> > static void spapr_machine_device_plug(HotplugHandler *hotplug_dev,
> > DeviceState *dev, Error **errp)
> > {
> > - sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(qdev_get_machine());
> > + MachineState *ms = MACHINE(hotplug_dev);
> > + sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(ms);
> >
> > if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> > int node;
> > @@ -3247,8 +3246,8 @@ static void spapr_machine_device_plug(HotplugHandler
> > *hotplug_dev,
> > static void spapr_machine_device_unplug_request(HotplugHandler
> > *hotplug_dev,
> > DeviceState *dev, Error
> > **errp)
> > {
> > - sPAPRMachineState *sms = SPAPR_MACHINE(qdev_get_machine());
> > - MachineClass *mc = MACHINE_GET_CLASS(qdev_get_machine());
> > + sPAPRMachineState *sms = SPAPR_MACHINE(OBJECT(hotplug_dev));
> > + MachineClass *mc = MACHINE_GET_CLASS(sms);
> >
> > if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> > if (spapr_ovec_test(sms->ov5_cas, OV5_HP_EVT)) {
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature