<div dir="ltr"><div>Hi Lyude,</div><div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 29, 2018 at 7:54 PM, Lyude Paul <span dir="ltr"><<a href="mailto:lyude@redhat.com" target="_blank">lyude@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span class="gmail-"><div>On Mon, 2018-05-28 at 09:11 +0200, Olivier Fourdan wrote:</div><blockquote type="cite" style="margin:0px 0px 0px 0.8ex;border-left:2px solid rgb(114,159,207);padding-left:1ex"><div dir="ltr">[...]<br><div class="gmail_extra">I agree that we need to check for the protocol availability of course, yet it doesn't mean we should ditch this patch.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Imagine (I say imagine, I do not know if that's plausible, nor if it's even possible), some vendor pushing for EGL streams realizing they could as well support GBM in the future, we would end up with both being supported, in which case we should probably still prefer GBM if we were to chose automatically.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">So, in this scenario, checking for GBM availability to decide whether or not we should enable one or the other backend is still a good thing, independently of the protocol availability.<br></div></div></blockquote></span><div>Oh! I should have looked closer at the patch, sorry about that-I had assumed it was checking for the EGLStream interfaces in the way that halfline had mentioned.</div><div><br></div><div>Anyway:</div><div>Reviewed-by: Lyude Paul <<a href="mailto:lyude@redhat.com" target="_blank">lyude@redhat.com</a>></div><span class="gmail-"></span><br></div></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks!</div><div class="gmail_extra"><br></div><div class="gmail_extra">This patch *is* the way Halfline had mentioned, it's useful in the (hypothetical) case where we would have *both* GBM and EGLstreams available, we should prefer GBM if not requested explicitly EGL Streams (from the command line).</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also, following Pekka (very good) point that we should not use strstr(), I shall post an updated version of this patch (v3) using epoxy_has_egl_extension() which does exactly what we want, the way we want it. I am therefore not adding your R-b to that new iteration of that patch since it's different.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_extra"></div><div class="gmail_extra">But it's not
 sufficient, we also need to check for Wayland interfaces availability 
which is another series that I shall post shortly.</div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Cheers,</div><div class="gmail_extra">Olivier<br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div></div>