<div dir="ltr">Hi, <div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 28, 2014 at 6:09 PM, Giovanni Campagna <span dir="ltr"><<a href="mailto:scampa.giovanni@gmail.com" target="_blank">scampa.giovanni@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="">On Thu, Aug 28, 2014 at 4:01 PM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br>

> On 22/08/14 17:41, Andreas Pokorny wrote:<br>
>> Allows using prime fds as display target and from display target.<br>
>> Test for PRIME capability after initializing kms_swrast screen.<br>
>><br>
> Hi Andreas,<br>
><br>
> I'm hoping that Giovanni will take a look. After all kms-dri is his creation.<br>
<br>
</div>I'm not sure I'm the right person to look at this code. After all I'm<br>
not a mesa developer.<br>
<br>
>From a cursory review, the patches look good to me, but what I'm not<br>
seeing are the necessary EGL changes to make use of prime fds with<br>
kms-swrast, eg. EGL_WL_bind_wayland_display is still not exposed, even<br>
though it could be implemented using prime fds.<br></blockquote><div><br></div><div>I have no clue on the current state of what wayland requires. There seem to be a lot of options now. I assumed the patch is one of the necessary pieces.<br>
 <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Another thing is, you briefly mentioned the generic GEM ioctls. If<br>
drivers really can use the generic GEM_FLINK/GEM_OPEN ioctl to give a<br>
name to dumb buffers, then indeed the kms-swrast driver is capable of<br>
buffer sharing (I always assumed that a somehow real GEM was<br>
necessary, and that the simple one in qxl/cirrus/simpledrm was not<br>
enough). I'd like to see that implemented first, which would allow to<br>
remove a good amount of complexity and special casing in gallium and<br>
egl (and would allow using kms-swrast for faster sw wayland too).<br>
<span class="HOEnZb"></span></blockquote><div><br></div><div>I was about to do that, then recent regressions in kvm utils and some low hanging fruits in i915 distracted me.<br></div><div>.. so real soon now.. <br><br></div>
<div>regards<br></div><div>Andreas<br></div><div> </div></div></div></div>