Re: [PATCH] perf trace: Handle task exit in BPF syscall summary

[Ian Rogers]

On Wed, Feb 25, 2026 at 5:49 PM Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
>
> Some system calls never return because it'd terminate the calling
> thread.  Let's hook the task exit path and update the duration of the
> last syscall.
>
> Before:
>   $ sudo perf trace -as --bpf-summary -- true |& grep exit
>   (nothing)
>
> After:
>   $ sudo perf trace -as --bpf-summary -- true |& grep exit
>      exit_group             1      0     0.004     0.004     0.004     0.004      0.00%
>
> Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Thanks,
Ian

> ---
>  tools/perf/util/bpf_skel/syscall_summary.bpf.c | 18 +++++++++++++++---
>  1 file changed, 15 insertions(+), 3 deletions(-)
>
> diff --git a/tools/perf/util/bpf_skel/syscall_summary.bpf.c b/tools/perf/util/bpf_skel/syscall_summary.bpf.c
> index 1bcd066a5199a476..4172f3c9fc483955 100644
> --- a/tools/perf/util/bpf_skel/syscall_summary.bpf.c
> +++ b/tools/perf/util/bpf_skel/syscall_summary.bpf.c
> @@ -118,13 +118,11 @@ int sys_enter(u64 *ctx)
>         return 0;
>  }
>
> -SEC("tp_btf/sys_exit")
> -int sys_exit(u64 *ctx)
> +static int do_exit(long ret)
>  {
>         int tid;
>         int key = 0;
>         u64 cgroup = 0;
> -       long ret = ctx[1]; /* return value of the syscall */
>         struct syscall_trace *st;
>         s64 delta;
>
> @@ -150,4 +148,18 @@ int sys_exit(u64 *ctx)
>         return 0;
>  }
>
> +SEC("tp_btf/sys_exit")
> +int sys_exit(u64 *ctx)
> +{
> +       long ret = ctx[1]; /* return value of the syscall */
> +
> +       return do_exit(ret);
> +}
> +
> +SEC("tp_btf/sched_process_exit")
> +int process_exit(u64 *ctx)
> +{
> +       return do_exit(0);
> +}
> +
>  char _license[] SEC("license") = "GPL";
> --
> 2.53.0.414.gf7e9f6c205-goog
>