[Libva] [PATCH 0/2] intel-driver: add support for vaPutImage()

[Xiang, Haihao]

On Fri, 2011-09-09 at 12:57 +0800, Gwenole Beauchesne wrote: 
> Hi,
> 
> 2011/9/6 Xiang, Haihao <haihao.xiang at intel.com>:
> 
> > There is an implementation of vaPutImage on vaapi-ext branch, the
> > implementation directly uses some internal post processing functions, it
> > supports pixel format conversion between NV12, YV12 and I420. Currently
> > it doesn't work on CTG due to lack of GPU kernels on CTG. Could you try
> > this implementation?
> 
> Well, my CTG platform is not very stable so it's not suitable for
> performance evaluation. Do you think the GPU kernels need to be
> written for CTG specifically? Actually, I could build them on Gen4 but
> my CTG support code is surely broken since the outcome is a GPU hang.
> :)

I think the GPU kernels can be used for CTG, but some commands need to
be changed. However I don't give a try. 

> 
> > I also have implemented vaGetImage with this way. Testing with VLC-vaapi
> > shows that the performance on Sandybridge is good (It has the same CPU
> > usage with mplayer-vaapi when playing a 1080p video) however the
> > performance on Ironlake is bad. So I don't push the implementation of
> > vaGetImage currently.
> 
> Can we make a mix as an interim solution? e.g.
> - SNB+ shaders for vaGetImage()
> - ILK+ shaders for vaPutImage()
Yeah, this is a good idea.

> 
> Thanks,
> Gwenole.