From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id o8A0abqF123532 for ; Thu, 9 Sep 2010 19:36:37 -0500 Received: from mail.internode.on.net (localhost [127.0.0.1]) by cuda.sgi.com (Spam Firewall) with ESMTP id 8FF685FA78 for ; Thu, 9 Sep 2010 17:37:21 -0700 (PDT) Received: from mail.internode.on.net (bld-mail16.adl2.internode.on.net [150.101.137.101]) by cuda.sgi.com with ESMTP id ME7xvq0fWq0Cfz5q for ; Thu, 09 Sep 2010 17:37:21 -0700 (PDT) Date: Fri, 10 Sep 2010 10:37:06 +1000 From: Dave Chinner Subject: Re: [RFC PATCH] fs,xfs: Use __GFP_MOVABLE for XFS buffers Message-ID: <20100910003706.GC7032@dastard> References: <20100909111131.GO29263@csn.ul.ie> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20100909111131.GO29263@csn.ul.ie> 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: Mel Gorman Cc: linux-mm@kvack.org, Alex Elder , linux-kernel@vger.kernel.org, xfs@oss.sgi.com On Thu, Sep 09, 2010 at 12:11:32PM +0100, Mel Gorman wrote: > Fragmentation avoidance in the kernel depends on reclaimable and movable > allocations being marked-up at page allocation time. Reclaimable allocations > refer to slab caches such as inode caches which can be reclaimed although > not necessarily in a targetted fashion. Movable pages are those pages that > can be moved to backing storage (during page reclaim) or migrated. > > When testing against XFS, it was noticed that large page allocation rates > against XFS were far lower than expected in comparison to ext3. Investigation > showed that buffer pages allocated by XFS are placed on the LRU but not > marked __GFP_MOVABLE at allocation time. > > This patch updates xb_to_gfp() to specify __GFP_MOVABLE and is correct iff > all pages allocated from a mask derived from xb_to_gfp() are guaranteed to > be movable be it via page reclaim or page migration. It needs an XFS expert > to make that determination but when applied, huge page allocation success > rates are similar to those seen on tests backed by ext3. > > Signed-off-by: Mel Gorman I don't see any problems with this, but I don't think it's going to be useful for very long given the work I'm doing on the XFS buffer cache right now - converting it to caching buffers with a shrinker traversed LRU for reclaim instead of using the page cache and hoping reclaim doesn't trash the working set. I'm hoping to have it done in time for the .37 merge window, so adding __GFP_MOVEABLE now might not to even see a release.... Cheers, Dave. -- Dave Chinner david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs