[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/11] qemu-iotests: add no-op streaming test
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 08/11] qemu-iotests: add no-op streaming test |
Date: |
Fri, 15 May 2015 10:44:34 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, 05/13 16:27, Alberto Garcia wrote:
> This patch tests that in a partial block-stream operation, no data is
> ever copied from the base image.
>
> Signed-off-by: Alberto Garcia <address@hidden>
> Reviewed-by: Max Reitz <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
> ---
> tests/qemu-iotests/030 | 18 ++++++++++++++++++
> tests/qemu-iotests/030.out | 4 ++--
> 2 files changed, 20 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030
> index 6e6cb5a..bc53885 100755
> --- a/tests/qemu-iotests/030
> +++ b/tests/qemu-iotests/030
> @@ -91,6 +91,24 @@ class TestSingleDrive(iotests.QMPTestCase):
> qemu_io('-f', iotests.imgfmt, '-c', 'map',
> test_img),
> 'image file map does not match backing file after
> streaming')
>
> + def test_stream_no_op(self):
> + self.assert_no_active_block_jobs()
> +
> + # The image map is empty before the operation
> + empty_map = qemu_io('-f', iotests.imgfmt, '-c', 'map', test_img)
> +
> + # This is a no-op: no data should ever be copied from the base image
> + result = self.vm.qmp('block-stream', device='drive0', base=mid_img)
> + self.assert_qmp(result, 'return', {})
> +
> + self.wait_until_completed()
> +
> + self.assert_no_active_block_jobs()
> + self.vm.shutdown()
> +
> + self.assertEqual(qemu_io('-f', iotests.imgfmt, '-c', 'map',
> test_img),
> + empty_map, 'image file map changed after a no-op')
> +
> def test_stream_partial(self):
> self.assert_no_active_block_jobs()
>
> diff --git a/tests/qemu-iotests/030.out b/tests/qemu-iotests/030.out
> index fa16b5c..6323079 100644
> --- a/tests/qemu-iotests/030.out
> +++ b/tests/qemu-iotests/030.out
> @@ -1,5 +1,5 @@
> -.............
> +..............
> ----------------------------------------------------------------------
> -Ran 13 tests
> +Ran 14 tests
>
> OK
> --
> 2.1.4
>
>
- [Qemu-devel] [PATCH v7 00/11] Support streaming to an intermediate layer, Alberto Garcia, 2015/05/13
- [Qemu-devel] [PATCH 09/11] qemu-iotests: test streaming to an intermediate layer, Alberto Garcia, 2015/05/13
- [Qemu-devel] [PATCH 08/11] qemu-iotests: add no-op streaming test, Alberto Garcia, 2015/05/13
- Re: [Qemu-devel] [PATCH 08/11] qemu-iotests: add no-op streaming test,
Fam Zheng <=
- [Qemu-devel] [PATCH 01/11] block: keep a list of block jobs, Alberto Garcia, 2015/05/13
- [Qemu-devel] [PATCH 05/11] block: Add QMP support for streaming to an intermediate layer, Alberto Garcia, 2015/05/13
- [Qemu-devel] [PATCH 07/11] qemu-iotests: fix test_stream_partial(), Alberto Garcia, 2015/05/13
- [Qemu-devel] [PATCH 04/11] block: Support streaming to an intermediate layer, Alberto Garcia, 2015/05/13