[MPlayer-dev-eng] [PATCH]Fix decoding with -vc ffvc1vdpau

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Feb 28 10:11:17 CET 2009


On Fri, Feb 27, 2009 at 10:25:05PM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> stud.uni-karlsruhe.de> writes:
> 
> > In light of future changes in FFmpeg, I'd suggest juat setting both
> > do_dr1 and do_slices to 1 one.
> > The next step would then be to remove the other special-cases that sets
> > do_dr1 based on CODEC_CAP_HWACCEL_VDPAU (if that does not work it needs
> > to be fixed).
> 
> Like the following (inlined)?
> Carl Eugen
> 
> Index: libmpcodecs/vd_ffmpeg.c
> ===================================================================
> --- libmpcodecs/vd_ffmpeg.c     (revision 28747)
> +++ libmpcodecs/vd_ffmpeg.c     (working copy)
> @@ -916,7 +916,8 @@
>          avctx->draw_horiz_band = draw_slice;
>          mp_msg(MSGT_DECVIDEO, MSGL_INFO, MSGTR_MPCODECS_XVMCAcceleratedMPEG2);
>          assert(ctx->do_dr1);//these are must to!
> -        assert(ctx->do_slices); //it is (vo_)ffmpeg bug if this fails
> +        ctx->do_slices = 1;
> +        assert(!IMGFMT_IS_XVMC(imgfmt) || ctx->do_slices);

If that assert can still fail your compiler is very, very, very broken,
so you should remove it.
Also (though you could do it in another patch) the suggestion was to set
do_dr1, too.



More information about the MPlayer-dev-eng mailing list