From bd88071d945585ba8d1d03ab2a23b0bd37e79c5f Mon Sep 17 00:00:00 2001 From: Shriram Rajagopalan Date: Mon, 25 Apr 2011 13:29:18 +0100 Subject: [PATCH] remus: fix incorrect error handling for switch_qemu_logdirty in checkpoint code c/s 22275: "tools: cleanup domain save switch_qemu_logdirty callback" introduced a whole bunch of error code fixups. In the process, it also ended up treating the success return code (0) from switch_qemu_logdirty as an error and vice versa. Signed-off-by: Shriram Rajagopalan xen-unstable changeset: 23202:67911039ac94 xen-unstable date: Tue Apr 12 13:28:51 2011 +0100 --- tools/python/xen/lowlevel/checkpoint/libcheckpoint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/python/xen/lowlevel/checkpoint/libcheckpoint.c b/tools/python/xen/lowlevel/checkpoint/libcheckpoint.c index f4d3e7fd1a..e96dc8f067 100644 --- a/tools/python/xen/lowlevel/checkpoint/libcheckpoint.c +++ b/tools/python/xen/lowlevel/checkpoint/libcheckpoint.c @@ -185,7 +185,7 @@ int checkpoint_start(checkpoint_state* s, int fd, hvm = s->domtype > dt_pv; if (hvm) { flags |= XCFLAGS_HVM; - if (!switch_qemu_logdirty(s, 1)) + if (switch_qemu_logdirty(s, 1)) return -1; }