Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: use nvidia-kernel-open module #43

Closed
wants to merge 3 commits into from
Closed

Conversation

mirkobrombin
Copy link
Member

@mirkobrombin mirkobrombin commented Jul 31, 2024

Should fix some issues with multiple-monitors (based on feedback from another user) and probably others.

Merging rules

  • Merging this PR makes Vanilla OS start supporting the nvidia-propietary
  • We have to take note that older GPUs will not work, they should be pointed to use the new nvidia-propietary
  • The installer must be updated to identify old GPUs (like 470.x or 390.x) and suggest the new nvidia-propietary
  • We have to find a way to automatically update existing installations (I was thinking of just making a new nvidia-open-image image instead of nvidia -propietary so as existing installations do not get affected, but that would change nothing since existing installations with issues will just keep having issues and needs to switch to the new nvidia-open manually which is the same as doing the opposite)
  • I suggest to wait for some more users testing the changes (by using the mirkobrombin/nvidia-open image, if this actually fixes some issues then it must be merged
  • DO NOT merge this without first following the previous steps

Where I tested

I do not have an external monitor to test. Anyway I tested this on 2 different laptops:

  • GTX 1660
  • RTX 4050

What I tested

everything works fine, especially:

  • switching PRIME profiles
  • Wayland
  • Xorg
  • games (Steam via Flatpak/Apx (Ubuntu)/Cpak verified via MangoHUD

Issues

had no issues of any type, also interesting point: boot was speeded up by about 3s, I don't know if it's related but I noticed it 4 out of 4 times

Custom Kargs

none, just the default ones (quiet splash bgrt_disable $vt_handoff lsm=integrity)

nvidia-smi

+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 555.42.06              Driver Version: 555.42.06      CUDA Version: N/A      |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  NVIDIA GeForce RTX 4050 ...    Off |   00000000:01:00.0 Off |                  N/A |
| N/A   45C    P3              8W /   35W |       8MiB /   6141MiB |      0%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+
                                                                                         
+-----------------------------------------------------------------------------------------+
| Processes:                                                                              |
|  GPU   GI   CI        PID   Type   Process name                              GPU Memory |
|        ID   ID                                                               Usage      |
|=========================================================================================|
|    0   N/A  N/A      3501      G   /usr/bin/gnome-shell                            1MiB |
+-----------------------------------------------------------------------------------------+

cur-gpu

OpenGL version: 4.6 (Compatibility Profile) Mesa 24.1.3-2
OpenGL vendor: Intel
OpenGL renderer: Mesa Intel(R) Graphics (ADL GT2)

using nrun:

OpenGL version: 4.6.0 NVIDIA 555.42.06
OpenGL vendor: NVIDIA Corporation
OpenGL renderer: NVIDIA GeForce RTX 4050 Laptop GPU/PCIe/SSE2

Does not fix/Does not change

I am including in this section some of the issues we are already tracking that does not get fixed and/or don't change with this change.

  • some setups (currently 3 reports) remain stuck on the vanilla logo

@mirkobrombin mirkobrombin changed the title chore: use nvidia-kernel-open module (should fix some issues with mul… chore: use nvidia-kernel-open module (should fix some issues with multiple-monitors) Jul 31, 2024
@mirkobrombin mirkobrombin changed the title chore: use nvidia-kernel-open module (should fix some issues with multiple-monitors) chore: use nvidia-kernel-open module Jul 31, 2024
@MeDerpWasTaken
Copy link

Won't this prevent the drivers from working on pre-Turing GPUs? (Including 10 and 9 series)

@mirkobrombin
Copy link
Member Author

mirkobrombin commented Aug 1, 2024

Won't this prevent the drivers from working on pre-Turing GPUs? (Including 10 and 9 series)

Yes. I am preparing another image for those GPUs, planning to detect them during the installation to switch to that image instead. NVIDIA setups are so delicate that providing at least 2 images should cover the majority of issues.

EDIT: just updated the main comment.

- enable fine-grained dynamic power management with NVreg_DynamicPowerManagement=0x02
- ensure video memory allocations are preserved across sleep cycles with NVreg_PreserveVideoMemoryAllocations=1
- set temporary file path for NVIDIA drivers to /var/tmp for improved file handling during power management
- enable framebuffer device emulation with nvidia-drm.fbdev=1 to maintain console visibility during early boot (might not be necessary)
@mirkobrombin mirkobrombin deleted the branch dev August 1, 2024 16:34
@mirkobrombin
Copy link
Member Author

Merged

@mirkobrombin mirkobrombin deleted the nvidia-open branch August 4, 2024 06:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants