From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Lameter Subject: Re: [patch 14/19] Use page_cache_xxx in ext2 Date: Wed, 28 Nov 2007 19:55:40 -0800 (PST) Message-ID: References: <20071129011052.866354847@sgi.com> <20071129011147.567317218@sgi.com> <20071129034521.GV119954183@sgi.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: akpm@linux-foundation.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Christoph Hellwig , Mel Gorman , William Lee Irwin III , Jens Axboe , Badari Pulavarty , Maxim Levitsky , Fengguang Wu , swin wang , totty.lu@gmail.com, hugh@veritas.com, joern@lazybastard.org To: David Chinner Return-path: Received: from relay1.sgi.com ([192.48.171.29]:48739 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751381AbXK2Dzl (ORCPT ); Wed, 28 Nov 2007 22:55:41 -0500 In-Reply-To: <20071129034521.GV119954183@sgi.com> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org ext2: Simplify some functions Signed-off-by: Christoph Lameter --- fs/ext2/dir.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) Index: mm/fs/ext2/dir.c =================================================================== --- mm.orig/fs/ext2/dir.c 2007-11-28 19:51:05.038882954 -0800 +++ mm/fs/ext2/dir.c 2007-11-28 19:53:59.074132710 -0800 @@ -63,8 +63,7 @@ static inline void ext2_put_page(struct static inline unsigned long dir_pages(struct inode *inode) { - return (inode->i_size+page_cache_size(inode->i_mapping)-1)>> - page_cache_shift(inode->i_mapping); + return page_cache_next(inode->i_mapping, inode->i_size); } /* @@ -74,13 +73,9 @@ static inline unsigned long dir_pages(st static unsigned ext2_last_byte(struct inode *inode, unsigned long page_nr) { - unsigned last_byte = inode->i_size; struct address_space *mapping = inode->i_mapping; - last_byte -= page_nr << page_cache_shift(mapping); - if (last_byte > page_cache_size(mapping)) - last_byte = page_cache_size(mapping); - return last_byte; + return inode->i_size - page_cache_pos(mapping, page_nr, 0); } static int ext2_commit_chunk(struct page *page, loff_t pos, unsigned len)