[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/4] qemu-iotests: disable VNC server for tes
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/4] qemu-iotests: disable VNC server for test 120 |
Date: |
Wed, 18 Nov 2015 17:56:46 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 04.11.2015 03:26, Bo Tu wrote:
> Ever since qemu-iotest 120 was introduced, its expected output didn't
> include the output from the built-in VNC server:
>
> QA output created by 120
> Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864
> QMP_VERSION
> +VNC server running on `::1:5900'
> {"return": {}}
> wrote 65536/65536 bytes at offset 0
> 64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>
> Since some architectures do not even have a graphical console, we just
> pass -nographic to avoid the output.
>
> Fixes: a68197ff5b11 ("iotests: Add tests for overriding
> BDRV_O_PROTOCOL")
>
> Reviewed-by: Sascha Silbe <address@hidden>
> Signed-off-by: Bo Tu <address@hidden>
> ---
> tests/qemu-iotests/120 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/120 b/tests/qemu-iotests/120
> index 9f13078..fb69efd 100755
> --- a/tests/qemu-iotests/120
> +++ b/tests/qemu-iotests/120
> @@ -49,7 +49,7 @@ echo "{'execute': 'qmp_capabilities'}
> {'execute': 'human-monitor-command',
> 'arguments': {'command-line': 'qemu-io drv \"write -P 42 0 64k\"'}}
> {'execute': 'quit'}" \
> - | $QEMU -qmp stdio -nodefaults \
> + | $QEMU -qmp stdio -nodefaults -nographic \
> -drive
> id=drv,if=none,file="$TEST_IMG",driver=raw,file.driver=$IMGFMT \
> | _filter_qmp | _filter_qemu_io
> $QEMU_IO -c 'read -P 42 0 64k' "$TEST_IMG" | _filter_qemu_io
Looks good, but I think we need the same for 119.
Max
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 0/4] Update tests/qemu-iotests failing cases for the s390 platform, Bo Tu, 2015/11/03
- [Qemu-devel] [PATCH v2 1/4] qemu-iotests: refine common.config, Bo Tu, 2015/11/03
- [Qemu-devel] [PATCH v2 4/4] qemu-iotests: disable VNC server for test 120, Bo Tu, 2015/11/03
- Re: [Qemu-devel] [PATCH v2 4/4] qemu-iotests: disable VNC server for test 120,
Max Reitz <=
- [Qemu-devel] [PATCH v2 3/4] qemu-iotests: s390x: fix test 068, Bo Tu, 2015/11/03
- [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, Bo Tu, 2015/11/03
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, Max Reitz, 2015/11/18
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, tu bo, 2015/11/19
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, Max Reitz, 2015/11/20
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, tu bo, 2015/11/22
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, Sascha Silbe, 2015/11/24
- Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051, Max Reitz, 2015/11/25