Bug 2329

Summary: i815 Xserver crashes on console switch w/ DRI enabled
Product: xorg Reporter: Matthias Hopf <mat>
Component: Driver/intelAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high CC: eich, roland.mainz
Version: git   
Hardware: x86 (IA32)   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
[FIXED_X11R68x] Fix for console switching bug roland.mainz: 6.8-branch+

Description Matthias Hopf 2005-01-19 09:08:32 UTC
I have a i815 system here that reproducably crashes when DRI is enabled. One can
switch to a console w/o problems, but as soon as one switches back to the
Xserver, it freezes, exits, and a new Xserver instance is started. This can be
repeated over and over again.

As it does not kill the machine, I do not set this to major.


The error message in the logfile:

Fatal server error:
EnterVT failed for screen 0

Please consult the The X.Org Foundation support 
         at http://wiki.X.Org
 for help. 
Please also check the log file at "/var/log/Xorg.0.log" for additional information.

Error in I810WaitLpRing(), now is 92443, start is 90442
pgetbl_ctl: 0x6540001 pgetbl_err: 0x0
ipeir: 0 iphdr: 0
LP ring tail: 8 head: 0 len: 0 start 0
eir: 0 esr: 0 emr: 3d
instdone: ff7b instpm: 0
memmode: 4 instps: 0
hwstam: 9ac7 ier: 0 imr: 9ac7 iir: 0
space: 65520 wanted 65528
(II) I810(0): [drm] removed 1 reserved context for kernel
(II) I810(0): [drm] unmapping 8192 bytes of SAREA 0xc89b0000 at 0xb7d9d000

FatalError re-entered, aborting
lockup
Comment 1 Matthias Hopf 2005-01-19 09:10:07 UTC
Created attachment 1717 [details] [review]
[FIXED_X11R68x] Fix for console switching bug

This small patch fixes the console switching bug for me. Appearantly it was
forgotten to release some amount of AGP memory, but requested again upon
VTEnter.
Comment 2 Egbert Eich 2005-01-24 07:02:51 UTC
Committed.

2005-01-24  Egbert Eich  <eich-at-freedesktop-dot-org>
        * programs/Xserver/hw/xfree86/drivers/i810/i810_dri.c:
        (I810DRILeave):
        Unmap XCursor AGP memory on VT switch to prevent system from lockup
        (Matthias Hopf).
Comment 3 Egbert Eich 2005-01-24 10:27:00 UTC
I've tried to test this. Unfortunately the kernel on my i810 box seems to be too
old. I'm unable to map memory for the hw cursor.
Comment 4 Roland Mainz 2005-01-24 16:15:41 UTC
Comment on attachment 1717 [details] [review]
[FIXED_X11R68x] Fix for console switching bug

Approval for commit into X11R6.8.x stable branch granted in the 2005-01-24
release-wranglers phone call (please don't commit, I'll do that myself in a few
mins).
Comment 5 Roland Mainz 2005-01-24 16:23:14 UTC
Comment on attachment 1717 [details] [review]
[FIXED_X11R68x] Fix for console switching bug

Patch checked-in into X11R6.8.x stable branch:
/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.365.2.132; previous revision: 1.365.2.131
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_dri.c,v  <-- 
i810_dri.c
new revision: 1.4.4.2; previous revision: 1.4.4.1
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
Mailing the commit message to xorg-commit@lists.freedesktop.org...

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.