| Message ID | 20231102075017.4060991-1-liuhangbin@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | kselftests: add per_test_log support | expand |
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 >
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 > >
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 ;;
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(-)