Re: [PATCHv2 5/10] xfs: encapsulate bbcount validity checking

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Fri, Apr 09, 2010 at 05:28:11PM -0500, Alex Elder wrote:
> Define a function that encapsulates checking the validity of a log
> block count.
> 
> (Updated from previous version--no longer includes error reporting in the
> encapsulated validation function.)
> 
> Signed-off-by: Alex Elder <aelder@xxxxxxx>

Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

but one small change needed:

>  
> +/*
> + * Verify the given count of basic blocks is valid number of blocks
> + * to specify for an operation involving the given XFS log buffer.
> + * Returns nonzero if the count is valid, 0 otherwise.
> + */
> +
> +inline static int

static inline int

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx