From ab2f573634d21cf5d2c0843960b98e890d495793 Mon Sep 17 00:00:00 2001 From: José Fonseca Date: Sun, 9 Dec 2012 10:15:19 +0000 Subject: llvmpipe: Drop PIPE_QUERY_TIME_ELAPSED support. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It was slightly wrong: we were computing the longest duration of the query among all the rasterizer tasks. Regardless, for tile-based implementations such as llvmpipe, time differences will never be very useful, because rendering before/during/after the query is all interleaved. And this is expected, see ARB_timer_query spec, issue 10. In particular, piglit ext_timer_query-time-elapsed still fails, because it makes assumptions that don't hold true in in tiled architectures. Not sure how to fix that though. Reviewed-by: Dave Airlie Reviewed-by: Marek Olšák Reviewed-by: Brian Paul --- src/gallium/drivers/llvmpipe/lp_query.c | 7 ------- 1 file changed, 7 deletions(-) (limited to 'src/gallium/drivers/llvmpipe/lp_query.c') diff --git a/src/gallium/drivers/llvmpipe/lp_query.c b/src/gallium/drivers/llvmpipe/lp_query.c index 7a62a809d58..01d5201a100 100644 --- a/src/gallium/drivers/llvmpipe/lp_query.c +++ b/src/gallium/drivers/llvmpipe/lp_query.c @@ -122,13 +122,6 @@ llvmpipe_get_query_result(struct pipe_context *pipe, *result += pq->count[i]; } break; - case PIPE_QUERY_TIME_ELAPSED: - for (i = 0; i < LP_MAX_THREADS; i++) { - if (pq->count[i] > *result) { - *result = pq->count[i]; - } - } - break; case PIPE_QUERY_TIMESTAMP: for (i = 0; i < LP_MAX_THREADS; i++) { if (pq->count[i] > *result) { -- cgit v1.2.3