r600g: fix RSQ of negative value on Cayman

[mesa.git] / src / gallium / drivers / r600 / r600_blit.c

diff --git a/src/gallium/drivers/r600/r600_blit.c b/src/gallium/drivers/r600/r600_blit.c
index a49b4530078fc69ec7725d6263fa921837d3b3bf..09fb6239221433550b3efed3108a1a8b4ca623f5 100644 (file)
--- a/src/gallium/drivers/r600/r600_blit.c
+++ b/src/gallium/drivers/r600/r600_blit.c
@@ -62,20 +62,16 @@ static void r600_blitter_begin(struct pipe_context *ctx, enum r600_blitter_op op
        util_blitter_save_rasterizer(rctx->blitter, rctx->states[R600_PIPE_STATE_RASTERIZER]);
 
        if (op & R600_SAVE_FRAGMENT_STATE) {
-               if (rctx->states[R600_PIPE_STATE_VIEWPORT]) {
-                       util_blitter_save_viewport(rctx->blitter, &rctx->viewport);
-               }
+               util_blitter_save_viewport(rctx->blitter, &rctx->viewport.state);
                util_blitter_save_fragment_shader(rctx->blitter, rctx->ps_shader);
                util_blitter_save_blend(rctx->blitter, rctx->states[R600_PIPE_STATE_BLEND]);
                util_blitter_save_depth_stencil_alpha(rctx->blitter, rctx->states[R600_PIPE_STATE_DSA]);
-               if (rctx->states[R600_PIPE_STATE_STENCIL_REF]) {
-                       util_blitter_save_stencil_ref(rctx->blitter, &rctx->stencil_ref);
-               }
+               util_blitter_save_stencil_ref(rctx->blitter, &rctx->stencil_ref.pipe_state);
                 util_blitter_save_sample_mask(rctx->blitter, rctx->sample_mask.sample_mask);
        }
 
        if (op & R600_SAVE_FRAMEBUFFER)
-               util_blitter_save_framebuffer(rctx->blitter, &rctx->framebuffer);
+               util_blitter_save_framebuffer(rctx->blitter, &rctx->framebuffer.state);
 
        if (op & R600_SAVE_TEXTURES) {
                util_blitter_save_fragment_sampler_states(
@@ -331,29 +327,46 @@ void r600_decompress_color_textures(struct r600_context *rctx,
        }
 }
 
+/* Helper for decompressing a portion of a color or depth resource before
+ * blitting if any decompression is needed.
+ * The driver doesn't decompress resources automatically while u_blitter is
+ * rendering. */
+static bool r600_decompress_subresource(struct pipe_context *ctx,
+                                       struct pipe_resource *tex,
+                                       unsigned level,
+                                       unsigned first_layer, unsigned last_layer)
+{
+       struct r600_context *rctx = (struct r600_context *)ctx;
+       struct r600_texture *rtex = (struct r600_texture*)tex;
+
+       if (rtex->is_depth && !rtex->is_flushing_texture) {
+               if (!r600_init_flushed_depth_texture(ctx, tex, NULL))
+                       return false; /* error */
+
+               r600_blit_decompress_depth(ctx, rtex, NULL,
+                                          level, level,
+                                          first_layer, last_layer,
+                                          0, u_max_sample(tex));
+       } else if (rctx->chip_class != CAYMAN && rtex->fmask_size && rtex->cmask_size) {
+               r600_blit_decompress_color(ctx, rtex, level, level,
+                                          first_layer, last_layer);
+       }
+       return true;
+}
+
 static void r600_copy_first_sample(struct pipe_context *ctx,
                                   const struct pipe_resolve_info *info)
 {
        struct r600_context *rctx = (struct r600_context *)ctx;
-       struct r600_texture *rsrc = (struct r600_texture*)info->src.res;
        struct pipe_surface *dst_view, dst_templ;
        struct pipe_sampler_view src_templ, *src_view;
        struct pipe_box box;
 
-       if (rsrc->is_depth && !rsrc->is_flushing_texture) {
-               if (!r600_init_flushed_depth_texture(ctx, info->src.res, NULL))
-                       return; /* error */
-
-               /* Decompress the first sample only. */
-               r600_blit_decompress_depth(ctx, rsrc, NULL,
-                                          0, 0,
-                                          info->src.layer, info->src.layer,
-                                          0, 0);
-       }
-       if (rctx->chip_class != CAYMAN && rsrc->fmask_size && rsrc->cmask_size) {
-               r600_blit_decompress_color(ctx, rsrc,
-                                          0, 0,
-                                          info->src.layer, info->src.layer);
+       /* The driver doesn't decompress resources automatically while
+        * u_blitter is rendering. */
+       if (!r600_decompress_subresource(ctx, info->src.res, 0,
+                                        info->src.layer, info->src.layer)) {
+               return; /* error */
        }
 
        /* this is correct for upside-down blits too */
@@ -496,7 +509,7 @@ static void r600_clear(struct pipe_context *ctx, unsigned buffers,
                       double depth, unsigned stencil)
 {
        struct r600_context *rctx = (struct r600_context *)ctx;
-       struct pipe_framebuffer_state *fb = &rctx->framebuffer;
+       struct pipe_framebuffer_state *fb = &rctx->framebuffer.state;
 
        r600_blitter_begin(ctx, R600_CLEAR);
        util_blitter_clear(rctx->blitter, fb->width, fb->height,
@@ -668,7 +681,6 @@ static void r600_resource_copy_region(struct pipe_context *ctx,
                                      const struct pipe_box *src_box)
 {
        struct r600_context *rctx = (struct r600_context *)ctx;
-       struct r600_texture *rsrc = (struct r600_texture*)src;
        struct texture_orig_info orig_info[2];
        struct pipe_box sbox;
        const struct pipe_box *psbox = src_box;
@@ -686,19 +698,11 @@ static void r600_resource_copy_region(struct pipe_context *ctx,
        assert(u_max_sample(dst) == u_max_sample(src));
        last_sample = u_max_sample(dst);
 
-       /* 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_decompress_depth(ctx, rsrc, NULL,
-                                          src_level, src_level,
-                                          src_box->z, src_box->z + src_box->depth - 1,
-                                          0, u_max_sample(src));
-       }
-       if (rctx->chip_class != CAYMAN && rsrc->fmask_size && rsrc->cmask_size) {
-               r600_blit_decompress_color(ctx, rsrc, src_level, src_level,
-                                          src_box->z, src_box->z + src_box->depth - 1);
+       /* The driver doesn't decompress resources automatically while
+        * u_blitter is rendering. */
+       if (!r600_decompress_subresource(ctx, src, src_level,
+                                        src_box->z, src_box->z + src_box->depth - 1)) {
+               return; /* error */
        }
 
        restore_orig[0] = restore_orig[1] = FALSE;