r600g: consolidate code for setting sampler views and fix bugs in the process
authorMarek Olšák <maraeo@gmail.com>
Sat, 14 Jul 2012 14:23:42 +0000 (16:23 +0200)
committerMarek Olšák <maraeo@gmail.com>
Tue, 17 Jul 2012 19:22:14 +0000 (21:22 +0200)
Issues fixed:

- set_vs_sampler_views for evergreen is now properly implemented.

- Added the missing inval_texture_cache call for evergreen.

- have_depth_texture was sometimes incorrectly set to false on evergreen even
  if there were depth textures in other shader stages. To fix this, set it
  to true once and never set it to false again. It's stupid, but it matches
  the r600 code. The proper fix is left to another patch.

- Optimizaton: The sampler views which aren't changed aren't updated.

src/gallium/drivers/r600/evergreen_state.c
src/gallium/drivers/r600/r600_pipe.h
src/gallium/drivers/r600/r600_state.c
src/gallium/drivers/r600/r600_state_common.c

index ad0eb2f..5885aec 100644 (file)
@@ -1099,58 +1099,20 @@ static struct pipe_sampler_view *evergreen_create_sampler_view(struct pipe_conte
        return &view->base;
 }
 
-static void evergreen_set_vs_sampler_view(struct pipe_context *ctx, unsigned count,
-                                       struct pipe_sampler_view **views)
+static void evergreen_set_vs_sampler_views(struct pipe_context *ctx, unsigned count,
+                                          struct pipe_sampler_view **views)
 {
        struct r600_context *rctx = (struct r600_context *)ctx;
-       struct r600_pipe_sampler_view **resource = (struct r600_pipe_sampler_view **)views;
-
-       for (int i = 0; i < count; i++) {
-               if (resource[i]) {
-                       r600_context_pipe_state_set_vs_resource(rctx, &resource[i]->state,
-                                                               i + R600_MAX_CONST_BUFFERS);
-               }
-       }
+       r600_set_sampler_views(rctx, &rctx->vs_samplers, count, views,
+                              r600_context_pipe_state_set_vs_resource);
 }
 
-static void evergreen_set_ps_sampler_view(struct pipe_context *ctx, unsigned count,
-                                       struct pipe_sampler_view **views)
+static void evergreen_set_ps_sampler_views(struct pipe_context *ctx, unsigned count,
+                                          struct pipe_sampler_view **views)
 {
        struct r600_context *rctx = (struct r600_context *)ctx;
-       struct r600_pipe_sampler_view **resource = (struct r600_pipe_sampler_view **)views;
-       int i;
-       int has_depth = 0;
-
-       for (i = 0; i < count; i++) {
-               if (&rctx->ps_samplers.views[i]->base != views[i]) {
-                       if (resource[i]) {
-                               if (((struct r600_resource_texture *)resource[i]->base.texture)->is_depth)
-                                       has_depth = 1;
-                               r600_context_pipe_state_set_ps_resource(rctx, &resource[i]->state,
-                                                                       i + R600_MAX_CONST_BUFFERS);
-                       } else
-                               r600_context_pipe_state_set_ps_resource(rctx, NULL,
-                                                                       i + R600_MAX_CONST_BUFFERS);
-
-                       pipe_sampler_view_reference(
-                               (struct pipe_sampler_view **)&rctx->ps_samplers.views[i],
-                               views[i]);
-               } else {
-                       if (resource[i]) {
-                               if (((struct r600_resource_texture *)resource[i]->base.texture)->is_depth)
-                                       has_depth = 1;
-                       }
-               }
-       }
-       for (i = count; i < NUM_TEX_UNITS; i++) {
-               if (rctx->ps_samplers.views[i]) {
-                       r600_context_pipe_state_set_ps_resource(rctx, NULL,
-                                                               i + R600_MAX_CONST_BUFFERS);
-                       pipe_sampler_view_reference((struct pipe_sampler_view **)&rctx->ps_samplers.views[i], NULL);
-               }
-       }
-       rctx->have_depth_texture = has_depth;
-       rctx->ps_samplers.n_views = count;
+       r600_set_sampler_views(rctx, &rctx->ps_samplers, count, views,
+                              r600_context_pipe_state_set_ps_resource);
 }
 
 static void evergreen_bind_ps_sampler(struct pipe_context *ctx, unsigned count, void **states)
@@ -1929,7 +1891,7 @@ void evergreen_init_state_functions(struct r600_context *rctx)
        rctx->context.set_blend_color = r600_set_blend_color;
        rctx->context.set_clip_state = evergreen_set_clip_state;
        rctx->context.set_constant_buffer = r600_set_constant_buffer;
-       rctx->context.set_fragment_sampler_views = evergreen_set_ps_sampler_view;
+       rctx->context.set_fragment_sampler_views = evergreen_set_ps_sampler_views;
        rctx->context.set_framebuffer_state = evergreen_set_framebuffer_state;
        rctx->context.set_polygon_stipple = evergreen_set_polygon_stipple;
        rctx->context.set_sample_mask = evergreen_set_sample_mask;
@@ -1937,7 +1899,7 @@ void evergreen_init_state_functions(struct r600_context *rctx)
        rctx->context.set_stencil_ref = r600_set_pipe_stencil_ref;
        rctx->context.set_vertex_buffers = r600_set_vertex_buffers;
        rctx->context.set_index_buffer = r600_set_index_buffer;
