Bug 17221 - Man page patch to clarify meaning of VideoRam option with i810/i815
Summary: Man page patch to clarify meaning of VideoRam option with i810/i815
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: git
Hardware: All All
: medium normal
Assignee: Wang Zhenyu
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-20 03:10 UTC by Fabio Pedretti
Modified: 2008-08-31 22:38 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Man page patch to clarify meaning of VideoRam option with i810/i815 (1.38 KB, patch)
2008-08-20 03:10 UTC, Fabio Pedretti 		no flags Details | Splinter Review
Updated patch: VideoRam is not an "Option", also note that DRI appears to require at least 16 MB. (1.46 KB, patch)
2008-08-20 03:53 UTC, Fabio Pedretti 		no flags Details | Splinter Review
Man page patch to clarify meaning of VideoRam option with i810/i815 (1.55 KB, patch)
2008-08-21 01:33 UTC, Fabio Pedretti 		no flags Details | Splinter Review
Show Obsolete (2) View All

Description Fabio Pedretti 2008-08-20 03:10:10 UTC 
Created attachment 18404 [details] [review]
Man page patch to clarify meaning of VideoRam option with i810/i815

The man page says that VideoRam for i810 is always 8 megabytes. This is not true, i810_driver.c specifies different amount of VideoRam based on actual RAM size. Later, it also says that this parameter is ignored, but this is related only to i830 and laters. This was causing some confusion on users, see for example https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-intel/+bug/243991 . 

This patch corrects this and also add the Option VideoRam in the i810 and i815 chipsets driver Options.
Comment 1 Fabio Pedretti 2008-08-20 03:53:34 UTC 
Created attachment 18405 [details] [review]
Updated patch: VideoRam is not an "Option", also note that DRI appears to require at least 16 MB.

Updated patch:

VideoRam has to be used with
VideoRam integer
and not as
Option VideoRam integer

Also, I noticed that DRI is enabled only when VideoRam is at least about 16000. This is now added as a comment.
Comment 2 Fabio Pedretti 2008-08-21 01:33:05 UTC 
Created attachment 18437 [details] [review]
Man page patch to clarify meaning of VideoRam option with i810/i815

Some cleanups.
Comment 3 Gordon Jin 2008-08-26 18:22:23 UTC 
Jesse thinks this reasonable. Zhenyu, any comments?
Comment 4 Wang Zhenyu 2008-08-31 22:38:36 UTC 
Thanks, pushed this.

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.