lkml.org 
[lkml]   [2007]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: Inotify fails to send IN_ATTRIB events
  Hi,

> I am seeing missing inotify IN_ATTRIB events in the following situation:
>
> 1. "touch foo"
>
> 2. Make inotify watch "foo"
>
> 3. "ln foo bar"
> --> Link count changed so I should have gotten an IN_ATTRIB.
>
> 4. "rm foo"
> --> Link count changed so I should have gotten an IN_ATTRIB. (Or
> IN_DELETE_SELF;
> I don't care which.)
>
> 5. "ln bar foo && rm bar"
> --> Still no events.
>
> 6. "mv foo bar"
> --> I get IN_MOVED_SELF. Good!
>
> 7. "mv bar foo"
> --> I get IN_MOVED_SELF. Good!
>
>
> 3+4 is pretty much the same as 6, so I really ought to be told that my
> file has changed
> name. I don't really care much about getting notified about 3, but
> for completeness
> it ought to be handled.
>
> As far as I can see, the only way to be told about 4 is to put a watch
> on the directory in
> which foo resides. That is inelegant and has an inherent race condition.
It looks sensible what you ask for ;) Wanna try the patch below?
Anybody has some objection to it?

Honza
--
Jan Kara <jack@suse.cz>
SuSE CR Labs

---

Send inotify events to the inode itself when its link count has changed.

Signed-off-by: Jan Kara <jack@suse.cz>

diff --git a/fs/namei.c b/fs/namei.c
index 3b993db..9a91130 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -2188,6 +2188,7 @@ int vfs_unlink(struct inode *dir, struct

/* We don't d_delete() NFS sillyrenamed files--they still exist. */
if (!error && !(dentry->d_flags & DCACHE_NFSFS_RENAMED)) {
+ fsnotify_link_count(dentry->d_inode);
d_delete(dentry);
}

@@ -2360,7 +2361,7 @@ int vfs_link(struct dentry *old_dentry,
error = dir->i_op->link(old_dentry, dir, new_dentry);
mutex_unlock(&old_dentry->d_inode->i_mutex);
if (!error)
- fsnotify_create(dir, new_dentry);
+ fsnotify_link(dir, new_dentry);
return error;
}

diff --git a/include/linux/fsnotify.h b/include/linux/fsnotify.h
index 2bd31fa..b21b818 100644
--- a/include/linux/fsnotify.h
+++ b/include/linux/fsnotify.h
@@ -92,6 +92,14 @@ static inline void fsnotify_inoderemove(
}

/*
+ * fsnotify_link_count - inode's link count changed
+ */
+static inline void fsnotify_link_count(struct inode *inode)
+{
+ inotify_inode_queue_event(inode, IN_ATTRIB, 0, NULL, NULL);
+}
+
+/*
* fsnotify_create - 'name' was linked in
*/
static inline void fsnotify_create(struct inode *inode, struct dentry *dentry)
@@ -103,6 +111,18 @@ static inline void fsnotify_create(struc
}

/*
+ * fsnotify_link - new hardlink in 'inode' directory
+ */
+static inline void fsnotify_link(struct inode *inode, struct dentry *new_dentry)
+{
+ inode_dir_notify(inode, DN_CREATE);
+ inotify_inode_queue_event(inode, IN_CREATE, 0, new_dentry->d_name.name,
+ new_dentry->d_inode);
+ fsnotify_link_count(new_dentry->d_inode);
+ audit_inode_child(new_dentry->d_name.name, new_dentry, inode);
+}
+
+/*
* fsnotify_mkdir - directory 'name' was created
*/
static inline void fsnotify_mkdir(struct inode *inode, struct dentry *dentry)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2007-11-22 16:37    [W:0.241 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site