From 15dee0fc1ddfc02ca5e073f67323907c87136263 Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Thu, 15 Sep 2016 12:07:18 -0600 Subject: [PATCH] svga: skip query flush if we already have the query result This reduces the number of times we flush in some situations (the arbocclude demo is one trivial example). Tested with Piglit, ETQW, Sauerbraten, arbocclude. Reviewed-by: Charmaine Lee --- src/gallium/drivers/svga/svga_pipe_query.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/svga/svga_pipe_query.c b/src/gallium/drivers/svga/svga_pipe_query.c index 74a7f5fa52b..5de3937474e 100644 --- a/src/gallium/drivers/svga/svga_pipe_query.c +++ b/src/gallium/drivers/svga/svga_pipe_query.c @@ -638,11 +638,11 @@ get_query_result_vgpu10(struct svga_context *svga, struct svga_query *sq, sws->query_get_result(sws, sq->gb_query, sq->offset, &queryState, result, resultLen); - if (!sq->fence) { - /* The query hasn't been submitted yet. We need to submit it now - * since the GL spec says "Querying the state for a given occlusion - * query forces that occlusion query to complete within a finite amount - * of time." + if (queryState != SVGA3D_QUERYSTATE_SUCCEEDED && !sq->fence) { + /* We don't have the query result yet, and the query hasn't been + * submitted. We need to submit it now since the GL spec says + * "Querying the state for a given occlusion query forces that + * occlusion query to complete within a finite amount of time." */ svga_context_flush(svga, &sq->fence); } -- 2.30.2