Mesa (master): i965: Delete more misleading comments.

Kenneth Graunke kwg at kemper.freedesktop.org
Wed Jan 31 19:34:02 UTC 2018


Module: Mesa
Branch: master
Commit: 876f1537e90685e76a1894f758c63ae9d98a8276
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=876f1537e90685e76a1894f758c63ae9d98a8276

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Wed Jan 31 06:47:02 2018 -0800

i965: Delete more misleading comments.

brw_bo_wait_rendering used to take a brw_context pointer for perf_debug
messages about stalls.  Chris eliminated that in 833108ac14ade91f54cc6e.
This message about passing NULL to avoid those warnings is no longer
relevant, and just adds confusion.  So, drop it.

---

 src/mesa/drivers/dri/i965/intel_batchbuffer.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
index 8e89963f2f..8a2b421860 100644
--- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c
+++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
@@ -804,9 +804,6 @@ throttle(struct brw_context *brw)
    if (brw->need_swap_throttle && brw->throttle_batch[0]) {
       if (brw->throttle_batch[1]) {
          if (!brw->disable_throttling) {
-            /* Pass NULL rather than brw so we avoid perf_debug warnings;
-             * stalling is common and expected here...
-             */
             brw_bo_wait_rendering(brw->throttle_batch[1]);
          }
          brw_bo_unreference(brw->throttle_batch[1]);




More information about the mesa-commit mailing list