summaryrefslogtreecommitdiffstats
path: root/src/gallium/drivers/radeonsi/si_state_draw.c
diff options
context:
space:
mode:
authorMarek Olšák <marek.olsak@amd.com>2016-09-08 01:42:06 +0200
committerMarek Olšák <marek.olsak@amd.com>2016-09-09 22:45:06 +0200
commita5a2cc530c1f493557c232557ad1910e607712c2 (patch)
tree0cfbf8c1b8242833ce29fdbfd513b5e582de0014 /src/gallium/drivers/radeonsi/si_state_draw.c
parenta67d81580b520aa6adf6c06ef29f27692b1aebe5 (diff)
downloadexternal_mesa3d-a5a2cc530c1f493557c232557ad1910e607712c2.zip
external_mesa3d-a5a2cc530c1f493557c232557ad1910e607712c2.tar.gz
external_mesa3d-a5a2cc530c1f493557c232557ad1910e607712c2.tar.bz2
radeonsi: fix the VGT performance tweak for small instances
Based on the VGT spec. The Vulkan driver doesn't do it optimally and they plan to fix it. Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
Diffstat (limited to 'src/gallium/drivers/radeonsi/si_state_draw.c')
-rw-r--r--src/gallium/drivers/radeonsi/si_state_draw.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c
index d3e6e1a..e44147f 100644
--- a/src/gallium/drivers/radeonsi/si_state_draw.c
+++ b/src/gallium/drivers/radeonsi/si_state_draw.c
@@ -318,14 +318,15 @@ static unsigned si_get_ia_multi_vgt_param(struct si_context *sctx,
wd_switch_on_eop = true;
/* Performance recommendation for 4 SE Gfx7-8 parts if
- * instances are smaller than a primgroup. Ignore the fact
- * primgroup_size is a primitive count, not vertex count.
- * Don't do anything for indirect draws.
+ * instances are smaller than a primgroup.
+ * Assume indirect draws always use small instances.
+ * This is needed for good VS wave utilization.
*/
if (sctx->b.chip_class <= VI &&
sctx->b.screen->info.max_se >= 4 &&
- !info->indirect &&
- info->instance_count > 1 && info->count < primgroup_size)
+ (info->indirect ||
+ (info->instance_count > 1 &&
+ si_num_prims_for_vertices(info) < primgroup_size)))
wd_switch_on_eop = true;
/* Required on CIK and later. */