From: Juan Quintela <quintela@redhat.com>
Date: Wed, 23 Feb 2011 19:17:45 +0000 (+0100)
Subject: migration: If there is one error, it makes no sense to continue
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=4fc7d8195f9c8433f8661b6156e5c57ff8d8e4fb;p=qemu.git

migration: If there is one error, it makes no sense to continue

Once there, add a comment about what each error mean.

Signed-off-by: Juan Quintela <quintela@redhat.com>
---

diff --git a/buffered_file.c b/buffered_file.c
index 486af575d5..94ecbbcea1 100644
--- a/buffered_file.c
+++ b/buffered_file.c
@@ -189,13 +189,19 @@ static int buffered_close(void *opaque)
     return ret;
 }
 
+/*
+ * The meaning of the return values is:
+ *   0: We can continue sending
+ *   1: Time to stop
+ *  -1: There has been an error
+ */
 static int buffered_rate_limit(void *opaque)
 {
     QEMUFileBuffered *s = opaque;
 
-    if (s->has_error)
-        return 0;
-
+    if (s->has_error) {
+        return -1;
+    }
     if (s->freeze_output)
         return 1;