FS#50399 - [linux] enable amdgpu support for Sea Islands cards

Attached to Project: Arch Linux
Opened by Em (nauticalnexus) - Monday, 15 August 2016, 05:04 GMT
Last edited by Doug Newgard (Scimmia) - Saturday, 14 January 2017, 00:24 GMT
Task Type Feature Request
Category Kernel
Status Closed
Assigned To Tobias Powalowski (tpowa)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

I built a kernel that enabled support for the hawaii chip for amdgpu, and probably for the other Sea Islands chips but I don't have anything but hawaii. The kernel is based on the kernel in the core repo. I'd like to have that put into core, because it's not that experimental anymore, in fact it works perfectly. I'll put the important bits of the config file down below, do with it as you will, love to help.

CONFIG_DRM_AMDGPU=m
CONFIG_DRM_AMDGPU_CIK=y
This task depends upon

Closed by  Doug Newgard (Scimmia)
Saturday, 14 January 2017, 00:24 GMT
Reason for closing:  Implemented
Comment by Doug Newgard (Scimmia) - Monday, 15 August 2016, 05:31 GMT
CONFIG_DRM_AMDGPU is already set, so this is really just CONFIG_DRM_AMDGPU_CIK. From what I can see, this is still experimental.
Comment by Em (nauticalnexus) - Monday, 15 August 2016, 05:34 GMT
I don't know why it's still listed as experimental. It works 100% perfectly for me, honestly, I've no idea why it's still considered experimental.
Comment by Doug Newgard (Scimmia) - Monday, 15 August 2016, 05:36 GMT
Sure it works, until you try something and your system catches fire :D
Comment by Em (nauticalnexus) - Monday, 15 August 2016, 05:38 GMT
No seriously, it works perfectly fine. It works perfectly fine for everyone else that's done it. I feel like this would benefit more Linux users that use AMD cards, because they have a choice instead of being stuck with radeon.
Comment by Doug Newgard (Scimmia) - Monday, 15 August 2016, 05:40 GMT
You say it works perfectly fine, but the authors don't.
Comment by Em (nauticalnexus) - Monday, 15 August 2016, 05:41 GMT
I don't understand why they don't believe so. It's pretty much non-experimental at this point, actually, it should've been out of experimental for a while now I believe.
Comment by Doug Newgard (Scimmia) - Monday, 15 August 2016, 05:43 GMT
Then you might want to figure out why they believe so, rather than blindly enabling it and hoping for the best.
Comment by Em (nauticalnexus) - Monday, 15 August 2016, 05:45 GMT
Blindly enabling it? I just said there's no issues with amdgpu for me, and the others that have enabled it manually, like the issues radeon has. I always got black screens that crashed my system with the radeon driver, and poor performance in games, and I haven't had any of that with the amdgpu driver. Performance is pretty solid, there's no reason this should still be experimental. It works perfectly.
Comment by novenary (Streetwalrus) - Saturday, 12 November 2016, 12:34 GMT
Just pitching in to state my interest in having this enabled for official kernels.
Experimental support for SI cards is coming to 4.9 with CONFIG_DRM_AMDGPU_SI, and as far as I know (from my own tests), the radeon module will be used by default unless explicitly configured not to (eg, by blacklisting it).
Enabling these options would allow using the experimental code without recompiling a kernel, while preserving the current setup by default.
Comment by Sebastiaan Lokhorst (lonaowna) - Friday, 23 December 2016, 13:39 GMT
Note that Southern Islands (CONFIG_DRM_AMDGPU_SI) is now enabled in the Arch package (4.9-1 in [testing]). https://git.archlinux.org/svntogit/packages.git/tree/trunk/config.x86_64?h=packages/linux#n4978
(Sea Islands/CIK is still disabled, even though it is considered just as "experimental")
Comment by Britt Yazel (brittyazel) - Sunday, 08 January 2017, 19:53 GMT
It does seem weird to enable AMDGPU_SI and not AMDGPU_CIK, when CIK is actually in better shape than SI. The way the kernel config stands now, AMDGPU is only enabled for cards older than Radeon 7000 and newer than r9-380, and there is a weird void where GCN 1.1 isn't enabled.

My two cents are to set AMDGPU_SI=y and AMDGPU_CIK=y, because even when both are enabled, the kernel will still default to the radeon kernel driver unless it is explicitly blacklisted. Thus, no unwitting victim will have a bad experience with an experimental driver (as they will still be defaulted to radeon), and those of us who want to test AMDGPU can do so by blacklisting the radeon driver. All that is accomplished by keeping SI and CIK disabled is forcing us to have to compile the kernel ourselves to change this one thing
Comment by Sebastiaan Lokhorst (lonaowna) - Sunday, 08 January 2017, 20:39 GMT
@brittyazel: completely agree
Comment by Em (nauticalnexus) - Monday, 09 January 2017, 15:01 GMT
Is this serious? They enabled Southern Islands, an even more experimental setting, but not Sea Islands, which is definitely working completely at this point?
Comment by Britt Yazel (brittyazel) - Tuesday, 10 January 2017, 18:58 GMT
Do you know if the Linux maintainers have seen this request? It would be a real shame to not have this addressed (in either one way or another) before the next kernel release, given that it is such a small change on their part
Comment by Sebastiaan Lokhorst (lonaowna) - Friday, 13 January 2017, 11:19 GMT Comment by Britt Yazel (brittyazel) - Friday, 13 January 2017, 21:01 GMT
I agree, testing AMDGPU on my CIK card is yielding fantastic results. Glad to see this problem solved

Loading...