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

SPU: Extend Inaccurate SPU reservations #12628

Merged
merged 2 commits into from Sep 12, 2022
Merged

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Sep 8, 2022

Unleashes performance in some game.
Implements #12030
Fixes #9476

@elad335 elad335 changed the title SPU: Extend Inaccurate SPU reservatikns SPU: Extend Inaccurate SPU reservations Sep 8, 2022
@Yahfz
Copy link
Contributor

Yahfz commented Sep 8, 2022

closes #9476 too

@Jonathan44062
Copy link

Master
image

PR
image

No difference?

@Yahfz
Copy link
Contributor

Yahfz commented Sep 8, 2022

Master image

PR image

No difference?

did you use the setting?

@Jonathan44062
Copy link

What setting exactly? there's the log the PR build
RPCS3.log.gz

@YuriNator557
Copy link

SPU Accurate Reservations must be set to false, generate a new custom configuration and find it in the custom config file

@solarmystic
Copy link

Yep I see the improvement, from 66 to 71 FPS on average, with Accurate SPU Reservations set to false.

PR
SUtest

Master
SUmaster

@Jonathan44062
Copy link

Yup, Slight improve performance with Accurate SPU Reservations to False in my CPU i7 8700
image

@Yahfz
Copy link
Contributor

Yahfz commented Sep 9, 2022

Yup, Slight improve performance with Accurate SPU Reservations to False in my CPU i7 8700

By looking at the screenshots on your issue, it seems like you had 34fps there. So it seems like there's another regression missing too?

@Jonathan44062
Copy link

No idea, it was the same performance for me, I can't really tell if it is an regression or not

Another place in performance, taking screenshot is kinda hard to get exact FPS, because is very random
Master - Accurate SPU Reservations: True
https://user-images.githubusercontent.com/78843189/189270847-bc5ad6e4-ee3d-4675-a2ed-29ca43d4e6df.mp4

PR - Accurate SPU Reservations: False
https://user-images.githubusercontent.com/78843189/189270927-8ae100c7-01f0-4330-9099-751c0d85106f.mp4

Slight improve performance

@Xcedf
Copy link

Xcedf commented Sep 9, 2022

Master/PR Accurate SPU reservations: true
32
Custom build with non-accurate puttluc
31
PR Accurate SPU reservations: false
30

@elad335 elad335 force-pushed the putlluc branch 5 times, most recently from 559abb8 to 74116bf Compare September 9, 2022 07:03
@elad335
Copy link
Contributor Author

elad335 commented Sep 9, 2022

Packed more stuff in this setting.

@elad335 elad335 force-pushed the putlluc branch 2 times, most recently from 279138d to 8464708 Compare September 9, 2022 08:49
@MsDarkLow
Copy link
Contributor

MsDarkLow commented Sep 11, 2022

i7-8700K @ 5GHz | Windows 11
Tested Yakuza Ishin Main menu, there is a big fps improvement in the PR when going from True to False

  • Testing TSX vs No TSX delivers similar results and its all within margin of error.
  • When comparing MLAA patch between Master and the PR its also all within margin of error (RSX bottleneck)
Yakuza Ishin Main Menu Accurate SPU True Accurate SPU False
PR Master PR Master
TSX Disabled 41.9 40.1 76.2 38.8
TSX 38.6 same as above 77.3 same as above
MLAA Patch 202.2 similar 206.0 similar

@Nerboruto
Copy link

Nerboruto commented Sep 12, 2022

  • Testing TSX vs No TSX delivers similar results and its all within margin of error.

I tried to enable tsx on xeon e3-1230v3 on motorstorm pacific game and in fact from what I could now understand tsx does not give any advantage of fps ...
if I can solve the problems with capframex that crashes I will be able to have precise numbers.

@Augusto7743
Copy link

GOWA not start if using
SPU Accurate Reservations = false

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: Sonic Unleashed lost massive performance (#8888)
10 participants