Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

nvidia additions #91

Closed
wants to merge 6 commits into from
Closed

nvidia additions #91

wants to merge 6 commits into from

Conversation

dougg0k
Copy link

@dougg0k dougg0k commented Sep 30, 2024

@1Naim
Copy link
Member

1Naim commented Sep 30, 2024

@Edgars-Cirulis
Copy link

Why not adding NVreg_EnablePCIeGen3=1 ? even though if it's not compatible it should do fallback

@ventureoo
Copy link
Member

Why not adding NVreg_EnablePCIeGen3=1 ? even though if it's not compatible it should do fallback

This is not required on desktop cards. The default PCIe Gen 2 mode of operation is only for Quadro, Tesla, and NVS Kepler, which is not a common case. See: #3

Copy link
Member

@ventureoo ventureoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I said before, please do not split the use of each option into separate lines.

@dougg0k
Copy link
Author

dougg0k commented Sep 30, 2024

It's much more readable to have in each separated. And they work the same way. Your common practice here seems just because someone else did like that first.

@dougg0k dougg0k changed the title Nvidia additions nvidia additions Sep 30, 2024
Copy link

@Edgars-Cirulis Edgars-Cirulis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since powermizer is dead , maybe remove it?

@ventureoo
Copy link
Member

Since powermizer is dead , maybe remove it?

We may want to remove this when we stop supporting older driver versions like 390xx.

@dougg0k
Copy link
Author

dougg0k commented Sep 30, 2024

Maybe this would be another to have as optional. NVreg_DynamicPowerManagementVideoMemoryThreshold=0

https://www.reddit.com/r/cachyos/comments/1dw2c73/fix_found_for_nvidia_gpu_fails_to_power_off/

@dougg0k
Copy link
Author

dougg0k commented Oct 2, 2024

Added hdmi_deepcolor as optional option. https://www.phoronix.com/news/NVIDIA-545.23.06-Linux-Beta

@ventureoo
Copy link
Member

Added hdmi_deepcolor as optional option. https://www.phoronix.com/news/NVIDIA-545.23.06-Linux-Beta

It was enabled by default in the 555 driver.

@dougg0k
Copy link
Author

dougg0k commented Oct 2, 2024

can be optionally enabled

@Edgars-Cirulis
Copy link

can be optionally enabled

To disable it? Why. Its good feature. It's enabled by default on 555.

@Edgars-Cirulis
Copy link

Good choice if it doesn't support 10bit it will fallback to 8bit :)

@Edgars-Cirulis
Copy link

Edgars-Cirulis commented Oct 3, 2024

https://gitlab.archlinux.org/archlinux/packaging/packages/nvidia-utils/-/commit/1b02daa2ccca6a69fa4355fb5a369c2115ec3e22

We will not need anymore to enable fbdev and modeset you can make it optimally to disable it.

@1Naim
Copy link
Member

1Naim commented Oct 3, 2024

https://gitlab.archlinux.org/archlinux/packaging/packages/nvidia-utils/-/commit/1b02daa2ccca6a69fa4355fb5a369c2115ec3e22

We will not need anymore to enable fbdev and modeset you can make it optimally to disable it.

We've already patched this into the driver for a long time already. Furthermore, we have our own packages for NVIDIA and do not depend on Arch.

@ventureoo
Copy link
Member

https://gitlab.archlinux.org/archlinux/packaging/packages/nvidia-utils/-/commit/1b02daa2ccca6a69fa4355fb5a369c2115ec3e22

We will not need anymore to enable fbdev and modeset you can make it optimally to disable it.

We still need it here for older driver versions such as 470.xx

@dougg0k
Copy link
Author

dougg0k commented Oct 15, 2024

This is not moving forward, if you decided to add, just merge, after.

@dougg0k dougg0k closed this Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants