summaryrefslogtreecommitdiffstats
path: root/src/gallium/drivers/nouveau/nv50
diff options
context:
space:
mode:
authorSamuel Pitoiset <[email protected]>2016-10-27 00:08:30 +0200
committerSamuel Pitoiset <[email protected]>2016-11-02 23:42:09 +0100
commit548b5fee6b4a71d1b7b52b2268ef809a2cb90149 (patch)
treeeb527d57c0372d565ee13134437621a3968b433d /src/gallium/drivers/nouveau/nv50
parentb6137f226c593a31f85a7551594ce161799992af (diff)
nv50,nvc0: stop limiting the number of active queries to 1
This limitation was initially here because AMD_performance_monitor doesn't allow to expose the real number of hardware counters. But this actually really annoying when profiling with qapitrace. Anyways, performance counters are mostly for developers and failures are expected if you try to monitor more queries than supported. This breaks amd_performance_monitor_measure but it's expected. Signed-off-by: Samuel Pitoiset <[email protected]>
Diffstat (limited to 'src/gallium/drivers/nouveau/nv50')
-rw-r--r--src/gallium/drivers/nouveau/nv50/nv50_query.c14
1 files changed, 6 insertions, 8 deletions
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_query.c b/src/gallium/drivers/nouveau/nv50/nv50_query.c
index 9a1397a9860..e57e3ac8c3b 100644
--- a/src/gallium/drivers/nouveau/nv50/nv50_query.c
+++ b/src/gallium/drivers/nouveau/nv50/nv50_query.c
@@ -208,13 +208,11 @@ nv50_screen_get_driver_query_group_info(struct pipe_screen *pscreen,
if (screen->base.class_3d >= NV84_3D_CLASS) {
info->name = "MP counters";
- /* Because we can't expose the number of hardware counters needed
- * for each different query, we don't want to allow more than one
- * active query simultaneously to avoid failure when the maximum
- * number of counters is reached. Note that these groups of GPU
- * counters are currently only used by AMD_performance_monitor.
- */
- info->max_active_queries = 1;
+ /* Expose the maximum number of hardware counters available,
+ * although some queries use more than one counter. Expect failures
+ * in that case but as performance counters are for developers,
+ * this should not have a real impact. */
+ info->max_active_queries = 4;
info->num_queries = NV50_HW_SM_QUERY_COUNT;
return 1;
}
@@ -224,7 +222,7 @@ nv50_screen_get_driver_query_group_info(struct pipe_screen *pscreen,
if (screen->compute) {
if (screen->base.class_3d >= NV84_3D_CLASS) {
info->name = "Performance metrics";
- info->max_active_queries = 1;
+ info->max_active_queries = 2; /* A metric uses at least 2 queries */
info->num_queries = NV50_HW_METRIC_QUERY_COUNT;
return 1;
}