[vlc-devel] [PATCH 7/7] egl_display: add GBM platform support

[Romain Vimont]

On Wed, Mar 03, 2021 at 10:30:18PM +0200, Rémi Denis-Courmont wrote:
> Le keskiviikkona 3. maaliskuuta 2021, 22.18.33 EET Romain Vimont a écrit :
> > On Wed, Mar 03, 2021 at 07:48:12PM +0200, Rémi Denis-Courmont wrote:
> > > Le keskiviikkona 3. maaliskuuta 2021, 19.36.40 EET Romain Vimont a écrit :
> > > > From: Alexandre Janniaux <ajanni at videolabs.io>
> > > > +    sys->fd = vlc_open("/dev/dri/card0", O_RDWR);
> > > 
> > > IMU, you're supposed to open render(N+128), not cardN, for head-less
> > > operations. Also N should probably be configurable.
> > 
> > Indeed, "/dev/dri/renderD128" works on my machine.
> > 
> > I suggest to keep "/dev/dri/card0" by default, and add an option to use
> > a custom path.
> 
> That won't work. Typically users don't have permission to card nodes if they 
> did not log in locally. But then, they might as well use the regular X11 or 
> Wayland platform display.
> 
> The VA decoder already defaults to render nodes.

OK, thanks, then I will do the same as:
https://code.videolan.org/videolan/vlc/-/blob/152883b6980fb7f6369c16a69e2bc1d65a7e120a/modules/hw/vaapi/decoder_device.c#L145

Regards