[FFmpeg-devel] [PATCH 3/3] doc/examples: add QSV decode GPU copy sample

[Fu, Linjie]

> -----Original Message-----
> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf
> Of Li, Zhong
> Sent: Wednesday, March 27, 2019 11:17
> To: FFmpeg development discussions and patches <ffmpeg-
> devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 3/3] doc/examples: add QSV decode
> GPU copy sample
> 
> > From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On
> Behalf
> > Of Michael Niedermayer
> > Sent: Wednesday, March 27, 2019 12:05 AM
> > To: FFmpeg development discussions and patches
> > <ffmpeg-devel at ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [PATCH 3/3] doc/examples: add QSV decode
> > GPU copy sample
> >
> > On Tue, Mar 26, 2019 at 01:39:49PM +0800, Linjie Fu wrote:
> > > Usage is:
> > >   ./qsvdec_gpucopy on/off input_stream output_stream
> > >
> > > For example:
> > >  - enable/disable gpu copy:
> > >   ./qsvdec_gpucopy on input.h264 output.yuv
> > >   ./qsvdec_gpucopy off input.h264 output.yuv
> > >
> > > Generate fps per second.
> > >
> > > Signed-off-by: Linjie Fu <linjie.fu at intel.com>
> > > ---
> > >  configure                     |   2 +
> > >  doc/examples/Makefile         |   1 +
> > >  doc/examples/qsvdec_gpucopy.c | 276
> > > ++++++++++++++++++++++++++++++++++
> > >  3 files changed, 279 insertions(+)
> > >  create mode 100644 doc/examples/qsvdec_gpucopy.c
> >
> > breaks examples build on arm
> >
> > In file included from src/doc/examples/qsvdec_gpucopy.c:43:0:
> > src/libavcodec/qsv.h:24:26: fatal error: mfx/mfxvideo.h: No such file or
> > directory compilation terminated.
> > make: *** [doc/examples/qsvdec_gpucopy.o] Error 1
> > make: *** Waiting for unfinished jobs....
> 
> It is caused by libmfx is not checked to configure this example in configure
> file.
> It is checked for doc/examples/qsvdec.c: qsvdec_example_deps="avcodec
> avutil libmfx h264_qsv_decoder"
> Similar checking is needed for this example.

Thanks, will update and check.

> However, I would like to suggestion to add gpu_copy option based on
> current qsvdec.c example (working in system memory path should be added
> too).

I'm not quite sure whether the example of qsvdec.c can enable the system memory path
without hwdownload.