Skip to contentSkip to conversation

docs: ftrace: fix the wrong path of tracefs

PATCH Applied Zhaoyu Liu 3 participants 3 messages Nov 13, 2021 – Nov 15, 2021

Subsystems (3)
lore
Share & tools
lore

Zhaoyu Liu

author
#0 lore raw

PATCH 1/1 docs: ftrace: fix the wrong path of tracefs

Delete "tracing" due to it has been included in /proc/mounts. Delete "echo nop > $tracefs/tracing/current_tracer", maybe this command is redundant.

---
 Documentation/trace/ftrace.rst | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
Documentation/trace/ftrace.rst +4 -5
index 4e5b26f03d5b..b3166c4a7867 100644
@@ -2442,11 +2442,10 @@ Or this simple script!
   #!/bin/bash
 
   tracefs=`sed -ne 's/^tracefs \(.*\) tracefs.*/\1/p' /proc/mounts`
-  echo nop > $tracefs/tracing/current_tracer
-  echo 0 > $tracefs/tracing/tracing_on
-  echo $$ > $tracefs/tracing/set_ftrace_pid
-  echo function > $tracefs/tracing/current_tracer
-  echo 1 > $tracefs/tracing/tracing_on
+  echo 0 > $tracefs/tracing_on
+  echo $$ > $tracefs/set_ftrace_pid
+  echo function > $tracefs/current_tracer
+  echo 1 > $tracefs/tracing_on
   exec "$@"
 
 
Signature · 1 line
2.17.1
maintainer Acked-by on v1 1/1
#1 lore raw
Quoted 5 lines · On Sat, 13 Nov 2021 21:37:34 +0800 Zhaoyu Liu <[email protected]> wrote:

Delete "tracing" due to it has been included in /proc/mounts. Delete "echo nop > $tracefs/tracing/current_tracer", maybe this command is redundant.

Jon,

Can you take this through your tree?

Thanks,

-- Steve

Quoted 24 lines
---
 Documentation/trace/ftrace.rst | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
Documentation/trace/ftrace.rst +4 -5
index 4e5b26f03d5b..b3166c4a7867 100644
@@ -2442,11 +2442,10 @@ Or this simple script!
   #!/bin/bash
 
   tracefs=`sed -ne 's/^tracefs \(.*\) tracefs.*/\1/p' /proc/mounts`
-  echo nop > $tracefs/tracing/current_tracer
-  echo 0 > $tracefs/tracing/tracing_on
-  echo $$ > $tracefs/tracing/set_ftrace_pid
-  echo function > $tracefs/tracing/current_tracer
-  echo 1 > $tracefs/tracing/tracing_on
+  echo 0 > $tracefs/tracing_on
+  echo $$ > $tracefs/set_ftrace_pid
+  echo function > $tracefs/current_tracer
+  echo 1 > $tracefs/tracing_on
   exec "$@"
 
 
maintainer Applied
#2 lore raw
Quoted 7 lines · Steven Rostedt
Quoted 4 lines · On Sat, 13 Nov 2021 21:37:34 +0800 Zhaoyu Liu <[email protected]> wrote:

Delete "tracing" due to it has been included in /proc/mounts. Delete "echo nop > $tracefs/tracing/current_tracer", maybe this command is redundant.

Quoted 7 lines
Quoted 1 line

Jon,

Can you take this through your tree?

Applied, thanks.

jon