* [PATCH] test-media: wrap vivid code around $vivid variable
@ 2021-02-17 18:11 Helen Koike
2021-02-17 18:22 ` Hans Verkuil
0 siblings, 1 reply; 3+ messages in thread
From: Helen Koike @ 2021-02-17 18:11 UTC (permalink / raw)
To: linux-media; +Cc: hverkuil, kernel, linux-kernel, Helen Koike
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@collabora.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
-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
--
2.30.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] test-media: wrap vivid code around $vivid variable
2021-02-17 18:11 [PATCH] test-media: wrap vivid code around $vivid variable Helen Koike
@ 2021-02-17 18:22 ` Hans Verkuil
2021-02-17 18:33 ` Helen Koike
0 siblings, 1 reply; 3+ messages in thread
From: Hans Verkuil @ 2021-02-17 18:22 UTC (permalink / raw)
To: Helen Koike, linux-media; +Cc: kernel, linux-kernel
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@collabora.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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] test-media: wrap vivid code around $vivid variable
2021-02-17 18:22 ` Hans Verkuil
@ 2021-02-17 18:33 ` Helen Koike
0 siblings, 0 replies; 3+ messages in thread
From: Helen Koike @ 2021-02-17 18:33 UTC (permalink / raw)
To: Hans Verkuil, linux-media; +Cc: kernel, linux-kernel
On 2/17/21 3:22 PM, Hans Verkuil wrote:
> 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@collabora.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.
I see, thanks, ignore this patch then.
At some point I had a doubt if the script was testing vimc or vivid for
some nodes, but this was my mistake.
Thanks
Helen
>
> 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
>>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-02-17 18:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-17 18:11 [PATCH] test-media: wrap vivid code around $vivid variable Helen Koike
2021-02-17 18:22 ` Hans Verkuil
2021-02-17 18:33 ` Helen Koike
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).