<div dir="ltr">Hi Emil, <div><br></div><div>I confirm I tested on lollipop-x86 with i965 on Intel HD4000, X3100 and nouveau with GT210,</div><div>and I saw no regressions on android-x86.<br></div><div><br></div><div>Mauro</div><div><br></div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-11-29 17:48 GMT+01:00 Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Otherwise the user has no way of using it, and we'll try to access the<br>
linear one.<br>
<br>
</span>v2:<br>
 - Bail out when KHR_gl_colorspace is missing and srgb is set (Marek)<br>
<span class=""><br>
Cc: Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>><br>
Cc: Chih-Wei Huang <<a href="mailto:cwhuang@android-x86.org">cwhuang@android-x86.org</a>><br>
</span>Cc: "11.0 11.1" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
<span class="">Fixes: c2c2e9ab604(egl: implement EGL_KHR_gl_colorspace (v2))<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=91596" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=91596</a><br>
Signed-off-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
---<br>
<br>
</span>Mauro, I thought you already tested this but I cannot find any traces.<br>
<br>
Thanks<br>
Emil<br>
<br>
 src/egl/drivers/dri2/egl_dri2.c | 2 ++<br>
 1 file changed, 2 insertions(+)<br>
<br>
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c<br>
index d34b161..8f50f0c 100644<br>
--- a/src/egl/drivers/dri2/egl_dri2.c<br>
+++ b/src/egl/drivers/dri2/egl_dri2.c<br>
@@ -235,6 +235,8 @@ dri2_add_config(_EGLDisplay *disp, const __DRIconfig *dri_config, int id,<br>
<br>
       case __DRI_ATTRIB_FRAMEBUFFER_SRGB_CAPABLE:<br>
<span class="">          srgb = value != 0;<br>
</span>+         if (!disp->Extensions.KHR_gl_colorspace && srgb)<br>
<div class="HOEnZb"><div class="h5">+            return NULL;<br>
          break;<br>
<br>
       default:<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
2.6.2<br>
<br>
</font></span></blockquote></div><br></div></div></div>