From 876f1537e90685e76a1894f758c63ae9d98a8276 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Wed, 31 Jan 2018 06:47:02 -0800 Subject: [PATCH] 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 8e89963f2f9..8a2b4218608 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]); -- 2.30.2