fanotify: drop the useless priority argument
The priority argument in fanotify is useless. Kill it. Signed-off-by: Eric Paris <eparis@redhat.com>
This commit is contained in:
parent
269ed32a9c
commit
08ae89380a
|
@ -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,
|
||||||
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)
|
||||||
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)
|
||||||
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 @@ out:
|
||||||
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)
|
||||||
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 {
|
||||||
* 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 *,
|
||||||
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);
|
||||||
|
|
Loading…
Reference in New Issue