Comment 5 for bug 1761372

Revision history for this message
Christian Ehrhardt  (paelzer) wrote : Re: backport needed for spectre/meltdown mitigation machine type qemu patches (kvm)

So all that, to provide a -sxxm type that is NOT the default (not in your patches, so I didn't make it the default in mine).

I wonder, what is the benefit of this then.
If users need to change the machine type, then they can just as well set the CFPC/SBBC/IBS caps right?

Since we are not changing the deafault (pseries-bionic) is what we essentially support.
We don't even have to wrap it into the 2.11 type.

Todays machine types in Bionic:
[Upstream types]
pseries-2.10 pSeries Logical Partition (PAPR compliant)
pseries-2.11 pSeries Logical Partition (PAPR compliant)
pseries-2.12 pSeries Logical Partition (PAPR compliant)
[...]
[Ubuntu recommended types (the defaults)]
pseries pSeries Logical Partition (PAPR compliant) (alias of pseries-bionic)
pseries-bionic pSeries Logical Partition (PAPR compliant) (default)
[...]

Your patch will add
pseries-2.12-sxxm based on the 2.12 that was added in the stable 2.11.1 patches.
I mentioned this before on other places.
That might be a nice simplification for users, but since 2.12 is not yet set in stone (at least it is on -rc now) we can't (and won't) make any guarantees about cross release migrations.

pseries-bionic is what we can control, and the default and what we consider that shall work across releases.
Since you provided the extra type as non-default convenience type I'll do the same.

IFF there are changes to 2.12 in the remaining upstream 2.12 or the coming Ubuntu 18.10 development cycle we will not try (too hard) to keep e.g. a Bionic pseries-2.12-sxxm in sync with that of 18.10.

This is just the same as 1001 extra qemu options, if you derived from the common case it won't be migratable (it easily is with shutdown, start, just live has issues).