[LTP] [PATCH V2 4/9] ftrace_stress: skip unsupported tests

[Cyril Hrubis]

Hi!
> > > +	local p=1;
> > > +	while [ $p -lt $NR_PIDS ]; do
> > > +                local kill_pid=pid${p}
> > > +                kill -KILL ${!kill_pid}
> >                                 ^
> > 				Bashism.
> 
> I am trying to change to use this way
> 
>     local pid_var=pid${p}
>                 eval local kill_pid=\$${pid_var}
>                 tst_resm TINFO "killing ${kill_pid}"
>                 eval kill -KILL $kill_pid
>                 wait ${kill_pid}
> 
> maybe i'm still using bashism?  thanks. if it's ok
> then i will use this way.
> 
> Ideally i should find a dash to try. but i have not
> found the package in rhel.

You can always compile it from source:

http://gondor.apana.org.au/~herbert/dash/files/

And there also seems to be debian devel perl script that can check for
bashism, maybe we can start using it to check shell scripts:

https://anonscm.debian.org/cgit/collab-maint/devscripts.git/tree/scripts/checkbashisms.pl

> > > +export_pids()
> > > +{
> > > +	local p=0
> > > +	while [ $p -lt $NR_PIDS ]; do
> > > +		export pid${p}
> > > +		p=$((p + 1))
> > > +	done
> > > +}
> > >  
> > > +cd ftrace_stress/
> > 
> > Why the cd? It does not seem to be needed.
> 
> as currently we are in ftrace_test/, but sub stresses
> cases are in ftrace_test/ftrace_stress/
> 
> in test_stress(), i start the sub script using
> 
>   sh ftrace_${target}.sh &
> 
> This make the path shorter. If you don't have objection,
> I will move the cd ftrace_stress/ into test_stress(). 
> What do you think of this? 

The stress tests are installed into $DESTDIR/testcases/bin/ aren't they?

If that is the case these scripts are in $PATH and could be executed
without fiddling with full path.

-- 
Cyril Hrubis
chrubis@suse.cz