summaryrefslogtreecommitdiffstats
path: root/fs/notify/group.c
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2010-07-28 10:18:39 -0400
committerEric Paris <eparis@redhat.com>2010-07-28 10:18:54 -0400
commit03930979afa63e079e9aefd4d3dd429240711027 (patch)
tree7df0097a8d51b9f71b4f10a0f49e773fb41d9ba5 /fs/notify/group.c
parent2612abb51b11ffd2d75c472b11178115f5808909 (diff)
downloadlinux-03930979afa63e079e9aefd4d3dd429240711027.tar.bz2
fsnotify: remove the global masks
Because we walk the object->fsnotify_marks list instead of the global fsnotify groups list we don't need the fsnotify_inode_mask and fsnotify_vfsmount_mask as these were simply shortcuts in fsnotify() for performance. They are now extra checks, rip them out. Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'fs/notify/group.c')
-rw-r--r--fs/notify/group.c39
1 files changed, 2 insertions, 37 deletions
diff --git a/fs/notify/group.c b/fs/notify/group.c
index 48d3a6d6e47a..8da532dd6026 100644
--- a/fs/notify/group.c
+++ b/fs/notify/group.c
@@ -34,54 +34,21 @@ static DEFINE_MUTEX(fsnotify_grp_mutex);
LIST_HEAD(fsnotify_inode_groups);
/* all groups registered to receive mount point filesystem notifications */
LIST_HEAD(fsnotify_vfsmount_groups);
-/* bitwise OR of all events (FS_*) interesting to some group on this system */
-__u32 fsnotify_inode_mask;
-/* bitwise OR of all events (FS_*) interesting to some group on this system */
-__u32 fsnotify_vfsmount_mask;
-
-/*
- * When a new group registers or changes it's set of interesting events
- * this function updates the fsnotify_mask to contain all interesting events
- */
-void fsnotify_recalc_global_mask(void)
-{
- struct fsnotify_group *group;
- __u32 inode_mask = 0;
- __u32 vfsmount_mask = 0;
-
- mutex_lock(&fsnotify_grp_mutex);
- list_for_each_entry_rcu(group, &fsnotify_inode_groups, inode_group_list)
- inode_mask |= group->mask;
- list_for_each_entry_rcu(group, &fsnotify_vfsmount_groups, vfsmount_group_list)
- vfsmount_mask |= group->mask;
-
- fsnotify_inode_mask = inode_mask;
- fsnotify_vfsmount_mask = vfsmount_mask;
-
- mutex_unlock(&fsnotify_grp_mutex);
-}
/*
* Update the group->mask by running all of the marks associated with this
- * group and finding the bitwise | of all of the mark->mask. If we change
- * the group->mask we need to update the global mask of events interesting
- * to the system.
+ * group and finding the bitwise | of all of the mark->mask.
*/
void fsnotify_recalc_group_mask(struct fsnotify_group *group)
{
__u32 mask = 0;
- __u32 old_mask = group->mask;
struct fsnotify_mark *mark;
spin_lock(&group->mark_lock);
list_for_each_entry(mark, &group->marks_list, g_list)
mask |= mark->mask;
- spin_unlock(&group->mark_lock);
-
group->mask = mask;
-
- if (old_mask != mask)
- fsnotify_recalc_global_mask();
+ spin_unlock(&group->mark_lock);
}
void fsnotify_add_vfsmount_group(struct fsnotify_group *group)
@@ -217,8 +184,6 @@ void fsnotify_put_group(struct fsnotify_group *group)
mutex_unlock(&fsnotify_grp_mutex);
- /* and now it is really dead. _Nothing_ could be seeing it */
- fsnotify_recalc_global_mask();
fsnotify_destroy_group(group);
}