Skip to content

Commit

Permalink
bpf: task_group_seq_get_next: kill next_task
Browse files Browse the repository at this point in the history
It only adds the unnecessary confusion and compicates the "retry" code.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Yonghong Song <yonghong.song@linux.dev>
Link: https://lore.kernel.org/r/20230905154654.GA24945@redhat.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
  • Loading branch information
Oleg Nesterov authored and Alexei Starovoitov committed Sep 8, 2023
1 parent 87abbf7 commit 0ee9808
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions kernel/bpf/task_iter.c
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ static struct task_struct *task_group_seq_get_next(struct bpf_iter_seq_task_comm
u32 *tid,
bool skip_if_dup_files)
{
struct task_struct *task, *next_task;
struct task_struct *task;
struct pid *pid;
u32 saved_tid;

Expand Down Expand Up @@ -68,10 +68,10 @@ static struct task_struct *task_group_seq_get_next(struct bpf_iter_seq_task_comm
return NULL;

retry:
next_task = next_thread(task);
task = next_thread(task);

saved_tid = *tid;
*tid = __task_pid_nr_ns(next_task, PIDTYPE_PID, common->ns);
*tid = __task_pid_nr_ns(task, PIDTYPE_PID, common->ns);
if (!*tid || *tid == common->pid) {
/* Run out of tasks of a process. The tasks of a
* thread_group are linked as circular linked list.
Expand All @@ -82,13 +82,11 @@ static struct task_struct *task_group_seq_get_next(struct bpf_iter_seq_task_comm

common->pid_visiting = *tid;

if (skip_if_dup_files && next_task->files == next_task->group_leader->files) {
task = next_task;
if (skip_if_dup_files && task->files == task->group_leader->files)
goto retry;
}

get_task_struct(next_task);
return next_task;
get_task_struct(task);
return task;
}

static struct task_struct *task_seq_get_next(struct bpf_iter_seq_task_common *common,
Expand Down

0 comments on commit 0ee9808

Please sign in to comment.