[LTP] [PATCH 1/1] cgroup_regression_test.sh: Fix TWARN usage
Wed Feb 27 12:10:55 CET 2019
> >> +++ b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> >> @@ -408,7 +408,7 @@ test_7_2()
> >> grep -q -w "cpu" /proc/cgroups
> >> if [ $? -ne 0 -o ! -e /proc/sched_debug ]; then
> >> - tst_res TWARN "skip rest of testing due possible oops triggered by reading /proc/sched_debug"
> >> + tst_res TCONF "skip rest of testing due possible oops triggered by reading /proc/sched_debug"
> > Right, TWARN is not good (IMHO TWARN is not popular in LTP testsuites due marking test failing),
> > but if you don't mind I'd merge it with TBROK (more appropriate in this case).
> mmm...I'm NOT so sure that a FAIL should be the outcome here ... we are skipping
> if cpu is not amongst cgroups or sched_debug is NOT available...BUT at this
> point we have run (or skipped) 7 tests so far and there are 3 more to go...so
> why can't we just TCONF ?
OK I got that :) /proc/sched_debug is enabled by CONFIG_SCHED_DEBUG, os it's
kind of configuration issue. Before I thought that failing "grep -q -w "cpu"
/proc/cgroups" means failure/bug found by test, but it's not.
So I'm ok with this change.
Acked-by: Petr Vorel <email@example.com>
> (beside the fact that as I said in the cover-letter one TCONF will lead to the
> whole 10-subtest croup suite to be reported as TCONF...)
IMHO that's not true (see my reply to cover letter).
More information about the ltp