fs: use inode_set_ctime_to_ts to set inode ctime to current time

Message ID 20240228173031.3208743-1-phind.uet@gmail.com
State New
Headers
Series fs: use inode_set_ctime_to_ts to set inode ctime to current time |

Commit Message

Nguyen Dinh Phi Feb. 28, 2024, 5:30 p.m. UTC
  The function inode_set_ctime_current simply retrieves the current time
and assigns it to the field __i_ctime without any alterations. Therefore,
it is possible to set ctime to now directly using inode_set_ctime_to_ts

Signed-off-by: Nguyen Dinh Phi <phind.uet@gmail.com>
---
 fs/inode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Christian Brauner Feb. 29, 2024, 10:22 a.m. UTC | #1
On Thu, 29 Feb 2024 01:30:31 +0800, Nguyen Dinh Phi wrote:
> The function inode_set_ctime_current simply retrieves the current time
> and assigns it to the field __i_ctime without any alterations. Therefore,
> it is possible to set ctime to now directly using inode_set_ctime_to_ts
> 
> 

Applied to the vfs.misc branch of the vfs/vfs.git tree.
Patches in the vfs.misc branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.misc

[1/1] fs: use inode_set_ctime_to_ts to set inode ctime to current time
      https://git.kernel.org/vfs/vfs/c/6adf169c32c2
  

Patch

diff --git a/fs/inode.c b/fs/inode.c
index 91048c4c9c9e..0b1327be581a 100644
--- a/fs/inode.c
+++ b/fs/inode.c
@@ -2509,7 +2509,7 @@  struct timespec64 inode_set_ctime_current(struct inode *inode)
 {
 	struct timespec64 now = current_time(inode);
 
-	inode_set_ctime(inode, now.tv_sec, now.tv_nsec);
+	inode_set_ctime_to_ts(inode, now);
 	return now;
 }
 EXPORT_SYMBOL(inode_set_ctime_current);