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

Increases default stack allocation for callbacks #5325

Merged
merged 2 commits into from
Nov 12, 2018

Conversation

RipleyTom
Copy link
Contributor

This change moves "Afrika" a bit further ingame.

Before it died because a callback stores a value at r1[0x30]

seg001:002604A8 stdu r1, -0x40(r1)
seg001:002604AC std r31, 0x38(r1)
seg001:002604B0 mr r31, r1
seg001:002604B4 std r3, 0x70(r31) <- stores a value at +0x30 over the previous stack value

With previous stack allocation r1[0x30] was the r1[0] for the previous caller which is where is stored the saved stack.

@RipleyTom RipleyTom force-pushed the ExtraStack branch 2 times, most recently from ef5cd98 to 9c1d4af Compare November 10, 2018 21:56
@digitaldude555
Copy link

screenshot 965

@Nekotekina Nekotekina merged commit fada8b6 into RPCS3:master Nov 12, 2018
elad335 added a commit to elad335/rpcs3 that referenced this pull request Feb 9, 2019
elad335 added a commit to elad335/rpcs3 that referenced this pull request Feb 9, 2019
elad335 added a commit to elad335/rpcs3 that referenced this pull request Feb 9, 2019
elad335 added a commit to elad335/rpcs3 that referenced this pull request Feb 9, 2019
elad335 added a commit to elad335/rpcs3 that referenced this pull request Feb 9, 2019
@RipleyTom RipleyTom deleted the ExtraStack branch June 4, 2019 17:19
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.

None yet

3 participants