[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/3] block: qemu-iotests for vhdx, read sampl
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/3] block: qemu-iotests for vhdx, read sample dynamic image |
Date: |
Fri, 27 Sep 2013 07:35:35 -0400 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Sep 27, 2013 at 11:14:55AM +0200, Kevin Wolf wrote:
> Am 25.09.2013 um 14:12 hat Jeff Cody geschrieben:
> > This adds the VHDX format to the qemu-iotests format, and adds
> > a read test. The test reads from an existing sample image, that
> > was created with Hyper-V under Windwos Server 2012.
> >
> > The image file is a 1GB dynamic image, with 32MB blocks.
> >
> > The pattern 0xa5 exists from 0MB-33MB (past a block size boundary)
> >
> > The pattern 0x96 exists from 33MB-66MB (past another block boundary,
> > and leaving a partial blank block)
> >
> > From 66MB-1024MB, all reads should return 0.
> >
> > Although 1GB dynamic image with 66MB of data, the bzip2'ed image
> > file size is only 874 bytes.
> >
> > Signed-off-by: Jeff Cody <address@hidden>
> > Reviewed-by: Stefan Hajnoczi <address@hidden>
>
> > diff --git a/tests/qemu-iotests/064.out b/tests/qemu-iotests/064.out
> > new file mode 100644
> > index 0000000..b9e8e4a
> > --- /dev/null
> > +++ b/tests/qemu-iotests/064.out
> > @@ -0,0 +1,14 @@
> > +QA output created by 064
> > +
> > +=== Verify pattern 0xa5, 0 - 33MB ===
> > +read 34603008/34603008 bytes at offset 0
> > +33 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> > +
> > +=== Verify pattern 0x96, 33M - 66M ===
> > +read 34603008/34603008 bytes at offset 34603008
> > +33 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> > +
> > +=== Verify pattern 0x00, 66M - 1024M ===
> > +read 1004535808/1004535808 bytes at offset 69206016
> > +958 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> > +*** done
>
> Doesn't work for me. Am I missing patches that need to be applied first?
>
I need to add -r to the QEMU_IO arguments. I had tested this on my
rebased vhdx branch, which has r/w support, so I missed that. QEMU on
qemu/master will refuse to open a VHDX image r/w.
Submitting a v4 with this change (just this patch since you applied
the other two)
> 064 1s ... [11:09:58] [11:09:59] - output mismatch (see 064.out.bad)
> --- 064.out 2013-09-27 11:00:37.077677734 +0200
> +++ 064.out.bad 2013-09-27 11:09:59.704298133 +0200
> @@ -1,14 +1,14 @@
> QA output created by 064
>
> === Verify pattern 0xa5, 0 - 33MB ===
> -read 34603008/34603008 bytes at offset 0
> -33 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +qemu-io: can't open device
> /home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx:
> Could not open
> '/home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx':
> Operation not supported
> +no file open, try 'help open'
>
> === Verify pattern 0x96, 33M - 66M ===
> -read 34603008/34603008 bytes at offset 34603008
> -33 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +qemu-io: can't open device
> /home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx:
> Could not open
> '/home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx':
> Operation not supported
> +no file open, try 'help open'
>
> === Verify pattern 0x00, 66M - 1024M ===
> -read 1004535808/1004535808 bytes at offset 69206016
> -958 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +qemu-io: can't open device
> /home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx:
> Could not open
> '/home/kwolf/source/qemu/tests/qemu-iotests/scratch/iotest-dynamic-1G.vhdx':
> Operation not supported
> +no file open, try 'help open'
> *** done
>
> Kevin
>
- [Qemu-devel] [PATCH v3 0/3] qemu-iotests with sample images, vhdx test, cleanup, Jeff Cody, 2013/09/25
- [Qemu-devel] [PATCH v3 1/3] block: qemu-iotests - add basic ability to use binary sample images, Jeff Cody, 2013/09/25
- [Qemu-devel] [PATCH v3 2/3] block: qemu-iotests for vhdx, read sample dynamic image, Jeff Cody, 2013/09/25
- [Qemu-devel] [PATCH v3 3/3] block: qemu-iotests - quote $TEST_IMG* and $TEST_DIR usage, Jeff Cody, 2013/09/25
- Re: [Qemu-devel] [PATCH v3 0/3] qemu-iotests with sample images, vhdx test, cleanup, Kevin Wolf, 2013/09/27