r300g: allow resolutions up to 1280x1024 with AA optimizations on 1-pipe cards
authorMarek Olšák <maraeo@gmail.com>
Sun, 20 Jan 2013 15:52:17 +0000 (16:52 +0100)
committerMarek Olšák <maraeo@gmail.com>
Mon, 21 Jan 2013 16:00:51 +0000 (17:00 +0100)
because single-pipe cards have bigger CMASK RAM

src/gallium/drivers/r300/r300_chipset.h
src/gallium/drivers/r300/r300_texture_desc.c

index 996491e..f8b5d4e 100644 (file)
@@ -30,7 +30,6 @@
 #define RV530_HIZ_LIMIT 15360
 
 /* rv3xx have only one pipe */
-#define PIPE_CMASK_SIZE 4096
 #define PIPE_ZMASK_SIZE 4096
 #define RV3xx_ZMASK_SIZE 5120
 
index 889b8e8..c5636ca 100644 (file)
@@ -414,7 +414,7 @@ static void r300_setup_cmask_properties(struct r300_screen *screen,
 {
     static unsigned cmask_align_x[4] = {16, 32, 48, 32};
     static unsigned cmask_align_y[4] = {16, 16, 16, 32};
-    unsigned pipes, stride, cmask_num_dw;
+    unsigned pipes, stride, cmask_num_dw, cmask_max_size;
 
     /* We need an AA colorbuffer, no mipmaps. */
     if (tex->b.b.nr_samples <= 1 ||
@@ -436,6 +436,10 @@ static void r300_setup_cmask_properties(struct r300_screen *screen,
     /* CMASK is part of raster pipes. The number of Z pipes doesn't matter. */
     pipes = screen->info.r300_num_gb_pipes;
 
+    /* The single-pipe cards have 5120 dwords of CMASK RAM,
+     * the other cards have 4096 dwords of CMASK RAM per pipe. */
+    cmask_max_size = pipes == 1 ? 5120 : pipes * 4096;
+
     stride = r300_stride_to_width(tex->b.b.format,
                                   tex->tex.stride_in_bytes[0]);
     stride = align(stride, 16);
@@ -446,7 +450,7 @@ static void r300_setup_cmask_properties(struct r300_screen *screen,
                                          cmask_align_y[pipes-1]);
 
     /* Check the CMASK size against the CMASK memory limit. */
-    if (cmask_num_dw <= PIPE_CMASK_SIZE * pipes) {
+    if (cmask_num_dw <= cmask_max_size) {
         tex->tex.cmask_dwords = cmask_num_dw;
         tex->tex.cmask_stride_in_pixels =
             util_align_npot(stride, cmask_align_x[pipes-1]);