-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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: prime render offload #66601
nvidia: prime render offload #66601
Conversation
Patched Device is detected,
Still broken
|
db99540
to
3fc022e
Compare
d12e300
to
5a80896
Compare
Confirmed PRIME synchronization and render offload works fine on my device.
|
e03a73c
to
d81c3a1
Compare
17c28dc
to
721f3f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like hashes are changing
@eadwu Devices
Configuration services.xserver.videoDrivers = [ "modesetting" "nvidia" ];
hardware.nvidia.prime.offload.enable = true;
hardware.nvidia.prime.nvidiaBusId = "PCI:1:0:0";
hardware.nvidia.prime.intelBusId = "PCI:0:2:0"; Interesting why X server is listed here?
Don't see any picture with these
|
721f3f8
to
aa55960
Compare
Not sure why the |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/fix-bumblebee-kernel-panic/3933/2 |
Got Vulkan to work eadwu#17 |
I'm probably only going to have time to look into this later during the weekend, depending on the time I finish university things. |
As of the current point, if there aren't any bugs anyone is experiencing, I would say this PR is good to go. |
Is @baracoder the only missing review? |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/114 |
Is there anything preventing this from being merged? I've been eagerly waiting for this to land in master. |
I'd love to see this merged! My battery life is suffering every day this isn't merged! :( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on a Dell G3 3579 (Intel Corporation UHD Graphics 630 / GeForce GTX 1050 Ti Mobile). Looks good to me.
Backport to 20.03 anyone? |
I was under the thought that new features shouldn't be backported? Though if some people want to be on 20.03 instead of unstable I can open a PR. |
@eadwu Imho this is 2019 year feature, would be cool to see it on 20.03 |
I'll open one up later today and see how it goes. |
nvidia: prime render offload (cherry picked from commit 74f5358)
@shyim I think your 2 edits to the Nvidia NixOS wiki article are in error:
I believe you may have missed the
mentioned there, and changed it back because those settings don't exist in 20.03 (this sentence also currently unfortunately is present in only 1 of the 2 sections that refer to 20.09-only attributes). I think we should undo the first change, and modify the second change so that it describes how to enable sync mode for both 20.09 (unstable) and 20.03. |
@eadwu Offloading seems to work well for me (I tested on 20.30 with your backport #86094 applied). But even in offload mode, resume-from-suspend still does not work: #73494 (comment) |
I have done this now, and also cleaned up some syntax: https://nixos.wiki/index.php?title=Nvidia&diff=4266&oldid=4211 |
Thanks for the hard work on this feature. I’m looking forward to reverse PRIME! I’ve updated the wiki page to clarify that just like offload mode, only some laptops need modesetting to be added to videoDrivers. For example, doing so on my X1 Extreme (20QV) breaks external monitors and glxinfo, makes picom segfault, and makes firefox and chrome flicker every few seconds. |
''; | ||
}; | ||
|
||
hardware.nvidia.prime.sync.enable = mkOption { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eadwu and others:
Soes prime.sync
have anything to do with PRIME Synchronization? (Arch wiki link and NVIDIA link)
Or is it just the old name for "Optimus Prime's sync mode" as described in https://nixos.wiki/wiki/Nvidia#sync_mode ?
If the former, we should probably rename it, or at write this in the docs here and into the NixOS wiki.
Do you know?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes prime.sync
refers to PRIME Synchronization. It was previously put as optimus_prime
https://github.com/NixOS/nixpkgs/pull/66601/files#diff-28ea4da3a9a595f8bd1a37a555c229f998f3a11b0715ebc5842ba266384eba73L55 as why the section has two different versions.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/laptop-graphics-card-not-detected-by-nixos-generate-config/22450/10 |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @