Bug 70053 - hard machine hang when switching to battery power with DPM enabled on Trinity APU
Summary: hard machine hang when switching to battery power with DPM enabled on Trinity...
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Radeon (show other bugs)
Version: DRI git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-02 20:37 UTC by Lucas Stach
Modified: 2014-05-21 11:59 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
dmesg before hang (94.64 KB, text/plain)
2013-10-02 20:37 UTC, Lucas Stach
no flags Details
Xorg.0.log (44.09 KB, text/plain)
2013-10-02 20:38 UTC, Lucas Stach
no flags Details
possible fix (1.48 KB, patch)
2013-10-02 21:47 UTC, Alex Deucher
no flags Details | Splinter Review
force battery state on DC (1.54 KB, patch)
2013-10-03 14:11 UTC, Alex Deucher
no flags Details | Splinter Review
better patch (2.29 KB, patch)
2013-10-03 14:29 UTC, Alex Deucher
no flags Details | Splinter Review

Description Lucas Stach 2013-10-02 20:37:52 UTC
Created attachment 87002 [details]
dmesg before hang

When booting with DPM enabled the machine hard hangs when switching from AC to battery power or when booting without AC connected as soon as GDM starts.

I can't reach it via ssh after the hang, so only dmesg before hang provided. Kernel is 3.12-rc2 with agd5f/linux/drm-fixes-3.12 merged in.
Comment 1 Lucas Stach 2013-10-02 20:38:19 UTC
Created attachment 87003 [details]
Xorg.0.log
Comment 2 Alex Deucher 2013-10-02 21:47:57 UTC
Created attachment 87006 [details] [review]
possible fix

Does this patch help?  Seems like maybe your system isn't getting ac events properly.  Can you attach the dmesg output of this patch?
Comment 3 Lucas Stach 2013-10-02 22:10:51 UTC
Yes, the patch fixes the immediate lockup. But it seems your analysis on why it fails isn't correct as I can now see your printks toggling between AC and DC when plugging in/out external power.
Comment 4 Alex Deucher 2013-10-03 14:11:17 UTC
Created attachment 87033 [details] [review]
force battery state on DC

Does this patch also fix the problem?
Comment 5 Alex Deucher 2013-10-03 14:29:22 UTC
Created attachment 87044 [details] [review]
better patch

Improved patch.  Please try this one instead.
Comment 6 Lucas Stach 2013-10-04 13:23:12 UTC
No this patch (v2) does not solve the problem.
Comment 7 Lucas Stach 2014-05-21 11:59:56 UTC
Already fixed in 3.12 final.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.