Update SAs when an inode is dirtied
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 12 Dec 2012 00:58:44 +0000 (16:58 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 14 Dec 2012 20:18:54 +0000 (12:18 -0800)
commit8780c53961e668211682d40ad36946294c3145d8
tree9e3bfc68eb71071ac8e40413746d296096ee9f92
parentbd192c4f4831fae78f9dc3264b8c1a9b6dc85307
Update SAs when an inode is dirtied

Revert the portion of commit d3aa3ea which always resulted in the
SAs being update when an mmap()'ed file was closed.  That change
accidentally resulted in unexpected ctime updates which upset tools
like git.  That was always a horrible hack and I'm happy it will
never make it in to a tagged release.

The right fix is something I initially resisted doing because I
was worried about the additional overhead.  However, in hindsight
the overhead isn't as bad as I feared.

This patch implemented the sops->dirty_inode() callback which is
unsurprisingly called when an inode is dirtied.  We leverage this
callback to keep the znode SAs strictly in sync with the inode.

However, for now we're going to go slowly to avoid introducing
any new unexpected issues by only updating the atime, mtime, and
ctime.  This will cover the callpath of most concern to us.

  ->filemap_page_mkwrite->file_update_time->update_time->
      mark_inode_dirty_sync->__mark_inode_dirty->dirty_inode

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #764
Closes #1140
config/kernel-dirty-inode.m4 [new file with mode: 0644]
config/kernel.m4
include/sys/zfs_vnops.h
module/zfs/zfs_vnops.c
module/zfs/zpl_super.c