From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id q5KFanNi096791 for ; Wed, 20 Jun 2012 10:36:49 -0500 Date: Wed, 20 Jun 2012 10:36:59 -0500 From: Ben Myers Subject: Re: [PATCH 2/9] xfs: convert internal buffer functions to pass maps Message-ID: <20120620153659.GI2676@sgi.com> References: <1339133914-11148-3-git-send-email-david@fromorbit.com> <20120618204325.GX4721@sgi.com> <20120618210713.GY4721@sgi.com> <20120619071552.GA23093@infradead.org> <20120619155946.GA2676@sgi.com> <20120619170304.GC7554@infradead.org> <20120619171153.GD2676@sgi.com> <20120620055619.GA30705@dastard> <20120620060400.GA5617@infradead.org> <20120620062931.GC30705@dastard> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20120620062931.GC30705@dastard> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: Christoph Hellwig , xfs@oss.sgi.com 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 _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs