Re: [Gnash-dev] Flash HD (H.264) video decoding acceleration

[Benjamin Wolsey]

> The agglomerated patch is available in attachment and at:
> <http://www.splitted-desktop.com/~gbeauchesne/gnash-vaapi/>
> 
> This is based on bzr rev 11514. I have tried to be the less intrusive as 
> possible. The resulting patch, without new files, is under 1000 lines. 
> It's not really clean yet as some dead code and debug code remain.
> 
> NOTE: it's also possible to use the AGG renderer but reading pixels back 
> from GPU, doing color space conversion and other things, and then 
> transfering the things back to GPU is an ineffective approach. You have a 
> gain because the encoded bitstream is submitted to the HW decoder. Its job 
> is to decode and display the decoded frames so everything is kept on the 
> GPU/VPU side.

I will certainly review parts of this patch. It makes changes to
GnashImage that look relatively sensible but (at a brief glance) appear
to duplicate what BitmapInfo does. That's not necessarily bad as it
might open up a way to improve BitmapInfo.

I won't be reviewing the OGL parts as I don't know enough about it.

The size of the patch almost certainly means that copyright assignment
is necessary before it can be applied, and only Rob Savoye can sort that
out.

I'll try to review by tomorrow.

bwy
--
The current release of Gnash is 0.8.5
http://www.gnu.org/software/gnash/

Benjamin Wolsey, Software Developer - http://benjaminwolsey.de

signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil