<div dir="ltr">Hi Emil,<div><br></div><div>Thx for the suggestion about kmscube and piglit. I tried them and I have sent a v3 patch.</div><div><br></div><div>Hi Chris,</div><div><br></div><div>I can see that in Ben's patch it calls <span style="color:rgb(0,0,0);white-space:pre-wrap">brw_bo_unreference instead of </span><font color="#000000"><span style="white-space:pre-wrap">brw_bo_unmap.</span></font></div><div><font color="#000000"><span style="white-space:pre-wrap">Also the existing brw_bo_unmap is a no op (strange it is implemented as just return 0)</span></font></div><div><font color="#000000"><span style="white-space:pre-wrap">Any idea ?</span></font></div><div><font color="#000000"><span style="white-space:pre-wrap"><br></span></font></div><div><font color="#000000"><span style="white-space:pre-wrap">Thx</span></font></div><div><font color="#000000"><span style="white-space:pre-wrap">Julien</span></font></div><div><font color="#000000"><span style="white-space:pre-wrap"><br></span></font></div><div class="gmail_extra"><br><div class="gmail_quote">On 8 November 2017 at 17:18, 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">Hi Julien,<br>
<span><br>
On 8 November 2017 at 16:55, Julien Isorce <<a href="mailto:julien.isorce@gmail.com" target="_blank">julien.isorce@gmail.com</a>> wrote:<br>
> v2: add early return if (flag & MAP_INTERNAL_MASK)<br>
><br>
> Already implemented for Gallium drivers.<br>
><br>
> Useful for gbm_bo_(un)map.<br>
><br>
> Tested by porting wayland/weston/clients/simple-<wbr>dmabuf-drm.c to GBM.<br>
><br>
</span>Ben did a slightly different implementation a while back [1], but it<br>
never got merged due to shortage of review.<br>
Glad to see some interest on the topic.<br>
<br>
On the testing side - the ext_image_dma_buf_import piglit tests +<br>
kmscube should help, alongside your weston* case.<br>
<br>
HTH<br>
Emil<br>
<br>
[1] <a href="https://lists.freedesktop.org/archives/mesa-dev/2017-April/152646.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>archives/mesa-dev/2017-April/1<wbr>52646.html</a><br>
</blockquote></div><br></div></div>