[Libva] [PATCH] driver_name_map: add nouveau and radeon for Mesa Gallium

[Sean V Kelley]

Hi Julien,

> On Dec 8, 2015, at 12:53 AM, Julien Isorce <julien.isorce at gmail.com> wrote:
> 
> Hi Sean,
> 
> Note that this patch "driver_name_map: add nouveau and radeon for Mesa Gallium" is only for VA_DISPLAY_DRM case.


Understood.

> 
> About your question, all patches regarding gallium backend for vaapi have landed: http://cgit.freedesktop.org/mesa/mesa/log/src/gallium/state_trackers/va <http://cgit.freedesktop.org/mesa/mesa/log/src/gallium/state_trackers/va>
> 
> But there are still 4 patches for nouveau driver under review, to support chunk decoding: https://github.com/CapOM/mesa/commits/st_va_nouveau_chunk_decoding <https://github.com/CapOM/mesa/commits/st_va_nouveau_chunk_decoding>


Thanks.  I can now track them.


> 
> I have only tested with Nvidia hardware (with nouveau driver so), with https://github.com/mpv-player/mpv <https://github.com/mpv-player/mpv> player and gstreamer-vaapi on videos available here: 
> http://download.blender.org/peach/bigbuckbunny_movies/ <http://download.blender.org/peach/bigbuckbunny_movies/>
> http://playready.directtaps.net/smoothstreaming/ <http://playready.directtaps.net/smoothstreaming/>
> https://github.com/malek0512/PLS/raw/master/Tests/centaur_2.mpg <https://github.com/malek0512/PLS/raw/master/Tests/centaur_2.mpg>
> http://www.h264info.com/clips.html <http://www.h264info.com/clips.html>
> 
> At least for radeon it is already ready, the gallium backend for vaapi is what AMD support and maintain. Maybe they also use the bridge http://cgit.freedesktop.org/vaapi/vdpau-driver <http://cgit.freedesktop.org/vaapi/vdpau-driver> for some comparisons. Christian would be able to give more details about it. But in long terms I think this bridge will be only used by the Nvidia proprietary driver (unless they decide to provide a vaapi backend too)


I’m happy to add both to the backend, but I would like Christian to comment if he has any concerns from a radeon perspective.

Thanks,

Sean


> 
> Regards
> Julien
> 
> 
> 
> On 3 December 2015 at 22:37, Sean V Kelley <seanvk at posteo.de <mailto:seanvk at posteo.de>> wrote:
> Hi Julien,
> 
> On 11/27/2015 12:53 AM, Julien Isorce wrote:
> > This patch allows va_getDriverName to succeed. It is useful in
> > vaInitialize to select the vaapi driver provided by Mesa Gallium:
> > lib/dri/gallium_drv_video.so
> >
> > Signed-off-by: Julien Isorce <j.isorce at samsung.com <mailto:j.isorce at samsung.com>> ---
> > va/drm/va_drm_utils.c | 2 ++ 1 file changed, 2 insertions(+)
> 
> Can you provide a status on the Gallium patch series, have they been
> merged?  What testing have you done with the gallium backend enabled?
> 
> Thanks,
> 
> Sean
> 
> 
> >
> > diff --git a/va/drm/va_drm_utils.c b/va/drm/va_drm_utils.c index
> > 71091fd..041be79 100644 --- a/va/drm/va_drm_utils.c +++
> > b/va/drm/va_drm_utils.c @@ -41,6 +41,8 @@ static const struct
> > driver_name_map g_driver_name_map[] = { { "pvrsrvkm",   8, "pvr"
> > }, // Intel UMG PVR driver { "emgd",       4, "emgd"   }, // Intel
> > ECG PVR driver { "hybrid",     6, "hybrid" }, // Intel OTC Hybrid
> > driver +    { "nouveau",    7, "gallium" }, // Mesa Gallium driver
> > +    { "radeon",     6, "gallium" }, // Mesa Gallium driver { NULL,
> > } };
> >
> >
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org <mailto:Libva at lists.freedesktop.org>
> http://lists.freedesktop.org/mailman/listinfo/libva <http://lists.freedesktop.org/mailman/listinfo/libva>
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libva/attachments/20151209/2a4360d7/attachment.html>