提交 08ae8938 编写于 作者: E Eric Paris

fanotify: drop the useless priority argument

The priority argument in fanotify is useless.  Kill it.
Signed-off-by: NEric Paris <eparis@redhat.com>
上级 269ed32a
...@@ -616,14 +616,13 @@ static int fanotify_add_inode_mark(struct fsnotify_group *group, ...@@ -616,14 +616,13 @@ static int fanotify_add_inode_mark(struct fsnotify_group *group,
} }
/* fanotify syscalls */ /* fanotify syscalls */
SYSCALL_DEFINE3(fanotify_init, unsigned int, flags, unsigned int, event_f_flags, SYSCALL_DEFINE2(fanotify_init, unsigned int, flags, unsigned int, event_f_flags)
unsigned int, priority)
{ {
struct fsnotify_group *group; struct fsnotify_group *group;
int f_flags, fd; int f_flags, fd;
pr_debug("%s: flags=%d event_f_flags=%d priority=%d\n", pr_debug("%s: flags=%d event_f_flags=%d\n",
__func__, flags, event_f_flags, priority); __func__, flags, event_f_flags);
if (event_f_flags) if (event_f_flags)
return -EINVAL; return -EINVAL;
...@@ -645,7 +644,6 @@ SYSCALL_DEFINE3(fanotify_init, unsigned int, flags, unsigned int, event_f_flags, ...@@ -645,7 +644,6 @@ SYSCALL_DEFINE3(fanotify_init, unsigned int, flags, unsigned int, event_f_flags,
if (IS_ERR(group)) if (IS_ERR(group))
return PTR_ERR(group); return PTR_ERR(group);
group->priority = priority;
#ifdef CONFIG_FANOTIFY_ACCESS_PERMISSIONS #ifdef CONFIG_FANOTIFY_ACCESS_PERMISSIONS
mutex_init(&group->fanotify_data.access_mutex); mutex_init(&group->fanotify_data.access_mutex);
init_waitqueue_head(&group->fanotify_data.access_waitq); init_waitqueue_head(&group->fanotify_data.access_waitq);
......
...@@ -90,7 +90,6 @@ void fsnotify_recalc_group_mask(struct fsnotify_group *group) ...@@ -90,7 +90,6 @@ void fsnotify_recalc_group_mask(struct fsnotify_group *group)
void fsnotify_add_vfsmount_group(struct fsnotify_group *group) void fsnotify_add_vfsmount_group(struct fsnotify_group *group)
{ {
struct fsnotify_group *group_iter; struct fsnotify_group *group_iter;
unsigned int priority = group->priority;
mutex_lock(&fsnotify_grp_mutex); mutex_lock(&fsnotify_grp_mutex);
...@@ -98,7 +97,7 @@ void fsnotify_add_vfsmount_group(struct fsnotify_group *group) ...@@ -98,7 +97,7 @@ void fsnotify_add_vfsmount_group(struct fsnotify_group *group)
list_for_each_entry(group_iter, &fsnotify_vfsmount_groups, list_for_each_entry(group_iter, &fsnotify_vfsmount_groups,
vfsmount_group_list) { vfsmount_group_list) {
/* insert in front of this one? */ /* insert in front of this one? */
if (priority < group_iter->priority) { if (group < group_iter) {
/* list_add_tail() insert in front of group_iter */ /* list_add_tail() insert in front of group_iter */
list_add_tail_rcu(&group->inode_group_list, list_add_tail_rcu(&group->inode_group_list,
&group_iter->inode_group_list); &group_iter->inode_group_list);
...@@ -118,15 +117,14 @@ void fsnotify_add_vfsmount_group(struct fsnotify_group *group) ...@@ -118,15 +117,14 @@ void fsnotify_add_vfsmount_group(struct fsnotify_group *group)
void fsnotify_add_inode_group(struct fsnotify_group *group) void fsnotify_add_inode_group(struct fsnotify_group *group)
{ {
struct fsnotify_group *group_iter; struct fsnotify_group *group_iter;
unsigned int priority = group->priority;
mutex_lock(&fsnotify_grp_mutex); mutex_lock(&fsnotify_grp_mutex);
/* add to global group list, priority 0 first, UINT_MAX last */ /* add to global group list */
if (!group->on_inode_group_list) { if (!group->on_inode_group_list) {
list_for_each_entry(group_iter, &fsnotify_inode_groups, list_for_each_entry(group_iter, &fsnotify_inode_groups,
inode_group_list) { inode_group_list) {
if (priority < group_iter->priority) { if (group < group_iter) {
/* list_add_tail() insert in front of group_iter */ /* list_add_tail() insert in front of group_iter */
list_add_tail_rcu(&group->inode_group_list, list_add_tail_rcu(&group->inode_group_list,
&group_iter->inode_group_list); &group_iter->inode_group_list);
...@@ -260,8 +258,6 @@ struct fsnotify_group *fsnotify_alloc_group(const struct fsnotify_ops *ops) ...@@ -260,8 +258,6 @@ struct fsnotify_group *fsnotify_alloc_group(const struct fsnotify_ops *ops)
spin_lock_init(&group->mark_lock); spin_lock_init(&group->mark_lock);
INIT_LIST_HEAD(&group->marks_list); INIT_LIST_HEAD(&group->marks_list);
group->priority = UINT_MAX;
group->ops = ops; group->ops = ops;
return group; return group;
......
...@@ -143,7 +143,6 @@ struct fsnotify_group { ...@@ -143,7 +143,6 @@ struct fsnotify_group {
* a group */ * a group */
struct list_head marks_list; /* all inode marks for this group */ struct list_head marks_list; /* all inode marks for this group */
unsigned int priority; /* order of this group compared to others */
/* prevents double list_del of group_list. protected by global fsnotify_grp_mutex */ /* prevents double list_del of group_list. protected by global fsnotify_grp_mutex */
bool on_inode_group_list; bool on_inode_group_list;
bool on_vfsmount_group_list; bool on_vfsmount_group_list;
......
...@@ -813,8 +813,7 @@ asmlinkage long sys_pselect6(int, fd_set __user *, fd_set __user *, ...@@ -813,8 +813,7 @@ asmlinkage long sys_pselect6(int, fd_set __user *, fd_set __user *,
asmlinkage long sys_ppoll(struct pollfd __user *, unsigned int, asmlinkage long sys_ppoll(struct pollfd __user *, unsigned int,
struct timespec __user *, const sigset_t __user *, struct timespec __user *, const sigset_t __user *,
size_t); size_t);
asmlinkage long sys_fanotify_init(unsigned int flags, unsigned int event_f_flags, asmlinkage long sys_fanotify_init(unsigned int flags, unsigned int event_f_flags);
unsigned int priority);
asmlinkage long sys_fanotify_mark(int fanotify_fd, unsigned int flags, asmlinkage long sys_fanotify_mark(int fanotify_fd, unsigned int flags,
u64 mask, int fd, u64 mask, int fd,
const char __user *pathname); const char __user *pathname);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册