diff options
author | Eric Paris <eparis@redhat.com> | 2010-07-28 10:18:39 -0400 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2010-07-28 10:18:53 -0400 |
commit | 2612abb51b11ffd2d75c472b11178115f5808909 (patch) | |
tree | 5657a07a347984cc1f34117d7ef88af73ea12b60 /fs/notify/dnotify | |
parent | 0215054f377ce5ac4ffc27b26b13b3f10e6410e6 (diff) |
fsnotify: cleanup should_send_event
The change to use srcu and walk the object list rather than the global
fsnotify_group list means that should_send_event is no longer needed for a
number of groups and can be simplified for others. Do that.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'fs/notify/dnotify')
-rw-r--r-- | fs/notify/dnotify/dnotify.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/fs/notify/dnotify/dnotify.c b/fs/notify/dnotify/dnotify.c index e3e855ff0dd8..c3dc15879a52 100644 --- a/fs/notify/dnotify/dnotify.c +++ b/fs/notify/dnotify/dnotify.c | |||
@@ -129,20 +129,11 @@ static bool dnotify_should_send_event(struct fsnotify_group *group, | |||
129 | struct fsnotify_mark *mark, __u32 mask, | 129 | struct fsnotify_mark *mark, __u32 mask, |
130 | void *data, int data_type) | 130 | void *data, int data_type) |
131 | { | 131 | { |
132 | bool send; | ||
133 | |||
134 | /* !dir_notify_enable should never get here, don't waste time checking | ||
135 | if (!dir_notify_enable) | ||
136 | return 0; */ | ||
137 | |||
138 | /* not a dir, dnotify doesn't care */ | 132 | /* not a dir, dnotify doesn't care */ |
139 | if (!S_ISDIR(inode->i_mode)) | 133 | if (!S_ISDIR(inode->i_mode)) |
140 | return false; | 134 | return false; |
141 | 135 | ||
142 | mask = (mask & ~FS_EVENT_ON_CHILD); | 136 | return true; |
143 | send = (mask & mark->mask); | ||
144 | |||
145 | return send; | ||
146 | } | 137 | } |
147 | 138 | ||
148 | static void dnotify_free_mark(struct fsnotify_mark *fsn_mark) | 139 | static void dnotify_free_mark(struct fsnotify_mark *fsn_mark) |