No need to log empty results, qmp_qemu_io() already checks for success.
Signed-off-by: Marc-André Lureau <[email protected]>
---
tests/qemu-iotests/236 | 7 +++----
tests/qemu-iotests/236.out | 8 --------
2 files changed, 3 insertions(+), 12 deletions(-)
diff --git a/tests/qemu-iotests/236 b/tests/qemu-iotests/236
index 3cf0de4043f6..0d2c1780df71 100755
--- a/tests/qemu-iotests/236
+++ b/tests/qemu-iotests/236
@@ -23,8 +23,7 @@
import iotests
from iotests import log
-iotests.script_initialize(supported_fmts=['generic'],
- require_hmp=True)
+iotests.script_initialize(supported_fmts=['generic'])
size = 64 * 1024 * 1024
granularity = 64 * 1024
@@ -62,7 +61,7 @@ with iotests.FilePath('img') as img_path, \
for p in patterns:
cmd = "write -P%s %s %s" % p
log(cmd)
- log(vm.hmp_qemu_io("drive0", cmd))
+ vm.qmp_qemu_io("drive0", cmd)
log(query_bitmaps(vm), indent=2)
@@ -100,7 +99,7 @@ with iotests.FilePath('img') as img_path, \
for p in overwrite:
cmd = "write -P%s %s %s" % p
log(cmd)
- log(vm.hmp_qemu_io("drive0", cmd))
+ vm.qmp_qemu_io("drive0", cmd)
log('\n--- Disabling A & C ---\n')
vm.qmp_log("transaction", indent=2, actions=[
diff --git a/tests/qemu-iotests/236.out b/tests/qemu-iotests/236.out
index 7448ceea027e..aedac37931d7 100644
--- a/tests/qemu-iotests/236.out
+++ b/tests/qemu-iotests/236.out
@@ -11,13 +11,9 @@
--- Emulating writes ---
write -P0x5d 0 64k
-{"return": ""}
write -P0xd5 1M 64k
-{"return": ""}
write -P0xdc 32M 64k
-{"return": ""}
write -P0xcd 0x3ff0000 64k
-{"return": ""}
{
"bitmaps": {
"drive0": [
@@ -150,13 +146,9 @@ write -P0xcd 0x3ff0000 64k
--- Emulating further writes ---
write -P0xab 0 64k
-{"return": ""}
write -P0xad 0x00f8000 64k
-{"return": ""}
write -P0x1d 0x2008000 64k
-{"return": ""}
write -P0xea 0x3fe0000 64k
-{"return": ""}
--- Disabling A & C ---
--
2.56.0.rc0.29.g47ce80527c56
