aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/common/container/unordered_map.h2
-rw-r--r--src/common/container/unordered_set.h2
-rw-r--r--src/common/container_hash.h22
-rw-r--r--src/common/range_sets.h3
-rw-r--r--src/common/range_sets.inc12
-rw-r--r--src/video_core/buffer_cache/buffer_cache.h6
6 files changed, 42 insertions, 5 deletions
diff --git a/src/common/container/unordered_map.h b/src/common/container/unordered_map.h
index 7d9e49fd80..c4d4a72663 100644
--- a/src/common/container/unordered_map.h
+++ b/src/common/container/unordered_map.h
@@ -9,7 +9,7 @@
namespace Common {
-template <class Key, class T, class Hash = std::hash<Key>, class Pred = std::equal_to<Key>,
+template <class Key, class T, class Hash = FastHash<Key>, class Pred = std::equal_to<Key>,
class Allocator = std::allocator<std::pair<const Key, T>>>
using unordered_map = boost::unordered::unordered_flat_map<Key, T, Hash, Pred, Allocator>;
diff --git a/src/common/container/unordered_set.h b/src/common/container/unordered_set.h
index 54d71406a1..690c33286f 100644
--- a/src/common/container/unordered_set.h
+++ b/src/common/container/unordered_set.h
@@ -9,7 +9,7 @@
namespace Common {
-template <class Key, class Hash = std::hash<Key>, class Pred = std::equal_to<Key>,
+template <class Key, class Hash = FastHash<Key>, class Pred = std::equal_to<Key>,
class Allocator = std::allocator<Key>>
using unordered_set = boost::unordered::unordered_flat_set<Key, Hash, Pred, Allocator>;
diff --git a/src/common/container_hash.h b/src/common/container_hash.h
index d7477f7e4f..f06f69ad8b 100644
--- a/src/common/container_hash.h
+++ b/src/common/container_hash.h
@@ -19,6 +19,28 @@
namespace Common {
+/// The default hash of Common::unordered_map/set. std::hash of an integer or a pointer is a
+/// byte-wise FNV-1a on MSVC, and boost's flat containers mix a hash that does not say it
+/// avalanches anyway: pass those keys through (the GPU thread's page maps looked keys up
+/// tens of thousands of times a second with it). Other keys keep std::hash.
+template <class Key>
+struct FastHash : std::hash<Key> {};
+
+template <class Key>
+ requires std::is_integral_v<Key>
+struct FastHash<Key> {
+ std::size_t operator()(Key key) const noexcept {
+ return static_cast<std::size_t>(key);
+ }
+};
+
+template <class T>
+struct FastHash<T*> {
+ std::size_t operator()(T* pointer) const noexcept {
+ return reinterpret_cast<std::size_t>(pointer);
+ }
+};
+
namespace detail {
template <typename T>
diff --git a/src/common/range_sets.h b/src/common/range_sets.h
index f8fcee4837..e5cb53b058 100644
--- a/src/common/range_sets.h
+++ b/src/common/range_sets.h
@@ -26,6 +26,9 @@ public:
void Clear();
bool Empty() const;
+ /// Whether any range overlaps [base_address, base_address + size): one lookup.
+ bool Intersects(AddressType base_address, size_t size) const;
+
template <typename Func>
void ForEach(Func&& func) const;
diff --git a/src/common/range_sets.inc b/src/common/range_sets.inc
index 57b07d05f6..aeda7bc351 100644
--- a/src/common/range_sets.inc
+++ b/src/common/range_sets.inc
@@ -221,6 +221,18 @@ bool RangeSet<AddressType>::Empty() const {
}
template <typename AddressType>
+bool RangeSet<AddressType>::Intersects(AddressType base_address, size_t size) const {
+ const auto& set = m_impl->m_ranges_set;
+ if (set.empty() || size == 0) {
+ return false;
+ }
+ const AddressType end_address = base_address + static_cast<AddressType>(size);
+ // The first range not entirely below the query; it overlaps when it starts before its end.
+ const auto it = set.lower_bound(typename RangeSetImpl::IntervalType{base_address, end_address});
+ return it != set.end() && it->lower() < end_address;
+}
+
+template <typename AddressType>
template <typename Func>
void RangeSet<AddressType>::ForEach(Func&& func) const {
m_impl->ForEach(std::move(func));
diff --git a/src/video_core/buffer_cache/buffer_cache.h b/src/video_core/buffer_cache/buffer_cache.h
index 0a6b53b33b..c6fc5989b7 100644
--- a/src/video_core/buffer_cache/buffer_cache.h
+++ b/src/video_core/buffer_cache/buffer_cache.h
@@ -746,9 +746,9 @@ void BufferCache<P>::PopAsyncBuffers() {
template <class P>
bool BufferCache<P>::IsRegionGpuModified(DAddr addr, size_t size) {
- bool is_dirty = false;
- gpu_modified_ranges.ForEachInRange(addr, size, [&](DAddr, DAddr) { is_dirty = true; });
- return is_dirty;
+ // One lookup instead of visiting every range inside (~4% of the GPU thread for title 1's
+ // inline uploads once GPU-written ranges are no longer cleared by downloads).
+ return gpu_modified_ranges.Intersects(addr, size);
}
template <class P>