ACK: [X][SRU][PATCH 1/1] selftests/cpu-hotplug: exit with failure when test occured unexpected behaviors

[Kleber Souza]

On 12/25/18 11:41 AM, Po-Hsu Lin wrote:
> From: Li Zhijian <lizhijian at cn.fujitsu.com>
>
> BugLink: https://bugs.launchpad.net/bugs/1809699
>
> Previously, 'make run_tests -C cpu-hotplug' always PASS since cpu-on-off-test.sh
> always exits 0 even though the test got some unexpected errors like below:
> root at debian9:/home/lizhijian/chroot/linux/tools/testing/selftests/cpu-hotplug# make run_tests
> pid 878's current affinity mask: 1
> pid 878's new affinity mask: 1
> CPU online/offline summary:
> Cpus in online state: 0
>         Cpus in offline state: 0
> Limited scope test: one hotplug cpu
>         (leaves cpu in the original state):
>         online to offline to online: cpu 0
> ./cpu-on-off-test.sh: line 83: /sys/devices/system/cpu/cpu0/online: Permission denied
> offline_cpu_expect_success 0: unexpected fail
> ./cpu-on-off-test.sh: line 78: /sys/devices/system/cpu/cpu0/online: Permission denied
> online_cpu_expect_success 0: unexpected fail
> selftests: cpu-on-off-test.sh [PASS]
>
> after this patch, the test will exit with failure once it occurs some unexpected behaviors
>
> Signed-off-by: Li Zhijian <lizhijian at cn.fujitsu.com>
> Signed-off-by: Shuah Khan <shuahkh at osg.samsung.com>
> (cherry picked from commit 8fe6e53a8ed5f043ff4e6a366ebb3a641114546b)
> Signed-off-by: Po-Hsu Lin <po-hsu.lin at canonical.com>

Clean cherry-pick, limited to selftests.


Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>

> ---
>  tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh b/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
> index 98b1d65..e5666dd 100755
> --- a/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
> +++ b/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
> @@ -89,8 +89,10 @@ online_cpu_expect_success()
>  
>  	if ! online_cpu $cpu; then
>  		echo $FUNCNAME $cpu: unexpected fail >&2
> +		exit 1
>  	elif ! cpu_is_online $cpu; then
>  		echo $FUNCNAME $cpu: unexpected offline >&2
> +		exit 1
>  	fi
>  }
>  
> @@ -100,8 +102,10 @@ online_cpu_expect_fail()
>  
>  	if online_cpu $cpu 2> /dev/null; then
>  		echo $FUNCNAME $cpu: unexpected success >&2
> +		exit 1
>  	elif ! cpu_is_offline $cpu; then
>  		echo $FUNCNAME $cpu: unexpected online >&2
> +		exit 1
>  	fi
>  }
>  
> @@ -111,8 +115,10 @@ offline_cpu_expect_success()
>  
>  	if ! offline_cpu $cpu; then
>  		echo $FUNCNAME $cpu: unexpected fail >&2
> +		exit 1
>  	elif ! cpu_is_offline $cpu; then
>  		echo $FUNCNAME $cpu: unexpected offline >&2
> +		exit 1
>  	fi
>  }
>  
> @@ -122,8 +128,10 @@ offline_cpu_expect_fail()
>  
>  	if offline_cpu $cpu 2> /dev/null; then
>  		echo $FUNCNAME $cpu: unexpected success >&2
> +		exit 1
>  	elif ! cpu_is_online $cpu; then
>  		echo $FUNCNAME $cpu: unexpected offline >&2
> +		exit 1
>  	fi
>  }
>