All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 01/24] selftests: android: ion: return Kselftest Skip code for skipped tests
@ 2018-05-05  1:13 ` shuah
  0 siblings, 0 replies; 123+ messages in thread
From: Shuah Khan (Samsung OSG) @ 2018-05-05  1:13 UTC (permalink / raw)
  To: shuah, pintu.ping; +Cc: linux-kselftest, linux-kernel

When ion test is skipped because of unmet dependencies and/or unsupported
configuration, it returns 0 which is treated as a pass by the Kselftest
framework. This leads to false positive result even when the test could
not be run.

Change it to return kselftest skip code when a test gets skipped to
clearly report that the test could not be run.

Kselftest framework SKIP code is 4 and the framework prints appropriate
messages to indicate that the test is skipped.

Signed-off-by: Shuah Khan (Samsung OSG) <shuah@kernel.org>
---
 tools/testing/selftests/android/ion/ion_test.sh | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/android/ion/ion_test.sh b/tools/testing/selftests/android/ion/ion_test.sh
index a1aff506f5e6..69e676cfc94e 100755
--- a/tools/testing/selftests/android/ion/ion_test.sh
+++ b/tools/testing/selftests/android/ion/ion_test.sh
@@ -4,6 +4,9 @@ heapsize=4096
 TCID="ion_test.sh"
 errcode=0
 
+# Kselftest framework requirement - SKIP code is 4.
+ksft_skip=4
+
 run_test()
 {
 	heaptype=$1
@@ -25,7 +28,7 @@ check_root()
 	uid=$(id -u)
 	if [ $uid -ne 0 ]; then
 		echo $TCID: must be run as root >&2
-		exit 0
+		exit $ksft_skip
 	fi
 }
 
@@ -35,7 +38,7 @@ check_device()
 	if [ ! -e $DEVICE ]; then
 		echo $TCID: No $DEVICE device found >&2
 		echo $TCID: May be CONFIG_ION is not set >&2
-		exit 0
+		exit $ksft_skip
 	fi
 }
 
-- 
2.14.1

^ permalink raw reply related	[flat|nested] 123+ messages in thread

end of thread, other threads:[~2018-05-11  6:09 UTC | newest]

Thread overview: 123+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-05  1:13 [PATCH 01/24] selftests: android: ion: return Kselftest Skip code for skipped tests Shuah Khan (Samsung OSG)
2018-05-05  1:13 ` Shuah Khan (Samsung OSG)
2018-05-05  1:13 ` shuah
2018-05-05  1:13 ` [PATCH 02/24] selftests: breakpoints: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 03/24] selftests: cpu-hotplug: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 04/24] selftests: cpufreq: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-07  5:01   ` Viresh Kumar
2018-05-07  5:01     ` Viresh Kumar
2018-05-07  5:01     ` viresh.kumar
2018-05-05  1:13 ` [PATCH 05/24] selftests: efivarfs: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 06/24] selftests: exec: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-08  0:03   ` Steve Muckle
2018-05-08  0:03     ` Steve Muckle
2018-05-08  0:03     ` smuckle.linux
2018-05-05  1:13 ` [PATCH 07/24] selftests: filesystems: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-06  8:15   ` Christian Brauner
2018-05-06  8:15     ` Christian Brauner
2018-05-06  8:15     ` christian.brauner
2018-05-05  1:13 ` [PATCH 08/24] selftests: firmware: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-09 19:27   ` Luis R. Rodriguez
2018-05-09 19:27     ` Luis R. Rodriguez
2018-05-09 19:27     ` mcgrof
2018-05-05  1:13 ` [PATCH 09/24] selftests: ftrace: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-07 15:17   ` Steven Rostedt
2018-05-07 15:17     ` Steven Rostedt
2018-05-07 15:17     ` rostedt
2018-05-08  3:29     ` Masami Hiramatsu
2018-05-08  3:29       ` Masami Hiramatsu
2018-05-08  3:29       ` mhiramat
2018-05-08  3:38   ` Masami Hiramatsu
2018-05-08  3:38     ` Masami Hiramatsu
2018-05-08  3:38     ` mhiramat
2018-05-10 20:18     ` Shuah Khan
2018-05-10 20:18       ` Shuah Khan
2018-05-10 20:18       ` shuah
2018-05-11  6:09       ` Masami Hiramatsu
2018-05-11  6:09         ` Masami Hiramatsu
2018-05-11  6:09         ` mhiramat
2018-05-05  1:13 ` [PATCH 10/24] selftests: gpio: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 11/24] selftests: intel_pstate: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 12/24] selftests: ipc: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 13/24] selftests: kmod: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-09 19:28   ` Luis R. Rodriguez
2018-05-09 19:28     ` Luis R. Rodriguez
2018-05-09 19:28     ` mcgrof
2018-05-05  1:13 ` [PATCH 14/24] selftests: kvm: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-07 11:46   ` Paolo Bonzini
2018-05-07 11:46     ` Paolo Bonzini
2018-05-07 11:46     ` pbonzini
2018-05-07 19:41     ` Shuah Khan
2018-05-07 19:41       ` Shuah Khan
2018-05-07 19:41       ` shuah
2018-05-07 20:19       ` Shuah Khan
2018-05-07 20:19         ` Shuah Khan
2018-05-07 20:19         ` shuah
2018-05-05  1:13 ` [PATCH 15/24] selftests: lib: add prime_numbers.sh test to Makefile Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 16/24] selftests: lib: return Kselftest Skip code for skipped tests Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 17/24] selftests: locking: add Makefile for locking test Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 18/24] selftests: locking: return Kselftest Skip code for skipped tests Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 19/24] selftests: media_tests: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 20/24] selftests: membarrier: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 21/24] selftests: memfd: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:52   ` Mike Kravetz
2018-05-05  1:52     ` Mike Kravetz
2018-05-05  1:52     ` mike.kravetz
2018-05-07 18:45     ` Shuah Khan
2018-05-07 18:45       ` Shuah Khan
2018-05-07 18:45       ` shuah
2018-05-07 21:16       ` Shuah Khan
2018-05-07 21:16         ` Shuah Khan
2018-05-07 21:16         ` shuah
2018-05-05  1:13 ` [PATCH 22/24] selftests: memory-hotplug: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 23/24] selftests: mqueue: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-05  1:13 ` [PATCH 24/24] selftests: net: " Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` Shuah Khan (Samsung OSG)
2018-05-05  1:13   ` shuah
2018-05-07 11:04 ` [PATCH 01/24] selftests: android: ion: " Pintu Kumar
2018-05-07 11:04   ` Pintu Kumar
2018-05-07 11:04   ` pintu.ping

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.