diff options
author | Fam Zheng <famz@redhat.com> | 2017-12-26 14:53:00 +0800 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2018-01-23 12:33:07 +0100 |
commit | f86428a1f4f91a460ed585682af70d3e8c31dc06 (patch) | |
tree | 1f5bfcf7a8e3e35ebeef547a845ddb0ca59d5005 | |
parent | ee86981bda9ecd40c8daf81b7307b1d2aff68174 (diff) | |
download | qemu-f86428a1f4f91a460ed585682af70d3e8c31dc06.zip qemu-f86428a1f4f91a460ed585682af70d3e8c31dc06.tar.gz qemu-f86428a1f4f91a460ed585682af70d3e8c31dc06.tar.bz2 |
osdep: Retry SETLK upon EINTR
We could hit lock failure if there is a signal that makes fcntl return
-1 and errno set to EINTR. In this case we should retry.
Cc: qemu-stable@nongnu.org
Signed-off-by: Fam Zheng <famz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
-rw-r--r-- | util/osdep.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/util/osdep.c b/util/osdep.c index 1231f9f..a73de0e 100644 --- a/util/osdep.c +++ b/util/osdep.c @@ -244,7 +244,9 @@ static int qemu_lock_fcntl(int fd, int64_t start, int64_t len, int fl_type) .l_type = fl_type, }; qemu_probe_lock_ops(); - ret = fcntl(fd, fcntl_op_setlk, &fl); + do { + ret = fcntl(fd, fcntl_op_setlk, &fl); + } while (ret == -1 && errno == EINTR); return ret == -1 ? -errno : 0; } |