[FFmpeg-devel] Broken DNxHD encoder and ASM optimizations activation
matthieu.bouron at gmail.com
Thu Sep 20 23:29:26 CEST 2012
On Thu, Sep 20, 2012 at 10:42 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> On Thu, Sep 20, 2012 at 06:42:17PM +0200, Matthieu Bouron wrote:
> > Hello,
> > The DNxHD encoder is broken in its current state with 8bit profiles.
> > After some investigation, a regression was spotted at
> > d211547dddf9deeb34b7dda9cd80b86f3f5cd07a revision with asm optimizations
> > enabled or not.
> > This commit moves some code, which enables some optimisations, from
> > ff_MPV_commont_init_x86 to ff_MPV_encode_init_x86.
> > ff_MPV_common_init_x86 is used by ff_dct_common_init. So the encoders
> > use ff_dct_common_init won't benefit anymore from the optimized
> > quantization functions.
> > It might be a good occasion to check if this kind of regression does not
> > affect other encoders.
> yes, i too think this would be quite important
> > Also, the C version of the quantization function ff_dct_quantize_c (which
> > is used for the DNxHD 8bit profiles) is broken and produces artefacts as
> > the following sample:
> > http://samples.lolito.fr/DNXHD/test_dnxhd_120M_yuv422p.mxf
> patch posted that fixes this, there are other ways to solve it but
> this is the simplest
More information about the ffmpeg-devel