openharmony/third_party_mesa3d
1
0
Fork 0
mirror of https://gitee.com/openharmony/third_party_mesa3d synced 2024-12-04 05:33:41 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

d3d12: Don't accumulate timestamp queries

Browse Source 
If an app re-issues a timestamp query a lot, but doesn't ever ask
for the results, we could end up running off the end of our query
heap. But we don't actually need to advance/accumulate, so just
use a single entry in the heap.

Reviewed By: Bill Kristiansen <billkris@microsoft.com>

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12920>
This commit is contained in:
Jesse Natalie 2021-09-17 18:53:02 -07:00 committed by Marge Bot
parent 34739cb6e2
commit 2d1f5e3dcb
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/gallium/drivers/d3d12/d3d12_query.cpp
View File

@ -117,9 +117,12 @@ d3d12_create_query(struct pipe_context *pctx,
query->num_queries = 16;
/* With timer queries we want a few more queries, especially since we need two slots
* per query for TIME_ELAPSED queries */
if (unlikely(query->d3d12qtype == D3D12_QUERY_TYPE_TIMESTAMP))
* per query for TIME_ELAPSED queries
* For TIMESTAMP, we don't need more than one slot, since there's nothing to accumulate */
if (unlikely(query_type == PIPE_QUERY_TIME_ELAPSED))
query->num_queries = 64;
else if (query_type == PIPE_QUERY_TIMESTAMP)
query->num_queries = 1;
query->curr_query = 0;
@ -357,6 +360,10 @@ end_query(struct d3d12_context *ctx, struct d3d12_query *q)
if (q->subquery)
end_query(ctx, q->subquery);
/* For TIMESTAMP, there's only one slot */
if (q->type == PIPE_QUERY_TIMESTAMP)
q->curr_query = 0;
/* With QUERY_TIME_ELAPSED we have recorded one value at
* (2 * q->curr_query), and now we record a value at (2 * q->curr_query + 1)
* and when resolving the query we subtract the latter from the former */