Re: [PATCH] kselftests: add per_test_log support

[Hangbin Liu <liuhangbin@gmail.com>]

Ping Shuah. Is there any update I need?

Thanks
Hangbin
On Thu, Nov 23, 2023 at 09:54:26AM +0800, Hangbin Liu wrote:
> Hi Shuah,
> 
> Any comments for this?
> 
> Thanks
> Hangbin
> On Thu, Nov 02, 2023 at 03:50:17PM +0800, Hangbin Liu wrote:
> > By default, all the test output will be printed to stdout or output.log if
> > -s supplied. The kselftest/runner.sh also supports per test log if the
> > variable per_test_logging is set. So add new option -p to set this
> > veriable. Note the -p option is conflict with -s option.
> > 
> > Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
> > ---
> >  tools/testing/selftests/run_kselftest.sh | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tools/testing/selftests/run_kselftest.sh b/tools/testing/selftests/run_kselftest.sh
> > index 92743980e553..965220a314ce 100755
> > --- a/tools/testing/selftests/run_kselftest.sh
> > +++ b/tools/testing/selftests/run_kselftest.sh
> > @@ -20,7 +20,8 @@ usage()
> >  {
> >  	cat <<EOF
> >  Usage: $0 [OPTIONS]
> > -  -s | --summary		Print summary with detailed log in output.log
> > +  -s | --summary		Print summary with detailed log in output.log (conflict with -p)
> > +  -p | --per_test_log		Print test log in /tmp with each test name (conflict with -s)
> >    -t | --test COLLECTION:TEST	Run TEST from COLLECTION
> >    -c | --collection COLLECTION	Run all tests from COLLECTION
> >    -l | --list			List the available collection:test entries
> > @@ -41,6 +42,9 @@ while true; do
> >  			logfile="$BASE_DIR"/output.log
> >  			cat /dev/null > $logfile
> >  			shift ;;
> > +		-p | --per_test_log)
> > +			per_test_logging=1
> > +			shift ;;
> >  		-t | --test)
> >  			TESTS="$TESTS $2"
> >  			shift 2 ;;
> > -- 
> > 2.41.0
> >