From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51190) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uz89x-0003hC-2t for qemu-devel@nongnu.org; Tue, 16 Jul 2013 12:31:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Uz89s-0006qo-3v for qemu-devel@nongnu.org; Tue, 16 Jul 2013 12:31:28 -0400 Received: from mail-qc0-x22e.google.com ([2607:f8b0:400d:c01::22e]:59126) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uz88b-0005ZD-0A for qemu-devel@nongnu.org; Tue, 16 Jul 2013 12:30:05 -0400 Received: by mail-qc0-f174.google.com with SMTP id m15so494317qcq.33 for ; Tue, 16 Jul 2013 09:30:04 -0700 (PDT) Sender: Paolo Bonzini From: Paolo Bonzini Date: Tue, 16 Jul 2013 18:29:18 +0200 Message-Id: <1373992168-26043-8-git-send-email-pbonzini@redhat.com> In-Reply-To: <1373992168-26043-1-git-send-email-pbonzini@redhat.com> References: <1373992168-26043-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [PATCH v2 07/17] qemu-img: always probe the input image for allocated sectors List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: pl@kamp.de, famz@redhat.com, stefanha@redhat.com qemu-img convert is assuming "that sectors which are unallocated in the input image are present in both the output's and input's base images", but it is only doing this if the output image is zero initialized. And checking if the output image is zero initialized does not make much sense if the output image is copy-on-write. Always do the test. Signed-off-by: Paolo Bonzini --- qemu-img.c | 26 ++++++++++++-------------- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/qemu-img.c b/qemu-img.c index a4957eb..3faffee 100644 --- a/qemu-img.c +++ b/qemu-img.c @@ -1476,21 +1476,19 @@ static int img_convert(int argc, char **argv) n = bs_offset + bs_sectors - sector_num; } - if (has_zero_init) { - /* If the output image is being created as a copy on write image, - assume that sectors which are unallocated in the input image - are present in both the output's and input's base images (no - need to copy them). */ - if (out_baseimg) { - if (!bdrv_is_allocated(bs[bs_i], sector_num - bs_offset, - n, &n1)) { - sector_num += n1; - continue; - } - /* The next 'n1' sectors are allocated in the input image. Copy - only those as they may be followed by unallocated sectors. */ - n = n1; + /* If the output image is being created as a copy on write image, + assume that sectors which are unallocated in the input image + are present in both the output's and input's base images (no + need to copy them). */ + if (out_baseimg) { + if (!bdrv_is_allocated(bs[bs_i], sector_num - bs_offset, + n, &n1)) { + sector_num += n1; + continue; } + /* The next 'n1' sectors are allocated in the input image. Copy + only those as they may be followed by unallocated sectors. */ + n = n1; } else { n1 = n; } -- 1.8.3.1