[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 08/11] qemu-iotests: add no-op streaming test
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [PATCH 08/11] qemu-iotests: add no-op streaming test |
Date: |
Fri, 24 Apr 2015 15:55:00 +0200 |
User-agent: |
Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu) |
On Fri 24 Apr 2015 03:19:54 PM CEST, Max Reitz wrote:
>> This patch updates test_stream_partial() to test that the
>> block-stream operation never copies data from the image specified as
>> base.
>
> I think it would be better to add this as an own test case. But I
> won't oppose adding it as a special case of test_stream_partial
> either, because strictly speaking, streaming nothing is a special case
> of streaming partially.
That was the idea, but of course it can be moved to a new test, it's
also fine with me.
>> + self.assertNotEqual(qemu_io('-f', iotests.imgfmt, '-c', 'map',
>> mid_img),
>> + qemu_io('-f', iotests.imgfmt, '-c', 'map',
>> test_img),
>> + 'image file map matches backing file after a
>> no-op')
>
> Well, you haven't really proven anything by doing this, except that
> the mid_img hasn't been streamed completely into test_img... What you
> want to prove is that test_img is empty after streaming, right?
That's right, but what scenario would that be? A streaming operation
that is completed without errors and leaves partial data on the
destination image?
There's only 512 bytes written in mid_img for this test case. If we
suspect that it can go wrong maybe we should start comparing more things
than just the qemu_io map output...
Berto
- [Qemu-block] [PATCH v5 00/11] Support streaming to an intermediate layer, Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 05/11] block: Add QMP support for streaming to an intermediate layer, Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 07/11] qemu-iotests: fix test_stream_partial(), Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 06/11] docs: Document how to stream to an intermediate layer, Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 10/11] qemu-iotests: test block-stream operations in parallel, Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 09/11] qemu-iotests: test streaming to an intermediate layer, Alberto Garcia, 2015/04/24
- [Qemu-block] [PATCH 04/11] block: Support streaming to an intermediate layer, Alberto Garcia, 2015/04/24