https://bugs.freedesktop.org/show_bug.cgi?id=67043
Priority: medium Bug ID: 67043 Assignee: dri-devel@lists.freedesktop.org Summary: Atombios stuck in a look during resume Severity: normal Classification: Unclassified OS: Linux (All) Reporter: parag.warudkar@gmail.com Hardware: x86-64 (AMD64) Status: NEW Version: unspecified Component: DRM/Radeon Product: DRI
With mainline git kernel build from yesterday (3.11.0-rc1+ #7 SMP Wed Jul 17 11:34:34 EDT) I see resume takes longer and the below is printed in system log -
[ 7647.285017] sd 0:0:0:0: [sda] Starting disk [ 7647.374821] [drm:atom_op_jump] *ERROR* atombios stuck in loop for more than 5secs aborting [ 7647.374822] [drm:atom_execute_table_locked] *ERROR* atombios stuck executing CD3E (len 55, WS 0, PS 0) @ 0xCD61
I think this started after DPM commits. Prior kernels don't show this error.
Hardware is
lspci |grep VGA 01:00.0 VGA compatible controller: Advanced Micro Devices [AMD] nee ATI Turks PRO [Radeon HD 7570]
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #1 from Alex Deucher agd5f@yahoo.com --- Please attach your dmesg output. Is this with dpm enabled? Do you still have the issue without dpm enabled? If so can you bisect?
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #2 from Parag parag.warudkar@gmail.com --- Created attachment 82603 --> https://bugs.freedesktop.org/attachment.cgi?id=82603&action=edit dmesg with dpm enabled
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #3 from Parag parag.warudkar@gmail.com --- Created attachment 82613 --> https://bugs.freedesktop.org/attachment.cgi?id=82613&action=edit dmesg from 3.10.1 kernel without the timeout issue
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #4 from Parag parag.warudkar@gmail.com --- Tried with radeon.dpm=0 on current mainline - no difference, still got the stuck message and delayed resume. Also tried with 3.8 Ubuntu stock kernel and 3.10.1 kernel from Ubunty daily mainline builds - the issue doesn't exist on both kernels. So it matches my vague memory that it started post 3.11 - unsure if that was after dpm or some other patchset prior to that.
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #5 from Alex Deucher agd5f@yahoo.com --- If it started in 3.11 can you bisect?
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #6 from Parag parag.warudkar@gmail.com --- I suck at bisecting, but yes, I will give it a shot.
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #7 from Parag parag.warudkar@gmail.com --- After a not-so-clean git bisect (compilation failures in vgacon.c, some TI driver I didn't need) this was the result - at least it is related to drm.
git bisect good 372835a8527f85b3eff20a18c2c339e827dfd4e4 is the first bad commit commit 372835a8527f85b3eff20a18c2c339e827dfd4e4 Author: Daniel Vetter daniel.vetter@ffwll.ch Date: Sat Jun 15 00:13:13 2013 +0200
drm/crtc-helper: explicit DPMS on after modeset
Atm the crtc helper implementation of set_config has really inconsisten semantics: If just an fb update is good enough, dpms state will be left as-is, but if we do a full modeset we force everything to dpms on.
This change has already been applied to the i915 modeset code in
commit e3de42b68478a8c95dd27520e9adead2af9477a5 Author: Imre Deak imre.deak@intel.com Date: Fri May 3 19:44:07 2013 +0200
drm/i915: force full modeset if the connector is in DPMS OFF mode
which according to Greg KH seems to aim for a new record in most Bugzilla: links in a commit message.
The history of this dpms forcing is pretty interesting. This patch here is an almost-revert of
commit 811aaa55ba21ab37407018cfc01770d6b037d3fb Author: Keith Packard keithp@keithp.com Date: Thu Feb 3 16:57:28 2011 -0800
drm: Only set DPMS ON when actually configuring a mode
which fixed the bug of trying to dpms on disabled outputs, but introduced the new discrepancy between an fb update only and full modesets. The actual introduction of this goes back to
commit bf9dc102e284a5aa78c73fc9d72e11d5ccd8669f Author: Keith Packard keithp@keithp.com Date: Fri Nov 26 10:45:58 2010 -0800
drm: Set connector DPMS status to ON in drm_crtc_helper_set_config
And if you'd dig around in the i915 driver code there's even more fun around forcing dpms on and losing our heads and temper of the resulting inconsistencies. Especially the DP re-training code had tons of funny stuff in it.
Signed-off-by: Daniel Vetter daniel.vetter@ffwll.ch Signed-off-by: Dave Airlie airlied@redhat.com
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #8 from Parag parag.warudkar@gmail.com --- Verified : Reverting 372835a8527f85b3eff20a18c2c339e827dfd4e4 from HEAD does make the problem go away.
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #9 from Alex Deucher agd5f@yahoo.com --- Created attachment 82700 --> https://bugs.freedesktop.org/attachment.cgi?id=82700&action=edit possible fix
Does the attached patch fix it?
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Parag parag.warudkar@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED
--- Comment #10 from Parag parag.warudkar@gmail.com --- Tested a couple times with HEAD plus patch in previous comment and it seems to have fixed the issue. Thanks!
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Alexandre Demers alexandre.f.demers@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Atombios stuck in a look |Atombios stuck in a loop |during resume |during resume
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Alex Deucher agd5f@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |johannes.hirte@fem.tu-ilmen | |au.de
--- Comment #11 from Alex Deucher agd5f@yahoo.com --- *** Bug 66767 has been marked as a duplicate of this bug. ***
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Mike Lothian mike@fireburn.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED |--- CC| |mike@fireburn.co.uk
--- Comment #12 from Mike Lothian mike@fireburn.co.uk --- This is happening still on a Trinity 7500G
00:01.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Trinity [Radeon HD 7500G] [1002:990a]
Even with this patch
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #13 from Mike Lothian mike@fireburn.co.uk --- Created attachment 83581 --> https://bugs.freedesktop.org/attachment.cgi?id=83581&action=edit Dmesg
This is running with the latest drm-fixes-3.11 branch with the above fix
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #14 from Parag parag.warudkar@gmail.com --- Note that the original one and its duplicate 66767 reported in this thread happened on suspend/resume whereas this one doesn't seem to be suspend/resume related.
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #15 from Mike Lothian mike@fireburn.co.uk --- I was actually in a VT at the time I think when this normally happens X is up and the whole laptop appears to freeze up
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Alex Deucher agd5f@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution|--- |FIXED
--- Comment #16 from Alex Deucher agd5f@yahoo.com --- I think you may be seeing a different issue that just happens to have a similar sympotom. Please open a new bug. This particular issue is fixed by the referenced patch.
dri-devel@lists.freedesktop.org