<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 20, 2015 at 6:35 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 20 October 2015 at 17:06, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>> wrote:<br>
><br>
><br>
> On 19 October 2015 at 17:16, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br>
>><br>
>> On 17 October 2015 at 00:14, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>><br>
>> wrote:<br>
>> > This patch allows to use gallium vaapi without requiring<br>
>> > a X server running for your second graphic card.<br>
>> ><br>
>> I've sent a lengthy series which should mitigate the need of some hunks.<br>
><br>
><br>
> Ok I'll wait for your patches to land before going further on this patch.<br>
> Should I expect vl_winsy_drm.c in your patches ? Not sure do understood that<br>
> part. Actually I though about having "vl_screen_create_drm" and renames<br>
> vl_screen_create to vl_screen_create_x11 (because it takes XDisplay in<br>
> params) but then I got confused because vl_winsys.h includes Xlib.h. Should<br>
> future vl_screen_create_drm be in another header, vl_drm.h ?<br>
><br>
</span>My series flattens the if GALLIUM_STATIC_TARGETS spaghetti. Although<br>
it's more of a FYI rather than "wait until they land".<br>
<br>
On the winsys_dri vs winsys_drm side - I'm not planning to do any work<br>
there, neither I did notice the Xlib.h dependency in vl_winsys.h.<br>
<br>
What I'm pondering is about having a 'proper' drm backend, although<br>
admittedly I haven't looked exactly what libva{-intel-driver,}'s<br>
definition of that is. I'd assume that moving the non-winsys specifics<br>
(from vl_winsys_dri.c) to vl_winsys.h and adding a<br>
vl_screen_texture_from_drawable() equivalent for drm (amongst others).<br>
As you can tell much of this is guesswork, so if you don't have the<br>
time and others are happy with the approach as is, feel free to<br>
ignore.<br></blockquote><div><br></div><div>A wayland backend would be nice too. Right now vainfo under wayland just crashes.</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Arnaud</div></div></div>
</div></div>