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

framework: workaround display issues on AMD GPUs #1236

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

blitz
Copy link
Contributor

@blitz blitz commented Nov 10, 2024

Description of changes

Framework users (including me) have issues with PSR. We can trade some battery life for stability here. So far there is no workaround except disabling PSR.

See:

Things done
  • Tested the changes in your own NixOS Configuration
  • Tested the changes end-to-end by using your fork of nixos-hardware and
    importing it via <nixos-hardware> or Flake input

@blitz blitz force-pushed the framework-display branch from a10ccb3 to 467f768 Compare November 10, 2024 09:58
@miketth
Copy link
Contributor

miketth commented Nov 10, 2024

Great to see this added here. I had this option in my config for ages and I was never sure if this was the one that fixed my GPU hangs and why it did.

@blitz
Copy link
Contributor Author

blitz commented Nov 11, 2024

There is some hope that this might be fixed in the next linux-firmware version: https://gitlab.freedesktop.org/drm/amd/-/issues/3647#note_2652400

If this doesn't pan out, I'd vote for just disabling PSR until AMD gets their act together.

@Mic92
Copy link
Member

Mic92 commented Nov 12, 2024

@mergify queue

Copy link
Contributor

mergify bot commented Nov 12, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at f6581f1

mergify bot added a commit that referenced this pull request Nov 12, 2024
@mergify mergify bot merged commit f6581f1 into NixOS:master Nov 12, 2024
3 checks passed
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