<div dir="ltr">Sorry about this - thank you for updating the patch!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 15, 2022 at 9:45 PM Gao Xiang <<a href="mailto:hsiangkao@linux.alibaba.com" target="_blank">hsiangkao@linux.alibaba.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi David,<br>
<br>
On Fri, Mar 11, 2022 at 12:47:20PM +0800, Gao Xiang wrote:<br>
> Hi David,<br>
> <br>
> On Fri, Mar 11, 2022 at 04:18:29AM +0000, David Anderson via Linux-erofs wrote:<br>
> > EROFS images should inherit modification time rather than creation time,<br>
> > since users and host tooling have no easy way to control creation time.<br>
> > To reflect the new timestamp meaning, i_ctime and i_ctime_nsec are<br>
> > renamed to i_mtime and i_mtime_nsec.<br>
> > <br>
> > Signed-off-by: David Anderson <<a href="mailto:dvander@google.com" target="_blank">dvander@google.com</a>><br>
> <br>
> Thanks for the patch!<br>
> <br>
> This patch looks good to me, yet, should we update<br>
> <a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/filesystems/erofs.rst#n43" rel="noreferrer" target="_blank">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/filesystems/erofs.rst#n43</a><br>
> <br>
> here as well? (I think "Inode timestamp" might be fine..)<br>
<br>
Since another -rc8 is out, would you mind revising the patch with<br>
the document update... Or would you mind if I could help revise it?<br>
<br>
Thanks,<br>
Gao Xiang<br>
<br>
> <br>
> Thanks,<br>
> Gao Xiang<br>
</blockquote></div>