<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-07-15 21:18 GMT+02:00 Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi Mauro,<br>
<span class=""><br>
On 14 July 2016 at 04:33, Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br>
> Sending patches to implement DRI2_Fence extension for i915,<br>
> to enable EGL_KHR_fence_sync and EGL_KHR_wait_sync.<br>
><br>
> It is the step-by-step porting to i915 of i965/sync,<br>
> plus patch to support {server,client}_wait_sync without bound context.<br>
><br>
> [PATCH 1/3] i915/sync: Replace prefix 'intel_sync' -> 'intel_gl_sync'<br>
> [PATCH 2/3] i915/sync: Implement DRI2_Fence extension<br>
</span>These two are<br>
Acked-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
<span class=""><br>
> [PATCH 3/3] i915: store reference to the context within struct<br>
><br>
</span>And this one needs a re-spin to include locking, as done my Tomasz.<br></blockquote><div><br></div><div>I'm already doing the re-spin, I will submit v2 by step by step replication of Tomasz patch for i915.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="">> Motivation is to support (web)chromium which requires EGL_KHR_{fence,wait}_sync<br>
> Tested with marshmallow-x86 build on Asus Eee PC 1015PEM<br>
><br>
</span>Did you perform any tests as stated in the i965 "Implement DRI2_Fence<br>
extension" patch ? Afaict the new functionality is nicely separated so<br>
there's little chance for regressions but giving it greater testing<br>
would be appreciated.<br></blockquote><div><br></div><div>Tests were performed by Paulo and they consisted in checking chromium browser that was crashing without those features.</div><div><br></div><div>I will test on intel G33 in the weekend and  perform Android CTS deqp EGL and report back.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<span class=""><br>
> Spotted by Paulo Travaglia <<a href="mailto:pstglia@gmail.com">pstglia@gmail.com</a>><br>
><br>
> Cc: "12.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
</span>I'm not sold that these are stable material - they add feature and<br>
don't fix bug(s). If anything Android could use some updates to honour<br>
the lack of the EGL (EGL_KHR_fence_sync and EGL_KHR_wait_sync)<br>
extensions.<br></blockquote><div><br></div><div>Thanks, I understand your point about additional features for mesa-stable.<br></div><div><br></div><div>It depends if we consider that these are step-by-step cloned from i965 ones,</div><div>with the only difference that 'brw' was renamed as 'intel' as in all other similar functions in i915</div><div><br></div><div>Also these capabilities are independent from underlying HW, if I understood correctly</div><div><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">That said if the Intel devs are OK with having these in stable I won't<br>
object too much ;-)<br>
<br>
Thanks<br>
<span class=""><font color="#888888">Emil<br></font></span></blockquote><div><br></div><div>Merging in mesa-dev will be ok, If patches not accepted for mesa-stable it is not a problem,</div><div>because they can be used as they are in android-x86 marshmallow-x86 branch.</div><div><br></div><div>M.</div></div><br></div></div>