Re: [PATCH 2/4] cyclictest: move debugfs init code to its own function

[John Kacur <jkacur@redhat.com>]

On Thu, 17 Mar 2016, Luiz Capitulino wrote:

> A function added by a future commit will want to call
> this code too.
> 
> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> ---
>  src/cyclictest/cyclictest.c | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
> 
> diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
> index 6070457..4e80831 100644
> --- a/src/cyclictest/cyclictest.c
> +++ b/src/cyclictest/cyclictest.c
> @@ -514,11 +514,8 @@ static int settracer(char *tracer)
>  	return -1;
>  }
>  
> -static void setup_tracer(void)
> +static void debugfs_prepare(void)
>  {
> -	if (!tracelimit || notrace)
> -		return;
> -
>  	if (mount_debugfs(NULL))
>  		fatal("could not mount debugfs");
>  
> @@ -533,6 +530,14 @@ static void setup_tracer(void)
>  			    "TRACERs not configured?\n", testname);
>  	} else
>  		fileprefix = procfileprefix;
> +}
> +
> +static void setup_tracer(void)
> +{
> +	if (!tracelimit || notrace)
> +		return;
> +
> +	debugfs_prepare();
>  
>  	if (kernelversion >= KV_26_33) {
>  		int ret;
> -- 
> 2.1.0
> 
> --

Signed-off-by: John Kacur <jkacur@redhat.com>