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

cellSaveData: Slow save completion (brings iconic game save icons back) #13492

Merged
merged 4 commits into from
Mar 7, 2023

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Mar 4, 2023

The gane's save thread completed saving too fast which led to save completion to go unnoticed due to of bad game code.
I also expect that this would bring back games' animated saving icons as the saving PPU finally rests for half a second (delaying half a second) letting other threads do work letting the icon time to be displayed.
Fixes #9936 for Jak II for me. This is timing dependant so it needs testing for others.

Fixes #9936
Fixes #10812
Fixes #10569
Fixes #12843
Fixes #8251
Fixes #6508
Fixes #10636

@Megamouse
Copy link
Contributor

Megamouse commented Mar 4, 2023

Are you sure the delay isn't expected due to the user prompt? (in case of CELL_SAVEDATA_CBRESULT_OK_LAST)

@Darkhost1999
Copy link
Contributor

Officially fixes
#10812

@Megamouse Megamouse added Firmware: HLE Savedata Anything that involves cellSaveData labels Mar 4, 2023
@elad335
Copy link
Contributor Author

elad335 commented Mar 4, 2023

It always return OK_LAST in its only savedata callback. But it also does it for when loading the game and I would not expect PS3 to confim loading savedata when booting the game. It also has its own confirmation screen and does not use PS3 savedata UI at all, I expect there is another condition which enables this screen but this game disables it in another way.

@Megamouse
Copy link
Contributor

Megamouse commented Mar 4, 2023

Is it autosave/load? Just curious. I wouldn't add additional dialogs to the autosave/load.
I'm fine with a delay. We do everything 100 times faster than the PS3 anyway.

@elad335
Copy link
Contributor Author

elad335 commented Mar 4, 2023

Yes these are autosave functions.

@kd-11
Copy link
Contributor

kd-11 commented Mar 4, 2023

Needs testing with DeS/Ds2. Last time we had slow save routines due to io and they became unplayable due to the autosave triggering every few seconds.

@elad335
Copy link
Contributor Author

elad335 commented Mar 4, 2023

Yes because the PPU wasn't sleeping which made the whole emulation get stuck. Real PS3 saving is pretty slow and may take a few seconds.

@elad335
Copy link
Contributor Author

elad335 commented Mar 4, 2023

Tested DeS, it brings its saving icon for half a second as expected and does not stutter.

@elad335 elad335 changed the title cellSaveData: Slow save completion cellSaveData: Slow save completion (brings iconic game save icons back) Mar 4, 2023
@YuriNator557
Copy link

Also fixes #10569

@jjsat
Copy link
Contributor

jjsat commented Mar 5, 2023

Fixes #8251 and fixes getting stuck on the option screen in Thor: God of Thunder.

@Shoegzer
Copy link

Shoegzer commented Mar 5, 2023

When testing this PR against #5968 I discovered RPCS3 crashes immediately on launch. It crashes on attempts to load any title, however. Master is okay.

pic

  • OS: Linux Mint OS 21.1 / kernel 5.15
  • CPU: Intel 10870H

@tylerjm917
Copy link

Fixes #9936 for me

@BrancuAlexandru
Copy link

This is amazing, thank you

@KabbaCallisto
Copy link

I'm sorry if this is silly but how do i actually apply this?

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