Some questions about xf86-video-intel driver

[Simon Pasch]

Eric Anholt wrote:
> If you want to decrease the amount of memory allocated in AGP, you'll
> need to reduce the size of the allocations the X Server does.  We try to
> choose reasonable allocations by default, and you can see what exactly
> we do in the "Memory Allocation Layout" printout in Xorg.0.log.
> 

Alright. So I hard-coded the pI830->FbMapSize-Value to 32MB.
Is this solution acceptable? Or could I get into problems with that?

--- i830_driver.c.old	2007-10-23 09:29:17.000000000 +0200
+++ i830_driver.c	2007-10-23 09:29:34.000000000 +0200
@@ -1218,8 +1218,7 @@
        }
     } else {
        if (IS_I9XX(pI830)) {
-	 pI830->FbMapSize = 1UL << pciGetBaseSize(pI830->PciTag, 2,TRUE,
-						  NULL);
+	 pI830->FbMapSize = 0x2000000;
        } else {
  	 /* 128MB aperture for later i8xx series. */
  	 pI830->FbMapSize = 0x8000000;



The memory allocation layout looks like that now:
(II) intel(0): Memory allocation layout:
(II) intel(0): 0x00000000-0x0001ffff: ring buffer (128 kB)
(II) intel(0): 0x00020000-0x00029fff: HW cursors (40 kB, 0x 
1f820000 physical)
(II) intel(0): 0x0002a000-0x00031fff: logical 3D context (32 kB)
(II) intel(0): 0x00032000-0x00032fff: overlay registers (4 kB, 0x 
  1f832000 physical)
(II) intel(0): 0x00040000-0x01258fff: front buffer (18532 kB)
(II) intel(0): 0x007f7000:            end of stolen memory
(II) intel(0): 0x01259000-0x01268fff: xaa scratch (64 kB)
(II) intel(0): 0x02000000:            end of aperture