aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2009-06-11 11:09:47 -0400
committerEric Paris <eparis@redhat.com>2009-06-11 14:57:54 -0400
commitce61856bd2aadb064f595e5c0444376a2b117c41 (patch)
tree4a438d07fbce1acbb3970b631211aaf445f2e41b
parent5ac697b793a3c45005c568df692518da6e690390 (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>
-rw-r--r--fs/notify/dnotify/dnotify.c3
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;