[Mesa-dev] [PATCH] i965: Use WARN_ONCE for the single-primitive-exceeded-aperture message.

Kenneth Graunke kenneth at whitecape.org
Tue Dec 30 12:23:04 PST 2014


This makes it show up via ARB_debug_output and is also less code.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
---
 src/mesa/drivers/dri/i965/brw_draw.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_draw.c b/src/mesa/drivers/dri/i965/brw_draw.c
index c581cc0..5cd3f01 100644
--- a/src/mesa/drivers/dri/i965/brw_draw.c
+++ b/src/mesa/drivers/dri/i965/brw_draw.c
@@ -520,15 +520,10 @@ retry:
 	    fail_next = true;
 	    goto retry;
 	 } else {
-	    if (intel_batchbuffer_flush(brw) == -ENOSPC) {
-	       static bool warned = false;
-
-	       if (!warned) {
-		  fprintf(stderr, "i965: Single primitive emit exceeded"
-			  "available aperture space\n");
-		  warned = true;
-	       }
-	    }
+            int ret = intel_batchbuffer_flush(brw);
+            WARN_ONCE(ret == -ENOSPC,
+                      "i965: Single primitive emit exceeded "
+                      "available aperture space\n");
 	 }
       }
 
-- 
2.2.1



More information about the mesa-dev mailing list