[Top] [All Lists]

Re: [RFC PATCH 10/11] xfs: update the finobt on inode free

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [RFC PATCH 10/11] xfs: update the finobt on inode free
From: Brian Foster <bfoster@xxxxxxxxxx>
Date: Sat, 07 Sep 2013 08:30:07 -0400
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130906212441.GB12541@dastard>
References: <1378232708-57156-1-git-send-email-bfoster@xxxxxxxxxx> <1378232708-57156-11-git-send-email-bfoster@xxxxxxxxxx> <20130905025421.GX23571@dastard> <5228AF00.7080700@xxxxxxxxxx> <20130906002847.GR12779@dastard> <5229BEE6.3060407@xxxxxxxxxx> <20130906212441.GB12541@dastard>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8
On 09/06/2013 05:24 PM, Dave Chinner wrote:
> On Fri, Sep 06, 2013 at 07:39:18AM -0400, Brian Foster wrote:
>> On 09/05/2013 08:28 PM, Dave Chinner wrote:
>>> On Thu, Sep 05, 2013 at 12:19:12PM -0400, Brian Foster wrote:
>>>> On 09/04/2013 10:54 PM, Dave Chinner wrote:
>>>>> On Tue, Sep 03, 2013 at 02:25:07PM -0400, Brian Foster wrote:
>> ...
>>> No worries. BTW, can you post your rudimentary userspace support so
>>> we can run tests that use this code, too?
>> Sure. My xfsprogs branch currently is the application of a slightly
>> older version of this set (pre-cleanups I made to make this post-worthy)
>> with some hacks to make it apply/compile and a few other patches on top
>> of that for mkfs, xfs_db and xfs_repair to work through some basic
>> things I ran into when running xfstests.
>> Would you prefer I drop the whole thing on the list?
> Drop it on the list, maybe just a as tarball rather than a patchset
> if it's not ready for review yet.

Ok, attached. This applies on top of the following commit in the
xfsprogs tree:

982e5c7e xfs_db: add header to freesp -d output

Use the following command to format a finobt enabled fs:

mkfs.xfs -m crc=1,finobt=1 <dev>

... and otherwise there are a couple random fixes for xfs_db and xfs_repair.


> Cheers,
> Dave.

Attachment: xfsprogs_finobt_rfc.tar.bz2
Description: application/bzip

<Prev in Thread] Current Thread [Next in Thread>