On Saturday, 2017-09-23 13:37:29 +0900, Tomasz Figa wrote: > On Wed, Sep 20, 2017 at 7:47 PM, Eric Engestrom > <[email protected]> wrote: > > On Tuesday, 2017-09-19 17:19:59 +0000, Emil Velikov wrote: > >> From: Emil Velikov <[email protected]> > >> > >> In order to build EGL, one has to use shared glapi - libglapi.so. > >> > >> Thus the dlopen/dlsym dance is no longer needed and we can link to the > >> library directly. > >> > >> This allows us to remove a handful of platform specific names of the > >> library. > >> > >> Cc: Jonathan Gray <[email protected]> > >> Cc: Jon Turney <[email protected]> > >> Cc: Julien Isorce <[email protected]> > >> Cc: Rob Herring <[email protected]> > >> Cc: Tomasz Figa <[email protected]> > >> Signed-off-by: Emil Velikov <[email protected]> > > > > Nice cleanup! > > Assuming the build systems stuff works (with Rob's suggested change?): > > Reviewed-by: Eric Engestrom <[email protected]> > > > > struct dri2_egl_driver now only contains the glFlush() pointer and > > _EGLDriver. Could we move the _glapi_get_proc_address() call to > > the two places that use glFlush() (ie. dri2_make_current() and > > dri2_client_wait_sync()), and get rid of this struct, as well as > > the whole dri2_load() function? > > > > I'm happy to do this, just want to check that it would be ok :) > > Wouldn't that mean going through the series of strcmp()s over all the > symbols every time EGL wants to call glFlush?
Well, I was thinking of making that call static, maybe even global (shared between the functions), so it would happen at most once for each of the two functions, but yeah you're right, it would do it more often than right now. Do you think this is an issue? I guess I'm dropping this idea if that's the case :] > > Best regards, > Tomasz > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
