|
 |
99cbc7 |
From 91b7c36af5d91f9ddfe47a3a2b1b35368ef171a4 Mon Sep 17 00:00:00 2001
|
|
|
99cbc7 |
Message-Id: <91b7c36af5d91f9ddfe47a3a2b1b35368ef171a4@dist-git>
|
|
|
99cbc7 |
From: Andrea Bolognani <abologna@redhat.com>
|
|
|
99cbc7 |
Date: Fri, 5 Apr 2019 10:57:40 +0200
|
|
|
99cbc7 |
Subject: [PATCH] qemu: Drop QEMU_CAPS_ENABLE_KVM
|
|
|
99cbc7 |
|
|
|
99cbc7 |
It was already available in 1.5.0.
|
|
|
99cbc7 |
|
|
|
99cbc7 |
Moreover, we're not even formatting it on the QEMU command
|
|
|
99cbc7 |
line, ever: we just use it as part of some logic that decides
|
|
|
99cbc7 |
whether KVM support should be advertised, and as it turns out
|
|
|
99cbc7 |
that logic is actually buggy and dropping this capability
|
|
|
99cbc7 |
fixes it.
|
|
|
99cbc7 |
|
|
|
99cbc7 |
https://bugzilla.redhat.com/show_bug.cgi?id=1628469
|
|
|
99cbc7 |
|
|
|
99cbc7 |
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
|
|
|
99cbc7 |
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
99cbc7 |
(cherry picked from commit 88983855d5496a74b97551860db737c2b17b100e)
|
|
|
99cbc7 |
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
|
|
|
99cbc7 |
Message-Id: <20190405085744.8179-4-abologna@redhat.com>
|
|
|
99cbc7 |
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
99cbc7 |
---
|
|
|
99cbc7 |
src/qemu/qemu_capabilities.c | 3 ---
|
|
|
99cbc7 |
src/qemu/qemu_capabilities.h | 2 +-
|
|
|
99cbc7 |
tests/qemuxml2argvtest.c | 11 +++++------
|
|
|
99cbc7 |
3 files changed, 6 insertions(+), 10 deletions(-)
|
|
|
99cbc7 |
|
|
|
99cbc7 |
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
|
|
|
99cbc7 |
index 4f2051a2bb..c31c991a6f 100644
|
|
|
99cbc7 |
--- a/src/qemu/qemu_capabilities.c
|
|
|
99cbc7 |
+++ b/src/qemu/qemu_capabilities.c
|
|
|
99cbc7 |
@@ -834,7 +834,6 @@ virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
|
|
|
99cbc7 |
|
|
|
99cbc7 |
if (virFileExists("/dev/kvm") &&
|
|
|
99cbc7 |
(virQEMUCapsGet(qemubinCaps, QEMU_CAPS_KVM) ||
|
|
|
99cbc7 |
- virQEMUCapsGet(qemubinCaps, QEMU_CAPS_ENABLE_KVM) ||
|
|
|
99cbc7 |
kvmbin))
|
|
|
99cbc7 |
haskvm = true;
|
|
|
99cbc7 |
|
|
|
99cbc7 |
@@ -2604,7 +2603,6 @@ virQEMUCapsProbeQMPKVMState(virQEMUCapsPtr qemuCaps,
|
|
|
99cbc7 |
virQEMUCapsClear(qemuCaps, QEMU_CAPS_KVM);
|
|
|
99cbc7 |
} else if (!enabled) {
|
|
|
99cbc7 |
virQEMUCapsClear(qemuCaps, QEMU_CAPS_KVM);
|
|
|
99cbc7 |
- virQEMUCapsSet(qemuCaps, QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
}
|
|
|
99cbc7 |
|
|
|
99cbc7 |
return 0;
|
|
|
99cbc7 |
@@ -3912,7 +3910,6 @@ virQEMUCapsIsValid(void *data,
|
|
|
99cbc7 |
priv->runUid, priv->runGid) == 0;
|
|
|
99cbc7 |
|
|
|
99cbc7 |
if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_KVM) &&
|
|
|
99cbc7 |
- virQEMUCapsGet(qemuCaps, QEMU_CAPS_ENABLE_KVM) &&
|
|
|
99cbc7 |
kvmUsable) {
|
|
|
99cbc7 |
VIR_DEBUG("KVM was not enabled when probing '%s', "
|
|
|
99cbc7 |
"but it should be usable now",
|
|
|
99cbc7 |
diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
|
|
|
99cbc7 |
index 9e8ad5f5c3..98668115d6 100644
|
|
|
99cbc7 |
--- a/src/qemu/qemu_capabilities.h
|
|
|
99cbc7 |
+++ b/src/qemu/qemu_capabilities.h
|
|
|
99cbc7 |
@@ -79,7 +79,7 @@ typedef enum { /* virQEMUCapsFlags grouping marker for syntax-check */
|
|
|
99cbc7 |
X_QEMU_CAPS_XEN_DOMID, /* -xen-domid */
|
|
|
99cbc7 |
X_QEMU_CAPS_MIGRATE_QEMU_UNIX, /* qemu migration via unix sockets */
|
|
|
99cbc7 |
X_QEMU_CAPS_CHARDEV, /* Is the new -chardev arg available */
|
|
|
99cbc7 |
- QEMU_CAPS_ENABLE_KVM, /* -enable-kvm flag */
|
|
|
99cbc7 |
+ X_QEMU_CAPS_ENABLE_KVM, /* -enable-kvm flag */
|
|
|
99cbc7 |
X_QEMU_CAPS_MONITOR_JSON, /* JSON mode for monitor */
|
|
|
99cbc7 |
|
|
|
99cbc7 |
/* 25 */
|
|
|
99cbc7 |
diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
|
|
|
99cbc7 |
index 7a731e2f40..f401076731 100644
|
|
|
99cbc7 |
--- a/tests/qemuxml2argvtest.c
|
|
|
99cbc7 |
+++ b/tests/qemuxml2argvtest.c
|
|
|
99cbc7 |
@@ -912,16 +912,15 @@ mymain(void)
|
|
|
99cbc7 |
DO_TEST("clock-france", NONE);
|
|
|
99cbc7 |
DO_TEST("clock-hpet-off", NONE);
|
|
|
99cbc7 |
DO_TEST("clock-catchup", QEMU_CAPS_KVM_PIT_TICK_POLICY);
|
|
|
99cbc7 |
- DO_TEST("cpu-kvmclock", QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
- DO_TEST("cpu-host-kvmclock", QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
+ DO_TEST("cpu-kvmclock", NONE);
|
|
|
99cbc7 |
+ DO_TEST("cpu-host-kvmclock", NONE);
|
|
|
99cbc7 |
DO_TEST("kvmclock", QEMU_CAPS_KVM);
|
|
|
99cbc7 |
DO_TEST("clock-timer-hyperv-rtc", QEMU_CAPS_KVM);
|
|
|
99cbc7 |
|
|
|
99cbc7 |
- DO_TEST("cpu-eoi-disabled", QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
- DO_TEST("cpu-eoi-enabled", QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
+ DO_TEST("cpu-eoi-disabled", NONE);
|
|
|
99cbc7 |
+ DO_TEST("cpu-eoi-enabled", NONE);
|
|
|
99cbc7 |
DO_TEST("controller-order",
|
|
|
99cbc7 |
QEMU_CAPS_KVM,
|
|
|
99cbc7 |
- QEMU_CAPS_ENABLE_KVM,
|
|
|
99cbc7 |
QEMU_CAPS_PIIX3_USB_UHCI,
|
|
|
99cbc7 |
QEMU_CAPS_CCID_PASSTHRU,
|
|
|
99cbc7 |
QEMU_CAPS_SPICE,
|
|
|
99cbc7 |
@@ -933,7 +932,7 @@ mymain(void)
|
|
|
99cbc7 |
DO_TEST("eoi-enabled", NONE);
|
|
|
99cbc7 |
DO_TEST("pv-spinlock-disabled", NONE);
|
|
|
99cbc7 |
DO_TEST("pv-spinlock-enabled", NONE);
|
|
|
99cbc7 |
- DO_TEST("kvmclock+eoi-disabled", QEMU_CAPS_ENABLE_KVM);
|
|
|
99cbc7 |
+ DO_TEST("kvmclock+eoi-disabled", NONE);
|
|
|
99cbc7 |
|
|
|
99cbc7 |
DO_TEST("hyperv", NONE);
|
|
|
99cbc7 |
DO_TEST("hyperv-off", NONE);
|
|
|
99cbc7 |
--
|
|
|
99cbc7 |
2.21.0
|
|
|
99cbc7 |
|