[FFmpeg-devel] [PATCH v2 3/3] tests/checkasm: add check for vf_exposure

[James Almer]

On 11/4/2021 1:18 AM, Wu Jianhua wrote:
> Signed-off-by: Wu Jianhua <jianhua.wu at intel.com>
> ---
>   tests/checkasm/Makefile      |  1 +
>   tests/checkasm/checkasm.c    |  3 ++
>   tests/checkasm/checkasm.h    |  1 +
>   tests/checkasm/vf_exposure.c | 62 ++++++++++++++++++++++++++++++++++++
>   tests/fate/checkasm.mak      |  1 +
>   5 files changed, 68 insertions(+)
>   create mode 100644 tests/checkasm/vf_exposure.c
> 
> diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile
> index 4ef5fa87da..7b86ffca6b 100644
> --- a/tests/checkasm/Makefile
> +++ b/tests/checkasm/Makefile
> @@ -37,6 +37,7 @@ AVFILTEROBJS-$(CONFIG_AFIR_FILTER) += af_afir.o
>   AVFILTEROBJS-$(CONFIG_BLEND_FILTER) += vf_blend.o
>   AVFILTEROBJS-$(CONFIG_COLORSPACE_FILTER) += vf_colorspace.o
>   AVFILTEROBJS-$(CONFIG_EQ_FILTER)         += vf_eq.o
> +AVFILTEROBJS-$(CONFIG_EXPOSURE_FILTER)   += vf_exposure.o
>   AVFILTEROBJS-$(CONFIG_GBLUR_FILTER)      += vf_gblur.o
>   AVFILTEROBJS-$(CONFIG_HFLIP_FILTER)      += vf_hflip.o
>   AVFILTEROBJS-$(CONFIG_THRESHOLD_FILTER)  += vf_threshold.o
> diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
> index b1353f7cbe..50961d9961 100644
> --- a/tests/checkasm/checkasm.c
> +++ b/tests/checkasm/checkasm.c
> @@ -169,6 +169,9 @@ static const struct {
>       #if CONFIG_EQ_FILTER
>           { "vf_eq", checkasm_check_vf_eq },
>       #endif
> +    #if CONFIG_EXPOSURE_FILTER
> +        { "vf_exposure", checkasm_check_vf_exposure },
> +    #endif
>       #if CONFIG_GBLUR_FILTER
>           { "vf_gblur", checkasm_check_vf_gblur },
>       #endif
> diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h
> index 68b0697d3e..b402894ad3 100644
> --- a/tests/checkasm/checkasm.h
> +++ b/tests/checkasm/checkasm.h
> @@ -78,6 +78,7 @@ void checkasm_check_utvideodsp(void);
>   void checkasm_check_v210dec(void);
>   void checkasm_check_v210enc(void);
>   void checkasm_check_vf_eq(void);
> +void checkasm_check_vf_exposure(void);
>   void checkasm_check_vf_gblur(void);
>   void checkasm_check_vf_hflip(void);
>   void checkasm_check_vf_threshold(void);
> diff --git a/tests/checkasm/vf_exposure.c b/tests/checkasm/vf_exposure.c
> new file mode 100644
> index 0000000000..14f0efed5f
> --- /dev/null
> +++ b/tests/checkasm/vf_exposure.c
> @@ -0,0 +1,62 @@
> +/*
> + * This file is part of FFmpeg.
> + *
> + * FFmpeg is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * FFmpeg is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License along
> + * with FFmpeg; if not, write to the Free Software Foundation, Inc.,
> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> + */
> +
> +#include <float.h>
> +#include <string.h>
> +#include "checkasm.h"
> +#include "libavfilter/exposure.h"
> +
> +#define PIXELS 256
> +#define BUF_SIZE (PIXELS * 4)
> +
> +#define randomize_buffers(buf, size)             \
> +    do {                                         \
> +        int j;                                   \
> +        float *tmp_buf = (float *)buf;           \
> +        for (j = 0; j < size; j++)               \
> +            tmp_buf[j] = (float)(rnd() & 0xFF); \
> +    } while (0)
> +
> +void checkasm_check_vf_exposure(void)
> +{
> +    float *dst_ref = av_malloc(BUF_SIZE);
> +    float *dst_new = av_malloc(BUF_SIZE);

Use stack for these. They are 1k bytes each.

> +    ExposureContext s;
> +
> +    s.exposure = 0.5f;
> +    s.black = 0.1f;
> +    s.scale = 1.f / (exp2f(-s.exposure) - s.black);
> +
> +    randomize_buffers(dst_ref, PIXELS);
> +    memcpy(dst_new, dst_ref, BUF_SIZE);
> +
> +    ff_exposure_init(&s);
> +
> +    if (check_func(s.exposure_func, "exposure")) {
> +        declare_func(void, float *dst, int length, float black, float scale);
> +        call_ref(dst_ref, PIXELS, s.black, s.scale);
> +        call_new(dst_new, PIXELS, s.black, s.scale);
> +        if (!float_near_abs_eps_array(dst_ref, dst_new, 0.01f, PIXELS))
> +            fail();
> +        bench_new(dst_new, PIXELS, s.black, s.scale);
> +    }
> +    report("exposure");
> +
> +    av_freep(&dst_ref);
> +    av_freep(&dst_new);
> +}
> diff --git a/tests/fate/checkasm.mak b/tests/fate/checkasm.mak
> index 6e7edbe655..4d4cd6cc88 100644
> --- a/tests/fate/checkasm.mak
> +++ b/tests/fate/checkasm.mak
> @@ -34,6 +34,7 @@ FATE_CHECKASM = fate-checkasm-aacpsdsp                                  \
>                   fate-checkasm-vf_blend                                  \
>                   fate-checkasm-vf_colorspace                             \
>                   fate-checkasm-vf_eq                                     \
> +                fate-checkasm-vf_exposure                               \
>                   fate-checkasm-vf_gblur                                  \
>                   fate-checkasm-vf_hflip                                  \
>                   fate-checkasm-vf_nlmeans                                \
>