aboutsummaryrefslogtreecommitdiffstats
path: root/fs/notify/inode_mark.c
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2009-05-21 17:01:29 -0400
committerEric Paris <eparis@redhat.com>2009-06-11 14:57:53 -0400
commitc28f7e56e9d95fb531dc3be8df2e7f52bee76d21 (patch)
treeefce13ed8c4f5b312ef0b77950489d52ef5a039a /fs/notify/inode_mark.c
parent3be25f49b9d6a97eae9bcb96d3292072b7658bd8 (diff)
fsnotify: parent event notification
inotify and dnotify both use a similar parent notification mechanism. We add a generic parent notification mechanism to fsnotify for both of these to use. This new machanism also adds the dentry flag optimization which exists for inotify to dnotify. Signed-off-by: Eric Paris <eparis@redhat.com> Acked-by: Al Viro <viro@zeniv.linux.org.uk> Cc: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/notify/inode_mark.c')
-rw-r--r--fs/notify/inode_mark.c17
1 files changed, 17 insertions, 0 deletions
diff --git a/fs/notify/inode_mark.c b/fs/notify/inode_mark.c
index cdc154146974..a39534845b28 100644
--- a/fs/notify/inode_mark.c
+++ b/fs/notify/inode_mark.c
@@ -131,6 +131,8 @@ void fsnotify_recalc_inode_mask(struct inode *inode)
131 spin_lock(&inode->i_lock); 131 spin_lock(&inode->i_lock);
132 fsnotify_recalc_inode_mask_locked(inode); 132 fsnotify_recalc_inode_mask_locked(inode);
133 spin_unlock(&inode->i_lock); 133 spin_unlock(&inode->i_lock);
134
135 __fsnotify_update_child_dentry_flags(inode);
134} 136}
135 137
136/* 138/*
@@ -190,6 +192,19 @@ void fsnotify_destroy_mark_by_entry(struct fsnotify_mark_entry *entry)
190 group->ops->freeing_mark(entry, group); 192 group->ops->freeing_mark(entry, group);
191 193
192 /* 194 /*
195 * __fsnotify_update_child_dentry_flags(inode);
196 *
197 * I really want to call that, but we can't, we have no idea if the inode
198 * still exists the second we drop the entry->lock.
199 *
200 * The next time an event arrive to this inode from one of it's children
201 * __fsnotify_parent will see that the inode doesn't care about it's
202 * children and will update all of these flags then. So really this
203 * is just a lazy update (and could be a perf win...)
204 */
205
206
207 /*
193 * it's possible that this group tried to destroy itself, but this 208 * it's possible that this group tried to destroy itself, but this
194 * this mark was simultaneously being freed by inode. If that's the 209 * this mark was simultaneously being freed by inode. If that's the
195 * case, we finish freeing the group here. 210 * case, we finish freeing the group here.
@@ -323,6 +338,8 @@ int fsnotify_add_mark(struct fsnotify_mark_entry *entry,
323 if (lentry) { 338 if (lentry) {
324 ret = -EEXIST; 339 ret = -EEXIST;
325 fsnotify_put_mark(lentry); 340 fsnotify_put_mark(lentry);
341 } else {
342 __fsnotify_update_child_dentry_flags(inode);
326 } 343 }
327 344
328 return ret; 345 return ret;