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

Regression: Battle Princess of Arcadias (on Mult-threaded CPUs) #4985

Closed
HerrHulaHoop opened this issue Aug 14, 2018 · 1 comment
Closed

Comments

@HerrHulaHoop
Copy link

Battle Princess of Arcadias [NPUB31486 & NPEB02001] crashes for me after PR #3714.

Specs: (I've got logs from my laptop but I can confirm that the exact same thing happens with PC as well.)
Laptop: Intel(R) Core(TM) i7-3630QM CPU @ 2.40GHz | 8 Threads | 7.89 GiB RAM | AVX+
PC: Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz | 8 Threads | 15.94 GiB RAM | AVX+

With the previous master build, rpcs3-v0.0.4-2017-11-14-9e0b881b_win64, it goes ingame without any issue and is playable:
BPoA-6077

With build rpcs3-v0.0.4-2017-11-15-c435b328_win64 (PR 3714) it crashes after the logos on this screen:
BPoA-6089

It crashes with this error:
F 0:00:16.228045 {PPU[0x1000000] Thread (main_thread) [0x001a62a0]} MEM: Access violation reading location 0x42b

Attaching log of both builds. The error remains the same even with the PPU Interpreters. Also, this game takes a few attempts to boot so the log will have about 4-5 instances of me attempting to boot the game. The last run will have the error.
BPoA-6077-works.log.gz
BPoA-6089-crash.log.gz

Interesting point to note: 2 other people, DD555 and whym, have tried this game and weirdly, it is playable for both even on recent master builds. Both of them have a 4C4T CPU so I tried to see if disabling HT on my PC would solve the problem for me. It did not. However, Alchemy_Meister also faced this issue on a 6C12T CPU. So this is probably some weird race condition.

I have verified my copies of the game and can confirm that they are not broken in any way (and it works with the build prior to 3714).

@HerrHulaHoop
Copy link
Author

HerrHulaHoop commented Dec 8, 2018

Fixed in latest master after #5399!

BPoA

Edit: Just to clarify, only the regression is fixed. The separate issue of requiring multiple attempts to boot the game still stands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants