Skip to content

Commit

Permalink
net: Use umd_cleanup_helper()
Browse files Browse the repository at this point in the history
bpfilter_umh_cleanup() is the same function as umd_cleanup_helper().
Drop the redundant function.

Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20230526112104.1044686-1-jarkko@kernel.org
  • Loading branch information
Jarkko Sakkinen authored and Daniel Borkmann committed May 31, 2023
1 parent ffadc37 commit 9b68f30
Showing 3 changed files with 2 additions and 12 deletions.
1 change: 0 additions & 1 deletion include/linux/bpfilter.h
Original file line number Diff line number Diff line change
@@ -11,7 +11,6 @@ int bpfilter_ip_set_sockopt(struct sock *sk, int optname, sockptr_t optval,
unsigned int optlen);
int bpfilter_ip_get_sockopt(struct sock *sk, int optname, char __user *optval,
int __user *optlen);
void bpfilter_umh_cleanup(struct umd_info *info);

struct bpfilter_umh_ops {
struct umd_info info;
2 changes: 1 addition & 1 deletion net/bpfilter/bpfilter_kern.c
Original file line number Diff line number Diff line change
@@ -21,7 +21,7 @@ static void shutdown_umh(void)
if (tgid) {
kill_pid(tgid, SIGKILL, 1);
wait_event(tgid->wait_pidfd, thread_group_exited(tgid));
bpfilter_umh_cleanup(info);
umd_cleanup_helper(info);
}
}

11 changes: 1 addition & 10 deletions net/ipv4/bpfilter/sockopt.c
Original file line number Diff line number Diff line change
@@ -12,15 +12,6 @@
struct bpfilter_umh_ops bpfilter_ops;
EXPORT_SYMBOL_GPL(bpfilter_ops);

void bpfilter_umh_cleanup(struct umd_info *info)
{
fput(info->pipe_to_umh);
fput(info->pipe_from_umh);
put_pid(info->tgid);
info->tgid = NULL;
}
EXPORT_SYMBOL_GPL(bpfilter_umh_cleanup);

static int bpfilter_mbox_request(struct sock *sk, int optname, sockptr_t optval,
unsigned int optlen, bool is_set)
{
@@ -38,7 +29,7 @@ static int bpfilter_mbox_request(struct sock *sk, int optname, sockptr_t optval,
}
if (bpfilter_ops.info.tgid &&
thread_group_exited(bpfilter_ops.info.tgid))
bpfilter_umh_cleanup(&bpfilter_ops.info);
umd_cleanup_helper(&bpfilter_ops.info);

if (!bpfilter_ops.info.tgid) {
err = bpfilter_ops.start();

0 comments on commit 9b68f30

Please sign in to comment.