qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Stefan Hajnoczi <stefanha@redhat.com>
To: Spencer Baugh <sbaugh@catern.com>
Cc: Grant Wu <grantwwu@gmail.com>, Kevin Wolf <kwolf@redhat.com>,
	qemu-devel@nongnu.org, qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] vpc: use current_size field for XenServer VHD images
Date: Tue, 22 Mar 2016 10:42:00 +0000	[thread overview]
Message-ID: <20160322104200.GD29248@stefanha-x1.localdomain> (raw)
In-Reply-To: <878u1blls5.fsf@earth.catern.com>

[-- Attachment #1: Type: text/plain, Size: 1486 bytes --]

On Mon, Mar 21, 2016 at 02:37:46PM -0400, Spencer Baugh wrote:
> Stefan Hajnoczi <stefanha@redhat.com> writes:
> > What output did you get from "qemu-img info /dev/dm-1"?
> 
> After the patching:
> 
> root@storage-4:~# 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 |............O@..|
> 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
> root@storage-4:~# 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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

  reply	other threads:[~2016-03-22 10:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-19 19:42 [Qemu-devel] [PATCH] vpc: use current_size field for XenServer VHD images Stefan Hajnoczi
2016-03-20  5:33 ` Grant Wu
2016-03-21  9:56   ` Stefan Hajnoczi
2016-03-21 18:37     ` Spencer Baugh
2016-03-22 10:42       ` Stefan Hajnoczi [this message]
2016-03-22 15:11         ` Grant Wu
2016-03-21 16:37 ` [Qemu-devel] [Qemu-block] " Jeff Cody

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160322104200.GD29248@stefanha-x1.localdomain \
    --to=stefanha@redhat.com \
    --cc=grantwwu@gmail.com \
    --cc=kwolf@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=sbaugh@catern.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).