From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40977) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WkOSN-00016l-Px for qemu-devel@nongnu.org; Tue, 13 May 2014 21:58:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WkOSE-00008N-9U for qemu-devel@nongnu.org; Tue, 13 May 2014 21:58:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:30350) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WkOSE-00007J-24 for qemu-devel@nongnu.org; Tue, 13 May 2014 21:57:58 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s4E1vvx9016024 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Tue, 13 May 2014 21:57:57 -0400 Date: Wed, 14 May 2014 09:58:07 +0800 From: Fam Zheng Message-ID: <20140514015807.GC1856@T430.nay.redhat.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH 2/2] block: add test for vhdx image created by Disk2VHD List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jeff Cody Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Tue, 05/13 10:00, Jeff Cody wrote: > This adds a test for VHDX images created by Microsoft's tool, Disk2VHD. Out of curiosity, is this an official tool from Microsoft? > > VHDX images created by this tool have 2 identical header sections, with > identical sequence numbers. This makes sure we detect VHDX images with > identical headers, and do not flag them as corrupt. > > Signed-off-by: Jeff Cody > --- > tests/qemu-iotests/070 | 7 +++++++ > tests/qemu-iotests/070.out | 7 +++++++ > tests/qemu-iotests/sample_images/test-disk2vhd.vhdx.bz2 | Bin 0 -> 1424 bytes > 3 files changed, 14 insertions(+) > create mode 100644 tests/qemu-iotests/sample_images/test-disk2vhd.vhdx.bz2 > > diff --git a/tests/qemu-iotests/070 b/tests/qemu-iotests/070 > index ce71fa4..ea0dae7 100755 > --- a/tests/qemu-iotests/070 > +++ b/tests/qemu-iotests/070 > @@ -72,6 +72,13 @@ echo "=== Verify open image read-only succeeds after log replay ===" > $QEMU_IO -r -c "read -pP 0xa5 0 18M" "$TEST_IMG" 2>&1 | _filter_testdir \ > | _filter_qemu_io > > +_cleanup_test_img > +_use_sample_img test-disk2vhd.vhdx.bz2 > + > +echo > +echo "=== Verify image created by Disk2VHD can be opened ===" > +$QEMU_IMG info "$TEST_IMG" 2>&1 | _filter_testdir | _filter_qemu > + _filter_qemu shouldn't make a difference for $QEMU_IMG. Anyway, Reviewed-by: Fam Zheng > # success, all done > echo "*** done" > rm -f $seq.full > diff --git a/tests/qemu-iotests/070.out b/tests/qemu-iotests/070.out > index 922d62c..15f1fc1 100644 > --- a/tests/qemu-iotests/070.out > +++ b/tests/qemu-iotests/070.out > @@ -18,4 +18,11 @@ No errors were found on the image. > === Verify open image read-only succeeds after log replay === > read 18874368/18874368 bytes at offset 0 > 18 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) > + > +=== Verify image created by Disk2VHD can be opened === > +image: TEST_DIR/test-disk2vhd.vhdx > +file format: vhdx > +virtual size: 256M (268435456 bytes) > +disk size: 260M > +cluster_size: 2097152 > *** done > diff --git a/tests/qemu-iotests/sample_images/test-disk2vhd.vhdx.bz2 b/tests/qemu-iotests/sample_images/test-disk2vhd.vhdx.bz2 > new file mode 100644 > index 0000000000000000000000000000000000000000..2891c9a6d62908e0195d3791817284016249aaee > GIT binary patch > literal 1424 > zcmZ>Y%CIzaj8qGbG}k{O$Xb_J|NsC0KmUCH-JDzhyZ-zC_y7OrRo{&NR|gc{RqOEm > z=imPqn0Gw*bmiUDGLEd7O!Jz9a-FYyG%@kz4`91;!B*gMCgZ|o{hrI3-5eMf7&2U( > ze2MLfRMjk#3m2F92b;9b3)~wx*_X}#(!AiWP7J&Z7Z*rLWn7%V=F8TQk#Sieg@GYO > z%H#s$1cuB7Co(4-Ibm{f0b4Ia3Io%!MYdbrr~3cDvda022}1^3FSCOb0|sEu@-&&Z > zd{yQz7v?J$7%p6>a$t6L0IPy1Hi78)%CahBQl z&Sh$|j1q($7cl#p@Y?z`Wbn)~;$8WKbK(Lv_Z21=7;XxsEqh$#HSI@WE6^;a!cbHI > zu?%7hkTh5`=j@LcDLKD{B^^#a3zjsRcFl8cicgpF_s*b{y^=;gXHGaUxHsk6Z1z!6 > zR8jOb5@75|V437mU-yJTMS)R+ zC)tvLS9QX(l#{Jfd<|w?_&irwSR~K$n-GH&gVWNbjzLvv#!ZvhEgL4x zhm(_slhXnI%U%l|_%|#KQ59iedZ^gAy6MTT^pA1#JRe_E5V)X}!2L#gS<#~h?Gtqv > zl&`O9bYKze@DSi|;D~C!8#&cpD`XX`!J3mz{!xVw{r4+rTyzx)bS+3;^k;kfgqh(R > zjB+v<0`Hud8Jn$R?UuFBl*37&r7&VvuZxF5ll`XNKxZk5#5RfKr9KHrSX^b=2 > zQf8bIGb{Zh_kHjC@8`d&KfM}#^|g=#lbQEvqpGz5Ed?!#9L1S`SKYn`R8T$Vx#hW1 > zCM6{$l_f4NE*^KTTb?Ukoc4OXmY1i>B+sy}&s%2&>@2zH!^iM2gHz2Te8X9{cMVoM > z;{=)JSmb5gwe(qeljqKyZ`#t=IU7>$_6lS~Td$G%SNN0X?aCy3waorU$G7pHk*Ma- > zYE<;TxbKbjzRG&1nl|xWH#- z%%mvCGRqI%Rym??0~1n(1@CMZGrbn8sA)Z0k$X~Q0^5#teF5#=oBpZXdBt9_ShVJu > z8-w~y8S_&dArUE&X2}KGr!EJkWlT{K;ttAXNYjmGtKHP1b&A_jYaOdaPOr<6nbTE{ > zuGQZxQ2p%4>&9b+6?O;aW$X+xPMFR3&d_YRW#T@i-OCo2#W_wCnEbBIs?xo=)HQ$R > z>o>}_dZp{atdD$}H#<@;Max;n$qQfE(*sioE9rtPc(Y*#s7 > zpKUfvXG`_DIWbL zx$nd!MiKQrW>eatZPZ^(zWe`GGwaHqI+yFZw>+A3{QKrRzAG9SB77d|ev|%_V)0-` > z-JZ?tMvvIkH2+6tf9dm9+PF?_erw&^`i@ni63vV(0;eV%=E*Bq#jok#6Tz6rbtWUl > zLX<`8`FCKbFRwON)jb?9dDZx@toqN7-~A3R)HZBdvAAbTO@<1S0Fy|{so18Xs4bHB > zS53EGJoP`Mc;n-Ibe}Q4&w=3p0|SQwgF%C@o6`c-m0S(e=_6mW?bJO+Z@mKy2Ux)N > dpW$4UIbn)dbcj%=_t85O{X|r_v(Ec8000NwOCSIM > > literal 0 > HcmV?d00001 > > -- > 1.8.3.1 > >