-
-
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
vulkan: 1.2.162.0 → 1.2.176.0 #122061
vulkan: 1.2.162.0 → 1.2.176.0 #122061
Conversation
nativeBuildInputs = [ | ||
pkg-config |
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.
pkg-config
doesn't seem necssary to build most of these Vulkan projects. Please correct me if I'm wrong.
Result of 40 packages marked as broken and skipped:
215 packages skipped due to time constraints:
40 packages built successfully:
|
With this update I've also noticed a slight performance improvement in games with my RX 590 graphics card. Especially with Valheim, which was previously unplayable when launched with Vulkan. |
That's probably a coincidence; none of the code updated in this PR is on a hot path, by design. |
@Ralith Oh yep sorry, it's definitely a coincidence. I reverted these changes, and I'm still noticing the performance improvements. Maybe from a recent |
Thanks for the updates! I can't test these soon but it looks good to me!
…On Sat, 8 May 2021, 7:49 am Kira Bruneau, ***@***.***> wrote:
@Ralith <https://github.com/Ralith> Oh yep sorry, it's definitely a
coincidence. I reverted these changes, and I'm still noticing the
performance improvements. Maybe from a recent linuxPackages_latest update?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#122061 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGRJXH7OMMGP2BLIDUA7M3TMR35HANCNFSM44KU463A>
.
|
Version 1.2.182.0 was just tagged as a SDK release, but I think it would make sense to handle that in a separate PR. |
Thanks! I've opened the 182 bump here: #131299 |
Motivation for this change
Update to the latest sdk version: https://github.com/KhronosGroup/Vulkan-Loader/releases/tag/sdk-1.2.176.0
This includes a change in
vulkan-loader
to support multiple Vulkan layers with the same name, but different library architectures: KhronosGroup/Vulkan-Loader#525. This can be used byMangoHud
&vkBasalt
to support 32bit Vulkan layers as a workaround for #101597.@Ralith @expipiplus1 @zeratax @Atemu
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
I tried this, but ran out of disk space, which is why I'm submitting this to staging. I was able to build my NixOS system with these changes and I haven't noticed any runtime problems.
./result/bin/
)nix path-info -S
before and after)