Skip to content

Remove setting NVreg_DynamicPowerManagement#465

Merged
jackpot51 merged 1 commit intomasterfrom
rm-explicit-nv-pm
Sep 23, 2025
Merged

Remove setting NVreg_DynamicPowerManagement#465
jackpot51 merged 1 commit intomasterfrom
rm-explicit-nv-pm

Conversation

@crawfxrd
Copy link
Copy Markdown
Member

For a while now, this has had an auto value (0x03) that is the default. Explicitly setting it is only required if the auto value does not work. Such cases can be handled in system76-driver as a quirk.

Requires: pop-os/system76-driver#335

@crawfxrd crawfxrd requested review from a team September 11, 2025 16:11
@crawfxrd
Copy link
Copy Markdown
Member Author

crawfxrd commented Sep 11, 2025

ubuntu-drivers-common is explicitly setting the value to 0x02 in /usr/lib/modprobe.d/nvidia-runtimepm.conf. Would need to remove this file to ensure that either the default is used or it's set by the user in /etc/modprobe.d.


Other things

  • nvidia-drm modeset=1 is set by the driver itself, so that can be dropped
    • Is this true for 470?
  • blacklist i2c_nvidia_gpu can be set by a quirk in system76-driver
    • Already is for some models

This would remove the need for custom modprobe rules for discrete and hybrid graphics modes.

For a while now, this has had an auto value (0x03) that is the default.
Explicitly setting it is only required if the auto value does not work.
Such cases can be handled in system76-driver as a quirk.

Signed-off-by: Tim Crawford <tcrawford@system76.com>
Copy link
Copy Markdown

@jacobktm jacobktm left a comment

Choose a reason for hiding this comment

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

This does resolve the GSP crash in the addw4 and I also did some testing in an oryp12, bonw16, and gaze20 and everything works normally in those systems.

@jackpot51 jackpot51 merged commit d602561 into master Sep 23, 2025
20 checks passed
@jackpot51 jackpot51 deleted the rm-explicit-nv-pm branch September 23, 2025 16:03
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.

3 participants