Bug 75651

Summary:	resume from suspend failing 3.15(rc1,rc2,rc3,rc4) bisected
Product:	Drivers	Reporter:	William Shuman (wshuman3)
Component:	Video(DRI - non Intel)	Assignee:	drivers_video-dri
Status:	RESOLVED PATCH_ALREADY_AVAILABLE
Severity:	high	CC:	deathsimple, wshuman3
Priority:	P1
Hardware:	All
OS:	Linux
Kernel Version:	3.15	Subsystem:
Regression:	No	Bisected commit-id:
Attachments:	bisect log dmesg (rc5) Possible fix dmesg after patch Possible fix.

Description William Shuman 2014-05-07 13:17:16 UTC

Created attachment 135311 [details]
bisect log

resume from suspend is failing for me since 3.15rc1

6d2f2944e95e504a7d33385eeeb9bb7fcca72592 is the first bad commit
commit 6d2f2944e95e504a7d33385eeeb9bb7fcca72592
Author: Christian König <christian.koenig@amd.com>
Date:   Thu Feb 20 13:42:17 2014 +0100

    drm/radeon: use normal BOs for the page tables v4
    
    No need to make it more complicated than necessary,
    just allocate the page tables as normal BO and
    flush whenever the address change.
    
    v2: update comments and function name
    v3: squash bug fixes, page directory and tables patch
    v4: rebased on Mareks changes
    
    Signed-off-by: Christian König <christian.koenig@amd.com>

:040000 040000 c7fa0e5e797db0e3384b2cd1450bf45efdb4c284 b05fc80d14d6a34db744ecee5d89035fa99ce1fe M	drivers

Comment 1 William Shuman 2014-05-10 15:57:12 UTC

3.15rc5 seems to be better I can get into a tty and restart X.  But intially it still fails.  Attaching dmesg.

Comment 2 William Shuman 2014-05-10 15:57:42 UTC

Created attachment 135721 [details]
dmesg (rc5)

Comment 3 Christian König 2014-05-10 16:07:02 UTC

Already working on this, looks like swapping out page tables doesn't work reliable.

Comment 4 Christian König 2014-05-10 16:40:54 UTC

Created attachment 135731 [details]
Possible fix

Complete shoot into the dark, but please give the attached patch a try.

Comment 5 William Shuman 2014-05-10 18:30:22 UTC

Same behavior after patch.

Comment 6 William Shuman 2014-05-10 18:30:43 UTC

Created attachment 135741 [details]
dmesg after patch

Comment 7 Christian König 2014-05-12 12:48:41 UTC

Created attachment 135881 [details]
Possible fix.

Please try this one instead. It fixes an issue with the commit mentioned, so it might fix this one as well.

Comment 8 William Shuman 2014-05-12 13:05:19 UTC

This fixed this issue.  Thanks for your work.

Comment 9 Christian König 2014-05-12 13:21:04 UTC

Thanks for the info, so we can close this bug now?