From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42837) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YJk7k-0007ct-MD for qemu-devel@nongnu.org; Fri, 06 Feb 2015 09:43:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YJk7f-0007IN-Iq for qemu-devel@nongnu.org; Fri, 06 Feb 2015 09:43:12 -0500 Received: from mx2.parallels.com ([199.115.105.18]:58924) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YJk7f-00076L-BD for qemu-devel@nongnu.org; Fri, 06 Feb 2015 09:43:07 -0500 Message-ID: <54D4CE6D.1000609@parallels.com> Date: Fri, 6 Feb 2015 17:23:41 +0300 From: Vladimir Sementsov-Ogievskiy MIME-Version: 1.0 References: <1421080265-2228-1-git-send-email-jsnow@redhat.com> <1421080265-2228-13-git-send-email-jsnow@redhat.com> In-Reply-To: <1421080265-2228-13-git-send-email-jsnow@redhat.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v11 12/13] qemu-iotests: Add tests for drive-backup sync=dirty-bitmap List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow , qemu-devel@nongnu.org Cc: kwolf@redhat.com, famz@redhat.com, armbru@redhat.com, mreitz@redhat.com, stefanha@redhat.com On 12.01.2015 19:31, John Snow wrote: > From: Fam Zheng > > Signed-off-by: Fam Zheng > Signed-off-by: John Snow > --- > tests/qemu-iotests/056 | 33 ++++++++++++++++++++++++++++++--- > tests/qemu-iotests/056.out | 4 ++-- > tests/qemu-iotests/iotests.py | 8 ++++++++ > 3 files changed, 40 insertions(+), 5 deletions(-) > > diff --git a/tests/qemu-iotests/056 b/tests/qemu-iotests/056 > index 54e4bd0..6f8aa9a 100755 > --- a/tests/qemu-iotests/056 > +++ b/tests/qemu-iotests/056 > @@ -23,17 +23,17 @@ > import time > import os > import iotests > -from iotests import qemu_img, qemu_io, create_image > +from iotests import qemu_img, qemu_img_map_assert, qemu_io, create_image > > backing_img = os.path.join(iotests.test_dir, 'backing.img') > test_img = os.path.join(iotests.test_dir, 'test.img') > target_img = os.path.join(iotests.test_dir, 'target.img') > > -class TestSyncModesNoneAndTop(iotests.QMPTestCase): > +class TestSyncModes(iotests.QMPTestCase): > image_len = 64 * 1024 * 1024 # MB > > def setUp(self): > - create_image(backing_img, TestSyncModesNoneAndTop.image_len) > + create_image(backing_img, TestSyncModes.image_len) > qemu_img('create', '-f', iotests.imgfmt, '-o', 'backing_file=%s' % backing_img, test_img) > qemu_io('-c', 'write -P0x41 0 512', test_img) > qemu_io('-c', 'write -P0xd5 1M 32k', test_img) > @@ -64,6 +64,33 @@ class TestSyncModesNoneAndTop(iotests.QMPTestCase): > self.assertTrue(iotests.compare_images(test_img, target_img), > 'target image does not match source after backup') > > + def test_sync_dirty_bitmap_missing(self): > + self.assert_no_active_block_jobs() > + result = self.vm.qmp('drive-backup', device='drive0', sync='dirty-bitmap', > + format=iotests.imgfmt, target=target_img) > + self.assert_qmp(result, 'error/class', 'GenericError') > + > + def test_sync_dirty_bitmap_not_found(self): > + self.assert_no_active_block_jobs() > + result = self.vm.qmp('drive-backup', device='drive0', sync='dirty-bitmap', > + bitmap='unknown', > + format=iotests.imgfmt, target=target_img) > + self.assert_qmp(result, 'error/class', 'GenericError') > + > + def test_sync_dirty_bitmap(self): > + self.assert_no_active_block_jobs() > + result = self.vm.qmp('block-dirty-bitmap-add', node_ref='drive0', name='bitmap0') > + self.assert_qmp(result, 'return', {}) > + self.vm.hmp_qemu_io('drive0', 'write -P0x5a 0 512') > + self.vm.hmp_qemu_io('drive0', 'write -P0x5a 48M 512') > + result = self.vm.qmp('drive-backup', device='drive0', sync='dirty-bitmap', > + bitmap='bitmap0', > + format=iotests.imgfmt, target=target_img) > + self.assert_qmp(result, 'return', {}) > + self.wait_until_completed(check_offset=False) > + self.assert_no_active_block_jobs() > + qemu_img_map_assert(target_img, [0, 0x3000000]) > + > def test_cancel_sync_none(self): > self.assert_no_active_block_jobs() > > diff --git a/tests/qemu-iotests/056.out b/tests/qemu-iotests/056.out > index fbc63e6..914e373 100644 > --- a/tests/qemu-iotests/056.out > +++ b/tests/qemu-iotests/056.out > @@ -1,5 +1,5 @@ > -.. > +..... > ---------------------------------------------------------------------- > -Ran 2 tests > +Ran 5 tests > > OK > diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py > index f57f154..95bb959 100644 > --- a/tests/qemu-iotests/iotests.py > +++ b/tests/qemu-iotests/iotests.py > @@ -55,6 +55,14 @@ def qemu_img_pipe(*args): > '''Run qemu-img and return its output''' > return subprocess.Popen(qemu_img_args + list(args), stdout=subprocess.PIPE).communicate()[0] > > +def qemu_img_map_assert(img, offsets): > + '''Run qemu-img map on img and check the mapped ranges''' > + offs = [] > + for line in qemu_img_pipe('map', img).splitlines()[1:]: > + offset, length, mapped, fname = line.split() > + offs.append(int(offset, 16)) > + assert set(offs) == set(offsets), "mapped offsets in image '%s' not equal to '%s'" % (str(offs), str(offsets)) > + > def qemu_io(*args): > '''Run qemu-io and return the stdout data''' > args = qemu_io_args + list(args) why not just check by qemu-io -c "write -P0x5a 0 512" for symmetry? -- Best regards, Vladimir