<div>On Friday, 27 January 2017, Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jan 27, 2017 at 10:35:34AM -0800, Chad Versace wrote:<br>
> On Fri 27 Jan 2017, Rafael Antognolli wrote:<br>
> > On Thu, Jan 26, 2017 at 07:18:24PM -0800, Ben Widawsky wrote:<br>
> > > On 17-01-23 15:32:32, Chad Versace wrote:<br>
> > > > On Fri 20 Jan 2017, Rafael Antognolli wrote:<br>
> > > > > I have tested this series with the branches that you mentioned, and with<br>
> > > > > piglit with the patches from my own branch:<br>
> > > > ><br>
> > > > > <a href="https://github.com/rantogno/piglit/tree/review/fences-v02" target="_blank">https://github.com/rantogno/<wbr>piglit/tree/review/fences-v02</a><br>
> > > > ><br>
> > > > > Everything seems to work fine. You can add:<br>
> > > > ><br>
> > > > > Tested-by: Rafael Antognolli <<a href="javascript:;" onclick="_e(event, 'cvml', 'rafael.antognolli@intel.com')">rafael.antognolli@intel.com</a>><br>
> > > > ><br>
> > > > > I also have gone through these patches several times while they were<br>
> > > > > under development, and they look good to me. So if you disregard my<br>
> > > > > shallow knowledge of Mesa, you could add:<br>
> > > > ><br>
> > > > > Reviewed-by: Rafael Antognolli <<a href="javascript:;" onclick="_e(event, 'cvml', 'rafael.antognolli@intel.com')">rafael.antognolli@intel.com</a>><br>
> > > ><br>
> > > > Thanks. I've added the rb's and tb's locally.<br>
> > ><br>
> > > So IMO, Rafael is a great person to review this. He's underselling his knowledge<br>
> > > to his detriment. Is there something preventing pushing these patches?<br>
> ><br>
> > I believe there's nothing technically blocking it from landing, but Chad<br>
> > wanted to push them only after the kernel patches had landed.<br>
><br>
> Right. I'm waiting for the kernel devs to agree to Chris's execbuffer<br>
> fence fd patch.<br>
><br>
> > Hi Chris, do we have everything we need now to have the kernel side to<br>
> > land?<br>
><br>
> Mesa is still blocked on this i915 patch: <a href="https://lists.freedesktop.org/archives/intel-gfx/2017-January/117998.html" target="_blank">https://lists.freedesktop.org/<wbr>archives/intel-gfx/2017-<wbr>January/117998.html</a><br>
<br>
Done. Pushed to explicit fencing to kernel and libdrm.</blockquote><div>You forgot to run make check/distcheck before releasing libdrm. Please update the symbol test(s).</div><div><br></div><div>Thanks,</div><div>Emil</div><div>P.S. Pardon the html formatting.</div></div>