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

Implement sys_spu_thread_tryreceive_event #5331

Merged
merged 2 commits into from Nov 12, 2018

Conversation

Projects
None yet
3 participants
@elad335
Copy link
Contributor

elad335 commented Nov 12, 2018

Fixes BLUS30026 , BLUS30049

elad335 and others added some commits Nov 12, 2018

@Nekotekina Nekotekina merged commit 2e1aec4 into RPCS3:master Nov 12, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@elad335 elad335 deleted the elad335:spu branch Nov 12, 2018

@legend800

This comment has been minimized.

Copy link

legend800 commented Dec 22, 2018

@elad335 @Nekotekina

This broke Ridge Racer 7 on SPU LLVM (which is the ONLY way to get clean audio on the game). Worked fine on the previous build (7495).

Now you get a semophore error/soft lock in the menus. Have to shut down the emu. 100% repro on the same spot (car selection) but you get it all over the place in the menu system. Have to turn on xfloat or ASMJit to avoid.

rr7.zip

@elad335

This comment has been minimized.

Copy link
Contributor Author

elad335 commented Dec 22, 2018

@legend800 code path added in this pr used to throw an exception before about unknown spu stop code. your testing is wrong.

@legend800

This comment has been minimized.

Copy link

legend800 commented Dec 22, 2018

Ugh, sorry. I clicked on the wrong link. It was the NEXT PR that broke RR7. Sorry for wasting your time.

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