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
mesa: build iris #65017
mesa: build iris #65017
Conversation
I'm not sure this change will work. I don't know meson's build language semantics exactly, but it looks like it overrides the list of drivers completely, if EDIT: Ref: #63912 <- this PR removed the logic to determine which drivers to build. I guess we would have to reinstate most of that in order to build EDIT2: I'm going to try and patch the |
cc: @worldofpeace |
Alright, somehow this works, even though its not pretty. I'm going to build my system with this change and I'll report back. |
@@ -5,7 +5,7 @@ | |||
, libelf, libvdpau, python3Packages | |||
, libglvnd | |||
, enableRadv ? true | |||
, galliumDrivers ? ["auto"] | |||
, galliumDrivers ? ["auto" "iris" ] |
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.
"iris" should just be built on linux
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.
It's not. I've confirmed myself and it's clear from reading the meson.build
that it's not.
…On Thu, Jul 18, 2019, 17:44 Matthew Bauer ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In pkgs/development/libraries/mesa/default.nix
<#65017 (comment)>:
> @@ -5,7 +5,7 @@
, libelf, libvdpau, python3Packages
, libglvnd
, enableRadv ? true
-, galliumDrivers ? ["auto"]
+, galliumDrivers ? ["auto" "iris" ]
"iris" should just be built on linux
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#65017?email_source=notifications&email_token=AACP25BLCYVO74D25RHRBJTQACFW7A5CNFSM4IEYG2E2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB64HM5I#pullrequestreview-263747189>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AACP25BR6UBPP4YCXENXWELQACFW7ANCNFSM4IEYG2EQ>
.
|
@colemickens Was iris built before #63912? If so, we definitely should make sure it stays the same. Otherwise, though, I would prefer to defer to what mesa includes by default. The alternative is that you can build these drivers yourself and add them to opengl extraPackages:
|
I think it's perfectly sensible to be able to build iris, and defer to what upstream has as defaults.
Iris is highly desirous atm so I wouldn't want users to just have to do that. |
I'm fine with building it myself, especially if I don't have to build the world. I'd be happy to document it on the wiki as well. However, I haven't found an override that works yet... Trying with
Trying with
Trying with
|
Looks like the Nix derivation always copies the XA state tracker pkg-config, and it looks like mesa's meson.build only builds it when certain gallium drivers are included. I'm not sure if this is intentional, or an oversight on upstream's part when they added iris. I've asked in #dri-devel. This seems to build, at least:
However, after system activation, I see this:
|
cc: @eadwu |
Before #63912, I'd probably expect Adding it to
|
I made this change to my system nixos configuration: colemickens/nixcfg@6a05bda. I rebuilt, and rebooted, and there is still no iris available for me: |
You want the |
Thanks for the fast response. Next try: colemickens/nixcfg@a8a0f80 Now I get a collision:
I feel like I ought to know how to get past this, at this point in my NixOS usage. :/ |
Probably can be solved by setting the priority? Though I would rather use |
cool, this works: colemickens/nixcfg@5307eca I am far from an expert, but from what I'm reading we can remove the s3tc stuff: https://arch-dev-public.archlinux.narkive.com/ApEYtwBN/removal-of-lib32-libtxc-dxtn-and-libtxc-dxtn-from-repositories |
(Thank you very much for the guidance @eadwu). I'm going to go ahead and close this for now. Here's a wiki entry: https://nixos.wiki/wiki/Intel_Graphics (we can correct it if we turn out to be wrong about s3tc stuff, but I figure it can't matter much since it's disabled by default anyway). |
@colemickens The example you put in the wiki doesn't point to the |
Fixed. Thanks @worldofpeace |
Can this PR be reopened? It would be nice if mesa with the iris driver will be as a separate entry in all-packages.nix at least. Mesa builds for a very long time on my hardware (Celeron N4000, yes). :( |
Upstream is even defaulting to iris on some HW. It will be in mesa >= 20, currently in RC1 phase, so we should soon get it anyway. |
Okay, that's good news |
Motivation for this change
Fixes #65016.
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)