Mesa (master): i965: Move cache_used_by_gpu flag setting to brw_finish_batch.

Kenneth Graunke kwg at kemper.freedesktop.org
Fri Nov 15 19:00:02 UTC 2013


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Tue Oct 15 19:21:34 2013 -0700

i965: Move cache_used_by_gpu flag setting to brw_finish_batch.

It really makes more sense here.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/drivers/dri/i965/intel_batchbuffer.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
index 6d1ae79..9cdbe9e 100644
--- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c
+++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
@@ -198,12 +198,6 @@ brw_new_batch(struct brw_context *brw)
 
    brw->ib.type = -1;
 
-   /* Mark that the current program cache BO has been used by the GPU.
-    * It will be reallocated if we need to put new programs in for the
-    * next batch.
-    */
-   brw->cache.bo_used_by_gpu = true;
-
    /* We need to periodically reap the shader time results, because rollover
     * happens every few seconds.  We also want to see results every once in a
     * while, because many programs won't cleanly destroy our context, so the
@@ -232,6 +226,12 @@ brw_finish_batch(struct brw_context *brw)
       drm_intel_bo_unreference(brw->curbe.curbe_bo);
       brw->curbe.curbe_bo = NULL;
    }
+
+   /* Mark that the current program cache BO has been used by the GPU.
+    * It will be reallocated if we need to put new programs in for the
+    * next batch.
+    */
+   brw->cache.bo_used_by_gpu = true;
 }
 
 /* TODO: Push this whole function into bufmgr.




More information about the mesa-commit mailing list