Messages in this thread | | | Date | Thu, 04 Dec 2014 08:14:01 +0900 | From | Masami Hiramatsu <> | Subject | Re: [PATCH 3/3] ftracetests: Add test to test event filter logic |
| |
(2014/12/03 18:34), Steven Rostedt wrote: > On Wed, 03 Dec 2014 18:26:43 +0900 > Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote: > >> (2014/12/03 12:13), Steven Rostedt wrote: >>> From: "Steven Rostedt (Red Hat)" <rostedt@goodmis.org> >>> >>> Add a test to test the event filter logic. It currently tests the >>> following filters against sched:sched_switch event. >>> >>> ( prev_pid != 0 ) >>> ( prev_pid == 0 ) >>> ( prev_pid < 100 ) >>> ( prev_pid <= $$ ) >>> ( prev_pid > 100 ) >>> ( prev_pid >= $$ ) >>> ! ( prev_pid != 0 ) >>> ! ( prev_pid == 0 ) >>> ! ( prev_pid < 100 ) >>> ! ( prev_pid <= $$ ) >>> ! ( prev_pid > 100 ) >>> ! ( prev_pid >= $$ ) >>> ( prev_pid != 0 && next_pid > 10 ) >>> ( prev_pid != 0 || next_pid > 10 ) >>> ! ( prev_pid != 0 && next_pid > 10 ) >>> ! ( prev_pid != 0 || next_pid > 10 ) >>> ( prev_pid & 1 ) >>> ( prev_pid & 2 ) >>> ( prev_pid & 4 ) >>> ( prev_pid & 8 ) >>> ( prev_pid & 16 ) >>> ! ( prev_pid & 1 ) >>> ! ( prev_pid & 2 ) >>> ! ( prev_pid & 4 ) >>> ! ( prev_pid & 8 ) >>> ! ( prev_pid & 16 ) >>> ( next_comm ~ "ftrace-test-fil" ) >>> ( next_comm != "ftrace-test-fil" ) >>> ! ( next_comm ~ "ftrace-test-fil" ) >>> ! ( next_comm != "ftrace-test-fil" ) >>> >> >> Hmm, this uses some bash-only syntax, here is the result of checkbashisms. >> >> # checkbashisms tools/testing/selftests/ftrace/test.d/ftrace/filter.tc >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 70 (should be 'b = a'): >> if [ "$not" == '!' ]; then >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 99 (should be 'b = a'): >> if [ "$not" == '!' ]; then >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 137 (let ...): >> let x=1 >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 148 (let ...): >> let val="$pid & $x" >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 155 (should be 'b = a'): >> elif [ "$not" == '!' ]; then >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 160 (let ...): >> let x="$x << 1" >> possible bashism in tools/testing/selftests/ftrace/test.d/ftrace/filter.tc line 182 (should be 'b = a'): >> if [ "$not" == '!' ]; then >> >> to allow run this on busybox or dash, we'd better clean it. > > Do you know how to fix this?
Yes, it's easy :). As the tool output, [ a == b ] can be changed to [ a = b ], and "let ..." can be changed to "$((...))"
> >> >>> Signed-off-by: Steven Rostedt <rostedt@goodmis.org> >>> --- >> [...] >>> +# Test more complex compares (&& and !!) >>> +test_cmp2 -ne 0 -a -gt 10 "!= 0" "> 10" "&&" '' >>> +test_cmp2 -ne 0 -o -gt 10 "!= 0" "> 10" "||" '' >> >> This might better be >> test_cmp2 -ne 0 -a -gt 10 "!= 0" "&&" "> 10" '' >> test_cmp2 -ne 0 -o -gt 10 "!= 0" "||" "> 10" '' >> >> :-) > > Sure. > > OK, I'll hold off on sending this patch then till 3.20.
OK.
> > I'll still add the update to the kernel for 3.19, but the testing for > it needs work. It passes my test suite, but I don't know how to handle > the busybox limitations.
If you are using Fedora, it provides dash and busybox packages too.
Thank you,
-- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Research Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
| |