[PATCH 0/9] use canonical ftrace path whenever possible

Ross Zwisler posted 9 patches 2 years, 7 months ago
There is a newer version of this series
include/linux/kernel.h                        |  2 +-
include/linux/tracepoint.h                    |  4 ++--
include/uapi/linux/bpf.h                      |  8 ++++----
kernel/trace/Kconfig                          | 20 +++++++++----------
kernel/trace/kprobe_event_gen_test.c          |  2 +-
kernel/trace/ring_buffer.c                    |  2 +-
kernel/trace/synth_event_gen_test.c           |  2 +-
kernel/trace/trace.c                          |  2 +-
samples/bpf/cpustat_kern.c                    |  4 ++--
samples/bpf/hbm.c                             |  4 ++--
samples/bpf/ibumad_kern.c                     |  4 ++--
samples/bpf/lwt_len_hist.sh                   |  2 +-
samples/bpf/offwaketime_kern.c                |  2 +-
samples/bpf/task_fd_query_user.c              |  4 ++--
samples/bpf/test_lwt_bpf.sh                   |  2 +-
samples/bpf/test_overhead_tp_kern.c           |  4 ++--
samples/user_events/example.c                 |  4 ++--
scripts/leaking_addresses.pl                  |  1 +
scripts/tracing/draw_functrace.py             |  6 +++---
scripts/tracing/ftrace-bisect.sh              |  4 ++--
tools/include/uapi/linux/bpf.h                |  8 ++++----
tools/kvm/kvm_stat/kvm_stat                   |  2 +-
tools/lib/api/fs/tracing_path.c               |  4 ++--
tools/lib/traceevent/event-parse.c            |  8 ++++----
tools/perf/Documentation/perf-list.txt        |  2 +-
tools/perf/Documentation/perf-script-perl.txt |  2 +-
.../perf/Documentation/perf-script-python.txt |  4 ++--
tools/power/pm-graph/sleepgraph.py            |  4 ++--
.../x86/amd_pstate_tracer/amd_pstate_trace.py |  4 ++--
.../intel_pstate_tracer.py                    | 10 +++++-----
.../selftests/bpf/get_cgroup_id_user.c        |  2 +-
.../bpf/prog_tests/kprobe_multi_test.c        |  2 +-
.../bpf/prog_tests/task_fd_query_tp.c         |  2 +-
.../bpf/prog_tests/tp_attach_query.c          |  2 +-
.../selftests/bpf/prog_tests/trace_printk.c   |  2 +-
.../selftests/bpf/prog_tests/trace_vprintk.c  |  2 +-
.../selftests/bpf/progs/test_stacktrace_map.c |  2 +-
.../selftests/bpf/progs/test_tracepoint.c     |  2 +-
tools/testing/selftests/bpf/test_ftrace.sh    |  2 +-
tools/testing/selftests/bpf/test_tunnel.sh    |  8 ++++----
tools/testing/selftests/bpf/trace_helpers.c   |  4 ++--
.../testing/selftests/user_events/dyn_test.c  |  2 +-
.../selftests/user_events/ftrace_test.c       | 10 +++++-----
.../testing/selftests/user_events/perf_test.c |  8 ++++----
tools/testing/selftests/vm/protection_keys.c  |  4 ++--
tools/tracing/latency/latency-collector.c     |  2 +-
tools/virtio/virtio-trace/README              |  2 +-
tools/virtio/virtio-trace/trace-agent.c       |  2 +-
48 files changed, 96 insertions(+), 95 deletions(-)
[PATCH 0/9] use canonical ftrace path whenever possible
Posted by Ross Zwisler 2 years, 7 months ago
The canonical location for the tracefs filesystem is at /sys/kernel/tracing.

But, from Documentation/trace/ftrace.rst:

  Before 4.1, all ftrace tracing control files were within the debugfs
  file system, which is typically located at /sys/kernel/debug/tracing.
  For backward compatibility, when mounting the debugfs file system,
  the tracefs file system will be automatically mounted at:

  /sys/kernel/debug/tracing

There are many places where this older debugfs path is still used in
code comments, selftests, examples and tools, so let's update them to
avoid confusion.

I've broken up the series as best I could by maintainer or directory,
and I've only sent people the patches that I think they care about to
avoid spamming everyone.

