From 59621a3e812fbc8267ebfb03001c82eff48cc86a Mon Sep 17 00:00:00 2001 From: Jocelyn Falempe Date: Fri, 26 Jun 2026 10:54:40 +0200 Subject: [PATCH 06/19] Revert "tu/autotune: Only lock RPs sustain certain mode for 30s" This reverts commit da089bf741b3c06fa7ae21c2a349f4f96d5c6230. --- src/freedreno/vulkan/tu_autotune.cc | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/src/freedreno/vulkan/tu_autotune.cc b/src/freedreno/vulkan/tu_autotune.cc index e1e4666768c..338f2b68018 100644 --- a/src/freedreno/vulkan/tu_autotune.cc +++ b/src/freedreno/vulkan/tu_autotune.cc @@ -1060,9 +1060,6 @@ struct tu_autotune::rp_history { bool locked = false; /* If true, the probability will no longer be updated. */ uint64_t seed[2] { 0x3bffb83978e24f88, 0x9238d5d56c71cd35 }; - bool is_sysmem_winning = false; - uint64_t winning_since_ts = 0; - public: profiled_algo(uint64_t hash) { @@ -1129,15 +1126,6 @@ struct tu_autotune::rp_history { constexpr uint32_t MIN_LOCK_DURATION_COUNT = 15; constexpr uint64_t MIN_LOCK_THRESHOLD = GPU_TICKS_PER_US * 1'000; /* 1ms */ constexpr uint32_t LOCK_PERCENT_DIFF = 30; - constexpr uint64_t LOCK_TIME_WINDOW_NS = 30'000'000'000; /* 30s */ - - uint64_t now = os_time_get_nano(); - bool current_sysmem_winning = avg_sysmem < avg_gmem; - - if (winning_since_ts == 0 || current_sysmem_winning != is_sysmem_winning) { - winning_since_ts = now; - is_sysmem_winning = current_sysmem_winning; - } bool has_resolved = sysmem_prob == SLOW_MAX_PROBABILITY || sysmem_prob == SLOW_MIN_PROBABILITY; bool enough_samples = @@ -1147,7 +1135,7 @@ struct tu_autotune::rp_history { uint64_t percent_diff = (100 * (max_avg - min_avg)) / min_avg; if (has_resolved && enough_samples && max_avg >= MIN_LOCK_THRESHOLD && - percent_diff >= LOCK_PERCENT_DIFF && (now - winning_since_ts) >= LOCK_TIME_WINDOW_NS) { + percent_diff >= LOCK_PERCENT_DIFF) { if (avg_gmem < avg_sysmem) sysmem_prob = 0; else -- 2.54.0