linux-kernel - Re: [PATCH] test-media: wrap vivid code around $vivid variable

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c9d01579-7200-2d37-d244-92895353c842@xs4all.nl>
Date:   Wed, 17 Feb 2021 19:22:36 +0100
From:   Hans Verkuil <hverkuil@...all.nl>
To:     Helen Koike <helen.koike@...labora.com>,
        linux-media@...r.kernel.org
Cc:     kernel@...labora.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] test-media: wrap vivid code around $vivid variable

On 17/02/2021 19:11, Helen Koike wrote:
> The script was trying to load vivid and run some commands on top of it
> even when $vivid = 0.
> Wrap all vivid code under $vivid variable.
> 
> Signed-off-by: Helen Koike <helen.koike@...labora.com>
> ---
>  contrib/test/test-media | 66 ++++++++++++++++++++---------------------
>  1 file changed, 33 insertions(+), 33 deletions(-)
> 
> diff --git a/contrib/test/test-media b/contrib/test/test-media
> index 10b7e89d..8cd8bc37 100755
> --- a/contrib/test/test-media
> +++ b/contrib/test/test-media
> @@ -146,29 +146,29 @@ if [ $kmemleak -eq 1 ]; then
>  	echo clear >/sys/kernel/debug/kmemleak
>  fi
>  
> -rmmod vivid 2&>/dev/null
> -modprobe vivid n_devs=3 multiplanar=1,2,2 cache_hints=1,0,0 #allocators=0,1,1

Ah, no. Vivid is also used to test dmabuf for vim2m, vimc and vicodec tests. It
functions as the allocator for the dma buffers in that case.

So even if vivid isn't given, but only vim2m, vimc or vicodec, it should still be loaded.

It isn't needed for vidtv, so I guess it could be disabled if only vidtv is tested.

Regards,

	Hans

> -sleep 1
> +if [ $vivid -eq 1 ]; then
> +	rmmod vivid 2&>/dev/null
> +	modprobe vivid n_devs=3 multiplanar=1,2,2 cache_hints=1,0,0 #allocators=0,1,1
> +	sleep 1
>  
> -tmp=`mktemp`
> +	tmp=`mktemp`
>  
> -if ! $v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-cap ; then
> -	echo "FAIL: the vivid module failed to load" | tee -a $tmp
> -	echo "Grand Total for vivid: Succeeded: 0, Failed: 1, Warnings: 0" | tee -a $tmp
> -	echo "Final Summary: 1, Succeeded: 0, Failed: 1, Warnings: 0"
> -	exit 0
> -fi
> +	if ! $v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-cap ; then
> +		echo "FAIL: the vivid module failed to load" | tee -a $tmp
> +		echo "Grand Total for vivid: Succeeded: 0, Failed: 1, Warnings: 0" | tee -a $tmp
> +		echo "Final Summary: 1, Succeeded: 0, Failed: 1, Warnings: 0"
> +		exit 0
> +	fi
>  
> -$v4l2_ctl -z platform:vivid-000 -d vivid-000-vid-cap -i3 -v width=3840,height=2160,pixelformat=NV24
> -$v4l2_ctl -z platform:vivid-000 -d vivid-000-vid-out -o1 -x width=3840,height=2160,pixelformat=NV24
> -$v4l2_ctl -z platform:vivid-001 -d vivid-001-vid-cap -i3 -v width=3840,height=2160,pixelformat=NM16
> -$v4l2_ctl -z platform:vivid-001 -d vivid-001-vid-out -o1 -x width=3840,height=2160,pixelformat=NM16
> -$v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-cap -i3 -v width=3840,height=2160,pixelformat=NV24
> -$v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-out -o1 -x width=3840,height=2160,pixelformat=NM16
> +	$v4l2_ctl -z platform:vivid-000 -d vivid-000-vid-cap -i3 -v width=3840,height=2160,pixelformat=NV24
> +	$v4l2_ctl -z platform:vivid-000 -d vivid-000-vid-out -o1 -x width=3840,height=2160,pixelformat=NV24
> +	$v4l2_ctl -z platform:vivid-001 -d vivid-001-vid-cap -i3 -v width=3840,height=2160,pixelformat=NM16
> +	$v4l2_ctl -z platform:vivid-001 -d vivid-001-vid-out -o1 -x width=3840,height=2160,pixelformat=NM16
> +	$v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-cap -i3 -v width=3840,height=2160,pixelformat=NV24
> +	$v4l2_ctl -z platform:vivid-002 -d vivid-002-vid-out -o1 -x width=3840,height=2160,pixelformat=NM16
>  
> -echo
> +	echo
>  
> -if [ $vivid -eq 1 ]; then
>  	dmesg -n notice
>  	echo
>  	echo vivid compliance tests, contiguous planes | tee /dev/kmsg
> @@ -287,6 +287,18 @@ if [ $vivid -eq 1 ]; then
>  	echo
>  	echo
>  	echo
> +
> +	date
> +	echo
> +	echo unbind vivid | tee /dev/kmsg
> +	echo
> +	echo -n vivid.0 >/sys/bus/platform/drivers/vivid/unbind
> +	sleep $unbind_time
> +	echo
> +	echo rmmod vivid | tee /dev/kmsg
> +	echo
> +	rmmod vivid
> +	sleep $rmmod_time
>  fi
>  
>  if [ $vim2m -eq 1 ]; then
> @@ -300,7 +312,7 @@ if [ $vim2m -eq 1 ]; then
>  		echo "FAIL: the vim2m module failed to load" | tee -a $tmp
>  		echo "Grand Total for vim2m: Succeeded: 0, Failed: 1, Warnings: 0" | tee -a $tmp
>  		echo "Final Summary: 1, Succeeded: 0, Failed: 1, Warnings: 0"
> -		rmmod vivid
> +		rmmod vim2m
>  		exit 0
>  	fi
>  
> @@ -373,7 +385,7 @@ if [ $vimc -eq 1 ]; then
>  		echo "FAIL: the vimc module failed to load" | tee -a $tmp
>  		echo "Grand Total for vimc: Succeeded: 0, Failed: 1, Warnings: 0" | tee -a $tmp
>  		echo "Final Summary: 1, Succeeded: 0, Failed: 1, Warnings: 0"
> -		rmmod vivid
> +		rmmod vimc
>  		exit 0
>  	fi
>  
> @@ -467,7 +479,7 @@ if [ $vicodec -eq 1 ]; then
>  		echo "FAIL: the vicodec module failed to load" | tee -a $tmp
>  		echo "Grand Total for vicodec: Succeeded: 0, Failed: 1, Warnings: 0" | tee -a $tmp
>  		echo "Final Summary: 1, Succeeded: 0, Failed: 1, Warnings: 0"
> -		rmmod vivid
> +		rmmod vicodec
>  		exit 0
>  	fi
>  
> @@ -603,18 +615,6 @@ if [ $vicodec -eq 1 ]; then
>  	echo
>  fi
>  
> -date
> -echo
> -echo unbind vivid | tee /dev/kmsg
> -echo
> -echo -n vivid.0 >/sys/bus/platform/drivers/vivid/unbind
> -sleep $unbind_time
> -echo
> -echo rmmod vivid | tee /dev/kmsg
> -echo
> -rmmod vivid
> -sleep $rmmod_time
> -
>  if [ $vidtv -eq 1 ]; then
>  	rmmod dvb_vidtv_bridge dvb_vidtv_tuner dvb_vidtv_demod 2&>/dev/null
>  	modprobe vidtv
>