[FFmpeg-devel] [PATCH] avcodec/mips: [loongson] mmi optimizations for VP9 put and avg functions
Shiyou Yin
yinshiyou-hf at loongson.cn
Thu Feb 21 03:55:14 EET 2019
>-----Original Message-----
>From: ffmpeg-devel-bounces at ffmpeg.org [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf Of gxw
>Sent: Tuesday, February 19, 2019 11:02 AM
>To: ffmpeg-devel at ffmpeg.org
>Cc: gxw
>Subject: [FFmpeg-devel] [PATCH] avcodec/mips: [loongson] mmi optimizations for VP9 put and avg
>functions
>
>VP9 decoding speed improved about 109.3%(from 32fps to 67fps, tested on loongson 3A3000).
>---
> libavcodec/mips/Makefile | 1 +
> libavcodec/mips/vp9_mc_mmi.c | 680 +++++++++++++++++++++++++++++++++++++
> libavcodec/mips/vp9dsp_init_mips.c | 42 +++
> libavcodec/mips/vp9dsp_mips.h | 50 +++
> 4 files changed, 773 insertions(+)
> create mode 100644 libavcodec/mips/vp9_mc_mmi.c
>
>diff --git a/libavcodec/mips/Makefile b/libavcodec/mips/Makefile
>index c827649..c5b54d5 100644
>--- a/libavcodec/mips/Makefile
>+++ b/libavcodec/mips/Makefile
>@@ -88,3 +88,4 @@ MMI-OBJS-$(CONFIG_VC1_DECODER) += mips/vc1dsp_mmi.o
> MMI-OBJS-$(CONFIG_WMV2DSP) += mips/wmv2dsp_mmi.o
> MMI-OBJS-$(CONFIG_HEVC_DECODER) += mips/hevcdsp_mmi.o
> MMI-OBJS-$(CONFIG_VP3DSP) += mips/vp3dsp_idct_mmi.o
>+MMI-OBJS-$(CONFIG_VP9_DECODER) += mips/vp9_mc_mmi.o
>diff --git a/libavcodec/mips/vp9_mc_mmi.c b/libavcodec/mips/vp9_mc_mmi.c
>new file mode 100644
>index 0000000..145bbff
>--- /dev/null
>+++ b/libavcodec/mips/vp9_mc_mmi.c
>@@ -0,0 +1,680 @@
>+/*
>+ * Copyright (c) 2019 gxw <guxiwei-hf at loongson.cn>
>+ *
>+ * This file is part of FFmpeg.
>+ *
>+ * FFmpeg is free software; you can redistribute it and/or
>+ * modify it under the terms of the GNU Lesser General Public
>+ * License as published by the Free Software Foundation; either
>+ * version 2.1 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
>+ * Lesser General Public License for more details.
>+ *
>+ * You should have received a copy of the GNU Lesser 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 "libavcodec/vp9dsp.h"
>+#include "libavutil/mips/mmiutils.h"
>+#include "vp9dsp_mips.h"
>+
>+#define GET_DATA_H_MMI \
>+ "pmaddhw %[ftmp4], %[ftmp4], %[filter1] \n\t" \
>+ "pmaddhw %[ftmp5], %[ftmp5], %[filter2] \n\t" \
>+ "paddw %[ftmp4], %[ftmp4], %[ftmp5] \n\t" \
>+ "punpckhwd %[ftmp5], %[ftmp4], %[ftmp0] \n\t" \
>+ "paddw %[ftmp4], %[ftmp4], %[ftmp5] \n\t" \
>+ "pmaddhw %[ftmp6], %[ftmp6], %[filter1] \n\t" \
>+ "pmaddhw %[ftmp7], %[ftmp7], %[filter2] \n\t" \
>+ "paddw %[ftmp6], %[ftmp6], %[ftmp7] \n\t" \
>+ "punpckhwd %[ftmp7], %[ftmp6], %[ftmp0] \n\t" \
>+ "paddw %[ftmp6], %[ftmp6], %[ftmp7] \n\t" \
>+ "punpcklwd %[srcl], %[ftmp4], %[ftmp6] \n\t" \
>+ "pmaddhw %[ftmp8], %[ftmp8], %[filter1] \n\t" \
>+ "pmaddhw %[ftmp9], %[ftmp9], %[filter2] \n\t" \
>+ "paddw %[ftmp8], %[ftmp8], %[ftmp9] \n\t" \
>+ "punpckhwd %[ftmp9], %[ftmp8], %[ftmp0] \n\t" \
>+ "paddw %[ftmp8], %[ftmp8], %[ftmp9] \n\t" \
>+ "pmaddhw %[ftmp10], %[ftmp10], %[filter1] \n\t" \
>+ "pmaddhw %[ftmp11], %[ftmp11], %[filter2] \n\t" \
>+ "paddw %[ftmp10], %[ftmp10], %[ftmp11] \n\t" \
>+ "punpckhwd %[ftmp11], %[ftmp10], %[ftmp0] \n\t" \
>+ "paddw %[ftmp10], %[ftmp10], %[ftmp11] \n\t" \
>+ "punpcklwd %[srch], %[ftmp8], %[ftmp10] \n\t"
>+
>+#define GET_DATA_V_MMI \
>+ "punpcklhw %[srcl], %[ftmp4], %[ftmp5] \n\t" \
>+ "pmaddhw %[srcl], %[srcl], %[filter10] \n\t" \
>+ "punpcklhw %[ftmp12], %[ftmp6], %[ftmp7] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter32] \n\t" \
>+ "paddw %[srcl], %[srcl], %[ftmp12] \n\t" \
>+ "punpcklhw %[ftmp12], %[ftmp8], %[ftmp9] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter54] \n\t" \
>+ "paddw %[srcl], %[srcl], %[ftmp12] \n\t" \
>+ "punpcklhw %[ftmp12], %[ftmp10], %[ftmp11] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter76] \n\t" \
>+ "paddw %[srcl], %[srcl], %[ftmp12] \n\t" \
>+ "punpckhhw %[srch], %[ftmp4], %[ftmp5] \n\t" \
>+ "pmaddhw %[srch], %[srch], %[filter10] \n\t" \
>+ "punpckhhw %[ftmp12], %[ftmp6], %[ftmp7] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter32] \n\t" \
>+ "paddw %[srch], %[srch], %[ftmp12] \n\t" \
>+ "punpckhhw %[ftmp12], %[ftmp8], %[ftmp9] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter54] \n\t" \
>+ "paddw %[srch], %[srch], %[ftmp12] \n\t" \
>+ "punpckhhw %[ftmp12], %[ftmp10], %[ftmp11] \n\t" \
>+ "pmaddhw %[ftmp12], %[ftmp12], %[filter76] \n\t" \
>+ "paddw %[srch], %[srch], %[ftmp12] \n\t"
>+
>+#define ROUND_POWER_OF_TWO_MMI \
>+ /* Add 64 */ \
>+ "li %[tmp0], 0x40 \n\t" \
>+ "dmtc1 %[tmp0], %[ftmp6] \n\t" \
>+ "punpcklwd %[ftmp6], %[ftmp6], %[ftmp6] \n\t" \
>+ "paddw %[srcl], %[srcl], %[ftmp6] \n\t" \
>+ "paddw %[srch], %[srch], %[ftmp6] \n\t" \
>+ /* Arithmetic right shift 7 bits */ \
>+ "li %[tmp0], 0x07 \n\t" \
>+ "dmtc1 %[tmp0], %[ftmp5] \n\t" \
>+ "psraw %[srcl], %[srcl], %[ftmp5] \n\t" \
>+ "psraw %[srch], %[srch], %[ftmp5] \n\t"
>+
>+#define CLIP_PIXEL_MMI \
>+ /* Staturated operation */ \
>+ "packsswh %[srcl], %[srcl], %[srch] \n\t" \
>+ "packushb %[ftmp12], %[srcl], %[ftmp0] \n\t"
Suggest moving ROUND_POWER_OF_TWO_MMI and CLIP_PIXEL_MMI into libavutil/mips/mmiutils.h and make it more universal.
More information about the ffmpeg-devel
mailing list