From d1277de226a4f840df0c20a9931a2c009b505d25 Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Wed, 24 Feb 2016 14:40:51 +0100 Subject: [PATCH] qemu: Allow setting pinning of emulator/iohtread with automatic placement We honour the placement bitmaps when starting up, so there's no point in having this check. Additionally the check was buggy since it checked vm->def all the time even if the user requested to modify the persistent definition which had different configuration. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1308317 --- src/qemu/qemu_driver.c | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 75f447dede..8a529afa3a 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -5240,13 +5240,6 @@ qemuDomainPinEmulator(virDomainPtr dom, if (virDomainPinEmulatorEnsureACL(dom->conn, vm->def, flags) < 0) goto cleanup; - if (vm->def->placement_mode == VIR_DOMAIN_CPU_PLACEMENT_MODE_AUTO) { - virReportError(VIR_ERR_OPERATION_INVALID, "%s", - _("Changing affinity for emulator thread dynamically " - "is not allowed when CPU placement is 'auto'")); - goto cleanup; - } - if (qemuDomainObjBeginJob(driver, vm, QEMU_JOB_MODIFY) < 0) goto cleanup; @@ -5714,13 +5707,6 @@ qemuDomainPinIOThread(virDomainPtr dom, if (virDomainPinIOThreadEnsureACL(dom->conn, vm->def, flags) < 0) goto cleanup; - if (vm->def->placement_mode == VIR_DOMAIN_CPU_PLACEMENT_MODE_AUTO) { - virReportError(VIR_ERR_OPERATION_INVALID, "%s", - _("Changing affinity for IOThread dynamically is " - "not allowed when CPU placement is 'auto'")); - goto cleanup; - } - if (qemuDomainObjBeginJob(driver, vm, QEMU_JOB_MODIFY) < 0) goto cleanup;