[libvirt] [PATCH 3/8] qemu: command: Drop obsolete comment

Peter Krempa pkrempa at redhat.com
Wed Apr 6 16:02:23 UTC 2016


The change that made qemu not add the memballoon by default happened
prior to 0.12.0. Additionaly the comment was misleading due to the code
that was added below. Since we always need to add a balloon on the
commandline drop the comment.
---
 src/qemu/qemu_command.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index bcd2408..5843516 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -3453,12 +3453,7 @@ qemuBuildMemballoonCommandLine(virCommandPtr cmd,
                                virQEMUCapsPtr qemuCaps)
 {
     char *optstr;
-    /* QEMU changed its default behavior to not include the virtio balloon
-     * device.  Explicitly request it to ensure it will be present.
-     *
-     * NB: Earlier we declared that VirtIO balloon will always be in
-     * slot 0x3 on bus 0x0
-     */
+
     if (STREQLEN(def->os.machine, "s390-virtio", 10) &&
         virQEMUCapsGet(qemuCaps, QEMU_CAPS_VIRTIO_S390) && def->memballoon)
         def->memballoon->model = VIR_DOMAIN_MEMBALLOON_MODEL_NONE;
-- 
2.8.0




More information about the libvir-list mailing list