diff options
author | Tim Chase <[email protected]> | 2013-12-10 16:36:42 -0600 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2013-12-12 10:23:58 -0800 |
commit | 84b0aac5fdab6daf8c4179dfba4abeb47e0d8b8e (patch) | |
tree | ccc6847d52119bddc85c6ebe7fd3156c0de4fd27 /module/zfs/zvol.c | |
parent | 5cb65efe2c3d4aaa77a5881be364c443c859bbc8 (diff) |
Fix atime handling.
Previously, the atime-modifying vnops called ZFS_ACCESSTIME_STAMP()
followed by zfs_inode_update() to update the atime. However, since atimes
are cached in the znode for delayed writing, the zfs_inode_update()
function would effectively ignore the cached atime by reading it from
the SA.
This commit moves the updating of the atime in the inode into
zfs_tstamp_update_setup() which is called by the ZFS_ACCESSTIME_STAMP()
macro and eliminates the call to zfs_inode_update() in the atime-modifying
vnops.
It's possible the same thing could have been done directly in
zfs_inode_update() but I wasn't sure that it was safe in all cases where
it is called.
The effect is that atime handling is as if "strictatime" were selected;
even if the filesystem is mounted with "relatime".
Signed-off-by: Brian Behlendorf <[email protected]>
Issue #1949
Diffstat (limited to 'module/zfs/zvol.c')
0 files changed, 0 insertions, 0 deletions