Re: [PATCH spice-server v3 2/2] Some automatic check on video encoders

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2017-03-22 at 16:20 +0000, Frediano Ziglio wrote:
> Stress a bit video encoders.
> This check different combination of
> - encoder type;
> - image formats;
> - image clipping (encoding partial frames);
> - handling frames split into chunks.
> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx>

> ---
>  configure.ac                |  1 +
>  server/tests/Makefile.am    |  7 +++++++
>  server/tests/video-encoders | 25 +++++++++++++++++++++++++
>  3 files changed, 33 insertions(+)
>  create mode 100755 server/tests/video-encoders
> 
> Changes since v2:
> - enable check only if --enable-extra-checks is used;
> - fix make syntax-check;
> - fix make dist.
> 
> diff --git a/configure.ac b/configure.ac
> index 42ce1c1..fa6faed 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -100,6 +100,7 @@ if test "x$enable_gstreamer" != "xno" && test
> "x$enable_gstreamer" != "x1.0"; th
>      ])
>  fi
>  AM_CONDITIONAL(HAVE_GSTREAMER, test "x$have_gstreamer_0_10" =
> "xyes" || test "x$have_gstreamer_1_0" = "xyes")
> +AM_CONDITIONAL(HAVE_GSTREAMER_1_0, test "x$have_gstreamer_1_0" =
> "xyes")
>  
>  AS_IF([test "x$enable_gstreamer" = "xyes"],
>        [AC_MSG_ERROR("GStreamer support requested but not found")],
> diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am
> index 6cd6f49..f1e28d6 100644
> --- a/server/tests/Makefile.am
> +++ b/server/tests/Makefile.am
> @@ -77,6 +77,13 @@ endif
>  TESTS = $(check_PROGRAMS)			\
>  	$(NULL)
>  
> +if ENABLE_EXTRA_CHECKS
> +if HAVE_GSTREAMER_1_0
> +TESTS += video-encoders
> +endif
> +endif
> +EXTRA_DIST += video-encoders
> +
>  noinst_LIBRARIES += \
>  	libtest-stat1.a \
>  	libtest-stat2.a \
> diff --git a/server/tests/video-encoders b/server/tests/video-
> encoders
> new file mode 100755
> index 0000000..ca955c7
> --- /dev/null
> +++ b/server/tests/video-encoders
> @@ -0,0 +1,25 @@
> +#!/bin/bash
> +
> +set -e
> +
> +base_test() {
> +    echo "Running test with options: $*"
> +    ./test-gst -i 'videotestsrc pattern=14 foreground-
> color=0x4080ff background-color=0x402000 kx=-2 ky=-4 kxy=14 kt=3
> num-buffers=100 ! video/x-raw,width=1024,height=768 ! videoconvert
> qos=false' "$@"
> +}
> +
> +# check different clippings
> +# note that due to some internal alignment we are using odd number
> for sizes
> +for clipping in '' '--clipping (10%,10%)x(409,307)'
> +do
> +    for encoder in mjpeg 'gstreamer:mjpeg --min-psnr 16'
> gstreamer:vp8 gstreamer:h264
> +    do
> +        for split in '' '--split-lines=40'
> +        do
> +            # TODO check if encoder is supported
> +            for format in 16BIT 24BIT 32BIT RGBA
> +            do
> +                base_test -f $format -e $encoder $clipping $split
> +            done
> +        done
> +    done
> +done
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]