[PATCH 5/7] xfs: encapsulate bbcount validity checking

Christoph Hellwig hch at infradead.org
Mon Apr 5 06:32:12 CDT 2010


On Thu, Mar 18, 2010 at 05:54:06PM -0500, Alex Elder wrote:
> This replaces is a duplicated block of essentially identical code
> in three spots with a call to a common helper routine.

I don't think I really like this.  It's adding a lot of indirections
for some simple error reporting.

> +	if (bbcount > 0 && bbcount <= log->l_logBBsize)
> +	    	return 1;

Adding a tiny helper just for this might be worthwhile, though.




More information about the xfs mailing list