[Top] [All Lists]

Re: [PATCH v5 00/11] xfs: introduce the free inode btree

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH v5 00/11] xfs: introduce the free inode btree
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 24 Apr 2014 15:51:01 +1000
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20140414094632.GC31578@dastard>
References: <1397146116-42546-1-git-send-email-bfoster@xxxxxxxxxx> <20140414094632.GC31578@dastard>
User-agent: Mutt/1.5.21 (2010-09-15)
On Mon, Apr 14, 2014 at 07:46:32PM +1000, Dave Chinner wrote:
> On Thu, Apr 10, 2014 at 12:08:25PM -0400, Brian Foster wrote:
> > Hi all,
> > 
> > This is a minor rebase of the finobt series on the latest for-next
> > branch. The previous version (v4) is here:
> > 
> > http://oss.sgi.com/archives/xfs/2014-02/msg00763.html
> > 
> > The most recent series with review feedback is v3:
> > 
> > http://oss.sgi.com/archives/xfs/2014-02/msg00121.html
> > 
> > Brian
> I've just had a look at this series, and I haven't found anything to
> complain about apart from a couple of stray whitespaces that I
> fixed. I'm going to run it through some finobt=0 test runs until i
> get a chance to look at the userspace patches. If I don't notice
> anything in the next couple of days, I'll merge the topic branch
> into the main tree....

All looks pretty good. I can fix the one little problem I noticed
myself, and so I think this is ready for merging. The userspace
stuff isn't directly necessary right away - that won't get merged
until 3.2 is released, but I don't see that as a problem for merging
the kernel code...


Dave Chinner

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