[Mesa-dev] [PATCH 0/7] i965: Implement EGL_ANDROID_native_fence_sync

Emil Velikov emil.l.velikov at gmail.com
Fri Jan 27 23:36:25 UTC 2017


On Friday, 27 January 2017, Chris Wilson <chris at chris-wilson.co.uk> wrote:

> On Fri, Jan 27, 2017 at 10:35:34AM -0800, Chad Versace wrote:
> > On Fri 27 Jan 2017, Rafael Antognolli wrote:
> > > On Thu, Jan 26, 2017 at 07:18:24PM -0800, Ben Widawsky wrote:
> > > > On 17-01-23 15:32:32, Chad Versace wrote:
> > > > > On Fri 20 Jan 2017, Rafael Antognolli wrote:
> > > > > > I have tested this series with the branches that you mentioned,
> and with
> > > > > > piglit with the patches from my own branch:
> > > > > >
> > > > > > https://github.com/rantogno/piglit/tree/review/fences-v02
> > > > > >
> > > > > > Everything seems to work fine. You can add:
> > > > > >
> > > > > > Tested-by: Rafael Antognolli <rafael.antognolli at intel.com
> <javascript:;>>
> > > > > >
> > > > > > I also have gone through these patches several times while they
> were
> > > > > > under development, and they look good to me. So if you disregard
> my
> > > > > > shallow knowledge of Mesa, you could add:
> > > > > >
> > > > > > Reviewed-by: Rafael Antognolli <rafael.antognolli at intel.com
> <javascript:;>>
> > > > >
> > > > > Thanks. I've added the rb's and tb's locally.
> > > >
> > > > So IMO, Rafael is a great person to review this. He's underselling
> his knowledge
> > > > to his detriment. Is there something preventing pushing these
> patches?
> > >
> > > I believe there's nothing technically blocking it from landing, but
> Chad
> > > wanted to push them only after the kernel patches had landed.
> >
> > Right. I'm waiting for the kernel devs to agree to Chris's execbuffer
> > fence fd patch.
> >
> > > Hi Chris, do we have everything we need now to have the kernel side to
> > > land?
> >
> > Mesa is still blocked on this i915 patch: https://lists.freedesktop.org/
> archives/intel-gfx/2017-January/117998.html
>
> Done. Pushed to explicit fencing to kernel and libdrm.

You forgot to run make check/distcheck before releasing libdrm. Please
update the symbol test(s).

Thanks,
Emil
P.S. Pardon the html formatting.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170127/cf1f5e88/attachment.html>


More information about the mesa-dev mailing list