On Fri, Feb 03, 2017 at 09:45:41AM -0800, Darrick J. Wong wrote: > Redundant is_dedupe tests here. > > I also wonder if we really /want/ to emulate the existing btrfs > behavior, which seems to be: > > reflink: update mtime & ctime > dedupe: do not update mtime or ctime > > In particular, dedupe changes the inode metadata, which should qualify > for a ctime update, right? That depends on your metadata definition. For Posix it's basically just about stat data, and that doesn't change with dedupe. -- 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