-       rctx->context.set_vertex_sampler_views = evergreen_set_vs_sampler_view;
+       rctx->context.set_vertex_sampler_views = evergreen_set_vs_sampler_views;
        rctx->context.set_viewport_state = evergreen_set_viewport_state;
        rctx->context.sampler_view_destroy = r600_sampler_view_destroy;
        rctx->context.texture_barrier = r600_texture_barrier;
index 4688b62..fd11134 100644 (file)
@@ -528,6 +528,11 @@ void r600_set_index_buffer(struct pipe_context *ctx,
 void r600_vertex_buffers_dirty(struct r600_context *rctx);
 void r600_set_vertex_buffers(struct pipe_context *ctx, unsigned count,
                             const struct pipe_vertex_buffer *input);
+void r600_set_sampler_views(struct r600_context *rctx,
+                           struct r600_textures_info *dst,
+                           unsigned count,
+                           struct pipe_sampler_view **views,
+                           void (*set_resource)(struct r600_context*, struct r600_pipe_resource_state*, unsigned));
 void *r600_create_vertex_elements(struct pipe_context *ctx,
                                  unsigned count,
                                  const struct pipe_vertex_element *elements);
index 0091fe7..508b6c6 100644 (file)
@@ -1124,48 +1124,6 @@ static struct pipe_sampler_view *r600_create_sampler_view(struct pipe_context *c
        return &view->base;
 }
 
-static void r600_set_sampler_views(struct r600_context *rctx,
-                                  struct r600_textures_info *dst,
-                                  unsigned count,
-                                  struct pipe_sampler_view **views,
-                                  void (*set_resource)(struct r600_context*, struct r600_pipe_resource_state*, unsigned))
-{
-       struct r600_pipe_sampler_view **rviews = (struct r600_pipe_sampler_view **)views;
-       unsigned i;
-
-       if (count)
-               r600_inval_texture_cache(rctx);
-
-       for (i = 0; i < count; i++) {
-               if (rviews[i]) {
-                       if (((struct r600_resource_texture *)rviews[i]->base.texture)->is_depth)
-                               rctx->have_depth_texture = true;
-
-                       /* Changing from array to non-arrays textures and vice versa requires updating TEX_ARRAY_OVERRIDE. */
-                       if ((rviews[i]->base.texture->target == PIPE_TEXTURE_1D_ARRAY ||
-                            rviews[i]->base.texture->target == PIPE_TEXTURE_2D_ARRAY) != dst->is_array_sampler[i])
-                               dst->samplers_dirty = true;
-
-                       set_resource(rctx, &rviews[i]->state, i + R600_MAX_CONST_BUFFERS);
-               } else {
-                       set_resource(rctx, NULL, i + R600_MAX_CONST_BUFFERS);
-               }
-
-               pipe_sampler_view_reference(
-                       (struct pipe_sampler_view **)&dst->views[i],
-                       views[i]);
-       }
-
-       for (i = count; i < dst->n_views; i++) {
-               if (dst->views[i]) {
-                       set_resource(rctx, NULL, i + R600_MAX_CONST_BUFFERS);
-                       pipe_sampler_view_reference((struct pipe_sampler_view **)&dst->views[i], NULL);
-               }
-       }
-
-       dst->n_views = count;
-}
-
 static void r600_set_vs_sampler_views(struct pipe_context *ctx, unsigned count,
                                      struct pipe_sampler_view **views)
 {
index 84c7d5e..76a28ff 100644 (file)
@@ -457,6 +457,53 @@ void r600_set_vertex_buffers(struct pipe_context *ctx, unsigned count,
        r600_vertex_buffers_dirty(rctx);
 }
 
+void r600_set_sampler_views(struct r600_context *rctx,
+                           struct r600_textures_info *dst,
+                           unsigned count,
+                           struct pipe_sampler_view **views,
+                           void (*set_resource)(struct r600_context*, struct r600_pipe_resource_state*, unsigned))
+{
+       struct r600_pipe_sampler_view **rviews = (struct r600_pipe_sampler_view **)views;
+       unsigned i;
+
+       if (count)
+               r600_inval_texture_cache(rctx);
+
+       for (i = 0; i < count; i++) {
+               if (rviews[i] == dst->views[i]) {
+                       continue;
+               }
+
+               if (rviews[i]) {
+                       if (((struct r600_resource_texture *)rviews[i]->base.texture)->is_depth)
+                               rctx->have_depth_texture = true;
+
+                       /* Changing from array to non-arrays textures and vice
+                        * versa requires updating TEX_ARRAY_OVERRIDE on R6xx-R7xx. */
+                       if (rctx->chip_class <= R700 &&
+                           (rviews[i]->base.texture->target == PIPE_TEXTURE_1D_ARRAY ||
+                            rviews[i]->base.texture->target == PIPE_TEXTURE_2D_ARRAY) != dst->is_array_sampler[i]) {
+                               dst->samplers_dirty = true;
+                       }
+
+                       set_resource(rctx, &rviews[i]->state, i + R600_MAX_CONST_BUFFERS);
+               } else {
+                       set_resource(rctx, NULL, i + R600_MAX_CONST_BUFFERS);
+               }
+
+               pipe_sampler_view_reference((struct pipe_sampler_view **)&dst->views[i], views[i]);
+       }
+
+       for (i = count; i < dst->n_views; i++) {
+               if (dst->views[i]) {
+                       set_resource(rctx, NULL, i + R600_MAX_CONST_BUFFERS);
+                       pipe_sampler_view_reference((struct pipe_sampler_view **)&dst->views[i], NULL);
+               }
+       }
+
+       dst->n_views = count;
+}
+
 void *r600_create_vertex_elements(struct pipe_context *ctx,
                                  unsigned count,
                                  const struct pipe_vertex_element *elements)