[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <315DE858-FDCD-49BD-8E4E-0EAD0D389B96@whamcloud.com>
Date: Fri, 11 Nov 2011 22:50:53 -0700
From: Andreas Dilger <adilger@...mcloud.com>
To: Christoph Hellwig <hch@...radead.org>
Cc: Bobi Jam <bobijam@...mcloud.com>, Theodore Tso <tytso@....EDU>,
linux-ext4@...r.kernel.org
Subject: Re: [PATCH v2] ext4: expand commit callback
On 2011-11-09, at 3:17 AM, Christoph Hellwig wrote:
> On Wed, Nov 09, 2011 at 05:49:53PM +0800, Bobi Jam wrote:
>> This patch has been rebased on v3.2-rc1-91.
>>
>> The per-commit callback was used by mballoc code to manage free
>> space bitmaps after deleted blocks have been released. This patch
>> expand it to contain multiple different callbacks.
>
> It fails to explain why you do it, both in text and by sending it
> together with a patch that actually makes use of it.
We discussed this patch previously on the ext4 concall. The code is
being restructured to allow multiple commit callbacks to be registered.
The block freeing code in ext4 is being modified in the patch to use
the updated commit callback. This is used to mark deleted blocks as
available for allocation after the journal transaction has committed.
Cheers, Andreas
--
Andreas Dilger
Principal Engineer
Whamcloud, Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists