[Mesa-dev] [PATCH] egl/drm: Don't "fall back" to /dev/dri/card0 if the first open fails
Eric Engestrom
eric.engestrom at imgtec.com
Fri Aug 25 09:24:24 UTC 2017
On Thursday, 2017-08-24 14:52:14 -0400, Adam Jackson wrote:
> The snprintf stuff here already constructs the right name for the device
> node, and if it doesn't, you configured Mesa wrong, don't do that.
Very good point.
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
>
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
> src/egl/drivers/dri2/platform_drm.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/src/egl/drivers/dri2/platform_drm.c b/src/egl/drivers/dri2/platform_drm.c
> index 259b1cd519..0ccbd9a30a 100644
> --- a/src/egl/drivers/dri2/platform_drm.c
> +++ b/src/egl/drivers/dri2/platform_drm.c
> @@ -667,8 +667,6 @@ dri2_initialize_drm(_EGLDriver *drv, _EGLDisplay *disp)
> int n = snprintf(buf, sizeof(buf), DRM_DEV_NAME, DRM_DIR_NAME, 0);
> if (n != -1 && n < sizeof(buf))
> dri2_dpy->fd = loader_open_device(buf);
> - if (dri2_dpy->fd < 0)
> - dri2_dpy->fd = loader_open_device("/dev/dri/card0");
> gbm = gbm_create_device(dri2_dpy->fd);
> if (gbm == NULL) {
> err = "DRI2: failed to create gbm device";
> --
> 2.13.5
>
More information about the mesa-dev
mailing list