From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55655) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WV6Zd-0006Q8-80 for qemu-devel@nongnu.org; Tue, 01 Apr 2014 17:50:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WV6ZB-00023d-2o for qemu-devel@nongnu.org; Tue, 01 Apr 2014 17:50:08 -0400 Received: from mail-yh0-x231.google.com ([2607:f8b0:4002:c01::231]:49332) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WV6ZA-00023R-UJ for qemu-devel@nongnu.org; Tue, 01 Apr 2014 17:49:57 -0400 Received: by mail-yh0-f49.google.com with SMTP id z6so9641167yhz.22 for ; Tue, 01 Apr 2014 14:49:56 -0700 (PDT) Date: Tue, 1 Apr 2014 17:49:54 -0400 From: Shwetha Mathangi Chandra Choodamani Message-ID: <20140401214954.GA2902@smaugslair> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Subject: [Qemu-devel] [PATCH v2] Fix for qemu-img info to supply FORMAT values for SPARSE extents List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, famz@redhat.com, stefanha@redhat.com This patch fixes the bug in qemu-img info that wouldn't populate the extent type for default formats. The extent type has now been set where necessary. This is the second version in the series after inputs from Fam Zheng(famz@redhat.com). Signed-off-by: Shwetha Mathangi Chandra Choodamani --- block/vmdk.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 49 insertions(+), 4 deletions(-) diff --git a/block/vmdk.c b/block/vmdk.c index b69988d..d4a37ad 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -499,6 +499,18 @@ static int vmdk_open_vmfs_sparse(BlockDriverState *bs, VMDK3Header header; VmdkExtent *extent; + char access[11]; + char type[11]; + char fname[512]; + int64_t sectors = 0; + int64_t flat_offset; + int64_t size; + size = bdrv_getlength(file); + char *buf; + buf = g_malloc0(size + 1); + bdrv_pread(file, sizeof(magic), buf, size); + sscanf(buf, "%10s %" SCNd64 " %10s \"%511[^\n\r\"]\" %" SCNd64, + access, §ors, type, fname, &flat_offset); ret = bdrv_pread(file, sizeof(magic), &header, sizeof(header)); if (ret < 0) { error_setg_errno(errp, -ret, @@ -515,6 +527,18 @@ static int vmdk_open_vmfs_sparse(BlockDriverState *bs, le32_to_cpu(header.granularity), &extent, errp); + while (strcmp(access, "RW")) { + while (*buf) { + sscanf(buf, "%10s %" SCNd64 " %10s \"%511[^\n\r\"]\" %" SCNd64, + access, §ors, type, fname, &flat_offset); + if (*buf == '\n') { + buf++; + break; + } + buf++; + } + } + extent->type = g_strdup(type); if (ret < 0) { return ret; } @@ -523,6 +547,7 @@ static int vmdk_open_vmfs_sparse(BlockDriverState *bs, /* free extent allocated by vmdk_add_extent */ vmdk_free_last_extent(bs); } + g_free(buf); return ret; } @@ -566,7 +591,11 @@ static int vmdk_open_vmdk4(BlockDriverState *bs, VmdkExtent *extent; BDRVVmdkState *s = bs->opaque; int64_t l1_backup_offset = 0; - + char access[11]; + char type[11]; + char fname[512]; + int64_t sectors = 0; + int64_t flat_offset; ret = bdrv_pread(file, sizeof(magic), &header, sizeof(header)); if (ret < 0) { error_setg_errno(errp, -ret, @@ -586,11 +615,9 @@ static int vmdk_open_vmdk4(BlockDriverState *bs, return ret; } } - if (!s->create_type) { s->create_type = g_strdup("monolithicSparse"); } - if (le64_to_cpu(header.gd_offset) == VMDK4_GD_AT_END) { /* * The footer takes precedence over the header, so read it in. The @@ -694,6 +721,25 @@ static int vmdk_open_vmdk4(BlockDriverState *bs, g_free(s->create_type); s->create_type = g_strdup("streamOptimized"); } + + if (!extent->type) { + uint64_t desc_offset = le64_to_cpu(header.desc_offset); + char *buf = vmdk_read_desc(file, desc_offset<<9, errp); + sscanf(buf, "%10s %" SCNd64 " %10s \"%511[^\n\r\"]\" %" SCNd64, + access, §ors, type, fname, &flat_offset); + while (strcmp(access, "RW")) { + while (*buf) { + sscanf(buf, "%10s %" SCNd64 " %10s \"%511[^\n\r\"]\" %" SCNd64, + access, §ors, type, fname, &flat_offset); + if (*buf == '\n') { + buf++; + break; + } + buf++; + } + } + } + extent->type = g_strdup(type); extent->has_marker = le32_to_cpu(header.flags) & VMDK4_FLAG_MARKER; extent->version = le32_to_cpu(header.version); extent->has_zero_grain = le32_to_cpu(header.flags) & VMDK4_FLAG_ZERO_GRAIN; @@ -711,7 +757,6 @@ static int vmdk_parse_description(const char *desc, const char *opt_name, { char *opt_pos, *opt_end; const char *end = desc + strlen(desc); - opt_pos = strstr(desc, opt_name); if (!opt_pos) { return VMDK_ERROR; -- 1.7.9.5