From dbd9572a5a03b1e0bb00946750cf866420a25596 Mon Sep 17 00:00:00 2001
From: Takoroka <137927095+Takoroka@users.noreply.github.com>
Date: Sat, 31 Oct 2026 20:04:22 -0400
Subject: texture_cache: delete images without walking every channel's tables

Deleting an image stamped every entry of every channel's view-id map
with CORRUPT_ID (validation, which is on in release builds) and cleared
every channel's descriptor table bitmaps on the spot. On Mario Kart
World's title screen (~5,400 entries in 5 channels) that took 362 us
per deleted image, with dozens deleted a frame while the collector runs:
~10% of the time in its frames over 100 ms.

The stamp is debug-only now. The tables are invalidated along with it,
so every entry is written again before it is read. Invalidate() marks a
table and its bitmap is cleared on the next read, once however many
images were deleted in between. Same title screen: 3.7 us per deleted
image.
---
 src/video_core/texture_cache/texture_cache.h | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'src/video_core/texture_cache/texture_cache.h')

diff --git a/src/video_core/texture_cache/texture_cache.h b/src/video_core/texture_cache/texture_cache.h
index c8aaf8dff5..559423dbee 100644
--- a/src/video_core/texture_cache/texture_cache.h
+++ b/src/video_core/texture_cache/texture_cache.h
@@ -1712,9 +1712,14 @@ void TextureCache<P>::InvalidateScale(Image& image) {
     for (size_t c : active_channel_ids) {
         auto& channel_info = channel_storage[c];
 
+#ifndef NDEBUG
+        // A walk over every entry (title 1: ~5,400 in 5 channels) for each deleted image: debug
+        // builds only. The tables are invalidated below, so each entry is written again before
+        // it is read.
         if constexpr (ENABLE_VALIDATION)
             for (auto& e : channel_info.image_view_ids)
                 e.second = CORRUPT_ID;
+#endif
         channel_info.graphics_image_table.Invalidate();
         channel_info.compute_image_table.Invalidate();
     }
@@ -2906,9 +2911,14 @@ void TextureCache<P>::DeleteImage(ImageId image_id, bool immediate_delete) {
     }
     for (size_t c : active_channel_ids) {
         auto& channel_info = channel_storage[c];
+#ifndef NDEBUG
+        // A walk over every entry (title 1: ~5,400 in 5 channels) for each deleted image: debug
+        // builds only. The tables are invalidated below, so each entry is written again before
+        // it is read.
         if constexpr (ENABLE_VALIDATION)
             for (auto& e : channel_info.image_view_ids)
                 e.second = CORRUPT_ID;
+#endif
         channel_info.graphics_image_table.Invalidate();
         channel_info.compute_image_table.Invalidate();
     }
-- 
cgit v1.2.3