| Bug ID | 101198 |
|---|---|
| Summary | Major performance regression for AMD card |
| Product | Mesa |
| Version | git |
| Hardware | x86-64 (AMD64) |
| OS | All |
| Status | NEW |
| Severity | major |
| Priority | medium |
| Component | Drivers/Gallium/radeonsi |
| Assignee | dri-devel@lists.freedesktop.org |
| Reporter | dark.shadow4@web.de |
| QA Contact | dri-devel@lists.freedesktop.org |
With the latest git mesa, there are major performance drops. F= irst noticed the issue with the 32bit steam game "Move or die", but it seems to af= fect the whole system. It basically makes the game unplayable. Bisecting pointed me to the following commit: commit 524d4fff9e0e58e812e71bc5ba2c92ffa73bee16 Author: Leo Liu <leo.liu@amd.= com> Date: Wed May 3 14:06:35 2017 -0400 amd/common: add vcn dec ip info query Signed-off-by: Leo Liu <leo.l= iu@amd.com> Reviewed-by: Christian K=C3=B6nig <christian.koenig@amd.com> I tried reverting it, but it doesn't apply cleanly. Running Arch Linux 64 Linux 4.11.2 System the issue was tested on: - Arch Linux 64bit - Linux 4.11.2, AMDGPU driver - Radeon R9 285
| What | Removed | Added |
|---|---|---|
| Keywords | bisected, regression |
Just noticed, this seems to be reverted already during the tim= e I need to setup and bisect mesa.
Hi Fabian, David reverted it and Leo Liu sent an update just in time: Please try with this one: https://lists.freedesktop.org/archives/mesa-dev/2017-May/156930.html<= /a> Dicussion was here: https://lists.freedesktop.org/archives/mesa-dev/2017-May/156907.html<= /a> https://lists.freedesktop.org/archives/mesa-dev/2017-May/156931.html<= /a> Cheers, Dieter
(In reply to Fabian Maurer from comment #0) > With the latest git mesa, [...] Can't be the latest, since the commit you've identified is already reverted: <https://cgit.freedesktop.org/mesa/mesa/co= mmit/?id=3De1409f730251bf9d7e70046f58392de6e4e98815>. I find it helpful giving exact commit IDs of the builds I'm using (or the version number, whatever is more accurate). Otherwise bug reports don't ten= d to age well. ;-) In any case, if you're bisection is correct, you should check a build of current HEAD (e1409f730251bf9d7e70046f58392de6e4e98815) and close this bug = if appropriate. Otherwise a new biscetion would be required.
(In reply to Dieter N=C3=BCtzel from comment #2) > Hi Fabian, >=20 > David reverted it and Leo Liu sent an update just in time: > Please try with this one: > https://lists.freedesktop.org/archives/mesa-dev/2017-May/156930.= html >=20 > Dicussion was here: > https://lists.freedesktop.org/archives/mesa-dev/2017-May/156907.= html > https://lists.freedesktop.org/archives/mesa-dev/2017-May/156931.= html >=20 > Cheers, > Dieter s/Discussion... Bonus: When you've tested it fine, you can send four (first) 'Tested-by:' ;-)
| What | Removed | Added |
|---|---|---|
| Status | NEW | RESOLVED |
| Resolution | --- | FIXED |
Thanks, just tested, it works fine after the commit was revert= ed. Marking this as resolved.
I can reproduce the problem with kernel that has no vcn patch,= and the fix sent to mesa-dev list was tested and have issue fixed. Thanks, Leo