[libvirt] [PATCH 1/4] Revert "qemu: emit error when trying to update blkiotune group_name in qemuDomainChangeDiskLive"

Ján Tomko posted 4 patches 6 years, 10 months ago
[libvirt] [PATCH 1/4] Revert "qemu: emit error when trying to update blkiotune group_name in qemuDomainChangeDiskLive"
Posted by Ján Tomko 6 years, 10 months ago
https://bugzilla.redhat.com/show_bug.cgi?id=1601677

This reverts commit 047cfb05ee949325e77726531fd381820be8dc62
Using numeric comparison on strings means we reject every update
that does include the group name, even if it's unchanged.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
---
 src/qemu/qemu_domain.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 9ca4ca33e5..bb3a672d47 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -9386,9 +9386,6 @@ qemuDomainDiskChangeSupported(virDomainDiskDefPtr disk,
     CHECK_EQ(blkdeviotune.size_iops_sec,
              "blkdeviotune size_iops_sec",
              true);
-    CHECK_EQ(blkdeviotune.group_name,
-             "blkdeviotune group_name",
-             true);
 
     if (disk->serial && STRNEQ_NULLABLE(disk->serial, orig_disk->serial)) {
         virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 1/4] Revert "qemu: emit error when trying to update blkiotune group_name in qemuDomainChangeDiskLive"
Posted by Laine Stump 6 years, 10 months ago
On 3/28/19 10:34 AM, Ján Tomko wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1601677
>
> This reverts commit 047cfb05ee949325e77726531fd381820be8dc62
> Using numeric comparison on strings means we reject every update
> that does include the group name, even if it's unchanged.


Oopsy!


>
> Signed-off-by: Ján Tomko <jtomko@redhat.com>


Reviewed-by: Laine Stump <laine@laine.org>


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list