[PATCH] kselftests: add per_test_log support

[PATCH] kselftests: add per_test_log support

[Hangbin Liu]

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

Re: [PATCH] kselftests: add per_test_log support

[Hangbin Liu]

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
> 

Re: [PATCH] kselftests: add per_test_log support

[Hangbin Liu]

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
> >