On Mon, Nov 03, 2014 at 11:12:12PM +0100, Fanael Linithien wrote: > 2014-11-03 17:12 GMT+01:00 Brian Foster <bfoster@xxxxxxxxxx>: > > > > +static inline bool xfs_inobt_issparse(uint16_t holemask) > > +{ > > + return holemask == 0 ? false : true; > > +} > > Surely that should be "return holemask != 0;"? > ir_holemask bits are set for holes in the inode chunk and unset for allocated regions. This means that ir_holemask == 0 for a normal, fully-allocated chunk and != 0 otherwise (some bits are set to indicate the chunk has a hole). Check out the commit log for patch 4 for reasoning. Brian > _______________________________________________ > xfs mailing list > xfs@xxxxxxxxxxx > http://oss.sgi.com/mailman/listinfo/xfs _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs