> if (unlikely(S_ISREG(ldip->di_mode))) { > if ((ldip->di_format != XFS_DINODE_FMT_EXTENTS) && > - (ldip->di_format != XFS_DINODE_FMT_BTREE)) { > + (ldip->di_format != XFS_DINODE_FMT_BTREE) && > + (ldip->di_format != XFS_DINODE_FMT_META_BTREE)) { Maybe drop the pointless inner braces here instead of adding more? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>