From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35992) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WkY6N-0002UD-TY for qemu-devel@nongnu.org; Wed, 14 May 2014 08:16:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WkY6H-0007xZ-FR for qemu-devel@nongnu.org; Wed, 14 May 2014 08:16:03 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49875) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WkY6H-0007xT-82 for qemu-devel@nongnu.org; Wed, 14 May 2014 08:15:57 -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 s4ECFtgL019714 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Wed, 14 May 2014 08:15:56 -0400 Date: Wed, 14 May 2014 08:15:53 -0400 From: Jeff Cody Message-ID: <20140514121553.GD8452@localhost.localdomain> References: <20140514015807.GC1856@T430.nay.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140514015807.GC1856@T430.nay.redhat.com> 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: Fam Zheng Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Wed, May 14, 2014 at 09:58:07AM +0800, Fam Zheng wrote: > 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? > Yes, it is a p2v tool from MS for Virtual PC / Hyper-V hosts: http://technet.microsoft.com/en-us/sysinternals/ee656415.aspx > > > > 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 > > > >