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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

SPU: Quick fixes #4331

Merged
merged 3 commits into from
Mar 24, 2018
Merged

SPU: Quick fixes #4331

merged 3 commits into from
Mar 24, 2018

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Mar 24, 2018

  • Check get start type proxy mfc commands :
    crysis 1 (nothing => Intro)
    crysis 2 (nothing => Inagme)

  • Dont check prio if spu group type is 0x20 in sys_spu_thread_group_create
    far cry 4 (loadable => Intro)
    F1 race stars (nothing => Intro)

  • Fix spu image loading address calculation
    harry potter 7 part 1 (nothing => Intro/Loadable (depends on luck lol))
    fixes Regression: Baja: Edge of Control (#3366)聽#3834

Originated from raw fixes pr but neko wants a quike review and merge.. 馃槀

by calculating mem_size after nsegs has being calculated
it's currently unknown whats the exact relationship between the prio and the group type SYS_SPU_THREAD_GROUP_TYPE_COOPERATE_WITH_SYSTEM (0x20).
tho we do know prio'es whom less than 16 are reserved for the system.
@Kravickas
Copy link
Contributor

crysis 1 was ingame too in raw spu pr

@Kravickas
Copy link
Contributor

i know

crysis 1 (nothing => Intro)

@Nekotekina Nekotekina merged commit 63fa500 into RPCS3:master Mar 24, 2018
@elad335 elad335 deleted the quike branch March 24, 2018 15:56
@Asinin3
Copy link
Contributor

Asinin3 commented Mar 24, 2018

F1 Race stars is actually ingame
image
And Toybox turbos is Ingame (maybe playable) from nothing!

@elad335 elad335 changed the title spu: quike fixes SPU: Quick fixes Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression: Baja: Edge of Control (#3366)
5 participants