Mesa (master): llvmpipe: don't clear unused bins

Keith Whitwell keithw at kemper.freedesktop.org
Sun Aug 22 13:56:02 UTC 2010


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

Author: Keith Whitwell <keithw at vmware.com>
Date:   Sun Aug 22 11:43:01 2010 +0100

llvmpipe: don't clear unused bins

If bins outside the current scene bounds are being corrupted, we'll
need to fix that separately.  Currently seems ok though.

---

 src/gallium/drivers/llvmpipe/lp_scene.c |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/llvmpipe/lp_scene.c b/src/gallium/drivers/llvmpipe/lp_scene.c
index f88a759..15a09b7 100644
--- a/src/gallium/drivers/llvmpipe/lp_scene.c
+++ b/src/gallium/drivers/llvmpipe/lp_scene.c
@@ -163,12 +163,15 @@ lp_scene_reset(struct lp_scene *scene )
 
    /* Free all but last binner command lists:
     */
-   for (i = 0; i < TILES_X; i++) {
-      for (j = 0; j < TILES_Y; j++) {
+   for (i = 0; i < scene->tiles_x; i++) {
+      for (j = 0; j < scene->tiles_y; j++) {
          lp_scene_bin_reset(scene, i, j);
       }
    }
 
+   /* If there are any bins which weren't cleared by the loop above,
+    * they will be caught (on debug builds at least) by this assert:
+    */
    assert(lp_scene_is_empty(scene));
 
    /* Free all but last binned data block:




More information about the mesa-commit mailing list