Bug 3769 - MAXFORMATS needs to die
Summary: MAXFORMATS needs to die
Status: NEW
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/DDX/Xorg (show other bugs)
Version: 6.9.0
Hardware: Other Linux (All)
: high major
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-14 08:30 UTC by Stuart Anderson
Modified: 2007-11-16 17:18 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stuart Anderson 2005-07-14 08:30:23 UTC
The change for bug #2791 to add support for 12 & 30 bit depths causes more 
than MAXFORMATS depths to be created when RENDER is enabled. That overflows 
GCperDepth in the Screen structure and causes a crash. 
 
 screen #0: 
   dimensions:    1024x768 pixels (260x195 millimeters) 
   resolution:    100x100 dots per inch 
   depths (9):    1, 4, 8, 12, 16, 24, 30, 32, 8 
 
an additional questions, is why is the screen's depth listed twice? 
 
Is it safe to just increase MAXFORMATS? or is there a more appropriate 
solution?
Comment 1 Adam Jackson 2005-11-19 04:01:34 UTC
12bpp and 30bpp are disabled in HEAD now, so this isn't an 7.0 issue.
Comment 2 Adam Jackson 2006-04-25 05:40:51 UTC
(In reply to comment #1)
> 12bpp and 30bpp are disabled in HEAD now, so this isn't an 7.0 issue.

Likewise for 7.1, moving out to 7.2.
Comment 3 Daniel Stone 2006-11-04 09:53:27 UTC
... and for 7.2 ...
Comment 4 Daniel Stone 2007-02-27 01:27:18 UTC
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 5 Adam Jackson 2007-11-16 17:18:48 UTC
I'm working on removing MAXFORMATS from the ABI structures.

Updating summary to reflect reality.


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.