Add a separate config file for loading nvidia modules

[Daemonratte]

Describe the feature

Some repos or installers will overwrite 10-nvidia.conf, resulting in broken glx for the xlibre session.
An example would be installing the proprietary nvidia driver from the xanmod repo.

Someone noted here that he had to symlink nvidia_drv.so into the xlibre module path for it to work and my guess is it's because of a driver installer overwriting /usr/share/X11/xorg.conf.d/10-nvidia.conf

My proposal:
add a separate config for loading the necessary modules.
/usr/share/X11/xorg.conf.d/11-nvidia-modules.conf

Section "OutputClass"
    Identifier "nvidia-modules"
    MatchDriver "nvidia-drm"
    Module "glx"
    Module "glxserver_nvidia"
    ModulePath "/usr/lib/x86_64-linux-gnu/nvidia/xorg"
EndSection

/usr/share/X11/xorg.conf.d/10-nvidia.conf can be left as is (it will be overwritten anyways)

Section "OutputClass"
    Identifier "nvidia"
    MatchDriver "nvidia-drm"
    Driver "nvidia"
    Option "AllowEmptyInitialConfiguration"
    ModulePath "/usr/lib/x86_64-linux-gnu/nvidia/xorg"
EndSection

This way the setup will (hopefully) be more bulletproof

It should be implemented because

Xlibre just won't work with any hardwareaccelaration without this fix on some setups.
Having a separate config prevents errors caused by installers overwriting 10-nvidia.conf.
Also prevents issues when updating nvidia drivers (could also overwrite 10-nvidia.conf)

What are the alternatives?

Symlinking everything in /usr/lib/x86_64-linux-gnu/nvidia/xorg to /usr/lib/xorg/modules lol

Additional context

Someone noted an issue here: https://github.com/X11Libre/xserver/wiki/Graphics-devices-&-drivers#nvidia
This might also be related to it: xlibre-debian/devuan#10

Extra fields

• I have checked the existing issues
• I have read the Contributing Guidelines
• I'd like to work on this issue

[NexusSfan]

Yeah, the 10-nvidia.conf file name does conflict with NVIDIA's Debian packages.

[chankongus]

I'd argue it should be called something like 11-xlibre-nvidia.conf just so that simliar thing doesn't happen in the future when nvidia/packagers decide to call 20-nvidia-modules.conf something else entirely.

[smj-cc]

I'd argue it should be called something like 11-xlibre-nvidia.conf just so that simliar thing doesn't happen in the future when nvidia/packagers decide to call something else entirely.

Do the settings in 11-xlibre-nvidia.conf override the settings in 10-nvidia.conf? 20-nvidia-modules.conf?
Are there never any conflicts anticipated between their respective settings?

[Daemonratte]

Do the settings in 11-xlibre-nvidia.conf override the settings in 10-nvidia.conf?

No, the settings in 11-xlibre-nvidia.conf append the settings in 10-nvidia.conf?
This is nice, because by adding this file we can make sure that Xlibre always finds the nvidia modules.

Edit: I removed Driver "nvidia" from 11-xlibre-nvidia.conf to make it even more robust in case you change to modesetting in 10-nvidia.conf
and renamed 20-nvidia-modules.conf to 11-xlibre-nvidia.conf

[cepelinas9000]

@Daemonratte could you provide instructions how you installed xserver-xorg-video-nvidia-595 - notably apt.list file as i coudn't get using default repositories ?

[Daemonratte]

@cepelinas9000 I installed it from the xanmod repo:
https://xanmod.org/
Afterwards I just edited the source to say "trixie" instead of "excalibur"
That's why I have xserver-xorg-video-nvidia-595 which isn't available in the default devuan or debian repositories.
Even though this isn't the "default way" to install nvidia drivers I still think it's worth it to load the modules via a separate config file as other people seem to have experienced the same issue.
Some one here noted that he had to symlink the nvidia modules into the xorg modules folder.

[chankongus]

I just checked, and it's slightly different on arch:
https://aur.archlinux.org/cgit/aur.git/tree/nvidia-drm-outputclass.conf?h=nvidia-580xx-utils
https://gitlab.archlinux.org/archlinux/packaging/packages/nvidia-utils/-/blob/main/nvidia-drm-outputclass.conf?ref_type=heads