[FFmpeg-devel] [PATCH] Convert XvMC to hwaccel v2
Reimar.Doeffinger at gmx.de
Thu Dec 19 13:40:05 CET 2013
Ivan Kalvachev <ikalvachev at gmail.com> wrote:
>In addition to the changes in the previous patch:
>- this one doesn't use the define CONFIG_MPEG_XVMC_DECODER, as the
>decoder is no longer needed and will be removed. The change involves
>- leave FF_API_XVMC around the CODEC_CAP_HWACCEL, it is not really
>- move the newly introduces decode_mb() to the end of AVHWAccel struct
>- write some explanation for pack_pblock variable.
>Now, this is the last chance for you to review the changes before they
>hit main repository and paint FATE in red. ;)
I did a cursory review and found no issues. In particular I tried to make sure there's nothing that would break VDPAU, but still it would be good if someone could check.
One small issue though: it seems to me you remove the old pixfmts, does that mean applications not updated will break?
Also what about IDCT vs. MC-only XvMC, how is that selected?
More information about the ffmpeg-devel