Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GameSettings: Add patch for Dead to Rights audio #11086

Merged
merged 1 commit into from Sep 27, 2022

Conversation

smurf3tte
Copy link
Contributor

This game follows the anti-pattern of calling memset on a buffer in the midst of being DMA copied to ARAM, then calling a DVD read function that effectively cancels the memset with dcbi instructions. Dolphin does not emulate dcache for performance reasons, so this patch removes the offending memset call.

Patches are included for two regions and are enabled by default.

https://bugs.dolphin-emu.org/issues/12759

This game follows the anti-pattern of calling memset on a buffer in the
midst of being DMA copied to ARAM, then calling a DVD read function that
effectively cancels the memset with dcbi instructions. Dolphin does not
emulate dcache for performance reasons, so this patch removes the
offending memset call.

Patches are included for two regions and are enabled by default.

https://bugs.dolphin-emu.org/issues/12759
@Zopolis4
Copy link
Contributor

This (presumably) would solve https://bugs.dolphin-emu.org/issues/12759.

@JMC47
Copy link
Contributor

JMC47 commented Sep 24, 2022

Can confirm the issue and patch make sense.

@AdmiralCurtiss
Copy link
Contributor

Oh, so this is basically another instance of the RE3 audio bug?

@AdmiralCurtiss
Copy link
Contributor

AdmiralCurtiss commented Sep 24, 2022

Is there a third version of this game too? Our wiki suggests there's three versions of this by listing three game IDs, but redump only lists an EU and a US one. Moreover, the EU version does not have a German language listed, which suggests there's a German version with removed blood or something like that, as was pretty common at the time, but that doesn't quite match up with the game IDs on the Wiki...

e: I cannot find any evidence of a German copy at all, maybe this just never came out in German?

@JosJuice
Copy link
Member

If there was a separate German version, it wouldn't have the game ID GDRP. To me this just looks like some wiki editor made up a game ID with the wrong maker ID. It has happened before.

@JMC47
Copy link
Contributor

JMC47 commented Sep 27, 2022

Is there anything else to do with this? It might be worth merging to get people playing the game to test it throughout the whole game instead of just focused tests from me.

@JMC47 JMC47 merged commit d9cd819 into dolphin-emu:master Sep 27, 2022
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants