On Fri, Jan 19, 2018 at 04:00:00PM +1100, Suraj Jitindar Singh wrote: > The vmstate description and the contained needed function for migration > of spapr_caps is the same for each cap, with the name of the cap > substituted. As such introduce a macro to allow for easier generation of > these. > > Convert the three existing spapr_caps (htm, vsx, and dfp) to use this > macro. > > Signed-off-by: Suraj Jitindar Singh <sjitindarsi...@gmail.com>
Applied to ppc-for-2.12, thanks. > > --- > > V5: > - Patch added to series > --- > hw/ppc/spapr_caps.c | 78 > +++++++++++++++++------------------------------------ > 1 file changed, 24 insertions(+), 54 deletions(-) > > diff --git a/hw/ppc/spapr_caps.c b/hw/ppc/spapr_caps.c > index d5c9ce774a..5d52969bd5 100644 > --- a/hw/ppc/spapr_caps.c > +++ b/hw/ppc/spapr_caps.c > @@ -228,62 +228,32 @@ int spapr_caps_post_migration(sPAPRMachineState *spapr) > return ok ? 0 : -EINVAL; > } > > -static bool spapr_cap_htm_needed(void *opaque) > -{ > - sPAPRMachineState *spapr = opaque; > - > - return spapr->cmd_line_caps[SPAPR_CAP_HTM] && > - (spapr->eff.caps[SPAPR_CAP_HTM] != > spapr->def.caps[SPAPR_CAP_HTM]); > -} > - > -const VMStateDescription vmstate_spapr_cap_htm = { > - .name = "spapr/cap/htm", > - .version_id = 1, > - .minimum_version_id = 1, > - .needed = spapr_cap_htm_needed, > - .fields = (VMStateField[]) { > - VMSTATE_UINT8(mig.caps[SPAPR_CAP_HTM], sPAPRMachineState), > - VMSTATE_END_OF_LIST() > - }, > -}; > - > -static bool spapr_cap_vsx_needed(void *opaque) > -{ > - sPAPRMachineState *spapr = opaque; > - > - return spapr->cmd_line_caps[SPAPR_CAP_VSX] && > - (spapr->eff.caps[SPAPR_CAP_VSX] != > spapr->def.caps[SPAPR_CAP_VSX]); > +/* Used to generate the migration field and needed function for a spapr cap > */ > +#define SPAPR_CAP_MIG_STATE(cap, ccap) \ > +static bool spapr_cap_##cap##_needed(void *opaque) \ > +{ \ > + sPAPRMachineState *spapr = opaque; \ > + \ > + return spapr->cmd_line_caps[SPAPR_CAP_##ccap] && \ > + (spapr->eff.caps[SPAPR_CAP_##ccap] != \ > + spapr->def.caps[SPAPR_CAP_##ccap]); \ > +} \ > + \ > +const VMStateDescription vmstate_spapr_cap_##cap = { \ > + .name = "spapr/cap/" #cap, \ > + .version_id = 1, \ > + .minimum_version_id = 1, \ > + .needed = spapr_cap_##cap##_needed, \ > + .fields = (VMStateField[]) { \ > + VMSTATE_UINT8(mig.caps[SPAPR_CAP_##ccap], \ > + sPAPRMachineState), \ > + VMSTATE_END_OF_LIST() \ > + }, \ > } > > -const VMStateDescription vmstate_spapr_cap_vsx = { > - .name = "spapr/cap/vsx", > - .version_id = 1, > - .minimum_version_id = 1, > - .needed = spapr_cap_vsx_needed, > - .fields = (VMStateField[]) { > - VMSTATE_UINT8(mig.caps[SPAPR_CAP_VSX], sPAPRMachineState), > - VMSTATE_END_OF_LIST() > - }, > -}; > - > -static bool spapr_cap_dfp_needed(void *opaque) > -{ > - sPAPRMachineState *spapr = opaque; > - > - return spapr->cmd_line_caps[SPAPR_CAP_DFP] && > - (spapr->eff.caps[SPAPR_CAP_DFP] != > spapr->def.caps[SPAPR_CAP_DFP]); > -} > - > -const VMStateDescription vmstate_spapr_cap_dfp = { > - .name = "spapr/cap/dfp", > - .version_id = 1, > - .minimum_version_id = 1, > - .needed = spapr_cap_dfp_needed, > - .fields = (VMStateField[]) { > - VMSTATE_UINT8(mig.caps[SPAPR_CAP_DFP], sPAPRMachineState), > - VMSTATE_END_OF_LIST() > - }, > -}; > +SPAPR_CAP_MIG_STATE(htm, HTM); > +SPAPR_CAP_MIG_STATE(vsx, VSX); > +SPAPR_CAP_MIG_STATE(dfp, DFP); > > void spapr_caps_reset(sPAPRMachineState *spapr) > { -- 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