|
 |
383d26 |
From e5d7e4bd2388cebecc64469ea957676caba9bbe1 Mon Sep 17 00:00:00 2001
|
|
|
383d26 |
From: John Snow <jsnow@redhat.com>
|
|
|
383d26 |
Date: Wed, 27 Mar 2019 17:23:07 +0100
|
|
|
383d26 |
Subject: [PATCH 129/163] iotests: Wait for qemu to end in 223
|
|
|
383d26 |
|
|
|
383d26 |
RH-Author: John Snow <jsnow@redhat.com>
|
|
|
383d26 |
Message-id: <20190327172308.31077-55-jsnow@redhat.com>
|
|
|
383d26 |
Patchwork-id: 85219
|
|
|
383d26 |
O-Subject: [RHEL-7.7 qemu-kvm-rhev PATCH 54/55] iotests: Wait for qemu to end in 223
|
|
|
383d26 |
Bugzilla: 1691009
|
|
|
383d26 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
383d26 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
383d26 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
383d26 |
|
|
|
383d26 |
From: Eric Blake <eblake@redhat.com>
|
|
|
383d26 |
|
|
|
383d26 |
When iotest 223 was first written, it didn't matter if we waited for
|
|
|
383d26 |
the qemu process to clean up. But with the introduction of a later
|
|
|
383d26 |
qemu-nbd process trying to reuse the same file, there is a race where
|
|
|
383d26 |
even though the asynchronous qemu process has responded to "quit", it
|
|
|
383d26 |
has not yet had time to unlock the file and exit, resulting in:
|
|
|
383d26 |
|
|
|
383d26 |
-[{ "start": 0, "length": 65536, "depth": 0, "zero": false, "data": false},
|
|
|
383d26 |
-{ "start": 65536, "length": 2031616, "depth": 0, "zero": false, "data": true},
|
|
|
383d26 |
-{ "start": 2097152, "length": 2097152, "depth": 0, "zero": false, "data": false}]
|
|
|
383d26 |
+qemu-nbd: Failed to blk_new_open 'tests/qemu-iotests/scratch/t.qcow2': Failed to get shared "write" lock
|
|
|
383d26 |
+Is another process using the image [tests/qemu-iotests/scratch/t.qcow2]?
|
|
|
383d26 |
+qemu-img: Could not open 'driver=nbd,server.type=unix,server.path=tests/qemu-iotests/scratch/qemu-nbd.sock,x-dirty-bitmap=qemu:dirty-bitmap:b': Failed to connect socket tests/qemu-iotests/scratch/qemu-nbd.sock: Connection refused
|
|
|
383d26 |
+./common.nbd: line 33: kill: (11122) - No such process
|
|
|
383d26 |
|
|
|
383d26 |
Fixes: ddd09448
|
|
|
383d26 |
Reported-by: Alberto Garcia <berto@igalia.com>
|
|
|
383d26 |
Signed-off-by: Eric Blake <eblake@redhat.com>
|
|
|
383d26 |
Message-Id: <20190305182908.13557-1-eblake@redhat.com>
|
|
|
383d26 |
Tested-by: Alberto Garcia <berto@igalia.com>
|
|
|
383d26 |
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
383d26 |
(cherry picked from commit 054be3605459d4342e9ee5a82ae0fcffeeb09e4d)
|
|
|
383d26 |
Signed-off-by: John Snow <jsnow@redhat.com>
|
|
|
383d26 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
383d26 |
---
|
|
|
383d26 |
tests/qemu-iotests/223 | 1 +
|
|
|
383d26 |
tests/qemu-iotests/223.out | 1 +
|
|
|
383d26 |
2 files changed, 2 insertions(+)
|
|
|
383d26 |
|
|
|
383d26 |
diff --git a/tests/qemu-iotests/223 b/tests/qemu-iotests/223
|
|
|
383d26 |
index f120a01..c0a4f9c 100755
|
|
|
383d26 |
--- a/tests/qemu-iotests/223
|
|
|
383d26 |
+++ b/tests/qemu-iotests/223
|
|
|
383d26 |
@@ -179,6 +179,7 @@ _send_qemu_cmd $QEMU_HANDLE '{"execute":"nbd-server-remove",
|
|
|
383d26 |
_send_qemu_cmd $QEMU_HANDLE '{"execute":"nbd-server-stop"}' "return"
|
|
|
383d26 |
_send_qemu_cmd $QEMU_HANDLE '{"execute":"nbd-server-stop"}' "error" # Again
|
|
|
383d26 |
_send_qemu_cmd $QEMU_HANDLE '{"execute":"quit"}' "return"
|
|
|
383d26 |
+wait=yes _cleanup_qemu
|
|
|
383d26 |
|
|
|
383d26 |
echo
|
|
|
383d26 |
echo "=== Use qemu-nbd as server ==="
|
|
|
383d26 |
diff --git a/tests/qemu-iotests/223.out b/tests/qemu-iotests/223.out
|
|
|
383d26 |
index 963ae28..0524ffb 100644
|
|
|
383d26 |
--- a/tests/qemu-iotests/223.out
|
|
|
383d26 |
+++ b/tests/qemu-iotests/223.out
|
|
|
383d26 |
@@ -89,6 +89,7 @@ read 2097152/2097152 bytes at offset 2097152
|
|
|
383d26 |
{"return": {}}
|
|
|
383d26 |
{"error": {"class": "GenericError", "desc": "NBD server not running"}}
|
|
|
383d26 |
{"return": {}}
|
|
|
383d26 |
+{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "SHUTDOWN", "data": {"guest": false}}
|
|
|
383d26 |
|
|
|
383d26 |
=== Use qemu-nbd as server ===
|
|
|
383d26 |
|
|
|
383d26 |
--
|
|
|
383d26 |
1.8.3.1
|
|
|
383d26 |
|