Re: [Mesa-dev] [PATCH v2] egl/wayland: Support for KHR_partial_update

2017-10-20 Thread Harish Krupo
Hi Emil, Emil Velikov writes: > On 20 October 2017 at 12:35, Harish Krupo wrote: > >> + else if (!n_rects || !try_damage_buffer(dri2_surf, rects, n_rects)) >>wl_surface_damage(dri2_surf->wl_surface_wrapper, >> 0, 0, INT32_MAX, INT32_MAX); >> > Please add a com

Re: [Mesa-dev] [PATCH v2] egl/wayland: Support for KHR_partial_update

2017-10-20 Thread Eric Engestrom
On Friday, 2017-10-20 11:35:40 +, Harish Krupo wrote: > This passes 33/37 deqp tests related to partial_update, 4 are not > supported. Tests not supported: > dEQP-EGL.functional.negative_partial_update.not_postable_surface > dEQP-EGL.functional.negative_partial_update.not_current_surface > dEQP

Re: [Mesa-dev] [PATCH v2] egl/wayland: Support for KHR_partial_update

2017-10-20 Thread Emil Velikov
On 20 October 2017 at 12:35, Harish Krupo wrote: > + else if (!n_rects || !try_damage_buffer(dri2_surf, rects, n_rects)) >wl_surface_damage(dri2_surf->wl_surface_wrapper, > 0, 0, INT32_MAX, INT32_MAX); > Please add a comment why SetDamageRegionCalled is honoured

[Mesa-dev] [PATCH v2] egl/wayland: Support for KHR_partial_update

2017-10-20 Thread Harish Krupo
This passes 33/37 deqp tests related to partial_update, 4 are not supported. Tests not supported: dEQP-EGL.functional.negative_partial_update.not_postable_surface dEQP-EGL.functional.negative_partial_update.not_current_surface dEQP-EGL.functional.negative_partial_update.buffer_preserved dEQP-EGL.fu