ilo: mark some states dirty when they are really changed
authorChia-I Wu <olvaffe@gmail.com>
Fri, 12 Jul 2013 21:54:20 +0000 (05:54 +0800)
committerChia-I Wu <olvaffe@gmail.com>
Fri, 12 Jul 2013 22:43:53 +0000 (06:43 +0800)
The checks may seem redundant because cso_context handles them, but
util_blitter does not have access to cso_context.

src/gallium/drivers/ilo/ilo_state.c

index c6790045c364bc057a75159e52ad099c85c64ab7..070bc288528ac496672502cfafede9ecc7c4fe24 100644 (file)
@@ -489,6 +489,10 @@ ilo_bind_gs_state(struct pipe_context *pipe, void *state)
 {
    struct ilo_context *ilo = ilo_context(pipe);
 
+   /* util_blitter may set this unnecessarily */
+   if (ilo->gs == state)
+      return;
+
    ilo->gs = state;
 
    ilo->dirty |= ILO_DIRTY_GS;
@@ -555,6 +559,10 @@ ilo_set_stencil_ref(struct pipe_context *pipe,
 {
    struct ilo_context *ilo = ilo_context(pipe);
 
+   /* util_blitter may set this unnecessarily */
+   if (!memcpy(&ilo->stencil_ref, state, sizeof(*state)))
+      return;
+
    ilo->stencil_ref = *state;
 
    ilo->dirty |= ILO_DIRTY_STENCIL_REF;
@@ -566,6 +574,10 @@ ilo_set_sample_mask(struct pipe_context *pipe,
 {
    struct ilo_context *ilo = ilo_context(pipe);
 
+   /* util_blitter may set this unnecessarily */
+   if (ilo->sample_mask == sample_mask)
+      return;
+
    ilo->sample_mask = sample_mask;
 
    ilo->dirty |= ILO_DIRTY_SAMPLE_MASK;
@@ -954,6 +966,10 @@ ilo_set_stream_output_targets(struct pipe_context *pipe,
    if (!targets)
       num_targets = 0;
 
+   /* util_blitter may set this unnecessarily */
+   if (!ilo->so.count && !num_targets)
+      return;
+
    for (i = 0; i < num_targets; i++)
       pipe_so_target_reference(&ilo->so.states[i], targets[i]);