diff options
author | Eric Paris <eparis@redhat.com> | 2009-06-11 11:09:47 -0400 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2009-06-11 14:57:54 -0400 |
commit | ce61856bd2aadb064f595e5c0444376a2b117c41 (patch) | |
tree | 4a438d07fbce1acbb3970b631211aaf445f2e41b /fs/notify/dnotify/dnotify.c | |
parent | 5ac697b793a3c45005c568df692518da6e690390 (diff) |
dnotify: do not bother to lock entry->lock when reading mask
entry->lock is needed to make sure entry->mask does not change while
manipulating it. In dnotify_should_send_event() we don't care if we get an
old or a new mask value out of this entry so there is no point it taking
the lock.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'fs/notify/dnotify/dnotify.c')
-rw-r--r-- | fs/notify/dnotify/dnotify.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/notify/dnotify/dnotify.c b/fs/notify/dnotify/dnotify.c index 5134e898f60d..ec459b6e8c64 100644 --- a/fs/notify/dnotify/dnotify.c +++ b/fs/notify/dnotify/dnotify.c | |||
@@ -153,9 +153,8 @@ static bool dnotify_should_send_event(struct fsnotify_group *group, | |||
153 | if (!entry) | 153 | if (!entry) |
154 | return false; | 154 | return false; |
155 | 155 | ||
156 | spin_lock(&entry->lock); | ||
157 | send = (mask & entry->mask); | 156 | send = (mask & entry->mask); |
158 | spin_unlock(&entry->lock); | 157 | |
159 | fsnotify_put_mark(entry); /* matches fsnotify_find_mark_entry */ | 158 | fsnotify_put_mark(entry); /* matches fsnotify_find_mark_entry */ |
160 | 159 | ||
161 | return send; | 160 | return send; |