diff options
| author | Ian Rogers <irogers@google.com> | 2025-06-06 23:12:38 -0700 |
|---|---|---|
| committer | Namhyung Kim <namhyung@kernel.org> | 2025-06-25 12:37:41 -0700 |
| commit | f0d0f978f3f5830ab06d71d1f37b3b30d47d6219 (patch) | |
| tree | a2fcf7b2705dcfb7163277c6d38435a793e401fd /tools/perf | |
| parent | 4d2eefd7fb91482f1327f28f14112201e0b45dff (diff) | |
| download | linux-next-f0d0f978f3f5830ab06d71d1f37b3b30d47d6219.tar.gz linux-next-f0d0f978f3f5830ab06d71d1f37b3b30d47d6219.zip | |
perf header: Don't write empty BPF/BTF info
If there are no values in bpf_prog_info or bpf_btf feature don't write
the data into the header.
Signed-off-by: Ian Rogers <irogers@google.com>
Link: https://lore.kernel.org/r/20250617223356.2752099-4-irogers@google.com
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf')
| -rw-r--r-- | tools/perf/util/header.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c index a9538bb1004d..487f663ed2de 100644 --- a/tools/perf/util/header.c +++ b/tools/perf/util/header.c @@ -1016,10 +1016,13 @@ static int write_bpf_prog_info(struct feat_fd *ff, struct perf_env *env = &ff->ph->env; struct rb_root *root; struct rb_node *next; - int ret; + int ret = 0; down_read(&env->bpf_progs.lock); + if (env->bpf_progs.infos_cnt == 0) + goto out; + ret = do_write(ff, &env->bpf_progs.infos_cnt, sizeof(env->bpf_progs.infos_cnt)); if (ret < 0) @@ -1058,10 +1061,13 @@ static int write_bpf_btf(struct feat_fd *ff, struct perf_env *env = &ff->ph->env; struct rb_root *root; struct rb_node *next; - int ret; + int ret = 0; down_read(&env->bpf_progs.lock); + if (env->bpf_progs.btfs_cnt == 0) + goto out; + ret = do_write(ff, &env->bpf_progs.btfs_cnt, sizeof(env->bpf_progs.btfs_cnt)); |