Bug 1944

Summary: Various I8xx driver fixes
Product: xorg Reporter: Egbert Eich <eich>
Component: Driver/intelAssignee: Egbert Eich <eich>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high CC: roland.mainz
Version: 6.8.1   
Hardware: x86 (IA32)   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
[FIXED_X11R68x] Fix roland.mainz: 6.8-branch+

Description Egbert Eich 2004-11-26 07:29:16 UTC
This patch includes:
Change order of SetDisplayDevice(), HWRestore(), UnbindGART()
and RestoreBIOSMemSize() to be exactly opposite to the Save
procedure in EnterVT(). Without this console restoration may not work
(provided by Matthias Hopf, Alan Hourihane).

Fix for default amount of of allocated video memory from

Fix for server crash on reset when a structure allocated in
PreInit() was freed on CloseScreen().

AGP for i810: Use 16MB if less than 192MB are installed

Fixed numerous warnings due to signed unsigned comparisons.
Comment 1 Egbert Eich 2004-11-26 07:31:06 UTC
Created attachment 1403 [details] [review]
[FIXED_X11R68x] Fix

See comment above.
Submitted to CVS already. Only for 6.8.2 nomination.
Comment 2 Roland Mainz 2004-12-08 05:58:30 UTC
Comment on attachment 1403 [details] [review]
[FIXED_X11R68x] Fix

Approved in the 2004-12-06 release-wranglers conference call.
Please do not commit it yourself, I'll do that later today...
Comment 3 Roland Mainz 2004-12-15 01:35:43 UTC
Comment on attachment 1403 [details] [review]
[FIXED_X11R68x] Fix

Patch checked-in into X11R6.8.x stable branch:

/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.365.2.90; previous revision: 1.365.2.89
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/common.h,v  <--  common.h
new revision: 1.3.4.1; previous revision: 1.3
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810.h,v	<--  i810.h
new revision: 1.4.4.1; previous revision: 1.4
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_accel.c,v  <-- 
i810_accel.c
new revision: 1.4.2.1; previous revision: 1.4
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_dri.c,v  <-- 
i810_dri.c
new revision: 1.4.4.1; previous revision: 1.4
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_driver.c,v  <-- 
i810_driver.c
new revision: 1.9.2.1; previous revision: 1.9
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_memory.c,v  <-- 
i810_memory.c
new revision: 1.4.2.1; previous revision: 1.4
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i810_reg.h,v  <-- 
i810_reg.h
new revision: 1.3.4.1; previous revision: 1.3
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i830.h,v	<--  i830.h
new revision: 1.5.4.1; previous revision: 1.5
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i830_driver.c,v  <-- 
i830_driver.c
new revision: 1.6.2.1; previous revision: 1.6
/cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/i810/i830_memory.c,v  <-- 
i830_memory.c
new revision: 1.4.4.1; previous revision: 1.4
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...
Comment 4 Roland Mainz 2004-12-15 01:36:58 UTC
Patch checked-in into Xorg trunk and X11R6.8.x stable branch...

... marking bug as FIXED (please reopen if there are any issues left) ...

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.