Re: [PATCH 07/15] refactor xfs_btree_readahead

[Christoph Hellwig <hch@xxxxxx>]

On Fri, Jul 25, 2008 at 09:36:55AM +1000, Dave Chinner wrote:
> On Wed, Jul 23, 2008 at 10:08:59PM +0200, Christoph Hellwig wrote:
> > From: Dave Chinner <dgc@xxxxxxx>
> 
> Can i review my own code? ;)

Sure..

> > +STATIC int
> > +xfs_btree_reada_corel(
> 
> I've been wondering if this is the best naming convention -
> appending a single s or l to indicate short or long btree ops.
> Perhaps this would be better as xfs_btree_readahead_short() and
> xfs_btree_readahead_long(), esp. as we dropped the 'core' from
> the caller to make it xfs_btree_readahead().
> 
> Otherwise it looks good.

I'll see if I can find a saner name.