From: Kevin Wolf <kwolf@redhat.com>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: Fam Zheng <famz@redhat.com>,
qemu-devel@nongnu.org,
Ilkka Tengvall <ilkka.tengvall@cybercom.com>
Subject: Re: [Qemu-devel] [PATCH 0/4] qemu-img: support compression regardless of cluster size
Date: Tue, 16 Apr 2013 11:00:29 +0200 [thread overview]
Message-ID: <20130416090029.GD2825@dhcp-200-207.str.redhat.com> (raw)
In-Reply-To: <1366039054-32536-1-git-send-email-stefanha@redhat.com>
Am 15.04.2013 um 17:17 hat Stefan Hajnoczi geschrieben:
> It was hard to find a short email subject line. Anyway, the problem is that
> qemu-img convert -c fails with the following error message if the input image
> length is not a multiple of the output cluster size:
>
> qemu-img: error while compressing sector 0: Input/output error
>
> Ilkka Tengvall <ilkka.tengvall@cybercom.com> reported the failure. kwolf and
> eblake suggested a fix which this patch series implements.
>
> qemu-img convert -c succeeds with these patches applied.
>
> The final patch adds qemu-iotests case 053 to protect against regressions.
Thanks, applied all to the block branch. If you're going to send an
updated test case, I'll replace it in my tree.
Kevin
prev parent reply other threads:[~2013-04-16 9:00 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-15 15:17 [Qemu-devel] [PATCH 0/4] qemu-img: support compression regardless of cluster size Stefan Hajnoczi
2013-04-15 15:17 ` [Qemu-devel] [PATCH 1/4] qcow2: allow sub-cluster compressed write to last cluster Stefan Hajnoczi
2013-04-15 15:17 ` [Qemu-devel] [PATCH 2/4] qcow: " Stefan Hajnoczi
2013-04-15 15:17 ` [Qemu-devel] [PATCH 3/4] qemu-img: do not zero-pad the compressed write buffer Stefan Hajnoczi
2013-04-15 15:17 ` [Qemu-devel] [PATCH 4/4] qemu-iotests: add 053 unaligned compressed image size test Stefan Hajnoczi
2013-04-15 15:38 ` Eric Blake
2013-04-16 8:36 ` Stefan Hajnoczi
2013-04-16 8:53 ` Kevin Wolf
2013-04-16 9:00 ` Kevin Wolf [this message]
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=20130416090029.GD2825@dhcp-200-207.str.redhat.com \
--to=kwolf@redhat.com \
--cc=famz@redhat.com \
--cc=ilkka.tengvall@cybercom.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.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).