xfs
[Top] [All Lists]

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

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 2/9] xfs: convert internal buffer functions to pass maps
From: Ben Myers <bpm@xxxxxxx>
Date: Wed, 20 Jun 2012 10:36:59 -0500
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <20120620062931.GC30705@dastard>
References: <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> <20120620062931.GC30705@dastard>
User-agent: Mutt/1.5.20 (2009-06-14)
Hey Fellas,

On Wed, Jun 20, 2012 at 04:29:31PM +1000, Dave Chinner wrote:
> 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.
> 
> No, other way around ;)

Right.  Dave's buffer changes don't apply without Christoph's inode allocator
patchset and Jan's overallocation fix.

> But I'm going to leave Jan kara's buffer cache over-allocation bug
> fix ahead my list, because that has to go into 3.5-rcX as it is a
> regression fix....

Last I saw Jan is going to respin his overallocation fix.

> FWIW, Ben, there's a few reviewed bug fixes on the list that are
> outstanding that need to go to Linus ASAP:
> 
> http://oss.sgi.com/archives/xfs/2012-06/msg00154.html
> http://oss.sgi.com/archives/xfs/2012-06/msg00134.html

I am testing these... 

> http://oss.sgi.com/archives/xfs/2012-06/msg00056.html (regression)

...waiting for a respin from Jan.

> Can you take care of them please?

Yep, just doing some basic testing first.  There are a few others that I also
will be looking to pull in.

> > 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.
> > 
> > In fact I'll start reviewing them in about 10 minutes after I've
> > finished catching up on email for this morning.
> 
> Cool. I'm just rebasing right now....

That'll be great... I too will make reviewing (and testing) these a high
priority.  Just keep in mind we don't control Jan's schedule...

-Ben

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