diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2017-09-22 14:50:12 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2017-09-26 14:46:23 +0200 |
| commit | f3adefb2cea1c63b7b198acaef5e40eb4b2d2d39 (patch) | |
| tree | 38b95ac286ddedae55d3b540c0a52db6c1b6984f /tests | |
| parent | 7a6ab45e19b615b9285b9cfa2bbc1fee012bc8d7 (diff) | |
| download | qemu-f3adefb2cea1c63b7b198acaef5e40eb4b2d2d39.zip qemu-f3adefb2cea1c63b7b198acaef5e40eb4b2d2d39.tar.gz qemu-f3adefb2cea1c63b7b198acaef5e40eb4b2d2d39.tar.bz2 | |
qemu-io: Drop write permissions before read-only reopen
qemu-io provides a 'reopen' command that allows switching from writable
to read-only access. We need to make sure that we don't try to keep
write permissions to a BlockBackend that becomes read-only, otherwise
things are going to fail.
This requires a bdrv_drain() call because otherwise in-flight AIO
write requests could issue new internal requests while the permission
has already gone away, which would cause assertion failures. Draining
the queue doesn't break AIO requests in any new way, bdrv_reopen() would
drain it anyway only a few lines later.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/qemu-iotests/187.out | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/qemu-iotests/187.out b/tests/qemu-iotests/187.out index 68fb944..30b987f7 100644 --- a/tests/qemu-iotests/187.out +++ b/tests/qemu-iotests/187.out @@ -12,7 +12,7 @@ Start from read-write wrote 65536/65536 bytes at offset 0 64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) -write failed: Operation not permitted +Block node is read-only wrote 65536/65536 bytes at offset 0 64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) *** done |