On 12/06/15 19:07, Derek Foreman wrote: > We do this right almost everywhere already, but the idiom is a little > gruesome. This moves it into loader.c and fixes dri2_initialize_drm() > which could open without CLOEXEC in some cases. > > Signed-off-by: Derek Foreman <der...@osg.samsung.com> > --- > src/egl/drivers/dri2/platform_drm.c | 4 ++-- > src/egl/drivers/dri2/platform_wayland.c | 11 +---------- > src/egl/drivers/dri2/platform_x11.c | 12 ++---------- > src/glx/dri2_glx.c | 10 +--------- > src/loader/loader.c | 6 +++--- > src/loader/loader.h | 3 +++ > 6 files changed, 12 insertions(+), 34 deletions(-)
Despite that it's on the trivial side, can we please split this into 3 patches: 1) s/static int drm_open_device/int loader_drm_open_device/ 2) Use in libEGL (new EGL platform - "surfaceless" will land on Monday, so better wait for that one to land as well) 3) Use in libGL I'll throw in with similar patches for VL (for the video targets/drivers) and pipe-loader (OpenCL). Thank you Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev