[FFmpeg-devel] [PATCH] libavcodec/qsv.c: Issue fixed: QSV engine does not release display handler under linux platform.

[Michael Niedermayer]

On Mon, Jul 13, 2015 at 11:26:41PM +0200, Gwenole Beauchesne wrote:
> Hi,
> 
> 2015-07-13 16:22 GMT+02:00 Ivan Uskov <ivan.uskov at nablet.com>:
> > Hello All,
> >
> > Current QSV engine implementation does allocate but never does release
> > a display handler under linux platforms.
> > The attached patch solved this issue, please review.
> 
> LGTM. However, missing newline IMHO before
> ff_qsv_close_internal_session() and after QSVSession definition.

newlines added
applied

thanks


> 
> Note it would also be interesting to provide a means to import a
> VADisplay from the user through an additional function. e.g. in view
> to interop VA decode to MSDK encode and/or video processing.
> 
> Regards,
> -- 
> Gwenole Beauchesne
> Intel Corporation SAS / 2 rue de Paris, 92196 Meudon Cedex, France
> Registration Number (RCS): Nanterre B 302 456 199
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150714/40680eab/attachment.sig>