Instead of writing to rctx->atoms directly use a helper to take
advantage of assert checks.
---
 src/gallium/drivers/r600/evergreen_state.c   |  4 ++--
 src/gallium/drivers/r600/r600_pipe.h         |  1 +
 src/gallium/drivers/r600/r600_state.c        |  4 ++--
 src/gallium/drivers/r600/r600_state_common.c | 15 +++++++++++----
 4 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/gallium/drivers/r600/evergreen_state.c 
b/src/gallium/drivers/r600/evergreen_state.c
index bb53383..f7a76f6 100644
--- a/src/gallium/drivers/r600/evergreen_state.c
+++ b/src/gallium/drivers/r600/evergreen_state.c
@@ -3470,8 +3470,8 @@ void evergreen_init_state_functions(struct r600_context 
*rctx)
        }
        r600_init_atom(rctx, &rctx->stencil_ref.atom, id++, 
r600_emit_stencil_ref, 4);
        r600_init_atom(rctx, &rctx->vertex_fetch_shader.atom, id++, 
evergreen_emit_vertex_fetch_shader, 5);
-       rctx->atoms[id++] = &rctx->b.streamout.begin_atom;
-       rctx->atoms[id++] = &rctx->b.streamout.enable_atom;
+       r600_add_atom(rctx, &rctx->b.streamout.begin_atom, id++);
+       r600_add_atom(rctx, &rctx->b.streamout.enable_atom, id++);
        r600_init_atom(rctx, &rctx->vertex_shader.atom, id++, r600_emit_shader, 
23);
        r600_init_atom(rctx, &rctx->pixel_shader.atom, id++, r600_emit_shader, 
0);
        r600_init_atom(rctx, &rctx->geometry_shader.atom, id++, 
r600_emit_shader, 0);
diff --git a/src/gallium/drivers/r600/r600_pipe.h 
b/src/gallium/drivers/r600/r600_pipe.h
index 3653e8f..8b61269 100644
--- a/src/gallium/drivers/r600/r600_pipe.h
+++ b/src/gallium/drivers/r600/r600_pipe.h
@@ -671,6 +671,7 @@ void r600_emit_clip_misc_state(struct r600_context *rctx, 
struct r600_atom *atom
 void r600_emit_stencil_ref(struct r600_context *rctx, struct r600_atom *atom);
 void r600_emit_viewport_state(struct r600_context *rctx, struct r600_atom 
*atom);
 void r600_emit_shader(struct r600_context *rctx, struct r600_atom *a);
+void r600_add_atom(struct r600_context *rctx, struct r600_atom *atom, unsigned 
id);
 void r600_init_atom(struct r600_context *rctx, struct r600_atom *atom, 
unsigned id,
                    void (*emit)(struct r600_context *ctx, struct r600_atom 
*state),
                    unsigned num_dw);
diff --git a/src/gallium/drivers/r600/r600_state.c 
b/src/gallium/drivers/r600/r600_state.c
index 7fb03b4..8488188 100644
--- a/src/gallium/drivers/r600/r600_state.c
+++ b/src/gallium/drivers/r600/r600_state.c
@@ -3074,8 +3074,8 @@ void r600_init_state_functions(struct r600_context *rctx)
        r600_init_atom(rctx, &rctx->config_state.atom, id++, 
r600_emit_config_state, 3);
        r600_init_atom(rctx, &rctx->stencil_ref.atom, id++, 
r600_emit_stencil_ref, 4);
        r600_init_atom(rctx, &rctx->vertex_fetch_shader.atom, id++, 
r600_emit_vertex_fetch_shader, 5);
-       rctx->atoms[id++] = &rctx->b.streamout.begin_atom;
-       rctx->atoms[id++] = &rctx->b.streamout.enable_atom;
+       r600_add_atom(rctx, &rctx->b.streamout.begin_atom, id++);
+       r600_add_atom(rctx, &rctx->b.streamout.enable_atom, id++);
        r600_init_atom(rctx, &rctx->vertex_shader.atom, id++, r600_emit_shader, 
23);
        r600_init_atom(rctx, &rctx->pixel_shader.atom, id++, r600_emit_shader, 
0);
        r600_init_atom(rctx, &rctx->geometry_shader.atom, id++, 
r600_emit_shader, 0);
diff --git a/src/gallium/drivers/r600/r600_state_common.c 
b/src/gallium/drivers/r600/r600_state_common.c
index 2d654dc..a4238a2 100644
--- a/src/gallium/drivers/r600/r600_state_common.c
+++ b/src/gallium/drivers/r600/r600_state_common.c
@@ -47,18 +47,25 @@ void r600_release_command_buffer(struct r600_command_buffer 
*cb)
        FREE(cb->buf);
 }
 
+void r600_add_atom(struct r600_context *rctx,
+                  struct r600_atom *atom,
+                  unsigned id)
+{
+       assert(id < R600_NUM_ATOMS);
+       assert(rctx->atoms[id] == NULL);
+       rctx->atoms[id] = atom;
+       atom->dirty = false;
+}
+
 void r600_init_atom(struct r600_context *rctx,
                    struct r600_atom *atom,
                    unsigned id,
                    void (*emit)(struct r600_context *ctx, struct r600_atom 
*state),
                    unsigned num_dw)
 {
-       assert(id < R600_NUM_ATOMS);
-       assert(rctx->atoms[id] == NULL);
-       rctx->atoms[id] = atom;
        atom->emit = (void*)emit;
        atom->num_dw = num_dw;
-       atom->dirty = false;
+       r600_add_atom(rctx, atom, id);
 }
 
 void r600_emit_cso_state(struct r600_context *rctx, struct r600_atom *atom)
-- 
1.9.1

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

Reply via email to