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

Mahou Shoujo Madoka Magica Portable in-game problem #6557

Closed
sum2012 opened this issue Jul 15, 2014 · 16 comments · Fixed by #15930
Closed

Mahou Shoujo Madoka Magica Portable in-game problem #6557

sum2012 opened this issue Jul 15, 2014 · 16 comments · Fixed by #15930
Labels
I/O Affected by I/O timing settings, or other kind of I/O issue.
Milestone

Comments

@sum2012
Copy link
Collaborator

sum2012 commented Jul 15, 2014

Load the save
https://drive.google.com/file/d/0B3OaSdeV0L8kb0ZRRTN2amltdDA/edit?usp=sharing

after die,W[MM]: MemmapFunctions.cpp:93 ReadFromHardware: Invalid address 00000000

v0.9.8-1614-g459e244
info log:
https://gist.github.com/sum2012/654b7f4370045f5fa286
debug log:
https://drive.google.com/file/d/0B3OaSdeV0L8kcGRBOER1NXBhcEU/edit?usp=sharing

@unknownbrackets
Copy link
Collaborator

Has it ever worked? Does it work in JPCSP?

-[Unknown]

@daniel229
Copy link
Collaborator

PPSSPP always hangs there,0.7 to the latest,and for JPCSP,the game go to intro and playable since this https://code.google.com/p/jpcsp/source/detail?r=2268 ,and does not has problem there.

@unknownbrackets
Copy link
Collaborator

Hmm, interesting. Well, I asked because I found out Gods Eater Burst also does not load real PSP savedata correctly, so I was wondering if it was related. But now I think I'm understanding that the savedata is just to get to the problem, and the problem is there either way.

Unknown GetPointer 00000000 PC 00000000 LR 0889f2ac

Seems bad. What's at 0889f2ac?

Might possibly be sas or dispatch thread related...

-[Unknown]

@daniel229
Copy link
Collaborator

01

@unknownbrackets
Copy link
Collaborator

Hmm, so the pointer at t9 is becoming 0. Most likely we're either corrupting memory there, not scheduling properly, and not writing something there we ought to...

-[Unknown]

@daniel229
Copy link
Collaborator

By the way,this is not a savedata issue.

@unknownbrackets
Copy link
Collaborator

So it seems this is io delay related, or possibly async io related.

-[Unknown]

@unknownbrackets
Copy link
Collaborator

Reported not fixed by the async IO changes in #7636.

-[Unknown]

@Panderner
Copy link
Contributor

Simulated UMD delays helps it to prevent crashes

@sum2012 sum2012 added the I/O Affected by I/O timing settings, or other kind of I/O issue. label Jan 25, 2021
@sum2012 sum2012 added this to the Future milestone Jan 25, 2021
@sum2012
Copy link
Collaborator Author

sum2012 commented Jan 25, 2021

Update savedata to easier to reproduce.
https://drive.google.com/file/d/1wvDHLaYCReY0wmH6DZscW1hoWN3kcN4n/view?usp=sharing
v1.10.3-1639-g4eee5df2a
bad log: https://drive.google.com/file/d/1vOw7m5_kpIYehpA5dc9PnoXB1GI-83L-/view?usp=sharing
Simulate UMD delay good log: https://drive.google.com/file/d/1c03HMxkfaqopsUxKrT8QbnFz_H1knTnu/view?usp=sharing

The game maybe related to I\O add sceKernelUnlockMutex,sceKernelLockMutex staff

@sum2012
Copy link
Collaborator Author

sum2012 commented Jan 25, 2021

@sum2012
Copy link
Collaborator Author

sum2012 commented Jan 26, 2021

@sum2012
Copy link
Collaborator Author

sum2012 commented Aug 29, 2022

v1.13.1-603-g59cb7a075
Still need Simulated UMD delays

@unknownbrackets
Copy link
Collaborator

@unknownbrackets Can you try translate it ? seem ppsspp don't have it. https://github.com/jpcsp/jpcsp/blob/master/src/jpcsp/HLE/kernel/managers/MutexManager.java#L154

Sorry, I think I never saw this before. PPSSPP does do that, here:

__KernelChangeReadyState(this, GetUID(), true);

-[Unknown]

sum2012 added a commit to sum2012/ppsspp that referenced this issue Aug 30, 2022
@sum2012
Copy link
Collaborator Author

sum2012 commented Aug 30, 2022

I was too deep to jpcsp's code ,don't think the game's code itself

@sum2012 sum2012 reopened this Sep 1, 2022
@sum2012
Copy link
Collaborator Author

sum2012 commented Sep 1, 2022

Reopen as it is wrong change

@hrydgard hrydgard removed this from the Future milestone Sep 7, 2023
@hrydgard hrydgard added this to the v1.17.0 milestone Sep 7, 2023
@sum2012 sum2012 closed this as completed Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I/O Affected by I/O timing settings, or other kind of I/O issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants