2 comments right before code wrongly state that if (c|m)time is younger than atime, then atime is updated (behavior is the other way around). Fix aligns comments with actual behavior, function description and documentation (man mount). Signed-off-by: Aurelien Thierry <aurelien.thierry@xxxxxxxxxxxx> --- fs/inode.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/inode.c b/fs/inode.c index df6542ec3b88..1e5489288aa9 100644 --- a/fs/inode.c +++ b/fs/inode.c @@ -1620,12 +1620,12 @@ static int relatime_need_update(struct vfsmount *mnt, struct inode *inode, if (!(mnt->mnt_flags & MNT_RELATIME)) return 1; /* - * Is mtime younger than atime? If yes, update atime: + * Is atime earlier than mtime? If yes, update atime: */ if (timespec64_compare(&inode->i_mtime, &inode->i_atime) >= 0) return 1; /* - * Is ctime younger than atime? If yes, update atime: + * Is atime earlier than ctime? If yes, update atime: */ if (timespec64_compare(&inode->i_ctime, &inode->i_atime) >= 0) return 1; -- 2.17.1
