diff options
| author |   Yipeng Zou <zouyipeng@huawei.com> | 2022-11-04 10:09:31 +0800 |
|---|---|---|
| committer |   Shuah Khan <skhan@linuxfoundation.org> | 2022-11-14 11:29:02 -0700 |
| commit | a1d6cd88c8973cfb08ee85722488b1d6d5d16327 (patch) | |
| tree | 3b3f59631c9e0cbcc7269eef38119aba4a9ce5df /tools/testing/selftests/ftrace | |
| parent | selftests/vDSO: Add riscv getcpu & gettimeofday test (diff) | |
| download | wireguard-linux-a1d6cd88c8973cfb08ee85722488b1d6d5d16327.tar.xz wireguard-linux-a1d6cd88c8973cfb08ee85722488b1d6d5d16327.zip | |
selftests/ftrace: event_triggers: wait longer for test_event_enable
In some platform, the schedule event may came slowly, delay 100ms can't
cover it.
I was notice that on my board which running in low cpu_freq,and this
selftests allways gose fail.
So maybe we can check more times here to wait longer.
Fixes: 43bb45da82f9 ("selftests: ftrace: Add a selftest to test event enable/disable func trigger")
Signed-off-by: Yipeng Zou <zouyipeng@huawei.com>
Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'tools/testing/selftests/ftrace')
| -rw-r--r-- | tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc index 8d26d5505808..3eea2abf68f9 100644 --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc @@ -38,11 +38,18 @@ cnt_trace() { test_event_enabled() { val=$1 + check_times=10 # wait for 10 * SLEEP_TIME at most - e=`cat $EVENT_ENABLE` - if [ "$e" != $val ]; then - fail "Expected $val but found $e" - fi + while [ $check_times -ne 0 ]; do + e=`cat $EVENT_ENABLE` + if [ "$e" == $val ]; then + return 0 + fi + sleep $SLEEP_TIME + check_times=$((check_times - 1)) + done + + fail "Expected $val but found $e" } run_enable_disable() { |