ppsspp icon indicating copy to clipboard operation
ppsspp copied to clipboard

Crash Persona 2 Innocent Sin after a while

Open jimkage opened this issue 3 years ago • 8 comments

Game or games this happens in

ULUS-10584 Persona 2 Innocent Sin

What area of the game / PPSSPP

Game.crashes after a while in some areas: City maps, Lisas Home, Theater and etc.

Save: https://www.mediafire.com/file/ckac69wab43gojw/ULUS105840000.zip/file

What should happen

Crash videos Lisa's house https://youtu.be/G41pO8eKNRE

Map select https://youtu.be/oK7X6Bu1IQE

City (Inside the store has no crash) https://youtu.be/JovmicRztK8

Logs

https://www.mediafire.com/file/vr2wq45xz3tbfkj/ppsspp.rar/file

Platform

Android

Mobile phone model or graphics card

Moto G100

PPSSPP version affected

v1.13.1-153-g4d519b49a

Last working version

Tested over a dozen builds and it still happens

Graphics backend (3D API)

OpenGL / GLES

Checklist

  • [X] Test in the latest git build in case it's already fixed.
  • [X] Search for other reports of the same issue.
  • [X] Try resetting settings or older versions and include if the issue is related.
  • [X] Try without any cheats and without loading any save states.
  • [X] Include logs or screenshots of issue.

jimkage avatar Aug 03 '22 19:08 jimkage

Hm, this pretty much has to be some kind of memory leak. Will investigate

hrydgard avatar Aug 03 '22 22:08 hrydgard

Tested over a dozen builds and it still happens

How about the last stable release, v1.12.3? Did it happen there? If no, that gives us a starting point and trying halfway between the working and broken would help identify.

-[Unknown]

unknownbrackets avatar Aug 07 '22 15:08 unknownbrackets

Just tested ppsspp-v1.12.3-1467-g6f4ec0fc4-android and it still happen.

jimkage avatar Aug 08 '22 00:08 jimkage

That contains almost all the changes in v1.13.x. That means it's v1.12.3 plus 1467 changes. v1.13.0 was v.12.3 plus 1469 changes, so that's virtually the same as testing v1.13.0.

-[Unknown]

unknownbrackets avatar Aug 08 '22 00:08 unknownbrackets

Alright so I went all the way down to ppsspp-v1.12.2-9-g11b65ca22-android and got the game running for 15 minutes without any crashes. I hope this helps.

jimkage avatar Aug 08 '22 00:08 jimkage

So now we know the error is between -9 and -1467. How about trying one right in the middle? That will halve the range. And once you find if it broke before or after, you can split the new range in the middle too. If you keep doing this, you'll find the problem in 10 tries..

hrydgard avatar Aug 08 '22 04:08 hrydgard

Alright heres the last working version: ppsspp-v1.12.3-8-gc75784351-android from ppsspp-v1.12.3-10-g1cd520ae3-android onwards it starts crashing.

jimkage avatar Aug 08 '22 14:08 jimkage

Huh, surprised it's that early in the sequence. but if it is, it is.

Now, looking at the list of builds, from 8 to 9, this is the difference: #15013 (just a translation, cannot possibly be the cause)

And from 9 to 10 (for some reason there are two 10), these make up the difference: 7d00b6ca9, 258a96a47. The first one is the only one that's even remotely possibly the cause, but don't see how it can cause a memory leak, nor any other kind of long-running-caused crash.

Strange...

hrydgard avatar Aug 08 '22 15:08 hrydgard

I believe that this is the same as #15788 , which is fixed now.

Can you please test with the latest builds @jimkage ?

hrydgard avatar Sep 16 '22 07:09 hrydgard

Cannot reproduce this crash in theater. Screenshot_2022-09-16-20-19-10-42

ghost avatar Sep 16 '22 12:09 ghost

For testing purposes, likely the bug and the fix only affect Android 12+ devices.

hrydgard avatar Sep 16 '22 12:09 hrydgard

Anyway, I'm gonna close. Let's re-open if we hear about this again.

hrydgard avatar Sep 18 '22 09:09 hrydgard