[Mesa-dev] [PATCH v2 3/8] egl: add EGL_EXT_device_drm support

[Emil Velikov]

Hi Mathias,

On Thu, 20 Sep 2018 at 15:12, Mathias Fröhlich
<Mathias.Froehlich at gmx.net> wrote:

> > @@ -141,6 +231,12 @@ _eglQueryDeviceStringEXT(_EGLDevice *dev, EGLint name)
> >     switch (name) {
> >     case EGL_EXTENSIONS:
> >        return dev->extensions;
> > +#ifdef HAVE_LIBDRM
> > +   case EGL_DRM_DEVICE_FILE_EXT:
> > +      if (_eglDeviceSupports(dev, _EGL_DEVICE_DRM))
> > +         return dev->device->nodes[DRM_NODE_PRIMARY];
> ... we probably want
> return _eglGetDRMDeviceRenderNode(dev);
>
That isn't quite possible, as discussed in 2016's thread
"EGL_EXT_*_drm - primary vs render node".

The extensions is (was?) not too clear that a card node must be
returned, yet there are applications depend on it.

As mentioned in said thread we could add another extension which adds
support for EGL_DRM_RENDER_DEVICE_FILE_EXT.
But I'd suggest keeping that as a follow-up - hence the comment above
_eglGetDRMDeviceRenderNode()

-Emil