[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v14 07/20] iotests: 030: Prepare for image locki
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v14 07/20] iotests: 030: Prepare for image locking |
Date: |
Mon, 24 Apr 2017 14:15:40 +0800 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Fri, 04/21 15:51, Kevin Wolf wrote:
> Am 21.04.2017 um 05:55 hat Fam Zheng geschrieben:
> > qemu-img and qemu-io commands when guest is running need "-U" option,
> > add it.
> >
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > tests/qemu-iotests/030 | 24 ++++++++++++------------
> > 1 file changed, 12 insertions(+), 12 deletions(-)
> >
> > diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030
> > index 0d472d5..5f1dce8 100755
> > --- a/tests/qemu-iotests/030
> > +++ b/tests/qemu-iotests/030
> > @@ -63,8 +63,8 @@ class TestSingleDrive(iotests.QMPTestCase):
> > def test_stream_intermediate(self):
> > self.assert_no_active_block_jobs()
> >
> > - self.assertNotEqual(qemu_io('-f', 'raw', '-c', 'map', backing_img),
> > - qemu_io('-f', iotests.imgfmt, '-c', 'map',
> > mid_img),
> > + self.assertNotEqual(qemu_io('-f', 'raw', '-rU', '-c', 'map',
> > backing_img),
> > + qemu_io('-f', iotests.imgfmt, '-rU', '-c',
> > 'map', mid_img),
> > 'image file map matches backing file before
> > streaming')
> >
> > result = self.vm.qmp('block-stream', device='mid',
> > job_id='stream-mid')
> > @@ -114,7 +114,7 @@ class TestSingleDrive(iotests.QMPTestCase):
> > 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)
> > + empty_map = qemu_io('-f', iotests.imgfmt, '-rU', '-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)
> > @@ -125,7 +125,7 @@ class TestSingleDrive(iotests.QMPTestCase):
> > self.assert_no_active_block_jobs()
> > self.vm.shutdown()
> >
> > - self.assertEqual(qemu_io('-f', iotests.imgfmt, '-c', 'map',
> > test_img),
> > + self.assertEqual(qemu_io('-f', iotests.imgfmt, '-c', 'map', '-r',
> > test_img),
> > empty_map, 'image file map changed after a no-op')
>
> This one doesn't seem strictly necessary, as you're only adding -r
> without -U. I still think it's a good idea to use -r where we can, but
> if we decide to do this, there are more places in this test that could
> be updated.
>
> Maybe a separate patch for adding -r without -U to the cases where
> qemu-io is run after shutting down the VM?
Yes, will split. (In a previous version -U is implied by -r, and the patch was
"add -r to all qemu-io invocations where writing is not needed.)
Fam
- Re: [Qemu-devel] [PATCH v14 05/20] qemu-img: Update documentation for --share-rw, (continued)
- [Qemu-devel] [PATCH v14 04/20] qemu-img: Add --share-rw option to subcommands, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 06/20] qemu-io: Add --share-rw option, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 07/20] iotests: 030: Prepare for image locking, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 08/20] iotests: 046: Prepare for image locking, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 09/20] iotests: 055: Don't attach the target image already for drive-backup, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 10/20] iotests: 085: Avoid image locking conflict, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 11/20] iotests: 087: Don't attach test image twice, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 12/20] iotests: 091: Quit QEMU before checking image, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 14/20] tests: Use null-co:// instead of /dev/null as the dummy image, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 13/20] iotests: 172: Use separate images for multiple devices, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 15/20] file-posix: Add 'locking' option, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 16/20] tests: Disable image lock in test-replication, Fam Zheng, 2017/04/20
- [Qemu-devel] [PATCH v14 17/20] block: Reuse bs as backing hd for drive-backup sync=none, Fam Zheng, 2017/04/20