buffer_cache: Fix size reductions not having in mind bind sizes
A buffer binding can change between shaders without changing the shaders. This lead to outdated bindings on OpenGL.
This commit is contained in:
parent
9e066dcb15
commit
838d7e4ca5
|
@ -405,8 +405,6 @@ private:
|
||||||
u32 written_compute_texture_buffers = 0;
|
u32 written_compute_texture_buffers = 0;
|
||||||
u32 image_compute_texture_buffers = 0;
|
u32 image_compute_texture_buffers = 0;
|
||||||
|
|
||||||
std::array<u32, NUM_STAGES> fast_bound_uniform_buffers{};
|
|
||||||
|
|
||||||
std::array<u32, 16> uniform_cache_hits{};
|
std::array<u32, 16> uniform_cache_hits{};
|
||||||
std::array<u32, 16> uniform_cache_shots{};
|
std::array<u32, 16> uniform_cache_shots{};
|
||||||
|
|
||||||
|
@ -416,6 +414,10 @@ private:
|
||||||
|
|
||||||
std::conditional_t<HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS, std::array<u32, NUM_STAGES>, Empty>
|
std::conditional_t<HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS, std::array<u32, NUM_STAGES>, Empty>
|
||||||
dirty_uniform_buffers{};
|
dirty_uniform_buffers{};
|
||||||
|
std::conditional_t<IS_OPENGL, std::array<u32, NUM_STAGES>, Empty> fast_bound_uniform_buffers{};
|
||||||
|
std::conditional_t<HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS,
|
||||||
|
std::array<std::array<u32, NUM_GRAPHICS_UNIFORM_BUFFERS>, NUM_STAGES>, Empty>
|
||||||
|
uniform_buffer_binding_sizes{};
|
||||||
|
|
||||||
std::vector<BufferId> cached_write_buffer_ids;
|
std::vector<BufferId> cached_write_buffer_ids;
|
||||||
|
|
||||||
|
@ -684,6 +686,7 @@ void BufferCache<P>::SetUniformBuffersState(const std::array<u32, NUM_STAGES>& m
|
||||||
fast_bound_uniform_buffers.fill(0);
|
fast_bound_uniform_buffers.fill(0);
|
||||||
}
|
}
|
||||||
dirty_uniform_buffers.fill(~u32{0});
|
dirty_uniform_buffers.fill(~u32{0});
|
||||||
|
uniform_buffer_binding_sizes.fill({});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
enabled_uniform_buffer_masks = mask;
|
enabled_uniform_buffer_masks = mask;
|
||||||
|
@ -1016,14 +1019,18 @@ void BufferCache<P>::BindHostGraphicsUniformBuffer(size_t stage, u32 index, u32
|
||||||
TouchBuffer(buffer);
|
TouchBuffer(buffer);
|
||||||
const bool use_fast_buffer = binding.buffer_id != NULL_BUFFER_ID &&
|
const bool use_fast_buffer = binding.buffer_id != NULL_BUFFER_ID &&
|
||||||
size <= uniform_buffer_skip_cache_size &&
|
size <= uniform_buffer_skip_cache_size &&
|
||||||
!buffer.IsRegionGpuModified(cpu_addr, size);
|
!buffer.IsRegionGpuModified(cpu_addr, size) && false;
|
||||||
if (use_fast_buffer) {
|
if (use_fast_buffer) {
|
||||||
if constexpr (IS_OPENGL) {
|
if constexpr (IS_OPENGL) {
|
||||||
if (runtime.HasFastBufferSubData()) {
|
if (runtime.HasFastBufferSubData()) {
|
||||||
// Fast path for Nvidia
|
// Fast path for Nvidia
|
||||||
if (!HasFastUniformBufferBound(stage, binding_index)) {
|
const bool should_fast_bind =
|
||||||
|
!HasFastUniformBufferBound(stage, binding_index) ||
|
||||||
|
uniform_buffer_binding_sizes[stage][binding_index] != size;
|
||||||
|
if (should_fast_bind) {
|
||||||
// We only have to bind when the currently bound buffer is not the fast version
|
// We only have to bind when the currently bound buffer is not the fast version
|
||||||
fast_bound_uniform_buffers[stage] |= 1U << binding_index;
|
fast_bound_uniform_buffers[stage] |= 1U << binding_index;
|
||||||
|
uniform_buffer_binding_sizes[stage][binding_index] = size;
|
||||||
runtime.BindFastUniformBuffer(stage, binding_index, size);
|
runtime.BindFastUniformBuffer(stage, binding_index, size);
|
||||||
}
|
}
|
||||||
const auto span = ImmediateBufferWithData(cpu_addr, size);
|
const auto span = ImmediateBufferWithData(cpu_addr, size);
|
||||||
|
@ -1033,6 +1040,7 @@ void BufferCache<P>::BindHostGraphicsUniformBuffer(size_t stage, u32 index, u32
|
||||||
}
|
}
|
||||||
if constexpr (IS_OPENGL) {
|
if constexpr (IS_OPENGL) {
|
||||||
fast_bound_uniform_buffers[stage] |= 1U << binding_index;
|
fast_bound_uniform_buffers[stage] |= 1U << binding_index;
|
||||||
|
uniform_buffer_binding_sizes[stage][binding_index] = size;
|
||||||
}
|
}
|
||||||
// Stream buffer path to avoid stalling on non-Nvidia drivers or Vulkan
|
// Stream buffer path to avoid stalling on non-Nvidia drivers or Vulkan
|
||||||
const std::span<u8> span = runtime.BindMappedUniformBuffer(stage, binding_index, size);
|
const std::span<u8> span = runtime.BindMappedUniformBuffer(stage, binding_index, size);
|
||||||
|
@ -1046,9 +1054,13 @@ void BufferCache<P>::BindHostGraphicsUniformBuffer(size_t stage, u32 index, u32
|
||||||
}
|
}
|
||||||
++uniform_cache_shots[0];
|
++uniform_cache_shots[0];
|
||||||
|
|
||||||
if (!needs_bind && !HasFastUniformBufferBound(stage, binding_index)) {
|
// Skip binding if it's not needed and if the bound buffer is not the fast version
|
||||||
// Skip binding if it's not needed and if the bound buffer is not the fast version
|
// This exists to avoid instances where the fast buffer is bound and a GPU write happens
|
||||||
// This exists to avoid instances where the fast buffer is bound and a GPU write happens
|
needs_bind |= HasFastUniformBufferBound(stage, binding_index);
|
||||||
|
if constexpr (HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS) {
|
||||||
|
needs_bind |= uniform_buffer_binding_sizes[stage][binding_index] != size;
|
||||||
|
}
|
||||||
|
if (!needs_bind) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const u32 offset = buffer.Offset(cpu_addr);
|
const u32 offset = buffer.Offset(cpu_addr);
|
||||||
|
@ -1060,6 +1072,9 @@ void BufferCache<P>::BindHostGraphicsUniformBuffer(size_t stage, u32 index, u32
|
||||||
const bool is_copy_bind = offset != 0 && !runtime.SupportsNonZeroUniformOffset();
|
const bool is_copy_bind = offset != 0 && !runtime.SupportsNonZeroUniformOffset();
|
||||||
dirty_uniform_buffers[stage] |= (is_copy_bind ? 1U : 0U) << index;
|
dirty_uniform_buffers[stage] |= (is_copy_bind ? 1U : 0U) << index;
|
||||||
}
|
}
|
||||||
|
if constexpr (HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS) {
|
||||||
|
uniform_buffer_binding_sizes[stage][binding_index] = size;
|
||||||
|
}
|
||||||
if constexpr (NEEDS_BIND_UNIFORM_INDEX) {
|
if constexpr (NEEDS_BIND_UNIFORM_INDEX) {
|
||||||
runtime.BindUniformBuffer(stage, binding_index, buffer, offset, size);
|
runtime.BindUniformBuffer(stage, binding_index, buffer, offset, size);
|
||||||
} else {
|
} else {
|
||||||
|
@ -1725,6 +1740,7 @@ template <class P>
|
||||||
void BufferCache<P>::NotifyBufferDeletion() {
|
void BufferCache<P>::NotifyBufferDeletion() {
|
||||||
if constexpr (HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS) {
|
if constexpr (HAS_PERSISTENT_UNIFORM_BUFFER_BINDINGS) {
|
||||||
dirty_uniform_buffers.fill(~u32{0});
|
dirty_uniform_buffers.fill(~u32{0});
|
||||||
|
uniform_buffer_binding_sizes.fill({});
|
||||||
}
|
}
|
||||||
auto& flags = maxwell3d.dirty.flags;
|
auto& flags = maxwell3d.dirty.flags;
|
||||||
flags[Dirty::IndexBuffer] = true;
|
flags[Dirty::IndexBuffer] = true;
|
||||||
|
|
Reference in New Issue