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

Author: Marek Olšák <mar...@gmail.com>
Date:   Wed Jul 18 00:17:46 2012 +0200

r600g: remove needless wrapper r600_texture_depth_flush

---

 src/gallium/drivers/r600/r600_blit.c     |   13 ++++++++---
 src/gallium/drivers/r600/r600_resource.h |    5 ----
 src/gallium/drivers/r600/r600_texture.c  |   32 +++++------------------------
 3 files changed, 15 insertions(+), 35 deletions(-)

diff --git a/src/gallium/drivers/r600/r600_blit.c 
b/src/gallium/drivers/r600/r600_blit.c
index 9429a1b..6147a9f 100644
--- a/src/gallium/drivers/r600/r600_blit.c
+++ b/src/gallium/drivers/r600/r600_blit.c
@@ -336,10 +336,15 @@ static void r600_resource_copy_region(struct pipe_context 
*ctx,
                return;
        }
 
-       if (rsrc->is_depth && !rsrc->is_flushing_texture)
-               r600_texture_depth_flush(ctx, src, NULL,
-                                        src_level, src_level,
-                                        src_box->z, src_box->z + 
src_box->depth - 1);
+       /* This must be done before entering u_blitter to avoid recursion. */
+       if (rsrc->is_depth && !rsrc->is_flushing_texture) {
+               if (!r600_init_flushed_depth_texture(ctx, src, NULL))
+                       return; /* error */
+
+               r600_blit_uncompress_depth(ctx, rsrc, NULL,
+                                          src_level, src_level,
+                                          src_box->z, src_box->z + 
src_box->depth - 1);
+       }
 
        restore_orig[0] = restore_orig[1] = FALSE;
 
diff --git a/src/gallium/drivers/r600/r600_resource.h 
b/src/gallium/drivers/r600/r600_resource.h
index 8e5225e..0d87230 100644
--- a/src/gallium/drivers/r600/r600_resource.h
+++ b/src/gallium/drivers/r600/r600_resource.h
@@ -92,11 +92,6 @@ static INLINE struct r600_resource *r600_resource(struct 
pipe_resource *r)
 bool r600_init_flushed_depth_texture(struct pipe_context *ctx,
                                     struct pipe_resource *texture,
                                     struct r600_resource_texture **staging);
-void r600_texture_depth_flush(struct pipe_context *ctx,
-                             struct pipe_resource *texture,
-                             struct r600_resource_texture **staging,
-                             unsigned first_level, unsigned last_level,
-                             unsigned first_layer, unsigned last_layer);
 
 /* r600_texture.c texture transfer functions. */
 struct pipe_transfer* r600_texture_get_transfer(struct pipe_context *ctx,
diff --git a/src/gallium/drivers/r600/r600_texture.c 
b/src/gallium/drivers/r600/r600_texture.c
index 5a6d755..3a5a912 100644
--- a/src/gallium/drivers/r600/r600_texture.c
+++ b/src/gallium/drivers/r600/r600_texture.c
@@ -775,28 +775,6 @@ bool r600_init_flushed_depth_texture(struct pipe_context 
*ctx,
        return true;
 }
 
-void r600_texture_depth_flush(struct pipe_context *ctx,
-                             struct pipe_resource *texture,
-                             struct r600_resource_texture **staging,
-                             unsigned first_level, unsigned last_level,
-                             unsigned first_layer, unsigned last_layer)
-{
-       struct r600_resource_texture *rtex = (struct 
r600_resource_texture*)texture;
-
-       if (!r600_init_flushed_depth_texture(ctx, texture, staging))
-               return;
-
-       if (staging) {
-               r600_blit_uncompress_depth(ctx, rtex, *staging,
-                                          first_level, last_level,
-                                          first_layer, last_layer);
-       } else {
-               r600_blit_uncompress_depth(ctx, rtex, NULL,
-                                          first_level, last_level,
-                                          first_layer, last_layer);
-       }
-}
-
 /* Needs adjustment for pixelformat:
  */
 static INLINE unsigned u_box_volume( const struct pipe_box *box )
@@ -860,15 +838,17 @@ struct pipe_transfer* r600_texture_get_transfer(struct 
pipe_context *ctx,
                */
                struct r600_resource_texture *staging_depth;
 
-               r600_texture_depth_flush(ctx, texture, &staging_depth,
-                                        level, level,
-                                        box->z, box->z + box->depth - 1);
-               if (!staging_depth) {
+               if (!r600_init_flushed_depth_texture(ctx, texture, 
&staging_depth)) {
                        R600_ERR("failed to create temporary texture to hold 
untiled copy\n");
                        pipe_resource_reference(&trans->transfer.resource, 
NULL);
                        FREE(trans);
                        return NULL;
                }
+
+               r600_blit_uncompress_depth(ctx, rtex, staging_depth,
+                                          level, level,
+                                          box->z, box->z + box->depth - 1);
+
                trans->transfer.stride = staging_depth->pitch_in_bytes[level];
                trans->offset = r600_texture_get_offset(staging_depth, level, 
box->z);
                trans->staging = (struct r600_resource*)staging_depth;

_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to