[FFmpeg-devel] [PATCH] lavc/qsvenc: disable CO3 for mpeg2
Carl Eugen Hoyos
ceffmpeg at gmail.com
Mon Apr 29 01:31:38 EEST 2019
2019-04-28 14:35 GMT+02:00, Zhong Li <zhong.li at intel.com>:
> Currenntly there is no any function of CO3 appled to mpeg2,
> and enabling for mpeg2 it will cause regression with some old
> libmfx libaries (see tiket #7839), so disable CO3 for mpeg2.
>
> Also add runtime version check for CO3.
>
> Signed-off-by: Zhong Li <zhong.li at intel.com>
> ---
> libavcodec/qsvenc.c | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c
> index a03ab69590..e0886aeedb 100644
> --- a/libavcodec/qsvenc.c
> +++ b/libavcodec/qsvenc.c
> @@ -750,15 +750,18 @@ FF_ENABLE_DEPRECATION_WARNINGS
> }
> #endif
> }
> +
> + if (avctx->codec_id != AV_CODEC_ID_MPEG2VIDEO &&
> QSV_RUNTIME_VERSION_ATLEAST(q->ver, 1, 11)) {
> #if QSV_HAVE_CO3
> - q->extco3.Header.BufferId = MFX_EXTBUFF_CODING_OPTION3;
> - q->extco3.Header.BufferSz = sizeof(q->extco3);
> + q->extco3.Header.BufferId = MFX_EXTBUFF_CODING_OPTION3;
> + q->extco3.Header.BufferSz = sizeof(q->extco3);
> #if QSV_HAVE_GPB
> - if (avctx->codec_id == AV_CODEC_ID_HEVC)
> - q->extco3.GPB = q->gpb ? MFX_CODINGOPTION_ON :
> MFX_CODINGOPTION_OFF;
> + if (avctx->codec_id == AV_CODEC_ID_HEVC)
> + q->extco3.GPB = q->gpb ? MFX_CODINGOPTION_ON :
> MFX_CODINGOPTION_OFF;
> #endif
> - q->extparam_internal[q->nb_extparam_internal++] = (mfxExtBuffer
> *)&q->extco3;
> + q->extparam_internal[q->nb_extparam_internal++] = (mfxExtBuffer
> *)&q->extco3;
> #endif
> + }
Please separate the cosmetic from the function change(s) to make
the patch more readable.
Thank you, Carl Eugen
More information about the ffmpeg-devel
mailing list