[Mesa-dev] [PATCH] egl/android: Update color_buffer[] querying for buffer age

Nataraj Deshpande nataraj.deshpande at intel.com
Mon Jul 8 18:58:11 UTC 2019


color_buffers[] is currently hard coded to 3 for android which fails
in droid_window_dequeue_buffer when ANativeWindow creates 4 color_buffers
on ChromeOS while querying buffer age during dEQP partial_update tests.

Fixes dEQP-EGL.functional.partial_update* tests on chromebooks with
enabling EGL_KHR_partial_update.

Fixes: 2acc69da8ce "EGL/Android: Add EGL_EXT_buffer_age extension"
Signed-off-by: Nataraj Deshpande <nataraj.deshpande at intel.com>
---
 src/egl/drivers/dri2/egl_dri2.h | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/egl/drivers/dri2/egl_dri2.h b/src/egl/drivers/dri2/egl_dri2.h
index 943ff18..635a100 100644
--- a/src/egl/drivers/dri2/egl_dri2.h
+++ b/src/egl/drivers/dri2/egl_dri2.h
@@ -322,13 +322,11 @@ struct dri2_egl_surface
    __DRIimage *dri_image_front;
 
    /* Used to record all the buffers created by ANativeWindow and their ages.
-    * Usually Android uses at most triple buffers in ANativeWindow
-    * so hardcode the number of color_buffers to 3.
     */
    struct {
       struct ANativeWindowBuffer *buffer;
       int age;
-   } color_buffers[3], *back;
+   } color_buffers[4], *back;
 #endif
 
    /* surfaceless and device */
-- 
2.7.4



More information about the mesa-dev mailing list