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

Fix preformance regression on TSX path #8114

Merged
merged 1 commit into from Apr 28, 2020
Merged

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Apr 28, 2020

Seen as PPU LDARX/LWARX log spam introduced by #8051 that wasn't before it in some games.

@Nekotekina
Copy link
Member

Nekotekina commented Apr 28, 2020

It won't spam if we don't check for lock bits. What was the problem with PPU reservation loads?

@elad335
Copy link
Contributor Author

elad335 commented Apr 28, 2020

Hmm I misinterpreted lock bits on tsx path as similar on non-tsx where always signifies on-going reservation modification. So it can be reverted.
SPU code side of it can stay the same for now.

@Nekotekina Nekotekina merged commit a505d87 into RPCS3:master Apr 28, 2020
@elad335 elad335 deleted the tsx-fixup branch April 28, 2020 17:49
elad335 added a commit to elad335/rpcs3 that referenced this pull request Apr 29, 2020
elad335 added a commit to elad335/rpcs3 that referenced this pull request Apr 29, 2020
elad335 added a commit to elad335/rpcs3 that referenced this pull request Apr 29, 2020
elad335 added a commit to elad335/rpcs3 that referenced this pull request Apr 29, 2020
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

2 participants