Mesa (master): Unreference sampler object when it' s currently bound to texture unit.

Alan Hourihane alanh at kemper.freedesktop.org
Wed Mar 6 18:10:23 UTC 2013


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

Author: Alan Hourihane <alanh at vmware.com>
Date:   Wed Mar  6 16:08:58 2013 +0000

Unreference sampler object when it's currently bound to texture unit.

This change specifically unbinds a sampler object from the texture unit
if it's bound to a unit. The spec calls for default object when deleting
sampler objects which are currently bound.

Note: this is a candidate for the stable branches

Signed-off-by: Alan Hourihane <alanh at vmware.com>
Reviewed-by: Brian Paul <brianp at vmware.com>

---

 src/mesa/main/samplerobj.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/mesa/main/samplerobj.c b/src/mesa/main/samplerobj.c
index 319a444..4664cc3 100644
--- a/src/mesa/main/samplerobj.c
+++ b/src/mesa/main/samplerobj.c
@@ -206,9 +206,19 @@ _mesa_DeleteSamplers(GLsizei count, const GLuint *samplers)
 
    for (i = 0; i < count; i++) {
       if (samplers[i]) {
+         GLuint j;
          struct gl_sampler_object *sampObj =
             _mesa_lookup_samplerobj(ctx, samplers[i]);
+   
          if (sampObj) {
+            /* If the sampler is currently bound, unbind it. */
+            for (j = 0; j < ctx->Const.MaxCombinedTextureImageUnits; j++) {
+               if (ctx->Texture.Unit[j].Sampler == sampObj) {
+                  FLUSH_VERTICES(ctx, _NEW_TEXTURE);
+                  _mesa_reference_sampler_object(ctx, &ctx->Texture.Unit[j].Sampler, NULL);
+               }
+            }
+
             /* The ID is immediately freed for re-use */
             _mesa_HashRemove(ctx->Shared->SamplerObjects, samplers[i]);
             /* But the object exists until its reference count goes to zero */




More information about the mesa-commit mailing list