[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v1 5/5] cpu, spapr: Use migration_id from ps
From: |
Bharata B Rao |
Subject: |
Re: [Qemu-devel] [RFC PATCH v1 5/5] cpu, spapr: Use migration_id from pseries-2.7 onwards |
Date: |
Wed, 6 Jul 2016 19:54:44 +0530 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Wed, Jul 06, 2016 at 01:45:54PM +0200, Igor Mammedov wrote:
> On Wed, 6 Jul 2016 14:29:21 +0530
> Bharata B Rao <address@hidden> wrote:
>
> > Turn on use-migration-id property. Starting from pseries-2.7, prefer
> > the use of migration_id (cpu_dt_id) over cpu_index for cpu vmstate
> > registration and in XICS code.
> >
> > This allows migration to work when CPU cores are not necessarily
> > unplugged in LIFO order.
> >
> > TODO: Property use-migration-id is by default turned on, check
> > if this needs to turned off for older machine type versions of
> > all archs.
> >
> > Signed-off-by: Bharata B Rao <address@hidden>
> > ---
> > hw/ppc/spapr.c | 5 +++++
> > qom/cpu.c | 1 +
> > 2 files changed, 6 insertions(+)
> >
> > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> > index 7f33a1b..be9af10 100644
> > --- a/hw/ppc/spapr.c
> > +++ b/hw/ppc/spapr.c
> > @@ -2501,6 +2501,11 @@ DEFINE_SPAPR_MACHINE(2_7, "2.7", true);
> > .driver = TYPE_SPAPR_PCI_HOST_BRIDGE,\
> > .property = "ddw",\
> > .value = stringify(off),\
> > + }, \
> > + { \
> > + .driver = TYPE_CPU,\
> > + .property = "use-migration-id",\
> > + .value = stringify(off),\
> nit,
> why not to use "off" directly, stringify looks odd here.
Was just following the previous property "ddw" did, will check if
stringify can be dropped.
Regards,
Bharata.
- Re: [Qemu-devel] [RFC PATCH v1 3/5] spapr: Implement CPUClass::get_migration_id() for PowerPC CPUs, (continued)