[Mesa-dev] [PATCH mesa 0/2] egl: wire up swap_with_damage() and enable it on Android
Eric Engestrom
eric.engestrom at imgtec.com
Mon Jun 12 13:30:12 UTC 2017
On Monday, 2017-06-12 09:39:15 +0300, Tapani Pälli wrote:
> Hi Eric;
>
> EGL_KHR_swap_buffers_with_damage is implemented by Android's EGL wrapper
> library so these tests are passing already, no implementation required in
> Mesa side.
Right, I keep forgetting that android intercepts some stuff and lets the
rest through to mesa.
So 2/2 is unnecessary, but should we still land 1/2?
At worst, it's an unnecessary call to a `return false` function, and at
best it sets the correct damage before swapping.
There's an alternative "worst", which would be a platform on which
swap_buffers_with_damage != set_damage_region + swap_buffers,
but on such a platform the fallback shouldn't be used anyway, so using
it would be the bug.
What do you think?
More information about the mesa-dev
mailing list