在 2025/6/21 02:27, Alexei Starovoitov 写道:
On Thu, Jun 19, 2025 at 10:10 PM Tao Chen <chen.dylane@xxxxxxxxx> wrote:The field run_time_ns can tell us the run time of the bpf_prog, and load_time_s can tell us how long the bpf_prog loaded on the machine. Signed-off-by: Tao Chen <chen.dylane@xxxxxxxxx> --- kernel/bpf/syscall.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c index 51ba1a7aa43..407841ea296 100644 --- a/kernel/bpf/syscall.c +++ b/kernel/bpf/syscall.c @@ -2438,6 +2438,7 @@ static void bpf_prog_show_fdinfo(struct seq_file *m, struct file *filp) const struct bpf_prog *prog = filp->private_data; char prog_tag[sizeof(prog->tag) * 2 + 1] = { }; struct bpf_prog_kstats stats; + u64 now = ktime_get_boottime_ns(); bpf_prog_get_stats(prog, &stats); bin2hex(prog_tag, prog->tag, sizeof(prog->tag)); @@ -2450,7 +2451,8 @@ static void bpf_prog_show_fdinfo(struct seq_file *m, struct file *filp) "run_time_ns:\t%llu\n" "run_cnt:\t%llu\n" "recursion_misses:\t%llu\n" - "verified_insns:\t%u\n", + "verified_insns:\t%u\n" + "load_time_s:\t%llu\n", prog->type, prog->jited, prog_tag, @@ -2459,7 +2461,8 @@ static void bpf_prog_show_fdinfo(struct seq_file *m, struct file *filp) stats.nsecs, stats.cnt, stats.misses, - prog->aux->verified_insns); + prog->aux->verified_insns, + (now - prog->aux->load_time) / NSEC_PER_SEC);I don't like where it's going. Soon fdinfo will be printing the xlated insns of the prog too, since why not? Let's stop here. We have syscall query api-s for that and bpftool to print such things. No more fdinfo "improvements".
Got it, Alexei, thanks for your patient explanation. -- Best Regards Tao Chen
