Bug 11720 - [945GM S3] resume crashes X
Summary: [945GM S3] resume crashes X
Status: RESOLVED DUPLICATE of bug 10798
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: 7.2 (2007.02)
Hardware: x86 (IA32) Linux (All)
: medium major
Assignee: Jesse Barnes
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
: 13202 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-07-25 01:46 UTC by Johannes Engel
Modified: 2007-12-11 13:52 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments
log of the crash (49.31 KB, text/plain)
2007-07-25 01:47 UTC, Johannes Engel
no flags Details
Suspend/resume support for i915 DRM driver (53.44 KB, text/x-patch)
2007-10-30 10:15 UTC, Jesse Barnes
no flags Details
last resume-crash with libdrm from git (42.09 KB, text/plain)
2007-11-21 07:28 UTC, Roland Tapken
no flags Details
gdb backtrace of the crash (6.17 KB, text/plain)
2007-11-22 11:58 UTC, Roland Tapken
no flags Details
gdb backtrace with debugging symbols (7.94 KB, text/plain)
2007-11-23 05:45 UTC, Roland Tapken
no flags Details

Description Johannes Engel 2007-07-25 01:46:54 UTC
Resuming from suspend2ram my laptop using an intel 945G chip tries to restart X.Org several times before it goes back to console. I will attach the log of one of these attempts.
I cannot start the X-Server by hand afterwards.
The problem occurs only if drm is compiled in kernel (not as a module).

Kernel 2.6.22.1
X.Org 7.2.0
intel-driver 2.1.0
Comment 1 Johannes Engel 2007-07-25 01:47:39 UTC
Created attachment 10876 [details]
log of the crash
Comment 2 Michael Fu 2007-10-25 18:59:46 UTC
a dup of bug# 11029?
Comment 3 Gordon Jin 2007-10-25 19:05:43 UTC
No, bug#11720 is in fact no hurt just with some error messages.
Comment 4 Gordon Jin 2007-10-25 19:06:49 UTC
Ooops, I should say bug#11029 is in fact no hurt just with some error messages.
Comment 5 Jesse Barnes 2007-10-30 10:11:41 UTC
Johannes, can you try the attached patch?  If it doesn't apply easily, you could also try the latest DRM tree (git://git.freedesktop.org/git/mesa/drm).

It fixes some of the suspend/resume problems people have been seeing.

There are also some fixes in the latest X driver (git://git.freedesktop.org/git/xorg/driver/xf86-video-intel) that might help in your case.
Comment 6 Jesse Barnes 2007-10-30 10:15:23 UTC
Created attachment 12265 [details]
Suspend/resume support for i915 DRM driver
Comment 7 Gordon Jin 2007-11-12 18:24:07 UTC
*** Bug 13202 has been marked as a duplicate of this bug. ***
Comment 8 Jesse Barnes 2007-11-13 12:58:06 UTC
Ok, given that 9xx suspend/resume (and even 8xx suspend/resume) is working for lots of people now, I think we can assume this one is fixed.  Please reopen or file a new bug if you find any problems.

Thanks,
Jesse
Comment 9 Roland Tapken 2007-11-21 07:27:16 UTC
Sorry, i don't think that this bug is resolved. See my report in bug 10798, it seems to be the same error. I just tried libdrm from git, but nothing changed.
Comment 10 Roland Tapken 2007-11-21 07:28:56 UTC
Created attachment 12669 [details]
last resume-crash with libdrm from git
Comment 11 Roland Tapken 2007-11-21 09:49:39 UTC
X doesn't crash when setting option "NoAccel" to true. But... no dri, no compiz and very slow window redraw (e.g. when moving windows). Just a temporary solution for me.
Comment 12 Jesse Barnes 2007-11-22 08:32:48 UTC
Can you get a backtrace of the crash?  There are some instructions that might help at http://wiki.x.org/wiki/Development/Documentation/ServerDebugging?highlight=%28debugging%29
Comment 13 Roland Tapken 2007-11-22 11:58:34 UTC
Created attachment 12692 [details]
gdb backtrace of the crash

I was not able to use gdb with the existing server after resume, but I found another way to reproduce the bug:

- Start X
- Kill X
- Suspend
- Resume
- Start X with GDB (options -dumpShed and -kb)

Please tell me if you need additional info.
Comment 14 Roland Tapken 2007-11-23 05:45:33 UTC
Created attachment 12702 [details]
gdb backtrace with debugging symbols

Just learned how to create backtraces with gdb-symbols under gentoo. Not to myself: "debug"-useflag does NOT mean to include debugging symbols!
Comment 15 Roland Tapken 2007-11-23 09:41:42 UTC
Sorry, I forgot to mention: the original crash with X running in background while suspending occurs when resuming and switching from vt1 to vt7. In my backtrace-szenario, I started the server on vt7 and switched back to vt1 -> crash.
Comment 16 Jesse Barnes 2007-11-29 13:29:34 UTC
Roland, does this problem still occur with the 2.2 driver release?

And just to clarify a few things, does regular VT switching work before you suspend?  The server only crashes after you resume your machine, regardless of whether it was running before or not?

Comment 17 Roland Tapken 2007-11-29 14:35:22 UTC
The server doesn't crash on normal vt-switch, but only since i applied the patch from the following report: http://bugs.gentoo.org/show_bug.cgi?id=191822. The resume-crash did not changed.

The patch can be reproduced in the following way:

* Start system without X
* optional: suspend and resume as often you want to.
* start and exit X, as often as you want to, no problems. Feel free to switch vt.
* suspend and resume the system
* start x -> crash

This is slightly different to the way used in my backtrace, but the error message in Xorg.0.log is the same, so I think this is the same problem.
Comment 18 Roland Tapken 2007-11-29 14:36:49 UTC
Oh, and yes, I just used the current git version for testing. So I think the bug is not fixed in 2.2.
Comment 19 Jesse Barnes 2007-12-04 17:00:20 UTC
Upping severity
Comment 20 Jesse Barnes 2007-12-11 13:52:43 UTC
I think there are actually two bugs here, the one reported by Johannes (which should be fixed now) and the one Roland is talking about, which I think is a DUP of 10798, so I'm going to mark this one as such.

*** This bug has been marked as a duplicate of bug 10798 ***


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.