On Wed, Aug 4, 2021 at 5:39 AM Vladimir Sementsov-Ogievskiy <
vsement...@virtuozzo.com> wrote:
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
> Reviewed-by: Max Reitz <mre...@redhat.com>
> ---
> tests/qemu-iotests/iotests.py | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
> index 025e288ddd..9d0031a0e8 100644
> --- a/tests/qemu-iotests/iotests.py
> +++ b/tests/qemu-iotests/iotests.py
> @@ -655,9 +655,10 @@ def resume_drive(self, drive: str) -> None:
> self.hmp(f'qemu-io {drive} "remove_break bp_{drive}"')
>
> def hmp_qemu_io(self, drive: str, cmd: str,
> - use_log: bool = False) -> QMPMessage:
> + use_log: bool = False, qdev: bool = False) ->
> QMPMessage:
> """Write to a given drive using an HMP command"""
> - return self.hmp(f'qemu-io {drive} "{cmd}"', use_log=use_log)
> + d = '-d ' if qdev else ''
> + return self.hmp(f'qemu-io {d}{drive} "{cmd}"', use_log=use_log)
>
> def flatten_qmp_object(self, obj, output=None, basestr=''):
> if output is None:
> --
> 2.29.2
>
>
Guess that's really the only flag that this HMP command supports. I was
gonna suggest abstracting to {args} ... but, uh, that's the only one, so...
sure!
Reviewed-by: John Snow <js...@redhat.com>
