xfs
[Top] [All Lists]

Re: [PATCH 7/7] xfs: factor out code to find the longest free extent in

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 7/7] xfs: factor out code to find the longest free extent in the AG
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Sun, 15 Mar 2009 18:58:24 +1100
Cc: xfs@xxxxxxxxxxx, Dave Chinner <dgc@xxxxxxx>
In-reply-to: <20090220085230.115970000@xxxxxxxxxxxxxxxxxxxxxx>
Mail-followup-to: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx, Dave Chinner <dgc@xxxxxxx>
References: <20090220085207.663702000@xxxxxxxxxxxxxxxxxxxxxx> <20090220085230.115970000@xxxxxxxxxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17)
On Fri, Feb 20, 2009 at 03:52:14AM -0500, Christoph Hellwig wrote:
> From: Dave Chinner <dgc@xxxxxxx>
> 
> Signed-off-by: Dave Chinner <dgc@xxxxxxx>

....

>  /*
> + * Find the length of the longest extent in an AG.
> + */
> +xfs_extlen_t
> +xfs_alloc_min_freelist(
> +     struct xfs_mount        *mp,
> +     struct xfs_perag        *pag)

Looking at this again, I'd change the name of this function
to xfs_alloc_ag_longest_free_extent(), because that is what it
is actually returning....

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>