Azahar-Mirror/azahar
1
0
Fork 0
mirror of https://github.com/azahar-emu/azahar synced 2025-11-06 23:19:57 +01:00
Code Issues Packages Projects Releases Wiki Activity

GSP: correct MemoryFill interrupt signaling (#1218)

Browse Source 
* GSP: correct MemoryFill interrupt signaling

---------

Co-authored-by: PabloMK7 <hackyglitch2@gmail.com>
This commit is contained in:
Nikki 2025-07-11 00:22:10 +02:00 committed by GitHub
parent d8bef418a7
commit 0453e4463a
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/video_core/gpu.cpp
View File

@ -123,19 +123,21 @@ void GPU::Execute(const Service::GSP::Command& command) {
auto& memfill = regs.memory_fill_config;
// Write to the memory fill GPU registers.
// If both buffers are set GSP dispatches PSC0 only.
const bool has_both_bufs = params.start1 != 0 && params.start2 != 0;
if (params.start1 != 0) {
memfill[0].address_start = VirtualToPhysicalAddress(params.start1) >> 3;
memfill[0].address_end = VirtualToPhysicalAddress(params.end1) >> 3;
memfill[0].value_32bit = params.value1;
memfill[0].control = params.control1;
MemoryFill(0);
MemoryFill(0, has_both_bufs ? std::numeric_limits<u32>::max() : 0);
}
if (params.start2 != 0) {
memfill[1].address_start = VirtualToPhysicalAddress(params.start2) >> 3;
memfill[1].address_end = VirtualToPhysicalAddress(params.end2) >> 3;
memfill[1].value_32bit = params.value2;
memfill[1].control = params.control2;
MemoryFill(1);
MemoryFill(1, has_both_bufs ? 0 : 1);
}
break;
}
@ -266,10 +268,10 @@ void GPU::WriteReg(VAddr addr, u32 data) {
// Handle registers that trigger GPU actions
switch (index) {
case GPU_REG_INDEX(memory_fill_config[0].trigger):
MemoryFill(0);
MemoryFill(0, 0);
break;
case GPU_REG_INDEX(memory_fill_config[1].trigger):
MemoryFill(1);
MemoryFill(1, 1);
break;
case GPU_REG_INDEX(display_transfer_config.trigger):
MemoryTransfer();
@ -347,7 +349,7 @@ void GPU::SubmitCmdList(u32 index) {
config.trigger[index] = 0;
}
void GPU::MemoryFill(u32 index) {
void GPU::MemoryFill(u32 index, u32 intr_index) {
// Check if a memory fill was triggered.
auto& config = impl->pica.regs.memory_fill_config[index];
if (!config.trigger) {
@ -362,9 +364,9 @@ void GPU::MemoryFill(u32 index) {
// It seems that it won't signal interrupt if "address_start" is zero.
// TODO: hwtest this
if (config.GetStartAddress() != 0) {
if (!index) {
if (intr_index == 0) {
impl->signal_interrupt(Service::GSP::InterruptId::PSC0);
} else {
} else if (intr_index == 1) {
impl->signal_interrupt(Service::GSP::InterruptId::PSC1);
}
}

3
src/video_core/gpu.h
View File

@ -99,7 +99,8 @@ public:
private:
void SubmitCmdList(u32 index);
void MemoryFill(u32 index);
// Interrupt index must be 0 or 1 to signal the relative PSC interrupt.
void MemoryFill(u32 index, u32 intr_index);
void MemoryTransfer();