<br><div class="gmail_quote">On Fri, Jan 28, 2011 at 4:52 PM, Chia-I Wu <span dir="ltr"><<a href="mailto:olvaffe@gmail.com">olvaffe@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <div class="im">On Fri, Jan 28, 2011 at 9:22 AM, Jammy Zhou <<a href="mailto:jammy.zhou@linaro.org">jammy.zhou@linaro.org</a>> wrote:<br> > Just to make sure that this patch was sent out, so replied again here.<br> ><br> > Hi Chia-I,<br> ><br> > Could you please review this small patch?<br> </div>Sorry for the slow response. I recently had a surgery and I was not<br> able to spend much time on my inbox. I will start catching up in the<br> following weekend.<br></blockquote><div><br>Take care and have a good rest. ;)<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <br> When _glapi_get_proc_address is missing, I think egl_dri2 should<br> dlopen libglapi.so. If --enable-shared-glapi is not given (libGL.so<br> does not use the shared libglapi.so), egl_dri2 should disable<br> EGL_OPENGL_BIT as libglapi.so and libGL.so are conflicting.<br></blockquote><div><br>Hmm... The problem happens for me when load libEGL.so before libGLESv2.so. Of course the problem can be avoided in application side to load libGLESv2.so first. But I think we should better solve the dependency of libEGL.so (egl_dri2) on <a href="http://libglapi.so/libGL.so">libglapi.so/libGL.so</a> in mesa. If we can make libGL.so use libglapi.so, the issue becomes simple. Otherwise, it may be not easy to handle, because in that case we don't know if libGL.so will be used or not when load libEGL.so. Do you have plan to make libGL.so use libglapi.so by default? And I'm curious for what is blocking it.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <div><div></div><div class="h5"><br> > Regards,<br> > Jammy<br> ><br> > On Thu, Jan 27, 2011 at 1:32 PM, Jammy Zhou <<a href="mailto:jammy.zhou@linaro.org">jammy.zhou@linaro.org</a>> wrote:<br> >><br> >> egl_dri2 has been made built-in driver of EGL recently, and it depends on<br> >> _glapi_get_proc_address symbol, which is defined in libglapi.so. So add<br> >> the dependency of libglapi.so to libEGL<br> >><br> >> Signed-off-by: Jammy Zhou <<a href="mailto:jammy.zhou@linaro.org">jammy.zhou@linaro.org</a>><br> >> ---<br> >> src/egl/main/Makefile | 2 +-<br> >> 1 files changed, 1 insertions(+), 1 deletions(-)<br> >><br> >> diff --git a/src/egl/main/Makefile b/src/egl/main/Makefile<br> >> index c710631..fcbce2b 100644<br> >> --- a/src/egl/main/Makefile<br> >> +++ b/src/egl/main/Makefile<br> >> @@ -58,7 +58,7 @@ LOCAL_LIBS =<br> >> ifeq ($(filter dri2, $(EGL_DRIVERS_DIRS)),dri2)<br> >> LOCAL_CFLAGS += -D_EGL_BUILT_IN_DRIVER_DRI2<br> >> LOCAL_LIBS += $(TOP)/src/egl/drivers/dri2/libegl_dri2.a<br> >> -EGL_LIB_DEPS += $(XCB_DRI2_LIBS) $(LIBUDEV_LIBS) $(DLOPEN_LIBS)<br> >> $(LIBDRM_LIB)<br> >> +EGL_LIB_DEPS += $(XCB_DRI2_LIBS) $(LIBUDEV_LIBS) $(DLOPEN_LIBS)<br> >> $(LIBDRM_LIB) -l$(GLAPI_LIB)<br> >> endif<br> >> ifeq ($(filter glx, $(EGL_DRIVERS_DIRS)),glx)<br> >> LOCAL_CFLAGS += -D_EGL_BUILT_IN_DRIVER_GLX<br> >> --<br> >> 1.7.1<br> >><br> ><br> ><br> <br> <br> <br> </div></div><font color="#888888">--<br> olv@LunarG.com<br> </font></blockquote></div><br>