[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qcow2: Remove abort on free_clusters failure
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH] qcow2: Remove abort on free_clusters failure |
Date: |
Tue, 27 Apr 2010 13:52:32 +0100 |
On Tue, Apr 27, 2010 at 11:35 AM, Kevin Wolf <address@hidden> wrote:
> --- a/block/qcow2-refcount.c
> +++ b/block/qcow2-refcount.c
> @@ -638,7 +638,7 @@ void qcow2_free_clusters(BlockDriverState *bs,
> ret = update_refcount(bs, offset, size, -1);
> if (ret < 0) {
> fprintf(stderr, "qcow2_free_clusters failed: %s\n", strerror(-ret));
> - abort();
> + /* TODO Remember the clusters to free them later and avoid leaking */
> }
> }
Has there been discussion on a fix mode for qemu-img check? For qcow2
it could write new refcounts, calculated by traversing the L1/L2
tables of the image and snapshots. Perhaps it would also dump out the
orphaned clusters to a lost+found.
Stefan