From mboxrd@z Thu Jan 1 00:00:00 1970 From: Petr Vorel Date: Tue, 26 Feb 2019 19:42:28 +0100 Subject: [LTP] [PATCH 1/1] cgroup_regression_test.sh: Fix TWARN usage In-Reply-To: <20190215145239.51806-2-cristian.marussi@arm.com> References: <20190215145239.51806-1-cristian.marussi@arm.com> <20190215145239.51806-2-cristian.marussi@arm.com> Message-ID: <20190226184228.GA1113@dell5510> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it Hi Cristian, > Using TWARN to report that a specific sub-testcase is skipped > caused the whole cgroup_regression_test.sh to be reported as > FAILING. Using TCONF instead. Fixes: 78bfa7e78 ("cgroup: Simplify check_kernel_bug usage") My commit, sorry :(. > Signed-off-by: Cristian Marussi > --- > testcases/kernel/controllers/cgroup/cgroup_regression_test.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > diff --git a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh > index 686860923..e197f5d3f 100755 > --- a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh > +++ 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). Other option would be to use TINFO and don't skip test (we're trying to find kernel crashes, don't we), that's probably a bit crazy. Kind regards, Petr