From 8cb683f3b9a419c2472ead6750621c671a5becff Mon Sep 17 00:00:00 2001 From: ameerj <52414509+ameerj@users.noreply.github.com> Date: Thu, 17 Dec 2020 14:22:46 -0500 Subject: [PATCH] Overwrite slots instead of queuing them, add disconnect signal Fix for Katana Zero and Yoshi's Crafted World --- .../hle/service/nvflinger/buffer_queue.cpp | 41 +++++++++---------- src/core/hle/service/nvflinger/buffer_queue.h | 3 +- src/core/hle/service/vi/vi.cpp | 20 +++++---- 3 files changed, 35 insertions(+), 29 deletions(-) diff --git a/src/core/hle/service/nvflinger/buffer_queue.cpp b/src/core/hle/service/nvflinger/buffer_queue.cpp index 191286ce99..101b004926 100644 --- a/src/core/hle/service/nvflinger/buffer_queue.cpp +++ b/src/core/hle/service/nvflinger/buffer_queue.cpp @@ -25,7 +25,7 @@ void BufferQueue::SetPreallocatedBuffer(u32 slot, const IGBPBuffer& igbp_buffer) LOG_WARNING(Service, "Adding graphics buffer {}", slot); free_buffers.push_back(slot); - queue.push_back({ + buffers[slot] = { .slot = slot, .status = Buffer::Status::Free, .igbp_buffer = igbp_buffer, @@ -33,7 +33,7 @@ void BufferQueue::SetPreallocatedBuffer(u32 slot, const IGBPBuffer& igbp_buffer) .crop_rect = {}, .swap_interval = 0, .multi_fence = {}, - }); + }; buffer_wait_event.writable->Signal(); } @@ -46,11 +46,11 @@ std::optional> BufferQueue::Dequeue } auto f_itr = free_buffers.begin(); - auto itr = queue.end(); + auto itr = buffers.end(); while (f_itr != free_buffers.end()) { auto slot = *f_itr; - itr = std::find_if(queue.begin(), queue.end(), [&](const Buffer& buffer) { + itr = std::find_if(buffers.begin(), buffers.end(), [&](const Buffer& buffer) { // Only consider free buffers. Buffers become free once again after they've been // Acquired and Released by the compositor, see the NVFlinger::Compose method. if (buffer.status != Buffer::Status::Free) { @@ -65,14 +65,14 @@ std::optional> BufferQueue::Dequeue return buffer.igbp_buffer.width == width && buffer.igbp_buffer.height == height; }); - if (itr != queue.end()) { + if (itr != buffers.end()) { free_buffers.erase(f_itr); break; } ++f_itr; } - if (itr == queue.end()) { + if (itr == buffers.end()) { return std::nullopt; } @@ -81,9 +81,9 @@ std::optional> BufferQueue::Dequeue } const IGBPBuffer& BufferQueue::RequestBuffer(u32 slot) const { - auto itr = std::find_if(queue.begin(), queue.end(), + auto itr = std::find_if(buffers.begin(), buffers.end(), [&](const Buffer& buffer) { return buffer.slot == slot; }); - ASSERT(itr != queue.end()); + ASSERT(itr != buffers.end()); ASSERT(itr->status == Buffer::Status::Dequeued); return itr->igbp_buffer; } @@ -91,9 +91,9 @@ const IGBPBuffer& BufferQueue::RequestBuffer(u32 slot) const { void BufferQueue::QueueBuffer(u32 slot, BufferTransformFlags transform, const Common::Rectangle& crop_rect, u32 swap_interval, Service::Nvidia::MultiFence& multi_fence) { - auto itr = std::find_if(queue.begin(), queue.end(), + auto itr = std::find_if(buffers.begin(), buffers.end(), [&](const Buffer& buffer) { return buffer.slot == slot; }); - ASSERT(itr != queue.end()); + ASSERT(itr != buffers.end()); ASSERT(itr->status == Buffer::Status::Dequeued); itr->status = Buffer::Status::Queued; itr->transform = transform; @@ -104,9 +104,9 @@ void BufferQueue::QueueBuffer(u32 slot, BufferTransformFlags transform, } void BufferQueue::CancelBuffer(u32 slot, const Service::Nvidia::MultiFence& multi_fence) { - const auto itr = std::find_if(queue.begin(), queue.end(), + const auto itr = std::find_if(buffers.begin(), buffers.end(), [slot](const Buffer& buffer) { return buffer.slot == slot; }); - ASSERT(itr != queue.end()); + ASSERT(itr != buffers.end()); ASSERT(itr->status != Buffer::Status::Free); itr->status = Buffer::Status::Free; itr->multi_fence = multi_fence; @@ -118,16 +118,16 @@ void BufferQueue::CancelBuffer(u32 slot, const Service::Nvidia::MultiFence& mult } std::optional> BufferQueue::AcquireBuffer() { - auto itr = queue.end(); + auto itr = buffers.end(); // Iterate to find a queued buffer matching the requested slot. - while (itr == queue.end() && !queue_sequence.empty()) { + while (itr == buffers.end() && !queue_sequence.empty()) { const u32 slot = queue_sequence.front(); - itr = std::find_if(queue.begin(), queue.end(), [&slot](const Buffer& buffer) { + itr = std::find_if(buffers.begin(), buffers.end(), [&slot](const Buffer& buffer) { return buffer.status == Buffer::Status::Queued && buffer.slot == slot; }); queue_sequence.pop_front(); } - if (itr == queue.end()) { + if (itr == buffers.end()) { return std::nullopt; } itr->status = Buffer::Status::Acquired; @@ -135,9 +135,9 @@ std::optional> BufferQueue::Ac } void BufferQueue::ReleaseBuffer(u32 slot) { - auto itr = std::find_if(queue.begin(), queue.end(), + auto itr = std::find_if(buffers.begin(), buffers.end(), [&](const Buffer& buffer) { return buffer.slot == slot; }); - ASSERT(itr != queue.end()); + ASSERT(itr != buffers.end()); ASSERT(itr->status == Buffer::Status::Acquired); itr->status = Buffer::Status::Free; free_buffers.push_back(slot); @@ -146,10 +146,9 @@ void BufferQueue::ReleaseBuffer(u32 slot) { } void BufferQueue::Disconnect() { - queue.clear(); + buffers.fill({}); queue_sequence.clear(); - id = 1; - layer_id = 1; + buffer_wait_event.writable->Signal(); } u32 BufferQueue::Query(QueryType type) { diff --git a/src/core/hle/service/nvflinger/buffer_queue.h b/src/core/hle/service/nvflinger/buffer_queue.h index e7517c7e1f..e610923cb5 100644 --- a/src/core/hle/service/nvflinger/buffer_queue.h +++ b/src/core/hle/service/nvflinger/buffer_queue.h @@ -21,6 +21,7 @@ class KernelCore; namespace Service::NVFlinger { +constexpr u32 buffer_slots = 0x40; struct IGBPBuffer { u32_le magic; u32_le width; @@ -114,7 +115,7 @@ private: u64 layer_id; std::list free_buffers; - std::vector queue; + std::array buffers; std::list queue_sequence; Kernel::EventPair buffer_wait_event; }; diff --git a/src/core/hle/service/vi/vi.cpp b/src/core/hle/service/vi/vi.cpp index 5d8841ae82..45cfffe066 100644 --- a/src/core/hle/service/vi/vi.cpp +++ b/src/core/hle/service/vi/vi.cpp @@ -282,18 +282,24 @@ public: void DeserializeData() override { [[maybe_unused]] const std::u16string token = ReadInterfaceToken(); data = Read(); - buffer = Read(); + if (data.contains_object != 0) { + buffer_container = Read(); + } } struct Data { u32_le slot; - INSERT_PADDING_WORDS(1); - u32_le graphic_buffer_length; - INSERT_PADDING_WORDS(1); + u32_le contains_object; }; - Data data; - NVFlinger::IGBPBuffer buffer; + struct BufferContainer { + u32_le graphic_buffer_length; + INSERT_PADDING_WORDS(1); + NVFlinger::IGBPBuffer buffer{}; + }; + + Data data{}; + BufferContainer buffer_container{}; }; class IGBPSetPreallocatedBufferResponseParcel : public Parcel { @@ -547,7 +553,7 @@ private: case TransactionId::SetPreallocatedBuffer: { IGBPSetPreallocatedBufferRequestParcel request{ctx.ReadBuffer()}; - buffer_queue.SetPreallocatedBuffer(request.data.slot, request.buffer); + buffer_queue.SetPreallocatedBuffer(request.data.slot, request.buffer_container.buffer); IGBPSetPreallocatedBufferResponseParcel response{}; ctx.WriteBuffer(response.Serialize());