Commit c6718543 authored by Miklos Szeredi's avatar Miklos Szeredi

Revert "vfs: update ovl inode before relatime check"

This reverts commit 598e3c8f.

Overlayfs no longer relies on the vfs correct atime handling.
Signed-off-by: default avatarMiklos Szeredi <mszeredi@redhat.com>
parent 88059de1
...@@ -1554,37 +1554,17 @@ sector_t bmap(struct inode *inode, sector_t block) ...@@ -1554,37 +1554,17 @@ sector_t bmap(struct inode *inode, sector_t block)
} }
EXPORT_SYMBOL(bmap); EXPORT_SYMBOL(bmap);
/*
* Update times in overlayed inode from underlying real inode
*/
static void update_ovl_inode_times(struct dentry *dentry, struct inode *inode,
bool rcu)
{
if (!rcu) {
struct inode *realinode = d_real_inode(dentry);
if (unlikely(inode != realinode) &&
(!timespec64_equal(&inode->i_mtime, &realinode->i_mtime) ||
!timespec64_equal(&inode->i_ctime, &realinode->i_ctime))) {
inode->i_mtime = realinode->i_mtime;
inode->i_ctime = realinode->i_ctime;
}
}
}
/* /*
* With relative atime, only update atime if the previous atime is * With relative atime, only update atime if the previous atime is
* earlier than either the ctime or mtime or if at least a day has * earlier than either the ctime or mtime or if at least a day has
* passed since the last atime update. * passed since the last atime update.
*/ */
static int relatime_need_update(const struct path *path, struct inode *inode, static int relatime_need_update(struct vfsmount *mnt, struct inode *inode,
struct timespec now, bool rcu) struct timespec now)
{ {
if (!(path->mnt->mnt_flags & MNT_RELATIME)) if (!(mnt->mnt_flags & MNT_RELATIME))
return 1; return 1;
update_ovl_inode_times(path->dentry, inode, rcu);
/* /*
* Is mtime younger than atime? If yes, update atime: * Is mtime younger than atime? If yes, update atime:
*/ */
...@@ -1655,8 +1635,7 @@ static int update_time(struct inode *inode, struct timespec64 *time, int flags) ...@@ -1655,8 +1635,7 @@ static int update_time(struct inode *inode, struct timespec64 *time, int flags)
* This function automatically handles read only file systems and media, * This function automatically handles read only file systems and media,
* as well as the "noatime" flag and inode specific "noatime" markers. * as well as the "noatime" flag and inode specific "noatime" markers.
*/ */
bool __atime_needs_update(const struct path *path, struct inode *inode, bool atime_needs_update(const struct path *path, struct inode *inode)
bool rcu)
{ {
struct vfsmount *mnt = path->mnt; struct vfsmount *mnt = path->mnt;
struct timespec64 now; struct timespec64 now;
...@@ -1682,7 +1661,7 @@ bool __atime_needs_update(const struct path *path, struct inode *inode, ...@@ -1682,7 +1661,7 @@ bool __atime_needs_update(const struct path *path, struct inode *inode,
now = current_time(inode); now = current_time(inode);
if (!relatime_need_update(path, inode, timespec64_to_timespec(now), rcu)) if (!relatime_need_update(mnt, inode, timespec64_to_timespec(now)))
return false; return false;
if (timespec64_equal(&inode->i_atime, &now)) if (timespec64_equal(&inode->i_atime, &now))
...@@ -1697,7 +1676,7 @@ void touch_atime(const struct path *path) ...@@ -1697,7 +1676,7 @@ void touch_atime(const struct path *path)
struct inode *inode = d_inode(path->dentry); struct inode *inode = d_inode(path->dentry);
struct timespec64 now; struct timespec64 now;
if (!__atime_needs_update(path, inode, false)) if (!atime_needs_update(path, inode))
return; return;
if (!sb_start_write_trylock(inode->i_sb)) if (!sb_start_write_trylock(inode->i_sb))
......
...@@ -133,13 +133,6 @@ extern long prune_icache_sb(struct super_block *sb, struct shrink_control *sc); ...@@ -133,13 +133,6 @@ extern long prune_icache_sb(struct super_block *sb, struct shrink_control *sc);
extern void inode_add_lru(struct inode *inode); extern void inode_add_lru(struct inode *inode);
extern int dentry_needs_remove_privs(struct dentry *dentry); extern int dentry_needs_remove_privs(struct dentry *dentry);
extern bool __atime_needs_update(const struct path *, struct inode *, bool);
static inline bool atime_needs_update_rcu(const struct path *path,
struct inode *inode)
{
return __atime_needs_update(path, inode, true);
}
/* /*
* fs-writeback.c * fs-writeback.c
*/ */
......
...@@ -1015,7 +1015,7 @@ const char *get_link(struct nameidata *nd) ...@@ -1015,7 +1015,7 @@ const char *get_link(struct nameidata *nd)
if (!(nd->flags & LOOKUP_RCU)) { if (!(nd->flags & LOOKUP_RCU)) {
touch_atime(&last->link); touch_atime(&last->link);
cond_resched(); cond_resched();
} else if (atime_needs_update_rcu(&last->link, inode)) { } else if (atime_needs_update(&last->link, inode)) {
if (unlikely(unlazy_walk(nd))) if (unlikely(unlazy_walk(nd)))
return ERR_PTR(-ECHILD); return ERR_PTR(-ECHILD);
touch_atime(&last->link); touch_atime(&last->link);
......
...@@ -2088,6 +2088,7 @@ enum file_time_flags { ...@@ -2088,6 +2088,7 @@ enum file_time_flags {
S_VERSION = 8, S_VERSION = 8,
}; };
extern bool atime_needs_update(const struct path *, struct inode *);
extern void touch_atime(const struct path *); extern void touch_atime(const struct path *);
static inline void file_accessed(struct file *file) static inline void file_accessed(struct file *file)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment