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

GameDB: Add DMABusy fix to Shining Wind #5683

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

KrossX
Copy link
Contributor

@KrossX KrossX commented Mar 17, 2022

Description of Changes

Booting Shining Wind results in a crash with the assert "Gif Path Buffer Overflow!". Either EETimingHack or DMABusyHack seems to fix booting, picked the DMA one to avoid Emotion.

Rationale behind Changes

0016

Suggested Testing Steps

Run the game with a master build, see if it crashes.
Run it again with manual game fixes, enable Handle DMAC, see if it boots.

@Mrlinkwii
Copy link
Contributor

 SLKA-25215
 SLPM-74261

the game also has these serials , can you add it to them is well?

@KrossX
Copy link
Contributor Author

KrossX commented Mar 17, 2022

Done, although I cannot test if it was necessary for those titles. I suppose the "The Best" release is the very same, but is the korean version the same as well?

@Mrlinkwii
Copy link
Contributor

Mrlinkwii commented Mar 17, 2022

Done, although I cannot test if it was necessary for those titles. I suppose the "The Best" release is the very same, but is the korean version the same as well?

usually games from different region usually have the same underlying issue , so its better off getting all region covered and if theirs a region that dosent need it , it can be removed later :) , it also help filling in missing serials in the Gamedb 😄

@refractionpcsx2 refractionpcsx2 merged commit a4932ed into PCSX2:master Mar 17, 2022
@KrossX KrossX deleted the gdfix_shining_wind branch March 17, 2022 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants