From 1c2ed2a072ba24a3d21a25dfa6cfcce1bf1ae905 Mon Sep 17 00:00:00 2001 From: Takoroka <137927095+Takoroka@users.noreply.github.com> Date: Thu, 29 Oct 2026 04:40:26 -0400 Subject: video_core: skip empty fences; one lock per cache per fence; single-pass overlap subtract Switch 2 titles signal a fence for each compute dispatch's release (5,000-7,000 a second in title 1). With fast GPU fences the callback runs at once, so a fence with no downloads, queries or queued operations does nothing: those (~20%) are no longer made (one in 64 still is, for the operations fences carry). Each cache's lock is now taken once per fence (the flush check took both together first). The overlap range set's Subtract restarted its search after every erase (quadratic); the fencing thread runs it for every downloaded range while holding the buffer cache. The GPU profiler counts fences, skipped fences and buffer download bytes. --- src/video_core/query_cache/query_cache.h | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'src/video_core/query_cache/query_cache.h') diff --git a/src/video_core/query_cache/query_cache.h b/src/video_core/query_cache/query_cache.h index 245ab1996c..60ca50eb1f 100644 --- a/src/video_core/query_cache/query_cache.h +++ b/src/video_core/query_cache/query_cache.h @@ -519,6 +519,16 @@ bool QueryCacheBase::HasUncommittedFlushes() const { return result; } +template +bool QueryCacheBase::HasPendingFenceWork() const { + bool result = false; + impl->ForEachStreamer([&result](StreamerInterface* streamer) { + result |= streamer->HasUnsyncedQueries() || streamer->HasPendingSync(); + return result; + }); + return result; +} + template bool QueryCacheBase::ShouldWaitAsyncFlushes() { std::scoped_lock lk(impl->flush_guard); -- cgit v1.2.3