[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v2 0/5] spapr: Fix regression in CPU alias handlin
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling |
Date: |
Wed, 10 Aug 2016 13:14:47 +1000 |
User-agent: |
Mutt/1.6.2 (2016-07-01) |
On Tue, Aug 09, 2016 at 06:59:56PM +0200, Thomas Huth wrote:
> There is a regression with the "-cpu" parameter which has been
> introduced by the spapr CPU hotplug code: We used to allow to specify
> a "CPU family" name with the "-cpu" parameter when running on KVM so
> that the user does not need to know the gory details of the exact
> CPU version of the host CPU. For example, it was possible to
> use "-cpu POWER8" on a POWER8E host CPU. This behavior does not
> work anymore with the new hot-pluggable spapr-cpu-core types.
> Since libvirt already heavily depends on the old behavior, this
> is quite a severe regression in the QEMU parameter interface, thus
> I think these patches should still go into 2.7 if possible, to avoid
> that we break the "upper layers" with the final 2.7 release.
>
> This patch series fixes the regression (and two more minor bugs)
> by registering a family type for the spapr-cpu-core type, too
> (see the last patch). However, since that name clashes with the
> alias types that have been defined in spapr_cpu_core.c, we first
> have to introduce there a better way to handle CPU aliases (see
> the first two patches).
> The third patch fixes a small memory leak along the way, and
> the fourth patch makes sure that we do not mess up the generic
> CPU family type registration anymore (which was another regression
> introduced with the spapr CPU hotplug code).
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1363812
Applied to ppc-for-2.7, thanks.
>
> v2:
> - Add the "remove extra type variable" patch from Cédric Le Goater.
> - Remove the "Do not leak the memory of the type string" patch
> since it got obsoleted by Cédric's patch
>
> Cédric Le Goater (1):
> spapr: remove extra type variable
>
> Thomas Huth (4):
> ppc: Introduce a function to look up CPU alias strings
> hw/ppc/spapr: Look up CPU alias names instead of hard-coding the
> aliases
> ppc/kvm: Do not mess up the generic CPU family registration
> ppc/kvm: Register also a generic spapr CPU core family type
>
> hw/ppc/spapr.c | 15 ++++++---------
> hw/ppc/spapr_cpu_core.c | 38 +++++++++++++++++++++-----------------
> target-ppc/cpu.h | 1 +
> target-ppc/kvm.c | 19 +++++++++++--------
> target-ppc/translate_init.c | 13 +++++++++++++
> 5 files changed, 52 insertions(+), 34 deletions(-)
>
--
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
- [Qemu-ppc] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling, Thomas Huth, 2016/08/09
- [Qemu-ppc] [PATCH v2 1/5] spapr: remove extra type variable, Thomas Huth, 2016/08/09
- [Qemu-ppc] [PATCH v2 2/5] ppc: Introduce a function to look up CPU alias strings, Thomas Huth, 2016/08/09
- [Qemu-ppc] [PATCH v2 3/5] hw/ppc/spapr: Look up CPU alias names instead of hard-coding the aliases, Thomas Huth, 2016/08/09
- [Qemu-ppc] [PATCH v2 4/5] ppc/kvm: Do not mess up the generic CPU family registration, Thomas Huth, 2016/08/09
- [Qemu-ppc] [PATCH v2 5/5] ppc/kvm: Register also a generic spapr CPU core family type, Thomas Huth, 2016/08/09
- Re: [Qemu-ppc] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling,
David Gibson <=