Re: [PATCH 2/2] selftest/cpuidle: Add support for cpuidle latency measurement

[Pratik Sampat]

[..snip..]
> > +
> > +ins_mod()
> > +{
> > +ÂÂÂ if [ ! -f "$MODULE" ]; then
> > +ÂÂÂÂÂÂÂ printf "$MODULE module does not exist. Exitting\n"
> > +ÂÂÂÂÂÂÂ exit 2
>
> Please use ksft_skip code to indicate the test is being skipped.
Sure thing I'll use ksft_skip exit code instead.
> > +ÂÂÂ fi
> > +ÂÂÂ printf "Inserting $MODULE module\n\n"
> > +ÂÂÂ insmod $MODULE
> > +ÂÂÂ if [ $? != 0 ]; then
> > +ÂÂÂÂÂÂÂ printf "Insmod $MODULE failed\n"
> > +ÂÂÂÂÂÂÂ exit 2
>
> This is fine since you expect to be able to load the module.
Thanks for the review.
Pratik

[..snip..]
> >
>
> thanks,
> -- Shuah