[libvirt] [PATCH 1/3] qemu: Clarify error message in qemuMigrationSrcIsSafe

Jiri Denemark jdenemar at redhat.com
Tue Aug 13 16:08:36 UTC 2019


The original message was logically incorrect: cache != none or cache !=
directsync is always true. But even replacing "or" with "and" doesn't
make it more readable for humans.

Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
---
 src/qemu/qemu_migration.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index c2570f7303..2bd1a464bf 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -1266,7 +1266,7 @@ qemuMigrationSrcIsSafe(virDomainDefPtr def,
 
         virReportError(VIR_ERR_MIGRATE_UNSAFE, "%s",
                        _("Migration may lead to data corruption if disks"
-                         " use cache != none or cache != directsync"));
+                         " use cache other than none or directsync"));
         return false;
     }
 
-- 
2.22.1




More information about the libvir-list mailing list