linux-kernel - Re: [PATCH -next v2 3/5] selftests/cpu-hotplug: Delete fault injection related code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <68b54215646146e384896e995b58bb30@huawei.com>
Date:   Mon, 5 Sep 2022 13:50:10 +0000
From:   zhaogongyi <zhaogongyi@...wei.com>
To:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-kselftest@...r.kernel.org" <linux-kselftest@...r.kernel.org>
CC:     "shuah@...nel.org" <shuah@...nel.org>,
        "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
        "akinobu.mita@...il.com" <akinobu.mita@...il.com>
Subject: Re: [PATCH -next v2 3/5] selftests/cpu-hotplug: Delete fault
 injection related code

Hi shuah,

I am so sorry! There are some mistaks in the patch that did not delete the config file, please skip the patches of v2 version.
And I have resubmit the patches with new version, please see: https://patchwork.kernel.org/project/linux-kselftest/patch/20220905133615.139026-4-zhaogongyi@huawei.com/

Thanks for your review!

Kind regards,
Gongyi

> 
> Delete fault injection related code since the module has been deleted.
> 
> Signed-off-by: Zhao Gongyi <zhaogongyi@...wei.com>
> ---
>  .../selftests/cpu-hotplug/cpu-on-off-test.sh  | 87 ++-----------------
>  1 file changed, 6 insertions(+), 81 deletions(-)
> 
> 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 bd2e791fe887..184156763ccb 100755
> --- a/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
> +++ b/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
> @@ -116,10 +116,10 @@ online_cpu_expect_fail()
> 
>  	if online_cpu $cpu 2> /dev/null; then
>  		echo $FUNCNAME $cpu: unexpected success >&2
> -		exit 1
> +		retval=1
>  	elif ! cpu_is_offline $cpu; then
>  		echo $FUNCNAME $cpu: unexpected online >&2
> -		exit 1
> +		retval=1
>  	fi
>  }
> 
> @@ -142,16 +142,14 @@ offline_cpu_expect_fail()
> 
>  	if offline_cpu $cpu 2> /dev/null; then
>  		echo $FUNCNAME $cpu: unexpected success >&2
> -		exit 1
> +		retval=1
>  	elif ! cpu_is_online $cpu; then
>  		echo $FUNCNAME $cpu: unexpected offline >&2
> -		exit 1
> +		retval=1
>  	fi
>  }
> 
> -error=-12
>  allcpus=0
> -priority=0
>  online_cpus=0
>  online_max=0
>  offline_cpus=0
> @@ -159,31 +157,20 @@ offline_max=0
>  present_cpus=0
>  present_max=0
> 
> -while getopts e:ahp: opt; do
> +while getopts ah opt; do
>  	case $opt in
> -	e)
> -		error=$OPTARG
> -		;;
>  	a)
>  		allcpus=1
>  		;;
>  	h)
> -		echo "Usage $0 [ -a ] [ -e errno ] [ -p notifier-priority ]"
> +		echo "Usage $0 [ -a ]"
>  		echo -e "\t default offline one cpu"
>  		echo -e "\t run with -a option to offline all cpus"
>  		exit
>  		;;
> -	p)
> -		priority=$OPTARG
> -		;;
>  	esac
>  done
> 
> -if ! [ "$error" -ge -4095 -a "$error" -lt 0 ]; then
> -	echo "error code must be -4095 <= errno < 0" >&2
> -	exit 1
> -fi
> -
>  prerequisite
> 
>  #
> @@ -231,66 +218,4 @@ for cpu in `hotplaggable_offline_cpus`; do
>  	online_cpu_expect_success $cpu
>  done
> 
> -#
> -# Test with cpu notifier error injection -#
> -
> -DEBUGFS=`mount -t debugfs | head -1 | awk '{ print $3 }'`
> -NOTIFIER_ERR_INJECT_DIR=$DEBUGFS/notifier-error-inject/cpu
> -
> -prerequisite_extra()
> -{
> -	msg="skip extra tests:"
> -
> -	/sbin/modprobe -q -r cpu-notifier-error-inject
> -	/sbin/modprobe -q cpu-notifier-error-inject priority=$priority
> -
> -	if [ ! -d "$DEBUGFS" ]; then
> -		echo $msg debugfs is not mounted >&2
> -		exit $ksft_skip
> -	fi
> -
> -	if [ ! -d $NOTIFIER_ERR_INJECT_DIR ]; then
> -		echo $msg cpu-notifier-error-inject module is not available >&2
> -		exit $ksft_skip
> -	fi
> -}
> -
> -prerequisite_extra
> -
> -#
> -# Offline all hot-pluggable CPUs
> -#
> -echo 0 >
> $NOTIFIER_ERR_INJECT_DIR/actions/CPU_DOWN_PREPARE/error
> -for cpu in `hotpluggable_online_cpus`; do
> -	offline_cpu_expect_success $cpu
> -done
> -
> -#
> -# Test CPU hot-add error handling (offline => online) -# -echo $error >
> $NOTIFIER_ERR_INJECT_DIR/actions/CPU_UP_PREPARE/error
> -for cpu in `hotplaggable_offline_cpus`; do
> -	online_cpu_expect_fail $cpu
> -done
> -
> -#
> -# Online all hot-pluggable CPUs
> -#
> -echo 0 > $NOTIFIER_ERR_INJECT_DIR/actions/CPU_UP_PREPARE/error
> -for cpu in `hotplaggable_offline_cpus`; do
> -	online_cpu_expect_success $cpu
> -done
> -
> -#
> -# Test CPU hot-remove error handling (online => offline) -# -echo $error >
> $NOTIFIER_ERR_INJECT_DIR/actions/CPU_DOWN_PREPARE/error
> -for cpu in `hotpluggable_online_cpus`; do
> -	offline_cpu_expect_fail $cpu
> -done
> -
> -echo 0 >
> $NOTIFIER_ERR_INJECT_DIR/actions/CPU_DOWN_PREPARE/error
> -/sbin/modprobe -q -r cpu-notifier-error-inject
> -
>  exit $retval
> --
> 2.17.1