From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [patch 16/44] rd convert to new aops Date: Tue, 24 Apr 2007 12:18:11 +0100 Message-ID: <20070424111811.GA18373@infradead.org> References: <20070424012346.696840000@suse.de> <20070424013434.737154000@suse.de> <20070424104647.GB13268@infradead.org> <20070424110549.GC32738@wotan.suse.de> <20070424111119.GA17085@infradead.org> <20070424111653.GB25413@wotan.suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , Andrew Morton , Linux Filesystems , Mark Fasheh To: Nick Piggin Return-path: Received: from pentafluge.infradead.org ([213.146.154.40]:52008 "EHLO pentafluge.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1031059AbXDXLSQ (ORCPT ); Tue, 24 Apr 2007 07:18:16 -0400 Content-Disposition: inline In-Reply-To: <20070424111653.GB25413@wotan.suse.de> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Tue, Apr 24, 2007 at 01:16:53PM +0200, Nick Piggin wrote: > > find_or_create_cache_page given that's it's like find_or_create_page + > > add_to_page_cache? > > find_or_create_page adds to page cache as well, though :P > > All those random little slightly different allocators scattered over > filemap.c are a bit annoying. Basically I think it would be better > to have a single variant that takes gfp_mask of both the pagecache > page, and the radix-tree insertion. Then serveral things can be > converted to use it. > > I was going to try doing that after this patchset. Or do you think it > would be better to get the __grab_cache_page name right in the > first place? If you plan to fix things up afterwards there's not point in doing the renaming first. Would be nice to get both into the same release in the end, though :)