Ross Zwisler (9):
  tracing: always use canonical ftrace path
  bpf: use canonical ftrace path
  selftests/bpf: use canonical ftrace path
  perf docs: use canonical ftrace path
  tools/power: use canonical ftrace path
  selftests: use canonical ftrace path
  tools/virtio: use canonical ftrace path
  leaking_addresses: also skip canonical ftrace path
  tools/kvm_stat: use canonical ftrace path

 include/linux/kernel.h                        |  2 +-
 include/linux/tracepoint.h                    |  4 ++--
 include/uapi/linux/bpf.h                      |  8 ++++----
 kernel/trace/Kconfig                          | 20 +++++++++----------
 kernel/trace/kprobe_event_gen_test.c          |  2 +-
 kernel/trace/ring_buffer.c                    |  2 +-
 kernel/trace/synth_event_gen_test.c           |  2 +-
 kernel/trace/trace.c                          |  2 +-
 samples/bpf/cpustat_kern.c                    |  4 ++--
 samples/bpf/hbm.c                             |  4 ++--
 samples/bpf/ibumad_kern.c                     |  4 ++--
 samples/bpf/lwt_len_hist.sh                   |  2 +-
 samples/bpf/offwaketime_kern.c                |  2 +-
 samples/bpf/task_fd_query_user.c              |  4 ++--
 samples/bpf/test_lwt_bpf.sh                   |  2 +-
 samples/bpf/test_overhead_tp_kern.c           |  4 ++--
 samples/user_events/example.c                 |  4 ++--
 scripts/leaking_addresses.pl                  |  1 +
 scripts/tracing/draw_functrace.py             |  6 +++---
 scripts/tracing/ftrace-bisect.sh              |  4 ++--
 tools/include/uapi/linux/bpf.h                |  8 ++++----
 tools/kvm/kvm_stat/kvm_stat                   |  2 +-
 tools/lib/api/fs/tracing_path.c               |  4 ++--
 tools/lib/traceevent/event-parse.c            |  8 ++++----
 tools/perf/Documentation/perf-list.txt        |  2 +-
 tools/perf/Documentation/perf-script-perl.txt |  2 +-
 .../perf/Documentation/perf-script-python.txt |  4 ++--
 tools/power/pm-graph/sleepgraph.py            |  4 ++--
 .../x86/amd_pstate_tracer/amd_pstate_trace.py |  4 ++--
 .../intel_pstate_tracer.py                    | 10 +++++-----
 .../selftests/bpf/get_cgroup_id_user.c        |  2 +-
 .../bpf/prog_tests/kprobe_multi_test.c        |  2 +-
 .../bpf/prog_tests/task_fd_query_tp.c         |  2 +-
 .../bpf/prog_tests/tp_attach_query.c          |  2 +-
 .../selftests/bpf/prog_tests/trace_printk.c   |  2 +-
 .../selftests/bpf/prog_tests/trace_vprintk.c  |  2 +-
 .../selftests/bpf/progs/test_stacktrace_map.c |  2 +-
 .../selftests/bpf/progs/test_tracepoint.c     |  2 +-
 tools/testing/selftests/bpf/test_ftrace.sh    |  2 +-
 tools/testing/selftests/bpf/test_tunnel.sh    |  8 ++++----
 tools/testing/selftests/bpf/trace_helpers.c   |  4 ++--
 .../testing/selftests/user_events/dyn_test.c  |  2 +-
 .../selftests/user_events/ftrace_test.c       | 10 +++++-----
 .../testing/selftests/user_events/perf_test.c |  8 ++++----
 tools/testing/selftests/vm/protection_keys.c  |  4 ++--
 tools/tracing/latency/latency-collector.c     |  2 +-
 tools/virtio/virtio-trace/README              |  2 +-
 tools/virtio/virtio-trace/trace-agent.c       |  2 +-
 48 files changed, 96 insertions(+), 95 deletions(-)

-- 
2.39.1.456.gfc5497dd1b-goog
Re: [PATCH 0/9] use canonical ftrace path whenever possible
Posted by Michael S. Tsirkin 2 years, 7 months ago
On Mon, Jan 30, 2023 at 11:19:06AM -0700, Ross Zwisler wrote:
> The canonical location for the tracefs filesystem is at /sys/kernel/tracing.


Acked-by: Michael S. Tsirkin <mst@redhat.com>

