From 6227a5d2d70dd396c428af4864ab38b1a32ef0f5 Mon Sep 17 00:00:00 2001 Message-Id: <6227a5d2d70dd396c428af4864ab38b1a32ef0f5@dist-git> From: Jiri Denemark Date: Fri, 15 Nov 2019 17:52:34 +0100 Subject: [PATCH] qemu: Drop disabled CPU features unknown to QEMU MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When a CPU definition wants to explicitly disable some features that are unknown to QEMU, we can safely drop them from the definition before starting QEMU. Naturally QEMU won't enable such features implicitly. Signed-off-by: Jiri Denemark Reviewed-by: Daniel P. Berrangé (cherry picked from commit ac34e141596fab70fbe91a396311f80db6cb57c5) https://bugzilla.redhat.com/show_bug.cgi?id=1749672 https://bugzilla.redhat.com/show_bug.cgi?id=1756156 https://bugzilla.redhat.com/show_bug.cgi?id=1721608 Conflicts: src/qemu/qemu_process.c - downstream still uses cleanup label Signed-off-by: Jiri Denemark Message-Id: <92f146f8754300266d688dd92dfe95e737dda7da.1573836581.git.jdenemar@redhat.com> Reviewed-by: Michal Privoznik --- src/qemu/qemu_process.c | 32 +++++++++++++++++++ ...-Icelake-Server-pconfig.x86_64-latest.args | 2 +- 2 files changed, 33 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index 0cd61f02bb..0700b054f3 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -5730,6 +5730,26 @@ qemuProcessSetupHotpluggableVcpus(virQEMUDriverPtr driver, } +static bool +qemuProcessDropUnknownCPUFeatures(const char *name, + virCPUFeaturePolicy policy, + void *opaque) +{ + const char **features = opaque; + + if (policy != VIR_CPU_FEATURE_DISABLE && + policy != VIR_CPU_FEATURE_FORBID) + return true; + + if (virStringListHasString(features, name)) + return true; + + /* Features unknown to QEMU are implicitly disabled, we can just drop them + * from the definition. */ + return false; +} + + static int qemuProcessUpdateGuestCPU(virDomainDefPtr def, virQEMUCapsPtr qemuCaps, @@ -5792,6 +5812,18 @@ qemuProcessUpdateGuestCPU(virDomainDefPtr def, virQEMUCapsGetCPUDefinitions(qemuCaps, def->virtType)) < 0) goto cleanup; + if (ARCH_IS_X86(def->os.arch)) { + VIR_AUTOSTRINGLIST features = NULL; + + if (virQEMUCapsGetCPUFeatures(qemuCaps, def->virtType, false, &features) < 0) + goto cleanup; + + if (features && + virCPUDefFilterFeatures(def->cpu, qemuProcessDropUnknownCPUFeatures, + features) < 0) + goto cleanup; + } + def->cpu->fallback = VIR_CPU_FALLBACK_FORBID; ret = 0; diff --git a/tests/qemuxml2argvdata/cpu-Icelake-Server-pconfig.x86_64-latest.args b/tests/qemuxml2argvdata/cpu-Icelake-Server-pconfig.x86_64-latest.args index 664db44e7b..de737bfed7 100644 --- a/tests/qemuxml2argvdata/cpu-Icelake-Server-pconfig.x86_64-latest.args +++ b/tests/qemuxml2argvdata/cpu-Icelake-Server-pconfig.x86_64-latest.args @@ -10,7 +10,7 @@ QEMU_AUDIO_DRV=none \ -object secret,id=masterKey0,format=raw,\ file=/tmp/lib/domain--1-test/master-key.aes \ -machine pc,accel=kvm,usb=off,dump-guest-core=off \ --cpu Icelake-Server,pconfig=off \ +-cpu Icelake-Server \ -m 214 \ -realtime mlock=off \ -smp 1,sockets=1,cores=1,threads=1 \ -- 2.24.0