]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
perf ftrace: Remove an unnecessary condition check in BPF
authorNamhyung Kim <namhyung@kernel.org>
Thu, 27 Feb 2025 19:12:22 +0000 (11:12 -0800)
committerNamhyung Kim <namhyung@kernel.org>
Thu, 13 Mar 2025 07:18:10 +0000 (00:18 -0700)
The bucket_num is set based on the {max,min}_latency already in
cmd_ftrace(), so no need to check it again in BPF.  Also I found
that it didn't pass the max_latency to BPF. :)

No functional changes intended.

Cc: Gabriele Monaco <gmonaco@redhat.com>
Link: https://lore.kernel.org/r/20250227191223.1288473-2-namhyung@kernel.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
tools/perf/util/bpf_skel/func_latency.bpf.c

index 3d3d9f427c20876e634f8d8bef2e65cfe0013b09..ad70547446786c54a9b8090d975a34542984517f 100644 (file)
@@ -125,8 +125,7 @@ int BPF_PROG(func_end)
                        // than the min latency desired.
                        if (val > 0) { // 1st entry: [ 1 unit .. bucket_range units )
                                key = val / bucket_range + 1;
-                               if (key >= bucket_num ||
-                                       val >= max_latency - min_latency)
+                               if (key >= bucket_num)
                                        key = bucket_num - 1;
                        }