> - ASSERT(xfs_sb_version_hasdirv2(&mp->m_sb)); > + ASSERT(mp->m_sb.sb_versionnum & XFS_SB_VERSION_DIRV2BIT); How is this going to work for v5 filesystems? Probably better to just kill the assert.. Otherwise looks good, Reviewed-by: Christoph Hellwig <hch at lst.de>