[FFmpeg-devel] [PATCH] avcodec/mips: [loongson] mmi optimizations for VP9 put and avg functions
gxw
guxiwei-hf at loongson.cn
Thu Feb 21 09:33:17 EET 2019
> 在 2019年2月21日，上午9:55，Shiyou Yin <yinshiyou-hf at loongson.cn> 写道：
>
>> -----Original Message-----
>> From: ffmpeg-devel-bounces at ffmpeg.org <mailto:ffmpeg-devel-bounces at ffmpeg.org> [mailto: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 <mailto: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.
>
Thanks, will apply.
Warmest regards.
More information about the ffmpeg-devel
mailing list