Re: [PATCH 5/5] btrfs: add no_mtime flag to btrfs-extent-same

Mark Fasheh
2015-06-25T18:12:25Z

On Thu, Jun 25, 2015 at 02:52:50PM +0200, David Sterba wrote:
> On Wed, Jun 24, 2015 at 04:17:32PM -0400, Zygo Blaxell wrote:
> > Is there any sane use case where we would _want_ EXTENT_SAME to change
> > the mtime? We do a lot of work to make sure that none of the files
> > involved have any sort of content change. Why do we need the flag at all?
>
> Good point, I don't see the usecase for updating MTIME.

Yeah there isn't one and I doubt anyone will be upset if we just always
ignore the mtime update. I'll send some new patches shortly.

Thanks for the suggestion Zygo.
--Mark

--
Mark Fasheh