[Mesa-dev] [PATCH] i965: Use WARN_ONCE instead of open coding it.

Kenneth Graunke kenneth at whitecape.org
Wed Mar 29 04:50:54 UTC 2017


---
 src/mesa/drivers/dri/i965/brw_compute.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_compute.c b/src/mesa/drivers/dri/i965/brw_compute.c
index 16b5df7ca4e..21da2ad17b5 100644
--- a/src/mesa/drivers/dri/i965/brw_compute.c
+++ b/src/mesa/drivers/dri/i965/brw_compute.c
@@ -219,15 +219,10 @@ brw_dispatch_compute_common(struct gl_context *ctx)
          fail_next = true;
          goto retry;
       } else {
-         if (intel_batchbuffer_flush(brw) == -ENOSPC) {
-            static bool warned = false;
-
-            if (!warned) {
-               fprintf(stderr, "i965: Single compute shader dispatch "
-                       "exceeded available aperture space\n");
-               warned = true;
-            }
-         }
+         int ret = intel_batchbuffer_flush(brw);
+         WARN_ONCE(ret == -ENOSPC,
+                   "i965: Single compute shader dispatch "
+                   "exceeded available aperture space\n");
       }
    }
 
-- 
2.12.1



More information about the mesa-dev mailing list