r300g: do not print a rejected CS if RADEON_DUMP_CS is not set
authorMarek Olšák <maraeo@gmail.com>
Sat, 10 Jul 2010 18:34:29 +0000 (20:34 +0200)
committerMarek Olšák <maraeo@gmail.com>
Sat, 10 Jul 2010 18:46:26 +0000 (20:46 +0200)
Also print relocation failures on non-debug builds too.

src/gallium/winsys/radeon/drm/radeon_drm_buffer.c
src/gallium/winsys/radeon/drm/radeon_r300.c

index cb4ec32feae397cee3da9a5fc436efdccd9fb081..c5f133e7b286877b9681e2cc7854b245079a75e1 100644 (file)
@@ -361,8 +361,8 @@ void radeon_drm_bufmgr_write_reloc(struct pb_buffer *_buf,
     retval = radeon_cs_write_reloc(buf->mgr->rws->cs,
                                   buf->bo, gem_rd, gem_wd, flags);
     if (retval) {
-        debug_printf("radeon: Relocation of %p (%d, %d, %d) failed!\n",
-                    buf, gem_rd, gem_wd, flags);
+        fprintf(stderr, "radeon: Relocation of %p (%d, %d, %d) failed!\n",
+                buf, gem_rd, gem_wd, flags);
     }
 }
 
index 51cfc0fd40027a6b2f387ddf82c5185fce19a7ed..af35497fd7c25d0c9affa882e5ddd47ae1264c50 100644 (file)
@@ -252,8 +252,13 @@ static void radeon_flush_cs(struct r300_winsys_screen *rws)
     /* Emit the CS. */
     retval = radeon_cs_emit(ws->cs);
     if (retval) {
-        debug_printf("radeon: Bad CS, dumping...\n");
-        radeon_cs_print(ws->cs, stderr);
+        if (debug_get_bool_option("RADEON_DUMP_CS", FALSE)) {
+            fprintf(stderr, "radeon: The kernel rejected CS, dumping...\n");
+            radeon_cs_print(ws->cs, stderr);
+        } else {
+            fprintf(stderr, "radeon: The kernel rejected CS, "
+                            "see dmesg for more information.\n");
+        }
     }
 
     /* Reset CS.