linux-kernel - Re: [PATCH] selftest: gpio: remove obsolete gpio-mockup test

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 for Android: free password hash cracker in your pocket

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

Message-ID: <6910812b-b568-484f-8d3b-7f78951f5c22@csgroup.eu>
Date: Tue, 5 Mar 2024 15:52:30 +0000
From: Christophe Leroy <christophe.leroy@...roup.eu>
To: Kent Gibson <warthog618@...il.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "linux-kselftest@...r.kernel.org"
	<linux-kselftest@...r.kernel.org>, "linux-gpio@...r.kernel.org"
	<linux-gpio@...r.kernel.org>, "bartosz.golaszewski@...aro.org"
	<bartosz.golaszewski@...aro.org>, "andriy.shevchenko@...ux.intel.com"
	<andriy.shevchenko@...ux.intel.com>, "shuah@...nel.org" <shuah@...nel.org>,
	"bamv2005@...il.com" <bamv2005@...il.com>
CC: Pengfei Xu <pengfei.xu@...el.com>, Yi Lai <yi1.lai@...el.com>
Subject: Re: [PATCH] selftest: gpio: remove obsolete gpio-mockup test



Le 05/03/2024 à 01:21, Kent Gibson a écrit :
> [Vous ne recevez pas souvent de courriers de warthog618@...il.com. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]
> 
> With the removal of the ARCH_NR_GPIOS, the number of available GPIOs
> is effectively unlimited, causing the gpio-mockup module load failure
> test that overflowed the number of GPIOs to unexpectedly succeed, and
> so fail.
> 
> The test is no longer relevant so remove it.
> Promote the "no lines defined" test so there is still one load
> failure test in the basic set.
> 
> Fixes: 7b61212f2a07 ("gpiolib: Get rid of ARCH_NR_GPIOS")
> Reported-by: Pengfei Xu <pengfei.xu@...el.com>
> Reported-by: Yi Lai <yi1.lai@...el.com>
> Closes: https://lore.kernel.org/linux-gpio/ZC6OHBjdwBdT4sSb@xpf.sh.intel.com/
> Signed-off-by: Kent Gibson <warthog618@...il.com>

Acked-by: Christophe Leroy <christophe.leroy@...roup.eu>

> ---
>   tools/testing/selftests/gpio/gpio-mockup.sh | 9 +++------
>   1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/tools/testing/selftests/gpio/gpio-mockup.sh b/tools/testing/selftests/gpio/gpio-mockup.sh
> index 0d6c5f7f95d2..fc2dd4c24d06 100755
> --- a/tools/testing/selftests/gpio/gpio-mockup.sh
> +++ b/tools/testing/selftests/gpio/gpio-mockup.sh
> @@ -377,13 +377,10 @@ if [ "$full_test" ]; then
>          insmod_test "0,32,32,44,-1,22,-1,31" 32 12 22 31
>   fi
>   echo "2.  Module load error tests"
> -echo "2.1 gpio overflow"
> -# Currently: The max number of gpio(1024) is defined in arm architecture.
> -insmod_test "-1,1024"
> +echo "2.1 no lines defined"
> +insmod_test "0,0"
>   if [ "$full_test" ]; then
> -       echo "2.2 no lines defined"
> -       insmod_test "0,0"
> -       echo "2.3 ignore range overlap"
> +       echo "2.2 ignore range overlap"
>          insmod_test "0,32,0,1" 32
>          insmod_test "0,32,1,5" 32
>          insmod_test "0,32,30,35" 32
> --
> 2.39.2
>