Mesa (master): intel: Remove long-dead comment about releasing texture heaps.

Eric Anholt anholt at kemper.freedesktop.org
Mon Jun 14 14:04:19 PDT 2010


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

Author: Eric Anholt <eric at anholt.net>
Date:   Sat Jun 12 21:51:37 2010 -0700

intel: Remove long-dead comment about releasing texture heaps.

BOs are stored in the bufmgr, which is freed as part of the screen
structure.

---

 src/mesa/drivers/dri/intel/intel_context.c |   15 ---------------
 1 files changed, 0 insertions(+), 15 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c
index dec4797..5f2035d 100644
--- a/src/mesa/drivers/dri/intel/intel_context.c
+++ b/src/mesa/drivers/dri/intel/intel_context.c
@@ -827,8 +827,6 @@ intelDestroyContext(__DRIcontext * driContextPriv)
 
    assert(intel);               /* should never be null */
    if (intel) {
-      GLboolean release_texture_heaps;
-
       INTEL_FIREVERTICES(intel);
 
       _mesa_meta_free(&intel->ctx);
@@ -837,7 +835,6 @@ intelDestroyContext(__DRIcontext * driContextPriv)
 
       intel->vtbl.destroy(intel);
 
-      release_texture_heaps = (intel->ctx.Shared->RefCount == 1);
       _swsetup_DestroyContext(&intel->ctx);
       _tnl_DestroyContext(&intel->ctx);
       _vbo_DestroyContext(&intel->ctx);
@@ -855,18 +852,6 @@ intelDestroyContext(__DRIcontext * driContextPriv)
       drm_intel_bo_unreference(intel->first_post_swapbuffers_batch);
       intel->first_post_swapbuffers_batch = NULL;
 
-      if (release_texture_heaps) {
-         /* Nothing is currently done here to free texture heaps;
-          * but we're not using the texture heap utilities, so I
-          * rather think we shouldn't.  I've taken a look, and can't
-          * find any private texture data hanging around anywhere, but
-          * I'm not yet certain there isn't any at all...
-          */
-         /* if (INTEL_DEBUG & DEBUG_TEXTURE)
-            fprintf(stderr, "do something to free texture heaps\n");
-          */
-      }
-
       driDestroyOptionCache(&intel->optionCache);
 
       /* free the Mesa context */



More information about the mesa-commit mailing list