Am 24.06.2014 um 06:25 hat Fam Zheng geschrieben:
> On Mon, 06/09 10:56, Fam Zheng wrote:
> > The current behavior of mirroring zero byte image is slightly different from
> > non-zero image: the BLOCK_JOB_READY event is skipped and job is completed
> > immediately. This is not a big problem for human user but only makes
> > management
> > software harder to write. Since we are focusing on an good API instead of
> > UI,
> > let's make the behavior more consistent and predictable.
> >
> > The first patch fixes the behavior. The following two patches add test case
> > for
> > the involved code path.
> >
> > Thanks for Eric Blake to report this!
>
> Ping. Shall we merge this before it rots?
I had it already applied, but now test case 041 fails for me, so I'm
going to remove it from my tree again:
041 7s ... [13:28:48] [13:29:27] [failed, exit status 1] - output
mismatch (see 041.out.bad)
--- 041.out 2014-06-24 13:23:13.597949977 +0200
+++ 041.out.bad 2014-06-24 13:29:27.024871226 +0200
@@ -1,5 +1,31 @@
-.....................................
+qemu-img: /home/kwolf/source/qemu/tests/qemu-iotests/scratch/target.img:
Cluster size must be a power of two between 512 and 2048k
+qemu-img: /home/kwolf/source/qemu/tests/qemu-iotests/scratch/target.img:
Cluster size must be a power of two between 512 and 2048k
+..........................F...F......
+======================================================================
+FAIL: test_large_cluster (__main__.TestSingleDriveZeroLength)
+----------------------------------------------------------------------
+Traceback (most recent call last):
+ File "./041", line 185, in test_large_cluster
+ self.assert_qmp(result, 'return', {})
+ File "/home/kwolf/source/qemu/tests/qemu-iotests/iotests.py", line 232, in
assert_qmp
+ result = self.dictpath(d, path)
+ File "/home/kwolf/source/qemu/tests/qemu-iotests/iotests.py", line 211, in
dictpath
+ self.fail('failed path traversal for "%s" in "%s"' % (path, str(d)))
+AssertionError: failed path traversal for "return" in "{u'error': {u'class':
u'GenericError', u'desc': u"Could not open
'/home/kwolf/source/qemu/tests/qemu-iotests/scratch/target.img': No such file
or directory"}}"
+
+======================================================================
+FAIL: test_small_buffer2 (__main__.TestSingleDriveZeroLength)
+----------------------------------------------------------------------
+Traceback (most recent call last):
+ File "./041", line 169, in test_small_buffer2
+ self.assert_qmp(result, 'return', {})
+ File "/home/kwolf/source/qemu/tests/qemu-iotests/iotests.py", line 232, in
assert_qmp
+ result = self.dictpath(d, path)
+ File "/home/kwolf/source/qemu/tests/qemu-iotests/iotests.py", line 211, in
dictpath
+ self.fail('failed path traversal for "%s" in "%s"' % (path, str(d)))
+AssertionError: failed path traversal for "return" in "{u'error': {u'class':
u'GenericError', u'desc': u"Could not open
'/home/kwolf/source/qemu/tests/qemu-iotests/scratch/target.img': No such file
or directory"}}"
+
----------------------------------------------------------------------
Ran 37 tests
-OK
+FAILED (failures=2)
Failures: 041
