[Top] [All Lists]

Re: [PATCH 2/9] xfs: convert internal buffer functions to pass maps

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 2/9] xfs: convert internal buffer functions to pass maps
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Wed, 20 Jun 2012 16:35:08 +1000
Cc: Ben Myers <bpm@xxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <20120620060400.GA5617@xxxxxxxxxxxxx>
References: <1339133914-11148-1-git-send-email-david@xxxxxxxxxxxxx> <1339133914-11148-3-git-send-email-david@xxxxxxxxxxxxx> <20120618204325.GX4721@xxxxxxx> <20120618210713.GY4721@xxxxxxx> <20120619071552.GA23093@xxxxxxxxxxxxx> <20120619155946.GA2676@xxxxxxx> <20120619170304.GC7554@xxxxxxxxxxxxx> <20120619171153.GD2676@xxxxxxx> <20120620055619.GA30705@dastard> <20120620060400.GA5617@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Jun 20, 2012 at 02:04:00AM -0400, Christoph Hellwig wrote:
> On Wed, Jun 20, 2012 at 03:56:19PM +1000, Dave Chinner wrote:
> > > they are rebased not to require the inode allocator patchset.  I think 
> > > that Jan
> > > is also going to repost his 'Fix overallocation in 
> > > xfs_buf_allocate_memory()'
> > > based upon Dave's suggestions.  It's best not to depend upon that either. 
> > >  What
> > > do you say, Dave?
> > 
> > I reorder my local patch set and repost it after running it through
> > some testing....
> Sorry, I totally misunderstood the initial issue - I though Ben had my
> series applied, and thus yours didn't apply.
> I still think reordering is better - I'm quite busy and I'd rather get
> your series reviewed first before redoing and retesting the inode
> allocator changes.

I stand corrected - it doesn't need a rebase - all it needs is to be
applied on top of Jan Kara's bug fix. Looking back, that's exactly
what I said in patch 0 of the series....

So it should be good to go as Jan's fix needs to go to Linus before
3.5 is released...


Dave Chinner

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