diff options
author | Nicolai Hähnle <[email protected]> | 2017-05-03 11:26:27 +0200 |
---|---|---|
committer | Nicolai Hähnle <[email protected]> | 2017-05-08 17:42:17 +0200 |
commit | f16b7558632d1d2d355a8251e969b8fc41f9c1e8 (patch) | |
tree | cb6d95a318dec700f9f3c1fd249749f29b58d1c2 | |
parent | b84b631c6381d9b36bca5d0e7cc67dd23af188c1 (diff) |
radeonsi: fix gl_PrimitiveID in tessellation with instanced draws on SI
Cc: [email protected]
Reviewed-by: Marek Olšák <[email protected]>
-rw-r--r-- | src/gallium/drivers/radeonsi/si_state_draw.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index 5f1371a1433..f0ed898cfb0 100644 --- a/src/gallium/drivers/radeonsi/si_state_draw.c +++ b/src/gallium/drivers/radeonsi/si_state_draw.c @@ -188,6 +188,20 @@ static void si_emit_derived_tess_state(struct si_context *sctx, if (sctx->b.chip_class == SI) { unsigned one_wave = 64 / MAX2(num_tcs_input_cp, num_tcs_output_cp); *num_patches = MIN2(*num_patches, one_wave); + + if (sctx->screen->b.info.max_se == 1) { + /* The VGT HS block increments the patch ID unconditionally + * within a single threadgroup. This results in incorrect + * patch IDs when instanced draws are used. + * + * The intended solution is to restrict threadgroups to + * a single instance by setting SWITCH_ON_EOI, which + * should cause IA to split instances up. However, this + * doesn't work correctly on SI when there is no other + * SE to switch to. + */ + *num_patches = 1; + } } sctx->last_num_patches = *num_patches; |