On Sun, Dec 08, 2013 at 11:03:16PM -0800, Christoph Hellwig wrote: > On Mon, Dec 09, 2013 at 09:33:59AM +1100, Dave Chinner wrote: > > While we are changing this, I think it makes sense to move it to > > being a static inline function given how simple it is.... > > I actually tried that first, but with XFS_DINODE_FMT_BTREE in > fs/xfs/xfs_dinode.h, struct xfsicdinode in xfs_log_format.h > and the resulting dependencies it didn't seem workable. My inode > reshuffle might provide relief eventually. Ok, makes sense to leave it like that for the moment. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs