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

Metal Gear Solid Peace Walker Chinese Patched blue screen #14127

Closed
sum2012 opened this issue Feb 12, 2021 · 10 comments
Closed

Metal Gear Solid Peace Walker Chinese Patched blue screen #14127

sum2012 opened this issue Feb 12, 2021 · 10 comments
Labels
HLE/Kernel Kernel, memory manager, other HLE issues Patched-game Occurs only on patched or modified version of a game.
Milestone

Comments

@sum2012
Copy link
Collaborator

sum2012 commented Feb 12, 2021

-v1.11.1-2-g104d7e1a6 log
https://gist.githubusercontent.com/sum2012/160297ac1c18d3faa1ba5b7566b844f4/raw/e098e9e96bacfc4d61a26c04fb729a934774d6ba/gistfile1.txt

@sum2012 sum2012 added the Patched-game Occurs only on patched or modified version of a game. label Feb 12, 2021
@sum2012 sum2012 changed the title Metal Gear Solid Peace Walker Chinese Patch blue screen Metal Gear Solid Peace Walker Chinese Patched blue screen Feb 12, 2021
@sum2012 sum2012 added the HLE/Kernel Kernel, memory manager, other HLE issues label Feb 12, 2021
@sum2012
Copy link
Collaborator Author

sum2012 commented Feb 12, 2021

Edit nothing

@unknownbrackets
Copy link
Collaborator

Did it ever previously work?

-[Unknown]

@sum2012
Copy link
Collaborator Author

sum2012 commented Feb 13, 2021

In the past, we are ignore invalid address to play.
Kernel only debug add this line
23:36:930 kfs io0 I[SCEMODULE]: hle\scekernelmodule.cpp:2507 322=sceKernelLoadModuleByID(4,flag=00000000,00000014,00000002,00000000,position = 00000001)

Jpcsp emulator run this game no problem without ignore invalid address

sum2012 added a commit to sum2012/ppsspp that referenced this issue Feb 14, 2021
Fix hrydgard#14127
It is from chinese transate team
@sum2012
Copy link
Collaborator Author

sum2012 commented Feb 14, 2021

@unknownbrackets
Copy link
Collaborator

jmpLibFont seems to be a custom font library that uses sceFont behind the scenes, so blocking it from loading seems like it would cause errors, based on that log.

The change I made recently to more correctly handle sceKernelModuleStart with no start address didn't help, did it?

Read Word: Invalid address 0a000000 PC 08e1b560 LR 08e1b560 looks like it's actually during relocations or module handling, because it's logged after the imports but before the exports. So this seems like something wrong with ELF loading or relocations...

Is this jmp prx specific to the Chinese patch? If so, is it open source? I'd like to look at it if it's not copyright restricted.

-[Unknown]

@sum2012
Copy link
Collaborator Author

sum2012 commented Mar 21, 2021

v1.11.2-355-g7db53d2f0-windows-amd64 still same problem.
log:
https://gist.github.com/sum2012/4109b087fa7b0ea535304263aef3e3b8
Chinese patched file are closed source.

@unknownbrackets
Copy link
Collaborator

The prx is different from this one right?
https://github.com/hz86/mgspw

Seems likely related, I guess the idea is it replaces the text using a lookup when being drawn through this prx.

-[Unknown]

@sum2012
Copy link
Collaborator Author

sum2012 commented Mar 21, 2021

I use binary compare.
Yes,It is same file

@unknownbrackets
Copy link
Collaborator

How does it work now?

-[Unknown]

@sum2012
Copy link
Collaborator Author

sum2012 commented Mar 29, 2021

PPSSPP v1.11.2-375-g50d5195b4
Fixed.log: https://gist.github.com/sum2012/9888cefec359143f0fbb335c8dd33c44
Thanks @unknownbrackets
Closing

@sum2012 sum2012 closed this as completed Mar 29, 2021
@sum2012 sum2012 added this to the v1.12.0 milestone Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HLE/Kernel Kernel, memory manager, other HLE issues Patched-game Occurs only on patched or modified version of a game.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants