On Tue, Mar 16, 2004 at 01:52:39PM -0300, Rafael Ávila de Espíndola wrote:
> The xserver-xfree86.config file tests if the driver is a i810, and if it is,
> sets the default VideoRam to 8192. This will prevent dri from working since
> it needs more ram. I think the VideoRam option should not be set by default
> when the driver is the i810. According to
> http://www.xfree86.org/~dawes/845driver.html, in the absence of this option
> the driver "defaults to 32MB of VideoRam when the DRI is enabled (and 8MB
> when DRI is not enabled)"
Thanks for pointing this out. I've checked the 4.3.0 source code, and
verified that you're correct for the i830_driver.c (which is used by the
i830 chipset and all supported later models).
For i810_driver.c, the VideoRam default inside the driver is now 8 megs.
824 /* Default to 4MB framebuffer, which is sufficient for all
825 * supported 2d resolutions. If the user has specified a different
826 * size in the XF86Config, use that amount instead.
827 *
828 * Changed to 8 Meg so we can have acceleration by default (Mark).
829 */
830 pScrn->videoRam = 8192;
831 from = X_DEFAULT;
832 if (pI810->pEnt->device->videoRam) {
833 pScrn->videoRam = pI810->pEnt->device->videoRam;
834 from = X_CONFIG;
835 }
836
837 mem = I810CheckAvailableMemory(pScrn);
838 if (mem > 0 && mem < pScrn->videoRam) {
839 xf86DrvMsg(pScrn->scrnIndex, X_WARNING, "%dk of memory was requested,"
840 " but the\n\t maximum AGP memory available is %dk.\n",
841 pScrn->videoRam, mem);
842 from = X_PROBED;
843 if (mem > (6 * 1024)) {
844 xf86DrvMsg(pScrn->scrnIndex, X_INFO,
845 "Reducing video memory to 4MB\n");
846 pScrn->videoRam = 4096;
847 } else {
848 xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "Less than 6MB of AGP memory"
849 "is available. Cannot proceed.\n");
850 I810FreeRec(pScrn);
851 return FALSE;
852 }
853 }
854
855 xf86DrvMsg(pScrn->scrnIndex, from,
856 "Will alloc AGP framebuffer: %d kByte\n", pScrn->videoRam);
Therefore, I agree with your recommendation.
--
G. Branden Robinson | Kissing girls is a goodness. It is
Debian GNU/Linux | a growing closer. It beats the
branden@debian.org | hell out of card games.
http://people.debian.org/~branden/ | -- Robert Heinlein
Attachment:
signature.asc
Description: Digital signature