Re: [FFmpeg-devel] [PATCH 2/5] lavc/qsvdec: Replace current parser with MFXVideoDECODE_DecodeHeader()

2019-01-25 Thread Hendrik Leppkes
On Fri, Jan 25, 2019 at 11:43 AM Li, Zhong wrote: > avctx->pix_fmt should be get from MFXVideoDECODE_DecodeHeader() but it is > needed to init MSDK session, and looks like a deadlock, :( > One solution is that we assume avctx->format has been parsed somewhere else > (such as

Re: [FFmpeg-devel] [PATCH 2/5] lavc/qsvdec: Replace current parser with MFXVideoDECODE_DecodeHeader()

2019-01-25 Thread Li, Zhong
> > +ret = MFXVideoDECODE_DecodeHeader(q->session, , ); > This function may potentially return MFX_ERR_MORE_DATA if provided > bitstream don't contain full header. I am not sure whether ffmpeg will > guarantee that... And the decoding error reported by Artie suggests that > something is wrong

Re: [FFmpeg-devel] [PATCH 2/5] lavc/qsvdec: Replace current parser with MFXVideoDECODE_DecodeHeader()

2019-01-24 Thread Rogozhkin, Dmitry V
On Thu, 2019-01-24 at 21:43 +, Rogozhkin, Dmitry V wrote: > On Mon, 2019-01-21 at 20:41 +0800, Zhong Li wrote: > > Using MSDK parser can improve qsv decoder pass rate in some cases > > (E.g: > > sps declares a wrong level_idc, smaller than it should be). > > And it is necessary for adding new

Re: [FFmpeg-devel] [PATCH 2/5] lavc/qsvdec: Replace current parser with MFXVideoDECODE_DecodeHeader()

2019-01-24 Thread Rogozhkin, Dmitry V
On Mon, 2019-01-21 at 20:41 +0800, Zhong Li wrote: > Using MSDK parser can improve qsv decoder pass rate in some cases > (E.g: > sps declares a wrong level_idc, smaller than it should be). > And it is necessary for adding new qsv decoders such as MJPEG and VP9 > since current parser can't provide

[FFmpeg-devel] [PATCH 2/5] lavc/qsvdec: Replace current parser with MFXVideoDECODE_DecodeHeader()

2019-01-21 Thread Zhong Li
Using MSDK parser can improve qsv decoder pass rate in some cases (E.g: sps declares a wrong level_idc, smaller than it should be). And it is necessary for adding new qsv decoders such as MJPEG and VP9 since current parser can't provide enough information. Actually using