[Bug 194761] amdgpu driver breaks on Oland (SI)

[bugzilla-daemon at bugzilla.kernel.org]

https://bugzilla.kernel.org/show_bug.cgi?id=194761

--- Comment #23 from Marek Olšák (maraeo at gmail.com) ---
Hi,

(BTW I'm from AMD)

First of all, if a kernel change breaks Mesa, the kernel change should be
reverted. Kernel changes should never ever break existing userspace. There are
released versions of Mesa out there that we can't change with commits to the
master branch. That's why. Kernel changes can't break Mesa ever!

Now about the commit. Yes, the commit is wrong! I've checked internal hw docs
and there are 2 variants of Oland: Oland64 and Oland128. The number may refer
to the bus width. Oland128 uses the same config as Cape Verde, which is
P4_8x16. Oland64 uses the same config as Hainan, which is P2. The commit only
switches the config from Oland128 to Oland64, so it seems to fix Oland64 and at
the same time break Oland128.

I think the real fix is to use the old version of the tile mode table for
Oland128, and the new version for Oland64.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.