[PATCH 34/37] xfs: clean up xfs_bit.h includes

Dave Chinner david at fromorbit.com
Sun Apr 29 21:40:47 CDT 2012


On Sun, Apr 29, 2012 at 05:55:25PM -0400, Christoph Hellwig wrote:
> >   * Visible inode allocation functions.
> >   */
> > +/*
> > + * Find a free (set) bit in the inode bitmask.
> > + */
> > +static inline int xfs_ialloc_find_free(xfs_inofree_t *fp)
> > +{
> > +	return xfs_lowbit64(*fp);
> > +}
> 
> I'd suggest killing this helper entirely - there is just a single
> caller, and using the lowbit helper directly actually is even more
> obvious.

Ok. Done.

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list