Re: [PATCH] btrfs: update device path inode time instead of bd_inode

Josef Bacik
2021-10-14T15:50:12Z

On Thu, Oct 14, 2021 at 05:33:47PM +0200, Christoph Hellwig wrote:
> On Thu, Oct 14, 2021 at 11:17:08AM -0400, Josef Bacik wrote:
> > + now = current_time(d_inode(path.dentry));
> > + generic_update_time(d_inode(path.dentry), &now, S_MTIME | S_CTIME);
>
> This is still broken as it won't call into ->update_time.
> generic_update_time is a helper/default for ->update_time, not something
> for an external caller.

Then we probably need to fix all the people currently calling it. In the
meantime I'll add a helper to use the thing that calls ->update_time instead.
Thanks,

Josef