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

Disable icache emulation for Monster House #10739

Merged
merged 1 commit into from
Jun 11, 2022

Conversation

JMC47
Copy link
Contributor

@JMC47 JMC47 commented Jun 10, 2022

This game has stale icache values in the logs during certain loadscreens, and after disabling them I was able to play through the entire game, with one crash that may have been related to savestates and memory cards since during that loadscreen it said the memory card did not match.

This does cause invalid reads to be spit out during parts of the game, but you can safely skip them. Enabling MMU seems to surpress the invalid reads without side-effects, but the aforementioned memcard/savestate crash was with MMU on.

I did 100% complete the game with this set in order to verify that we can finally mark this annoying game as playable.

This game has stale icache values in the logs during certain
loadscreens, and after disabling them I was able to play through the
entire game, with one crash that may have been related to savestates and
memory cards since during that loadscreen it said the memory card did
not match.

This does cause invalid reads to be spit out during parts of the game,
but you can safely skip them.  Enabling MMU seems to surpress the
invalid reads without side-effects, but the aforementioned
memcard/savestate crash was with MMU on.
@dolphin-emu-bot
Copy link
Contributor

FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:

automated-fifoci-reporter

@JMC47 JMC47 merged commit ef384ca into dolphin-emu:master Jun 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants