Re: [PATCH v7 09/15] fs: Split off inode_needs_update_time and __file_update_time

Stefan Roesch
2022-06-02T21:04:43Z

On 6/2/22 5:57 AM, Matthew Wilcox wrote:
> On Wed, Jun 01, 2022 at 02:01:35PM -0700, Stefan Roesch wrote:
>> + /**
>> + * file_update_time - update mtime and ctime time
>> + * @file: file accessed
>> + *
>> + * Update the mtime and ctime members of an inode and mark the inode for
>> + * writeback. Note that this function is meant exclusively for usage in
>> + * the file write path of filesystems, and filesystems may choose to
>> + * explicitly ignore updates via this function with the _NOCMTIME inode
>> + * flag, e.g. for network filesystem where these imestamps are handled
>> + * by the server. This can return an error for file systems who need to
>> + * allocate space in order to update an inode.
>> + *
>> + * Return: 0 on success, negative errno on failure.
>> + */
>
> Can you remove the extra leading space from each of these lines?
>

I removed the extra space. Thanks for catching it.