From 9a18e0069ba06ee3adebb7b0232d9bc016459f7d Mon Sep 17 00:00:00 2001 From: Uri Lublin Date: Mon, 8 Jun 2009 19:27:21 +0300 Subject: [PATCH] exec-migration: handle EINTR in popen_get_buffer() Sometimes, upon interrupt, fread returns with no data, and the (incoming exec) migration fails. Fix by retrying on such a case. Signed-off-by: Uri Lublin Signed-off-by: Anthony Liguori --- savevm.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/savevm.c b/savevm.c index 61edd7e..17da35a 100644 --- a/savevm.c +++ b/savevm.c @@ -215,7 +215,14 @@ static int popen_put_buffer(void *opaque, const uint8_t *buf, int64_t pos, int s static int popen_get_buffer(void *opaque, uint8_t *buf, int64_t pos, int size) { QEMUFilePopen *s = opaque; - return fread(buf, 1, size, s->popen_file); + FILE *fp = s->popen_file; + int bytes; + + do { + clearerr(fp); + bytes = fread(buf, 1, size, fp); + } while ((bytes == 0) && ferror(fp) && (errno == EINTR)); + return bytes; } static int popen_close(void *opaque) -- 2.7.4