Re: [Qemu-block] [PATCH] vpc: use current_size field for XenServer VHD images

[Stefan Hajnoczi]

On Mon, Mar 21, 2016 at 02:37:46PM -0400, Spencer Baugh wrote:
> Stefan Hajnoczi <address@hidden> writes:
> > What output did you get from "qemu-img info /dev/dm-1"?
> 
> After the patching:
> 
> address@hidden:~# hexdump -C -n 512 /dev/vgstorage/tartanfsbackup
> 00000000  63 6f 6e 65 63 74 69 78  00 00 00 02 00 01 00 00 |conectix........|
> 00000010  00 00 00 00 00 00 02 00  14 10 4b 1f 71 65 6d 32 |..........K.qem2|
> 00000020  00 01 00 03 00 00 00 00  00 00 01 c4 4f 40 00 00 |address@hidden|
> 00000030  00 00 01 f4 00 00 00 00  ff ff 10 ff 00 00 00 03 |................|
> 00000040  ff ff ed 81 6d 0d 68 bd  9c fd 4d 65 a3 17 c7 6c |....m.h...Me...l|
> 00000050  06 a6 aa bf 00 00 00 00  00 00 00 00 00 00 00 00 |................|
> 00000060  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00 |................|
> *
> 00000200
> address@hidden:~# qemu-img info /dev/vgstorage/tartanfsbackup
> block-vpc: The header checksum of '/dev/vgstorage/tartanfsbackup' is 
> incorrect.

This is a warning message that does not cause opening the file to fail.
It makes sense since the header was modified without setting the new
checksum value.

> qemu-img: Could not open '/dev/vgstorage/tartanfsbackup': Could not open 
> '/dev/vgstorage/tartanfsbackup': Invalid argument

Please also post the output of "hexdump -C -n 512 -s 512
/dev/vgstorage/tartanfsbackup".  This is another header structure and it
gets parsed while opening the file.

Thanks,
Stefan

signature.asc
Description: PGP signature