[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling
From: |
Thomas Huth |
Subject: |
[Qemu-devel] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling |
Date: |
Tue, 9 Aug 2016 18:59:56 +0200 |
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
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(-)
--
1.8.3.1
- [Qemu-devel] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling,
Thomas Huth <=
- [Qemu-devel] [PATCH v2 1/5] spapr: remove extra type variable, Thomas Huth, 2016/08/09
- [Qemu-devel] [PATCH v2 2/5] ppc: Introduce a function to look up CPU alias strings, Thomas Huth, 2016/08/09
- [Qemu-devel] [PATCH v2 3/5] hw/ppc/spapr: Look up CPU alias names instead of hard-coding the aliases, Thomas Huth, 2016/08/09
- [Qemu-devel] [PATCH v2 4/5] ppc/kvm: Do not mess up the generic CPU family registration, Thomas Huth, 2016/08/09
- [Qemu-devel] [PATCH v2 5/5] ppc/kvm: Register also a generic spapr CPU core family type, Thomas Huth, 2016/08/09
- Re: [Qemu-devel] [PATCH v2 0/5] spapr: Fix regression in CPU alias handling, David Gibson, 2016/08/10