summaryrefslogtreecommitdiff
path: root/fs/notify
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2010-07-28 18:18:38 +0400
committerEric Paris <eparis@redhat.com>2010-07-28 18:18:53 +0400
commitc496313fcc35a41e176e3f19cdda2544ea3a32a6 (patch)
tree442a90a3a2ac7b7106c3e7ba0583735b17f7153d /fs/notify
parent7f6b6117e1803777fcf48fe31bd236a7fbf740db (diff)
downloadlinux-c496313fcc35a41e176e3f19cdda2544ea3a32a6.tar.xz
dnotify: use the mark in handler functions
dnotify now gets a mark in the should_send_event and handle_event functions. Rather than look up the mark themselves dnotify should just use the mark it was handed. Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'fs/notify')
-rw-r--r--fs/notify/dnotify/dnotify.c22
1 files changed, 5 insertions, 17 deletions
diff --git a/fs/notify/dnotify/dnotify.c b/fs/notify/dnotify/dnotify.c
index 2cae9be120db..e3e855ff0dd8 100644
--- a/fs/notify/dnotify/dnotify.c
+++ b/fs/notify/dnotify/dnotify.c
@@ -86,7 +86,6 @@ static int dnotify_handle_event(struct fsnotify_group *group,
struct fsnotify_mark *mark,
struct fsnotify_event *event)
{
- struct fsnotify_mark *fsn_mark = NULL;
struct dnotify_mark *dn_mark;
struct inode *to_tell;
struct dnotify_struct *dn;
@@ -96,12 +95,9 @@ static int dnotify_handle_event(struct fsnotify_group *group,
to_tell = event->to_tell;
- fsn_mark = fsnotify_find_inode_mark(group, to_tell);
- if (unlikely(!fsn_mark))
- return 0;
- dn_mark = container_of(fsn_mark, struct dnotify_mark, fsn_mark);
+ dn_mark = container_of(mark, struct dnotify_mark, fsn_mark);
- spin_lock(&fsn_mark->lock);
+ spin_lock(&mark->lock);
prev = &dn_mark->dn;
while ((dn = *prev) != NULL) {
if ((dn->dn_mask & test_mask) == 0) {
@@ -115,12 +111,11 @@ static int dnotify_handle_event(struct fsnotify_group *group,
else {
*prev = dn->dn_next;
kmem_cache_free(dnotify_struct_cache, dn);
- dnotify_recalc_inode_mask(fsn_mark);
+ dnotify_recalc_inode_mask(mark);
}
}
- spin_unlock(&fsn_mark->lock);
- fsnotify_put_mark(fsn_mark);
+ spin_unlock(&mark->lock);
return 0;
}
@@ -134,7 +129,6 @@ static bool dnotify_should_send_event(struct fsnotify_group *group,
struct fsnotify_mark *mark, __u32 mask,
void *data, int data_type)
{
- struct fsnotify_mark *fsn_mark;
bool send;
/* !dir_notify_enable should never get here, don't waste time checking
@@ -145,14 +139,8 @@ static bool dnotify_should_send_event(struct fsnotify_group *group,
if (!S_ISDIR(inode->i_mode))
return false;
- fsn_mark = fsnotify_find_inode_mark(group, inode);
- if (!fsn_mark)
- return false;
-
mask = (mask & ~FS_EVENT_ON_CHILD);
- send = (mask & fsn_mark->mask);
-
- fsnotify_put_mark(fsn_mark); /* matches fsnotify_find_inode_mark */
+ send = (mask & mark->mask);
return send;
}