[FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_gblur: add x86 SIMD optimizations
Reimar.Doeffinger at gmx.de
Wed Jun 12 08:50:46 EEST 2019
On 12.06.2019, at 03:00, Ruiling Song <git at videolan.org> wrote:
> ffmpeg | branch: master | Ruiling Song <ruiling.song at intel.com> | Wed May 15 17:54:10 2019 +0800| [83f9da77684e7ea0d8e9f9712ec716424140043a] | committer: Ruiling Song
> avfilter/vf_gblur: add x86 SIMD optimizations
> The horizontal pass get ~2x performance with the patch
> under single thread.
> Tested overall performance using the command(avx2 enabled):
> ./ffmpeg -i 1080p.mp4 -vf gblur -f null /dev/null
> ./ffmpeg -i 1080p.mp4 -vf gblur=threads=1 -f null /dev/null
> For single thread, the fps improves from 43 to 60, about 40%.
> For multi-thread, the fps improves from 110 to 130, about 20%.
> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> libavfilter/gblur.h | 55 ++++++++++++
> libavfilter/vf_gblur.c | 71 +++++++--------
> libavfilter/x86/Makefile | 2 +
> libavfilter/x86/vf_gblur.asm | 185 ++++++++++++++++++++++++++++++++++++++++
> libavfilter/x86/vf_gblur_init.c | 36 ++++++++
> 5 files changed, 310 insertions(+), 39 deletions(-)
> diff --git a/libavfilter/gblur.h b/libavfilter/gblur.h
> new file mode 100644
> index 0000000000..87129801de
> --- /dev/null
> +++ b/libavfilter/gblur.h
> @@ -0,0 +1,55 @@
> + * Copyright (c) 2011 Pascal Getreuer
> + * Copyright (c) 2016 Paul B Mahol
> + *
> + * Redistribution and use in source and binary forms, with or without modification,
> + * are permitted provided that the following conditions are met:
> + *
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above
> + * copyright notice, this list of conditions and the following
> + * disclaimer in the documentation and/or other materials provided
> + * with the distribution.
Where does this license come from?
Is that even GPL-compatible?
I mean how is someone compiling ffmpeg even supposed to know
they have to put this license text in their documentation?
More information about the ffmpeg-devel