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

Suggest tweaking the setting "Stub PPU Traps" after PPU trap #10921

Merged
merged 3 commits into from Sep 30, 2021

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Sep 24, 2021

Description is in the code itself.

@@ -1493,7 +1493,7 @@ void ppu_trap(ppu_thread& ppu, u64 addr)
// If stubbing is enabled, check current instruction and the following
if (!add || !vm::check_addr(ppu.cia, vm::page_executable) || !vm::check_addr(ppu.cia + add, vm::page_executable))
{
fmt::throw_exception("PPU Trap!");
fmt::throw_exception("PPU Trap! Sometimes tweaking the setting \"Stub PPU Traps\" can workaround this crash.\nBest values depend on game code, if unsure try 1.");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
fmt::throw_exception("PPU Trap! Sometimes tweaking the setting \"Stub PPU Traps\" can workaround this crash.\nBest values depend on game code, if unsure try 1.");
fmt::throw_exception("PPU Trap! Sometimes tweaking the setting \"Stub PPU Traps\" can prevent this crash.\nBest values depend on game code, if unsure try 1.");

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@Nekotekina Nekotekina merged commit 71b15c1 into RPCS3:master Sep 30, 2021
@elad335 elad335 deleted the patch-38 branch September 19, 2022 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants