diff mbox series

selftests/ftrace: Update comment and clean up

Message ID 20230311124151.43691-1-p4ranlee@gmail.com (mailing list archive)
State New
Headers show
Series selftests/ftrace: Update comment and clean up | expand

Commit Message

Paran Lee March 11, 2023, 12:41 p.m. UTC
Update renamed function comment
and clean up indent from 4 space to tab.

Signed-off-by: Paran Lee <p4ranlee@gmail.com>
---
 .../testing/selftests/ftrace/test.d/functions | 202 +++++++++---------
 1 file changed, 101 insertions(+), 101 deletions(-)

Comments

Steven Rostedt March 13, 2023, 2:30 p.m. UTC | #1
On Sat, 11 Mar 2023 21:41:53 +0900
paranlee <p4ranlee@gmail.com> wrote:

> Update renamed function comment
> and clean up indent from 4 space to tab.

This isn't C code, it's a shell script where editors do things differently.

I only care that the file is constant in its indenting.

I won't nack this, but I won't ack it nor take it myself.

-- Steve
Paran Lee March 15, 2023, 4:09 a.m. UTC | #2
2023-03-13 오후 11:30에 Steven Rostedt 이(가) 쓴 글:
> On Sat, 11 Mar 2023 21:41:53 +0900
> paranlee <p4ranlee@gmail.com> wrote:
> 
>> Update renamed function comment
>> and clean up indent from 4 space to tab.
> 
> This isn't C code, it's a shell script where editors do things differently.
> 
> I only care that the file is constant in its indenting.
> 
> I won't nack this, but I won't ack it nor take it myself.
> 
> -- Steve

Hi Steve, thanks for the code review,
it's nice to me have solid guidelines a tool related script works.

The first thing I noticed was that the a comment where "_do_fork was
changed to kernel_fork" instead of "kernel_clone",
so I fixed that as well.

Would it be better to send that typo fix along with the other fixes?

BR
Paran Lee
Steven Rostedt March 15, 2023, 1:47 p.m. UTC | #3
On Wed, 15 Mar 2023 13:09:51 +0900
Paran Lee <p4ranlee@gmail.com> wrote:

> The first thing I noticed was that the a comment where "_do_fork was
> changed to kernel_fork" instead of "kernel_clone",
> so I fixed that as well.
> 
> Would it be better to send that typo fix along with the other fixes?

Yes. And I'm glad I didn't ack this patch. A patch changing formatting
should *never* include any other changes. No typo fixes or what not. It
makes it impossible to review what exactly changed.

One reason I hate format changes is that they can sneak other changes in
undetected.

-- Steve
diff mbox series

Patch

diff --git a/tools/testing/selftests/ftrace/test.d/functions b/tools/testing/selftests/ftrace/test.d/functions
index 779f3e62ec90..d0946cccb8f0 100644
--- a/tools/testing/selftests/ftrace/test.d/functions
+++ b/tools/testing/selftests/ftrace/test.d/functions
@@ -1,195 +1,195 @@ 
 clear_trace() { # reset trace output
-    echo > trace
+	echo > trace
 }
 
 disable_tracing() { # stop trace recording
-    echo 0 > tracing_on
+	echo 0 > tracing_on
 }
 
 enable_tracing() { # start trace recording
-    echo 1 > tracing_on
+	echo 1 > tracing_on
 }
 
 reset_tracer() { # reset the current tracer
-    echo nop > current_tracer
+	echo nop > current_tracer
 }
 
 reset_trigger_file() {
-    # remove action triggers first
-    grep -H ':on[^:]*(' $@ |
-    while read line; do
-        cmd=`echo $line | cut -f2- -d: | cut -f1 -d"["`
+	# remove action triggers first
+	grep -H ':on[^:]*(' $@ |
+	while read line; do
+		cmd=`echo $line | cut -f2- -d: | cut -f1 -d"["`
 	file=`echo $line | cut -f1 -d:`
 	echo "!$cmd" >> $file
-    done
-    grep -Hv ^# $@ |
-    while read line; do
-        cmd=`echo $line | cut -f2- -d: | cut -f1 -d"["`
+	done
+	grep -Hv ^# $@ |
+	while read line; do
+		cmd=`echo $line | cut -f2- -d: | cut -f1 -d"["`
 	file=`echo $line | cut -f1 -d:`
 	echo "!$cmd" > $file
-    done
+	done
 }
 
 reset_trigger() { # reset all current setting triggers
-    if [ -d events/synthetic ]; then
-        reset_trigger_file events/synthetic/*/trigger
-    fi
-    reset_trigger_file events/*/*/trigger
+	if [ -d events/synthetic ]; then
+		reset_trigger_file events/synthetic/*/trigger
+	fi
+	reset_trigger_file events/*/*/trigger
 }
 
 reset_events_filter() { # reset all current setting filters
-    grep -v ^none events/*/*/filter |
-    while read line; do
+	grep -v ^none events/*/*/filter |
+	while read line; do
 	echo 0 > `echo $line | cut -f1 -d:`
-    done
+	done
 }
 
 reset_ftrace_filter() { # reset all triggers in set_ftrace_filter
-    if [ ! -f set_ftrace_filter ]; then
-      return 0
-    fi
-    echo > set_ftrace_filter
-    grep -v '^#' set_ftrace_filter | while read t; do
+	if [ ! -f set_ftrace_filter ]; then
+	  return 0
+	fi
+	echo > set_ftrace_filter
+	grep -v '^#' set_ftrace_filter | while read t; do
 	tr=`echo $t | cut -d: -f2`
 	if [ "$tr" = "" ]; then
-	    continue
+		continue
 	fi
 	if ! grep -q "$t" set_ftrace_filter; then
 		continue;
 	fi
 	name=`echo $t | cut -d: -f1 | cut -d' ' -f1`
 	if [ $tr = "enable_event" -o $tr = "disable_event" ]; then
-	    tr=`echo $t | cut -d: -f2-4`
-	    limit=`echo $t | cut -d: -f5`
+		tr=`echo $t | cut -d: -f2-4`
+		limit=`echo $t | cut -d: -f5`
 	else
-	    tr=`echo $t | cut -d: -f2`
-	    limit=`echo $t | cut -d: -f3`
+		tr=`echo $t | cut -d: -f2`
+		limit=`echo $t | cut -d: -f3`
 	fi
 	if [ "$limit" != "unlimited" ]; then
-	    tr="$tr:$limit"
+		tr="$tr:$limit"
 	fi
 	echo "!$name:$tr" > set_ftrace_filter
-    done
+	done
 }
 
 disable_events() {
-    echo 0 > events/enable
+	echo 0 > events/enable
 }
 
 clear_synthetic_events() { # reset all current synthetic events
-    grep -v ^# synthetic_events |
-    while read line; do
-        echo "!$line" >> synthetic_events
-    done
+	grep -v ^# synthetic_events |
+	while read line; do
+		echo "!$line" >> synthetic_events
+	done
 }
 
 clear_dynamic_events() { # reset all current dynamic events
-    again=1
-    stop=1
-    # loop mulitple times as some events require other to be removed first
-    while [ $again -eq 1 ]; do
+	again=1
+	stop=1
+	# loop mulitple times as some events require other to be removed first
+	while [ $again -eq 1 ]; do
 	stop=$((stop+1))
 	# Prevent infinite loops
 	if [ $stop -gt 10 ]; then
-	    break;
+		break;
 	fi
 	again=2
 	grep -v '^#' dynamic_events|
 	while read line; do
-	    del=`echo $line | sed -e 's/^.\([^ ]*\).*/-\1/'`
-	    if ! echo "$del" >> dynamic_events; then
+		del=`echo $line | sed -e 's/^.\([^ ]*\).*/-\1/'`
+		if ! echo "$del" >> dynamic_events; then
 		again=1
-	    fi
+		fi
+	done
 	done
-    done
 }
 
 initialize_ftrace() { # Reset ftrace to initial-state
 # As the initial state, ftrace will be set to nop tracer,
 # no events, no triggers, no filters, no function filters,
 # no probes, and tracing on.
-    disable_tracing
-    reset_tracer
-    reset_trigger
-    reset_events_filter
-    reset_ftrace_filter
-    disable_events
-    clear_dynamic_events
-    [ -f set_event_pid ] && echo > set_event_pid
-    [ -f set_ftrace_pid ] && echo > set_ftrace_pid
-    [ -f set_ftrace_notrace ] && echo > set_ftrace_notrace
-    [ -f set_graph_function ] && echo | tee set_graph_*
-    [ -f stack_trace_filter ] && echo > stack_trace_filter
-    [ -f kprobe_events ] && echo > kprobe_events
-    [ -f uprobe_events ] && echo > uprobe_events
-    [ -f synthetic_events ] && echo > synthetic_events
-    [ -f snapshot ] && echo 0 > snapshot
+	disable_tracing
+	reset_tracer
+	reset_trigger
+	reset_events_filter
+	reset_ftrace_filter
+	disable_events
+	clear_dynamic_events
+	[ -f set_event_pid ] && echo > set_event_pid
+	[ -f set_ftrace_pid ] && echo > set_ftrace_pid
+	[ -f set_ftrace_notrace ] && echo > set_ftrace_notrace
+	[ -f set_graph_function ] && echo | tee set_graph_*
+	[ -f stack_trace_filter ] && echo > stack_trace_filter
+	[ -f kprobe_events ] && echo > kprobe_events
+	[ -f uprobe_events ] && echo > uprobe_events
+	[ -f synthetic_events ] && echo > synthetic_events
+	[ -f snapshot ] && echo 0 > snapshot
 
 # Stop tracing while reading the trace file by default, to prevent
 # the test results while checking it and to avoid taking a long time
 # to check the result.
-    [ -f options/pause-on-trace ] && echo 1 > options/pause-on-trace
+	[ -f options/pause-on-trace ] && echo 1 > options/pause-on-trace
 
-    clear_trace
-    enable_tracing
+	clear_trace
+	enable_tracing
 }
 
 finish_ftrace() {
-    initialize_ftrace
+	initialize_ftrace
 # And recover it to default.
-    [ -f options/pause-on-trace ] && echo 0 > options/pause-on-trace
+	[ -f options/pause-on-trace ] && echo 0 > options/pause-on-trace
 }
 
 check_requires() { # Check required files and tracers
-    for i in "$@" ; do
+	for i in "$@" ; do
 	p=${i%:program}
-        r=${i%:README}
-        t=${i%:tracer}
+		r=${i%:README}
+		t=${i%:tracer}
 	if [ $p != $i ]; then
-	    if ! which $p ; then
-                echo "Required program $p is not found."
-                exit_unresolved
-	    fi
-        elif [ $t != $i ]; then
-            if ! grep -wq $t available_tracers ; then
-                echo "Required tracer $t is not configured."
-                exit_unsupported
-            fi
-        elif [ "$r" != "$i" ]; then
-            if ! grep -Fq "$r" README ; then
-                echo "Required feature pattern \"$r\" is not in README."
-                exit_unsupported
-            fi
-        elif [ ! -e $i ]; then
-            echo "Required feature interface $i doesn't exist."
-            exit_unsupported
-        fi
-    done
+		if ! which $p ; then
+				echo "Required program $p is not found."
+				exit_unresolved
+		fi
+		elif [ $t != $i ]; then
+			if ! grep -wq $t available_tracers ; then
+				echo "Required tracer $t is not configured."
+				exit_unsupported
+			fi
+		elif [ "$r" != "$i" ]; then
+			if ! grep -Fq "$r" README ; then
+				echo "Required feature pattern \"$r\" is not in README."
+				exit_unsupported
+			fi
+		elif [ ! -e $i ]; then
+			echo "Required feature interface $i doesn't exist."
+			exit_unsupported
+		fi
+	done
 }
 
 LOCALHOST=127.0.0.1
 
 yield() {
-    ping $LOCALHOST -c 1 || sleep .001 || usleep 1 || sleep 1
+	ping $LOCALHOST -c 1 || sleep .001 || usleep 1 || sleep 1
 }
 
 # The fork function in the kernel was renamed from "_do_fork" to
