[FFmpeg-devel] [PATCH v2 2/3] checkasm: add h264 chroma test
Andreas Rheinhardt
andreas.rheinhardt at outlook.com
Thu Aug 5 18:52:59 EEST 2021
J. Dekker:
> Signed-off-by: J. Dekker <jdek at itanimul.li>
> ---
> tests/checkasm/Makefile | 1 +
> tests/checkasm/checkasm.c | 3 +
> tests/checkasm/checkasm.h | 1 +
> tests/checkasm/h264chroma.c | 109 ++++++++++++++++++++++++++++++++++++
> tests/fate/checkasm.mak | 1 +
> 5 files changed, 115 insertions(+)
> create mode 100644 tests/checkasm/h264chroma.c
>
> diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile
> index 4ef5fa87da..41222c3827 100644
> --- a/tests/checkasm/Makefile
> +++ b/tests/checkasm/Makefile
> @@ -6,6 +6,7 @@ AVCODECOBJS-$(CONFIG_BSWAPDSP) += bswapdsp.o
> AVCODECOBJS-$(CONFIG_FLACDSP) += flacdsp.o
> AVCODECOBJS-$(CONFIG_FMTCONVERT) += fmtconvert.o
> AVCODECOBJS-$(CONFIG_G722DSP) += g722dsp.o
> +AVCODECOBJS-$(CONFIG_H264CHROMA) += h264chroma.o
> AVCODECOBJS-$(CONFIG_H264DSP) += h264dsp.o
> AVCODECOBJS-$(CONFIG_H264PRED) += h264pred.o
> AVCODECOBJS-$(CONFIG_H264QPEL) += h264qpel.o
> diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
> index b1353f7cbe..154c4a5c01 100644
> --- a/tests/checkasm/checkasm.c
> +++ b/tests/checkasm/checkasm.c
> @@ -104,6 +104,9 @@ static const struct {
> #if CONFIG_G722DSP
> { "g722dsp", checkasm_check_g722dsp },
> #endif
> + #if CONFIG_H264CHROMA
> + { "h264chroma", checkasm_check_h264chroma },
> + #endif
> #if CONFIG_H264DSP
> { "h264dsp", checkasm_check_h264dsp },
> #endif
> diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h
> index 68b0697d3e..ac2f22af05 100644
> --- a/tests/checkasm/checkasm.h
> +++ b/tests/checkasm/checkasm.h
> @@ -56,6 +56,7 @@ void checkasm_check_flacdsp(void);
> void checkasm_check_float_dsp(void);
> void checkasm_check_fmtconvert(void);
> void checkasm_check_g722dsp(void);
> +void checkasm_check_h264chroma(void);
> void checkasm_check_h264dsp(void);
> void checkasm_check_h264pred(void);
> void checkasm_check_h264qpel(void);
> diff --git a/tests/checkasm/h264chroma.c b/tests/checkasm/h264chroma.c
> new file mode 100644
> index 0000000000..1f9773e345
> --- /dev/null
> +++ b/tests/checkasm/h264chroma.c
> @@ -0,0 +1,109 @@
> +/*
> + * Copyright (c) 2015 Henrik Gramner
> + * Copyright (c) 2021 J. Dekker
> + *
> + * 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 <string.h>
> +#include "checkasm.h"
> +#include "libavcodec/avcodec.h"
What is this header needed for?
> +#include "libavcodec/h264chroma.h"
> +#include "libavutil/common.h"
> +#include "libavutil/internal.h"
And these two?
> +#include "libavutil/intreadwrite.h"
> +#include "libavutil/mem_internal.h"
> +
> +// static const int codec_ids[4] = { AV_CODEC_ID_H264, AV_CODEC_ID_VP8, AV_CODEC_ID_RV40, AV_CODEC_ID_SVQ3 };
> +static const uint32_t pixel_mask[3] = { 0xffffffff, 0x01ff01ff, 0x03ff03ff };
> +
> +#define SIZEOF_PIXEL ((bit_depth + 7) / 8)
> +#define BUF_SIZE (3 * 16 * 17)
> +
> +#define randomize_buffers() \
> + do { \
> + uint32_t mask = pixel_mask[bit_depth - 8]; \
> + int i; \
> + for (i = 0; i < BUF_SIZE; i += 4) { \
> + uint32_t r = rnd() & mask; \
> + AV_WN32A(buf0 + i, r); \
> + AV_WN32A(buf1 + i, r); \
> + } \
> + } while (0)
> +
> +#define src0 (buf0 + 4 * 16) /* Offset to allow room for top and left */
> +#define src1 (buf1 + 4 * 16)
> +
> +static void check_avg(void)
> +{
> + int i, bit_depth, x, y;
> + LOCAL_ALIGNED_16(uint8_t, buf0, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, buf1, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, dst0, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, dst1, [BUF_SIZE]);
> + H264ChromaContext h;
> +
> + declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, uint8_t *src, ptrdiff_t stride, int h, int x, int y);
> + for (bit_depth = 8; bit_depth <= 10; bit_depth++) {
> + ff_h264chroma_init(&h, bit_depth);
> + for (i = 0; i < 4; i++) {
> + if (check_func(h.avg_h264_chroma_pixels_tab[i], "avg_chroma_mc%d_%d", 1 << (3 - i), bit_depth)) {
> + randomize_buffers();
> + x = rnd() & 0x7; y = rnd() & 0x7;
> + call_ref(dst0, src0, 8, 4, x, y);
> + call_new(dst1, src1, 8, 4, x, y);
> + if (memcmp(buf0, buf1, BUF_SIZE))
> + fail();
> + bench_new(dst1, src1, 8, 4, x, y);
> + }
> + }
> + }
> +}
> +
> +static void check_put(void)
> +{
> + int i, bit_depth, x, y;
> + LOCAL_ALIGNED_16(uint8_t, buf0, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, buf1, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, dst0, [BUF_SIZE]);
> + LOCAL_ALIGNED_16(uint8_t, dst1, [BUF_SIZE]);
> + H264ChromaContext h;
> +
> + declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, uint8_t *src, ptrdiff_t stride, int h, int x, int y);
> + for (bit_depth = 8; bit_depth <= 10; bit_depth++) {
> + ff_h264chroma_init(&h, bit_depth);
> + for (i = 0; i < 4; i++) {
> + if (check_func(h.put_h264_chroma_pixels_tab[i], "put_chroma_mc%d_%d", 1 << (3 - i), bit_depth)) {
> + randomize_buffers();
> + x = rnd() & 0x7; y = rnd() & 0x7;
> + call_ref(dst0, src0, 8, 4, x, y);
> + call_new(dst1, src1, 8, 4, x, y);
> + if (memcmp(buf0, buf1, BUF_SIZE))
> + fail();
> + bench_new(dst1, src1, 8, 4, x, y);
> + }
> + }
> + }
> +}
> +
> +void checkasm_check_h264chroma(void)
> +{
> + check_avg();
> + report("avg");
> + check_put();
> + report("put");
> +}
> diff --git a/tests/fate/checkasm.mak b/tests/fate/checkasm.mak
> index cec6d28286..449f8f89f9 100644
> --- a/tests/fate/checkasm.mak
> +++ b/tests/fate/checkasm.mak
> @@ -11,6 +11,7 @@ FATE_CHECKASM = fate-checkasm-aacpsdsp \
> fate-checkasm-float_dsp \
> fate-checkasm-fmtconvert \
> fate-checkasm-g722dsp \
> + fate-checkasm-h264chroma \
> fate-checkasm-h264dsp \
> fate-checkasm-h264pred \
> fate-checkasm-h264qpel \
>
More information about the ffmpeg-devel
mailing list