diff options
author | Jan Kara <jack@suse.cz> | 2020-03-24 19:35:07 +0100 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2020-03-25 10:27:16 +0100 |
commit | 01affd5471dcab04c6cb0c2acaf132a20488f86f (patch) | |
tree | f3f42caaadc86a377c61ca3fa1d8fc62c6f03d9f /fs | |
parent | d766b553615ce67f33848ec4f6672b3158198a5c (diff) |
fanotify: Drop fanotify_event_has_fid()
When some events have directory id and some object id,
fanotify_event_has_fid() becomes mostly useless and confusing because we
usually need to know which type of file handle the event has. So just
drop the function and use fanotify_event_object_fh() instead.
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/notify/fanotify/fanotify.c | 2 | ||||
-rw-r--r-- | fs/notify/fanotify/fanotify.h | 5 | ||||
-rw-r--r-- | fs/notify/fanotify/fanotify_user.c | 4 |
3 files changed, 3 insertions, 8 deletions
diff --git a/fs/notify/fanotify/fanotify.c b/fs/notify/fanotify/fanotify.c index 74676228f784..ef39664e389c 100644 --- a/fs/notify/fanotify/fanotify.c +++ b/fs/notify/fanotify/fanotify.c @@ -387,7 +387,7 @@ init: __maybe_unused else event->pid = get_pid(task_tgid(current)); - if (fanotify_event_has_fid(event)) { + if (fanotify_event_object_fh(event)) { ffe->object_fh.len = 0; if (fsid) ffe->fsid = *fsid; diff --git a/fs/notify/fanotify/fanotify.h b/fs/notify/fanotify/fanotify.h index 3b50ee44a0cd..eecf4be3bfd1 100644 --- a/fs/notify/fanotify/fanotify.h +++ b/fs/notify/fanotify/fanotify.h @@ -83,11 +83,6 @@ FANOTIFY_FE(struct fanotify_event *event) return container_of(event, struct fanotify_fid_event, fae); } -static inline bool fanotify_event_has_fid(struct fanotify_event *event) -{ - return event->type == FANOTIFY_EVENT_TYPE_FID; -} - static inline __kernel_fsid_t *fanotify_event_fsid(struct fanotify_event *event) { if (event->type == FANOTIFY_EVENT_TYPE_FID) diff --git a/fs/notify/fanotify/fanotify_user.c b/fs/notify/fanotify/fanotify_user.c index 3a11f6a0339f..b93585406aad 100644 --- a/fs/notify/fanotify/fanotify_user.c +++ b/fs/notify/fanotify/fanotify_user.c @@ -275,7 +275,7 @@ static ssize_t copy_event_to_user(struct fsnotify_group *group, metadata.mask = event->mask & FANOTIFY_OUTGOING_EVENTS; metadata.pid = pid_vnr(event->pid); - if (fanotify_event_has_fid(event)) { + if (fanotify_event_object_fh(event)) { metadata.event_len += fanotify_event_info_len(event); } else if (path && path->mnt && path->dentry) { fd = create_fd(group, path, &f); @@ -300,7 +300,7 @@ static ssize_t copy_event_to_user(struct fsnotify_group *group, if (f) { fd_install(fd, f); - } else if (fanotify_event_has_fid(event)) { + } else if (fanotify_event_object_fh(event)) { ret = copy_fid_to_user(fanotify_event_fsid(event), fanotify_event_object_fh(event), buf + FAN_EVENT_METADATA_LEN); |