-# "kernel_fork". As older tests should still work with older kernels
+# "kernel_clone". As older tests should still work with older kernels
 # as well as newer kernels, check which version of fork is used on this
 # kernel so that the tests can use the fork function for the running kernel.
 FUNCTION_FORK=`(if grep '\bkernel_clone\b' /proc/kallsyms > /dev/null; then
-                echo kernel_clone; else echo '_do_fork'; fi)`
+				echo kernel_clone; else echo '_do_fork'; fi)`
 
 # Since probe event command may include backslash, explicitly use printf "%s"
 # to NOT interpret it.
 ftrace_errlog_check() { # err-prefix command-with-error-pos-by-^ command-file
-    pos=$(printf "%s" "${2%^*}" | wc -c) # error position
-    command=$(printf "%s" "$2" | tr -d ^)
-    echo "Test command: $command"
-    echo > error_log
-    (! printf "%s" "$command" >> "$3" ) 2> /dev/null
-    grep "$1: error:" -A 3 error_log
-    N=$(tail -n 1 error_log | wc -c)
-    # "  Command: " and "^\n" => 13
-    test $(expr 13 + $pos) -eq $N
+	pos=$(printf "%s" "${2%^*}" | wc -c) # error position
+	command=$(printf "%s" "$2" | tr -d ^)
+	echo "Test command: $command"
+	echo > error_log
+	(! printf "%s" "$command" >> "$3" ) 2> /dev/null
+	grep "$1: error:" -A 3 error_log
+	N=$(tail -n 1 error_log | wc -c)
+	# "  Command: " and "^\n" => 13
+	test $(expr 13 + $pos) -eq $N
 }