On 18/01/16 20:28, Nicolai Hähnle wrote:
From: Nicolai Hähnle <nicolai.haeh...@amd.com>

Apparently, nobody has combined stippling with a fragment shader
containing immediates in almost five years...

Fixes a bug in Kodi with radeonsi reported by Christian König.

Good find.

I don't know how to interpret this: if the fact it wasn't caught for so long corroborates the fact that nobody mixes legacy and new OpenGL, or if the fact it was found proves people do mix it!

One thing is sure, we don't really have much test coverage of it, as most of our piglit tests tend to aim to the minimum GL a feature was introduced, hence rarely cover interactions between older and newer features.

Cc: "11.0 11.1" <mesa-sta...@lists.freedesktop.org>
Tested-by: Christian König <christian.koe...@amd.com>
Reviewed-by: Marek Olšák <marek.ol...@amd.com>
---
  src/gallium/auxiliary/util/u_pstipple.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/src/gallium/auxiliary/util/u_pstipple.c 
b/src/gallium/auxiliary/util/u_pstipple.c
index 08dec13..3428172 100644
--- a/src/gallium/auxiliary/util/u_pstipple.c
+++ b/src/gallium/auxiliary/util/u_pstipple.c
@@ -230,6 +230,7 @@ pstip_transform_immed(struct tgsi_transform_context *ctx,
     struct pstip_transform_context *pctx =
        (struct pstip_transform_context *) ctx;
     pctx->numImmed++;
+   ctx->emit_immediate(ctx, immed);
  }


Ironically, this was fixed in src/gallium/auxiliary/draw/draw_pipe_pstipple.c by Brian in 7f16246acef4089570abca76a59580691ec6cf68


I think we should compare and see if there's been anything else that diverged in this files that should be crossported.

Or alternatively, see if there's some way we could have the code once (e.g, have draw call u_pstipple.c? Or force all drivers that use draw to also use u_pstipple?)


BTW, maybe it would be better to move the emit_immediate() call one line up, so it lines up with draw_pipe_pstipple.c.

Otherwise,

Reviewed-by: Jose Fonseca <jfons...@vmware.com>

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

Reply via email to