<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="h5">
> > -<br>
> > -   for ((i = 0; i < $MAX_LOOP; i++))<br>
> > -   {<br>
> > -           echo 0 > "$TRACING_PATH"/tracing_enabled<br>
> > -           echo 1 > "$TRACING_PATH"/tracing_enabled<br>
><br>
> Maybe we should keep this file but switch it to "tracing_on"?<br>
><br>
> -- Steve<br>
<br>
</div></div>Thanks for kind review the patch. As the case was created in 2010 when the two files exit together.There is another case for testing tracing_on : ftrace_tracing_on.sh.<br>
<br>
I think we need some checker to check whether tracing_on, tracing_enabled or other trace files are there, And then decide which sub stress scripts to run. And for some new trace file without test code,I'm also considering add some new stress script for them.<br></blockquote><div><br></div><div>Sounds good, feel free to format patch adding them. <br><br></div><div>To make the progress extendable, we could also take consider that reorganize the stress test to ftrace general test, then adding some function or regression testcases.<br><br></div><div>something like:<br>---------<br></div><div>tracing<br>- ftrace_test<br>   - ftrace_lib.sh<br></div><div>   - ftrace_function_test.sh<br>   - ftrace_stress_test.sh<br>   - ftrace_regression_test.sh<br>   - frace testcase directory...<br clear="all"></div></div>   - Makefile<br></div><div class="gmail_extra"><br><br><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Li Wang<br></div></div></div></div></div>
</div></div>