Ian Romanick wrote:
Manuel Bilderbeek wrote:

(II) RADEON(0): [agp] GART texture map handle = 0xe0302000 (II)
RADEON(0): Using 5 MB for GART textures (II) RADEON(0): Will use
2752 kb for textures at offset 0x1d50000

The card has 32MB RAM and I'm usually running at 1600x1200x24....
is this reasonable? It seems the Radeon driver doesn't have the
fallbacks you mention, then?


Of that 32MB, almost 23MB are taken up by your framebuffer and depthbuffer. That doesn't leave a lot of room for textures. You
should try increasing your GART memory to 32MB or 64MB. Add a line
like:


Option "GARTSize" "64M"

to your device section.
This doesn't work for me, the driver ignores all values supplied to that
parameter (dri tree). It accepts though values supplied to the old,
deprecated (?) "AGPSize" option.
Quick fix attached. A simpler option would be to axe support for AGPSize...
It doesn't seem to change the reported maximum texture sizes though for
the r200 at least (could be different on radeon I guess), does it
actually support using agp memory for texturing? Some environment
variable suggests that so though.


Can we make the driver use a larger GART size by default when DRI is
 enabled?  5MB is just about worthless.  I would suggest 16MB as a
bare minimum.
Couldn't it just use the largest GART size possible (set by the bios),
or would this have some negative consequences? I vaguely remember there
was some discussion about gart size some time before, but can't remember
any details. Currently, if you set the gart size manually higher than
what's possible (set in bios), dri will just get disabled due to missing agp support, which I consider bad behaviour, and that you get a useless error message in that case doesn't help neither.
(II) RADEON(0): [agp] 262144 kB allocated with handle 0x00000001
(EE) RADEON(0): [agp] Could not bind
(EE) RADEON(0): [agp] AGP failed to initialize. Disabling the DRI.
(II) RADEON(0): [agp] You may want to make sure the agpgart kernel module is loaded before the radeon kernel module.


Roland


Index: radeon_driver.c
===================================================================
RCS file: /cvs/dri/xc/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c,v
retrieving revision 1.86
diff -u -r1.86 radeon_driver.c
--- a/radeon_driver.c   29 Mar 2004 14:55:11 -0000      1.86
+++ b/radeon_driver.c   4 Jun 2004 01:50:30 -0000
@@ -132,6 +132,7 @@
     OPTION_AGP_MODE,
     OPTION_AGP_FW,
     OPTION_GART_SIZE,
+    OPTION_GART_SIZE_OLD,
     OPTION_RING_SIZE,
     OPTION_BUFFER_SIZE,
     OPTION_DEPTH_MOVE,
@@ -167,7 +168,7 @@
     { OPTION_USEC_TIMEOUT,   "CPusecTimeout",    OPTV_INTEGER, {0}, FALSE },
     { OPTION_AGP_MODE,       "AGPMode",          OPTV_INTEGER, {0}, FALSE },
     { OPTION_AGP_FW,         "AGPFastWrite",     OPTV_BOOLEAN, {0}, FALSE },
-    { OPTION_GART_SIZE,      "AGPSize",          OPTV_INTEGER, {0}, FALSE },
+    { OPTION_GART_SIZE_OLD,  "AGPSize",          OPTV_INTEGER, {0}, FALSE },
     { OPTION_GART_SIZE,      "GARTSize",         OPTV_INTEGER, {0}, FALSE },
     { OPTION_RING_SIZE,      "RingSize",         OPTV_INTEGER, {0}, FALSE },
     { OPTION_BUFFER_SIZE,    "BufferSize",       OPTV_INTEGER, {0}, FALSE },
@@ -3790,8 +3791,10 @@
        }
     }
 
-    if (xf86GetOptValInteger(info->Options,
-                            OPTION_GART_SIZE, (int *)&(info->gartSize))) {
+    if ((xf86GetOptValInteger(info->Options,
+                            OPTION_GART_SIZE, (int *)&(info->gartSize))) ||
+                            (xf86GetOptValInteger(info->Options,
+                            OPTION_GART_SIZE_OLD, (int *)&(info->gartSize)))) {
        switch (info->gartSize) {
        case 4:
        case 8:

Reply via email to