summaryrefslogtreecommitdiff
path: root/libavcodec
diff options
context:
space:
mode:
authorWenbin Chen <wenbin.chen-at-intel.com@ffmpeg.org>2022-03-18 14:25:09 +0800
committerHaihao Xiang <haihao.xiang@intel.com>2022-04-06 17:12:26 +0800
commit40f38659d777af3e0cf2487df81f4a22884e2806 (patch)
tree9223431fefecc15bf7773716aab2aeda020f14bd /libavcodec
parentad21f2649200cf88e45b645162a367d8a6907a1f (diff)
downloadffmpeg-40f38659d777af3e0cf2487df81f4a22884e2806.tar.gz
libavcodec/qsvdec: reinit decoder according to decode() return value
FFmpeg-qsv decoder reinit codec when width and height change, but there are not only resolution change need to reinit codec. I change it to use return value from DecodeFrameAsync() to decide whether decoder need to be reinitialized. Signed-off-by: Wenbin Chen <wenbin.chen@intel.com> Signed-off-by: Guangxin Xu <guangxin.xu@intel.com>
Diffstat (limited to 'libavcodec')
-rw-r--r--libavcodec/qsvdec.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/libavcodec/qsvdec.c b/libavcodec/qsvdec.c
index de4af1754d..e03d8f2a11 100644
--- a/libavcodec/qsvdec.c
+++ b/libavcodec/qsvdec.c
@@ -631,6 +631,13 @@ static int qsv_decode(AVCodecContext *avctx, QSVContext *q,
} while (ret == MFX_WRN_DEVICE_BUSY || ret == MFX_ERR_MORE_SURFACE);
+ if (ret == MFX_ERR_INCOMPATIBLE_VIDEO_PARAM) {
+ q->reinit_flag = 1;
+ av_log(avctx, AV_LOG_DEBUG, "Video parameter change\n");
+ av_freep(&sync);
+ return 0;
+ }
+
if (ret != MFX_ERR_NONE &&
ret != MFX_ERR_MORE_DATA &&
ret != MFX_WRN_VIDEO_PARAM_CHANGED &&
@@ -783,9 +790,9 @@ static int qsv_process_data(AVCodecContext *avctx, QSVContext *q,
ret = qsv_decode_header(avctx, q, pkt, pix_fmt, &param);
- if (ret >= 0 && (q->orig_pix_fmt != ff_qsv_map_fourcc(param.mfx.FrameInfo.FourCC) ||
+ if (q->reinit_flag || (ret >= 0 && (q->orig_pix_fmt != ff_qsv_map_fourcc(param.mfx.FrameInfo.FourCC) ||
avctx->coded_width != param.mfx.FrameInfo.Width ||
- avctx->coded_height != param.mfx.FrameInfo.Height)) {
+ avctx->coded_height != param.mfx.FrameInfo.Height))) {
AVPacket zero_pkt = {0};
if (q->buffered_count) {