lkml.org 
[lkml]   [2014]   [Nov]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ftracetest: Take the first debugfs mount found
(2014/11/01 7:06), Steven Rostedt wrote:
>
> Running ftracetests on a box that mounted debugfs in two locations
> made the ftracetests fail. This is because the tests uses a grep
> of debugfs from the /proc/mounts file to find the debugfs mount
> point, and then appends "/tracing" to that string to get the tracing
> directory.
>
> If the debugfs directory is mounted twice, then that grep will return
> two answers and appending "/tracing" to a string with two lines will
> not work.
>
> Use "head -1" to only take the first mount point found.

Indeed :)

>
> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>

Thank you!


> ---
> tools/testing/selftests/ftrace/ftracetest | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ftrace/ftracetest b/tools/testing/selftests/ftrace/ftracetest
> index a8f81c782856..515247601df4 100755
> --- a/tools/testing/selftests/ftrace/ftracetest
> +++ b/tools/testing/selftests/ftrace/ftracetest
> @@ -82,7 +82,7 @@ parse_opts() { # opts
> }
>
> # Parameters
> -DEBUGFS_DIR=`grep debugfs /proc/mounts | cut -f2 -d' '`
> +DEBUGFS_DIR=`grep debugfs /proc/mounts | cut -f2 -d' ' | head -1`
> TRACING_DIR=$DEBUGFS_DIR/tracing
> TOP_DIR=`absdir $0`
> TEST_DIR=$TOP_DIR/test.d
>


--
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@hitachi.com




\
 
 \ /
  Last update: 2014-11-03 10:01    [W:0.032 / U:0.220 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site