From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53561) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W8R6d-0002ZL-AQ for qemu-devel@nongnu.org; Wed, 29 Jan 2014 04:06:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W8R6Y-0004ri-Df for qemu-devel@nongnu.org; Wed, 29 Jan 2014 04:06:47 -0500 Received: from mx1.redhat.com ([209.132.183.28]:56772) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W8R6Y-0004rd-4X for qemu-devel@nongnu.org; Wed, 29 Jan 2014 04:06:42 -0500 Date: Wed, 29 Jan 2014 17:06:35 +0800 From: Fam Zheng Message-ID: <20140129090635.GA16936@T430.redhat.com> References: <1390984456-9429-1-git-send-email-pl@kamp.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1390984456-9429-1-git-send-email-pl@kamp.de> Subject: Re: [Qemu-devel] [PATCHv2] block/vmdk: add basic .bdrv_check support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Lieven Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Wed, 01/29 09:34, Peter Lieven wrote: > this adds a basic vmdk corruption check. it should detect severe > table corruptions and file truncation. > > Signed-off-by: Peter Lieven > --- > v2: - use %PRId64 instead of %ld [Stefan] > - extend -> extent [Stefan] > - fixed overlong lines > > block/vmdk.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > > diff --git a/block/vmdk.c b/block/vmdk.c > index 99ca60f..e809e2e 100644 > --- a/block/vmdk.c > +++ b/block/vmdk.c > @@ -1942,6 +1942,53 @@ static ImageInfo *vmdk_get_extent_info(VmdkExtent *extent) > return info; > } > > +static int vmdk_check(BlockDriverState *bs, BdrvCheckResult *result, > + BdrvCheckMode fix) > +{ > + BDRVVmdkState *s = bs->opaque; > + VmdkExtent *extent = NULL; > + int64_t sector_num = 0; > + int64_t total_sectors = bdrv_getlength(bs) / BDRV_SECTOR_SIZE; > + int ret; > + uint64_t cluster_offset; > + > + if (fix) { > + return -ENOTSUP; > + } > + > + for (;;) { > + if (sector_num >= total_sectors) { > + return 0; > + } > + extent = find_extent(s, sector_num, extent); > + if (!extent) { > + fprintf(stderr, > + "ERROR: could not find extent for sector %" PRId64 "\n", > + sector_num); > + break; > + } > + ret = get_cluster_offset(bs, extent, NULL, > + sector_num << BDRV_SECTOR_BITS, > + 0, &cluster_offset); > + if (ret == VMDK_ERROR) { > + fprintf(stderr, > + "ERROR: could not get cluster_offset for sector %" > + PRId64 "\n", sector_num); > + break; > + } > + if (ret == VMDK_OK && cluster_offset >= bdrv_getlength(extent->file)) { > + fprintf(stderr, > + "ERROR: cluster offset for sector %" > + PRId64 " points after EOF\n", sector_num); > + break; > + } > + sector_num += extent->cluster_sectors; > + } > + > + result->corruptions++; > + return 0; > +} > + > static ImageInfoSpecific *vmdk_get_specific_info(BlockDriverState *bs) > { > int i; > @@ -2015,6 +2062,7 @@ static BlockDriver bdrv_vmdk = { > .instance_size = sizeof(BDRVVmdkState), > .bdrv_probe = vmdk_probe, > .bdrv_open = vmdk_open, > + .bdrv_check = vmdk_check, > .bdrv_reopen_prepare = vmdk_reopen_prepare, > .bdrv_read = vmdk_co_read, > .bdrv_write = vmdk_co_write, > -- > 1.7.9.5 > Reviewed-by: Fam Zheng