st/mesa: add readpix_cache structure

Reviewed-by: Marek Olšák <marek.olsak@amd.com>

diff --git a/src/mesa/state_tracker/st_cb_readpixels.c b/src/mesa/state_tracker/st_cb_readpixels.c
index ef575ba..a501b7b 100644 (file)
--- a/src/mesa/state_tracker/st_cb_readpixels.c
+++ b/src/mesa/state_tracker/st_cb_readpixels.c
@@ -218,6 +218,14 @@ fail:
    return success;
 }
 
+/* Invalidate the readpixels cache to ensure we don't read stale data.
+ */
+void st_invalidate_readpix_cache(struct st_context *st)
+{
+   pipe_resource_reference(&st->readpix_cache.src, NULL);
+   pipe_resource_reference(&st->readpix_cache.cache, NULL);
+}
+
 /**
  * Create a staging texture and blit the requested region to it.
  */

diff --git a/src/mesa/state_tracker/st_context.c b/src/mesa/state_tracker/st_context.c
index 308094a..4721215 100644 (file)
--- a/src/mesa/state_tracker/st_context.c
+++ b/src/mesa/state_tracker/st_context.c
@@ -184,6 +184,9 @@ st_destroy_context_priv(struct st_context *st)
    free(st->drawpix_cache.image);
    pipe_resource_reference(&st->drawpix_cache.texture, NULL);
 
+   /* free glReadPixels cache data */
+   st_invalidate_readpix_cache(st);
+
    cso_destroy_context(st->cso_context);
    free( st );
 }

diff --git a/src/mesa/state_tracker/st_context.h b/src/mesa/state_tracker/st_context.h
index e3bee60..cc25e06 100644 (file)
--- a/src/mesa/state_tracker/st_context.h
+++ b/src/mesa/state_tracker/st_context.h
@@ -228,6 +228,16 @@ struct st_context
       struct pipe_resource *texture;
    } drawpix_cache;
 
+   /** for glReadPixels */
+   struct {
+      struct pipe_resource *src;
+      struct pipe_resource *cache;
+      enum pipe_format dst_format;
+      unsigned level;
+      unsigned layer;
+      unsigned hits;
+   } readpix_cache;
+
    /** for glClear */
    struct {
       struct pipe_rasterizer_state raster;
@@ -306,6 +316,7 @@ extern void st_init_driver_functions(struct pipe_screen *screen,
 
 void st_invalidate_state(struct gl_context * ctx, GLbitfield new_state);
 
+void st_invalidate_readpix_cache(struct st_context *st);
 
 
 #define Y_0_TOP 1