> But, from Documentation/trace/ftrace.rst:
> 
>   Before 4.1, all ftrace tracing control files were within the debugfs
>   file system, which is typically located at /sys/kernel/debug/tracing.
>   For backward compatibility, when mounting the debugfs file system,
>   the tracefs file system will be automatically mounted at:
> 
>   /sys/kernel/debug/tracing
> 
> There are many places where this older debugfs path is still used in
> code comments, selftests, examples and tools, so let's update them to
> avoid confusion.
> 
> I've broken up the series as best I could by maintainer or directory,
> and I've only sent people the patches that I think they care about to
> avoid spamming everyone.
> 
> Ross Zwisler (9):
>   tracing: always use canonical ftrace path
>   bpf: use canonical ftrace path
>   selftests/bpf: use canonical ftrace path
>   perf docs: use canonical ftrace path
>   tools/power: use canonical ftrace path
>   selftests: use canonical ftrace path
>   tools/virtio: use canonical ftrace path
>   leaking_addresses: also skip canonical ftrace path
>   tools/kvm_stat: use canonical ftrace path
> 
>  include/linux/kernel.h                        |  2 +-
>  include/linux/tracepoint.h                    |  4 ++--
>  include/uapi/linux/bpf.h                      |  8 ++++----
>  kernel/trace/Kconfig                          | 20 +++++++++----------
>  kernel/trace/kprobe_event_gen_test.c          |  2 +-
>  kernel/trace/ring_buffer.c                    |  2 +-
>  kernel/trace/synth_event_gen_test.c           |  2 +-
>  kernel/trace/trace.c                          |  2 +-
>  samples/bpf/cpustat_kern.c                    |  4 ++--
>  samples/bpf/hbm.c                             |  4 ++--
>  samples/bpf/ibumad_kern.c                     |  4 ++--
>  samples/bpf/lwt_len_hist.sh                   |  2 +-
>  samples/bpf/offwaketime_kern.c                |  2 +-
>  samples/bpf/task_fd_query_user.c              |  4 ++--
>  samples/bpf/test_lwt_bpf.sh                   |  2 +-
>  samples/bpf/test_overhead_tp_kern.c           |  4 ++--
>  samples/user_events/example.c                 |  4 ++--
>  scripts/leaking_addresses.pl                  |  1 +
>  scripts/tracing/draw_functrace.py             |  6 +++---
>  scripts/tracing/ftrace-bisect.sh              |  4 ++--
>  tools/include/uapi/linux/bpf.h                |  8 ++++----
>  tools/kvm/kvm_stat/kvm_stat                   |  2 +-
>  tools/lib/api/fs/tracing_path.c               |  4 ++--
>  tools/lib/traceevent/event-parse.c            |  8 ++++----
>  tools/perf/Documentation/perf-list.txt        |  2 +-
>  tools/perf/Documentation/perf-script-perl.txt |  2 +-
>  .../perf/Documentation/perf-script-python.txt |  4 ++--
>  tools/power/pm-graph/sleepgraph.py            |  4 ++--
>  .../x86/amd_pstate_tracer/amd_pstate_trace.py |  4 ++--
>  .../intel_pstate_tracer.py                    | 10 +++++-----
>  .../selftests/bpf/get_cgroup_id_user.c        |  2 +-
>  .../bpf/prog_tests/kprobe_multi_test.c        |  2 +-
>  .../bpf/prog_tests/task_fd_query_tp.c         |  2 +-
>  .../bpf/prog_tests/tp_attach_query.c          |  2 +-
>  .../selftests/bpf/prog_tests/trace_printk.c   |  2 +-
>  .../selftests/bpf/prog_tests/trace_vprintk.c  |  2 +-
>  .../selftests/bpf/progs/test_stacktrace_map.c |  2 +-
>  .../selftests/bpf/progs/test_tracepoint.c     |  2 +-
>  tools/testing/selftests/bpf/test_ftrace.sh    |  2 +-
>  tools/testing/selftests/bpf/test_tunnel.sh    |  8 ++++----
>  tools/testing/selftests/bpf/trace_helpers.c   |  4 ++--
>  .../testing/selftests/user_events/dyn_test.c  |  2 +-
>  .../selftests/user_events/ftrace_test.c       | 10 +++++-----
>  .../testing/selftests/user_events/perf_test.c |  8 ++++----
>  tools/testing/selftests/vm/protection_keys.c  |  4 ++--
>  tools/tracing/latency/latency-collector.c     |  2 +-
>  tools/virtio/virtio-trace/README              |  2 +-
>  tools/virtio/virtio-trace/trace-agent.c       |  2 +-
>  48 files changed, 96 insertions(+), 95 deletions(-)
> 
> -- 
> 2.39.1.456.gfc5497dd1b-goog