[Mesa-dev] [Mesa-stable] i915: Enable EGL_KHR_{fence, wait}_sync

[Emil Velikov]

Hi Mauro,

On 14 July 2016 at 04:33, Mauro Rossi <issor.oruam at gmail.com> wrote:
> Sending patches to implement DRI2_Fence extension for i915,
> to enable EGL_KHR_fence_sync and EGL_KHR_wait_sync.
>
> It is the step-by-step porting to i915 of i965/sync,
> plus patch to support {server,client}_wait_sync without bound context.
>
> [PATCH 1/3] i915/sync: Replace prefix 'intel_sync' -> 'intel_gl_sync'
> [PATCH 2/3] i915/sync: Implement DRI2_Fence extension
These two are
Acked-by: Emil Velikov <emil.l.velikov at gmail.com>

> [PATCH 3/3] i915: store reference to the context within struct
>
And this one needs a re-spin to include locking, as done my Tomasz.

> Motivation is to support (web)chromium which requires EGL_KHR_{fence,wait}_sync
> Tested with marshmallow-x86 build on Asus Eee PC 1015PEM
>
Did you perform any tests as stated in the i965 "Implement DRI2_Fence
extension" patch ? Afaict the new functionality is nicely separated so
there's little chance for regressions but giving it greater testing
would be appreciated.

> Spotted by Paulo Travaglia <pstglia at gmail.com>
>
> Cc: "12.0" <mesa-stable at lists.freedesktop.org>
I'm not sold that these are stable material - they add feature and
don't fix bug(s). If anything Android could use some updates to honour
the lack of the EGL (EGL_KHR_fence_sync and EGL_KHR_wait_sync)
extensions.

That said if the Intel devs are OK with having these in stable I won't
object too much ;-)

Thanks
Emil