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.

Sign up for GitHub

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

Jump to bottom

mesa: 17.1.10 -> 17.2.2 #30196

Merged
merged 1 commit into from Oct 8, 2017
Merged

mesa: 17.1.10 -> 17.2.2 #30196

merged 1 commit into from Oct 8, 2017

Conversation

gebner
Copy link
Member

@gebner gebner commented Oct 7, 2017

Motivation for this change

The 17.1.10 version in master hangs my graphics card (works fine on the release-17.09 branch though).

I have enabled the enableRadv option by default, since it seems to work just fine with my RX480. At least the smoketest demo runs at 1kFPS:

$(nix-build -A vulkan-loader.demos)/bin/smoketest

I'll merge this on Monday if there are no objections.
cc @vcunat @eduarrrd

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@vcunat
Copy link
Member

vcunat commented Oct 7, 2017

Yes, this was in my plans. Since it's larger update, it would be nice to also test with some other GPU. I can test nouveau tomorrow (probably).

@vcunat
Copy link
Member

vcunat commented Oct 7, 2017

@gebner: what driver do you use?

@gebner
Copy link
Member Author

gebner commented Oct 7, 2017

@vcunat I've tested it on a Radeon RX480 using the amdgpu driver.

@vcunat
Copy link
Member

vcunat commented Oct 7, 2017

It's not directly related, but can you also test this amdgpu bump? vcunat@550a292f3c9 (apply wherever you want) I don't even have any compatible hardware.

@gebner
Copy link
Member Author

gebner commented Oct 7, 2017

The xf86-video-amdgpu bump to 1.4.0 works fine here.

vcunat
vcunat approved these changes Oct 8, 2017
View reviewed changes
Copy link
Member

@vcunat vcunat left a comment

Choose a reason for hiding this comment

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

nouveau works OK for me, at least along with a bunch of related changes.

@vcunat vcunat merged commit e16d95b into NixOS:staging Oct 8, 2017
vcunat added a commit that referenced this pull request Oct 8, 2017
@vcunat
Merge #30196: mesa: 17.1.10 -> 17.2.2
0df6a8f
@vcunat vcunat mentioned this pull request Oct 9, 2017
Closed
@gebner gebner mentioned this pull request Nov 2, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vcunat vcunat vcunat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@gebner @vcunat