<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 29, 2015 at 11:28 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"><div class="HOEnZb"><div class="h5">On 20 October 2015 at 17:40, Arnaud Vrac <<a href="mailto:rawoul@gmail.com">rawoul@gmail.com</a>> wrote:<br>
> On Tue, Oct 20, 2015 at 6:35 PM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On 20 October 2015 at 17:06, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>><br>
>> 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>><br>
>> > 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<br>
>> >> hunks.<br>
>> ><br>
>> ><br>
>> > Ok I'll wait for your patches to land before going further on this<br>
>> > patch.<br>
>> > Should I expect vl_winsy_drm.c in your patches ? Not sure do understood<br>
>> > 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.<br>
>> > Should<br>
>> > future vl_screen_create_drm be in another header, vl_drm.h ?<br>
>> ><br>
>> 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>
><br>
><br>
> A wayland backend would be nice too.<br>
</div></div>I'm afraid not many of us have the time and/or interest to work on<br>
that. Patches implementing it will be kindly accepted :-)<br>
<span class=""><br>
> Right now vainfo under wayland just<br>
> crashes.<br>
><br>
</span>I guess we can separate the VA_DISPLAY_foo switch statement into a<br>
separate patch which can also go into stable. Personally I don't mind<br>
either way.<br></blockquote><div><br></div><div>Julien added the VA_DISPLAY_WAYLAND in the switch in the latest patches, so at least an error is now returned instead of crashing.</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Arnaud</div></div></div>
</div></div>