[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] spapr/kvm: Set default cpu model for all machine classes
From: |
Greg Kurz |
Subject: |
Re: [PATCH] spapr/kvm: Set default cpu model for all machine classes |
Date: |
Wed, 13 Nov 2019 16:09:00 +0100 |
On Wed, 13 Nov 2019 15:43:44 +0100
Jiri Denemark <address@hidden> wrote:
> Hi David.
>
> On Wed, Oct 30, 2019 at 17:32:43 +0100, David Gibson wrote:
> > We have to set the default model of all machine classes, not just for the
> > active one. Otherwise, "query-machines" will indicate the wrong CPU model
> > ("qemu-s390x-cpu" instead of "host-s390x-cpu") as "default-cpu-type".
> >
> > s390x already fixed this in de60a92e "s390x/kvm: Set default cpu model for
> > all machine classes". This patch applies a similar fix for the pseries-*
> > machine types on ppc64.
> >
> > Doing a
> > {"execute":"query-machines"}
> > under KVM now results in
> > {
> > "hotpluggable-cpus": true,
> > "name": "pseries-4.2",
> > "numa-mem-supported": true,
> > "default-cpu-type": "host-powerpc64-cpu",
> > "is-default": true,
> > "cpu-max": 1024,
> > "deprecated": false,
> > "alias": "pseries"
> > },
> > {
> > "hotpluggable-cpus": true,
> > "name": "pseries-4.1",
> > "numa-mem-supported": true,
> > "default-cpu-type": "host-powerpc64-cpu",
> > "cpu-max": 1024,
> > "deprecated": false
> > },
> > ...
> >
> > Libvirt probes all machines via "-machine none,accel=kvm:tcg" and will
> > currently see the wrong CPU model under KVM.
>
> Will this patch make it into 4.2.0?
>
David is away until the 19th of November, which is the release date
of rc2 according to the planning [*]. Then we have rc3 the 26th, and
final release (or rc4) the 3rd of December, so it should be ok.
If David cannot make it for some reason, I guess Laurent Vivier or
myself can send a PR with this patch, as already suggested by David
in the past.
Cheers,
--
Greg
[*] https://wiki.qemu.org/Planning/4.2
> Jirka
>
>