[FFmpeg-devel] [PATCH] Add VDA support to ffmpeg

[Reimar Döffinger]

On Wed, Jul 18, 2012 at 04:48:33PM +0200, Hendrik Leppkes wrote:
> On Wed, Jul 18, 2012 at 4:31 PM, Xunzhen Quan <quanxunzhen at gmail.com> wrote:
> 
> > On Wed, Jul 18, 2012 at 10:27 PM, Jean-Baptiste Kempf <jb at videolan.org
> > >wrote:
> >
> > > On Wed, Jul 18, 2012 at 09:27:52PM +0800, Xunzhen Quan wrote :
> > > > This code seem to work well for me, but I think it may need more tests.
> > >
> > > What is the difference with the existing HWAccel?
> > >
> >
> > The existing HWAccel seems not complete and I cannot find the way to make
> > it work, so I implemented a decoder instead.
> >
> >
> HWAccels need support in the application that uses them, which is why the
> ffmpeg application cannot use it, it lacks support for this (don't know
> about mplayer)

At least for some like DXVA2, VDA and VDPAU that is just a design
failure on FFmpeg's part.
And I will strongly object to patches that implement HWAccel support
but hacking lots of code into FFmpeg that is not reusable by anyone
else.
Because the only case where the current situation of needing special
code in the application is defensible is when directly displaying the
video.
For the copy-back case e.g. for on-CPU filtering there is no good reason
for exposing any of the HWAccel internals to an application.