Return MFX_ERR_INCOMPATIBLE_VIDEO_PARAM once parameter is changed #41
Conversation
|
Improper Commit Message |
sysopenci
added
Pending Developer Approval
Shao-Feng
force-pushed
the
commit_feb_13
branch
from
c7ac087 to
7bc0bcd
Compare
|
Improper Commit Message |
Shao-Feng
force-pushed
the
commit_feb_13
branch
from
7bc0bcd to
5c819a8
Compare
Shao-Feng
changed the title
|
Improper Commit Message |
Shao-Feng
force-pushed
the
commit_feb_13
branch
from
5c819a8 to
042d521
Compare
sysopenci
requested review from
dyang23,
feijiang1,
gkdeepa,
TianmiChen,
yaoxinshi-intel,
YuanjunHuang and
yyao3
|
@nanazhan @TianmiChen @lsun30 Please help review the patches |
| @@ -1050,7 +1050,8 @@ mfxStatus VideoDECODEH265::DecodeFrameCheck(mfxBitstream *bs, mfxFrameSurface1 * | |||
| { | |||
| if (!m_isFirstRun) | |||
| { | |||
| sts = MFX_WRN_VIDEO_PARAM_CHANGED; | |||
There was a problem hiding this comment.
I'm not suggesting change this logic in VPL. MFX_WRN_VIDEO_PARAM_CHANGED means the decoder detected a new sequence header. Some parameters may be changed in the new sequence. For some parameters changed, decoder can still work fine without re-init decoder. Please try to call MFXVideoDECODE_GetVideoParam(session, &newParams) to get the new parameters after MFXVideoDECODE_DecodeFrameAsync returned MFX_WRN_VIDEO_PARAM_CHANGED in Codec2.0, if color aspects really differ, then close the decoder and re-init decoder.
For AVC and HEVC decoder, once parameter including colorAspect and bitDepth is changed, MFX_ERR_INCOMPATIBLE_VIDEO_PARAM is returned. In order to inform c2-plugin to re-init decoder for parameter changes Tracked-On: OAM-130444 Signed-off-by: Shao-Feng <[email protected]>
Return MFX_ERR_INCOMPATIBLE_VIDEO_PARAM once parameter is changed
For AVC and HEVC decoder, once parameter including colorAspect and
bitDepth is changed, MFX_ERR_INCOMPATIBLE_VIDEO_PARAM is returned.
In order to inform c2-plugin to re-init decoder for parameter changes