On Fri, Oct 07, 2016 at 04:07:38PM -0700, Darrick J. Wong wrote: > The loop in _reflink_clear_inode_flag isn't necessary since we > jump out if any part of any extent is shared. Remove the loop > and we no longer need two maps, so we can save some stack use. I have a patch that gets rid of the xfs_bmapi_read calls entirely and instead does an initial xfs_bmap_search_extents and then iterates over the extent index. It also uses xfs_reflink_trim_around_shared instead of opencoding it. It didn't plan to submit it now as it seemed to superficial for 4.9, I can't dust it off if you want. -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
