PlayStation Core Full Savestate Support

Xbytez
Site Admin
Posts: 469
Joined: Wed May 20, 2020 3:36 pm
Has thanked: 214 times
Been thanked: 787 times

PlayStation Core Full Savestate Support

Unread post by Xbytez »

Amazing! So much talent! Robert you rock! :D

FoxbatStargazer
Top Contributor
Posts: 994
Joined: Thu Dec 10, 2020 5:44 pm
Has thanked: 309 times
Been thanked: 228 times

Re: PlayStation Core Full Savestate Support

Unread post by FoxbatStargazer »

If there's enough FPGA resources to have save states on the PS1, it suggests that nearly every core could support them with enough effort.

Almost enough to make me crawl through the Genesis code. :P
FPGAzumSpass
Core Developer
Posts: 380
Joined: Sat May 23, 2020 12:55 pm
Has thanked: 38 times
Been thanked: 383 times

Re: PlayStation Core Full Savestate Support

Unread post by FPGAzumSpass »

In retrospective i can say that the costs for savestates on the PS1 are about 25%.
This is only for the core, not the framework.

So i wouldn't try it on any core that is above 65-70% logic usage already.
User avatar
Caldor
Top Contributor
Posts: 930
Joined: Sat Jul 25, 2020 11:20 am
Has thanked: 112 times
Been thanked: 111 times

Re: PlayStation Core Full Savestate Support

Unread post by Caldor »

FPGAzumSpass wrote: Sat Dec 04, 2021 3:33 pm In retrospective i can say that the costs for savestates on the PS1 are about 25%.
This is only for the core, not the framework.

So i wouldn't try it on any core that is above 65-70% logic usage already.
The savestate feature alone uses 25% of the FPGA logic? I figured the PS1 core would on its own be close to using the full logic space of the DE10-Nano?

Wont the sound then be a problem when it comes to having audio and the savestate feature in the core at the same time? Unless you already reserved the required space for it or something. Although I do not know how much of the space you already used.
MostroW
Posts: 323
Joined: Tue Aug 18, 2020 3:32 pm
Has thanked: 140 times
Been thanked: 43 times

Re: PlayStation Core Full Savestate Support

Unread post by MostroW »

Robert just keeps on speeding along progress at high speed.
FPGAzumSpass
Core Developer
Posts: 380
Joined: Sat May 23, 2020 12:55 pm
Has thanked: 38 times
Been thanked: 383 times

Re: PlayStation Core Full Savestate Support

Unread post by FPGAzumSpass »

Caldor wrote: Sun Dec 05, 2021 9:33 pm The savestate feature alone uses 25% of the FPGA logic? I figured the PS1 core would on its own be close to using the full logic space of the DE10-Nano?

Wont the sound then be a problem when it comes to having audio and the savestate feature in the core at the same time? Unless you already reserved the required space for it or something. Although I do not know how much of the space you already used.
25% for the core, nothing for framework, so about 20% of the FPGA, yes.

There is still ~20% of the FPGA left.
Biggest modules currently are GPU with 14%, GTE with 10% and CPU with 9%.

So the chances are quiet good for SPU to fit in, as i would assume it to be around 10% max as well.
CD might need another 5% for supporting Audio CDs and Memory Cards another 1-2%.

Overall it's tight, but i have good hope that i don't need to remove the savestate feature to fit everything in.
hiddenbyleaves
Posts: 121
Joined: Mon Nov 02, 2020 11:25 am
Has thanked: 48 times
Been thanked: 19 times

Re: PlayStation Core Full Savestate Support

Unread post by hiddenbyleaves »

Unbelievable!
User avatar
NinjaWarrior
Posts: 38
Joined: Fri Nov 19, 2021 11:48 pm
Has thanked: 22 times
Been thanked: 14 times

Re: PlayStation Core Full Savestate Support

Unread post by NinjaWarrior »

Indeed good news!
User avatar
Phoenix
Posts: 24
Joined: Tue May 11, 2021 8:08 pm
Has thanked: 5 times
Been thanked: 6 times

Re: PlayStation Core Full Savestate Support

Unread post by Phoenix »

Not really related but I'm not sure if my question deserves a topic on its own...

So if I understand correctly, ps1 core won't use 100% of DE10-Nano capacity, does it also mean that there is no risk of overheating?
Because that's my most fear about "3d" cores, I don't know if it could cause overheating because of the charge needed to run those games...
RalphJB
Posts: 47
Joined: Fri Aug 28, 2020 1:29 am
Been thanked: 7 times

Re: PlayStation Core Full Savestate Support

Unread post by RalphJB »

FPGAzumSpass wrote: Mon Dec 06, 2021 5:09 am
Caldor wrote: Sun Dec 05, 2021 9:33 pm The savestate feature alone uses 25% of the FPGA logic? I figured the PS1 core would on its own be close to using the full logic space of the DE10-Nano?

Wont the sound then be a problem when it comes to having audio and the savestate feature in the core at the same time? Unless you already reserved the required space for it or something. Although I do not know how much of the space you already used.
25% for the core, nothing for framework, so about 20% of the FPGA, yes.

There is still ~20% of the FPGA left.
Biggest modules currently are GPU with 14%, GTE with 10% and CPU with 9%.

So the chances are quiet good for SPU to fit in, as i would assume it to be around 10% max as well.
CD might need another 5% for supporting Audio CDs and Memory Cards another 1-2%.

Overall it's tight, but i have good hope that i don't need to remove the savestate feature to fit everything in.
STOP!
I can only get so erect!
User avatar
Chris23235
Top Contributor
Posts: 841
Joined: Sun May 24, 2020 8:45 pm
Has thanked: 106 times
Been thanked: 167 times

Re: PlayStation Core Full Savestate Support

Unread post by Chris23235 »

Phoenix wrote: Mon Dec 06, 2021 4:10 pm Not really related but I'm not sure if my question deserves a topic on its own...

So if I understand correctly, ps1 core won't use 100% of DE10-Nano capacity, does it also mean that there is no risk of overheating?
Because that's my most fear about "3d" cores, I don't know if it could cause overheating because of the charge needed to run those games...
Even if a core utilises 100%, it does not lead to overheat. The FPGA part of the DE 10 always runs cool. The heat comes ftom the ARM CPU running at fullspeed all the time.

If you use your MiSTer you always reach maximum temperatures because of the ARM, regardless of the core you use.
ToothbrushThreepwood
Posts: 89
Joined: Sun May 24, 2020 8:39 pm
Has thanked: 30 times
Been thanked: 32 times

Re: PlayStation Core Full Savestate Support

Unread post by ToothbrushThreepwood »

Chris23235 wrote: Mon Dec 06, 2021 5:54 pm
Phoenix wrote: Mon Dec 06, 2021 4:10 pm Not really related but I'm not sure if my question deserves a topic on its own...

So if I understand correctly, ps1 core won't use 100% of DE10-Nano capacity, does it also mean that there is no risk of overheating?
Because that's my most fear about "3d" cores, I don't know if it could cause overheating because of the charge needed to run those games...
Even if a core utilises 100%, it does not lead to overheat. The FPGA part of the DE 10 always runs cool. The heat comes ftom the ARM CPU running at fullspeed all the time.

If you use your MiSTer you always reach maximum temperatures because of the ARM, regardless of the core you use.
And these heat levels are harmless, even without heatsink and fan. Worst case, a few cores with imperfect timings may bug out due to elevated temperatures, but there is zero risk of frying any hardware.
User avatar
Phoenix
Posts: 24
Joined: Tue May 11, 2021 8:08 pm
Has thanked: 5 times
Been thanked: 6 times

Re: PlayStation Core Full Savestate Support

Unread post by Phoenix »

Thanks a lot for your answers, what a great community! :-)
held
Posts: 209
Joined: Sun Sep 26, 2021 2:18 pm
Has thanked: 27 times
Been thanked: 31 times

Re: PlayStation Core Full Savestate Support

Unread post by held »

Brilliant :D

@FPGAzumSpass: Just an idea: send bugreports with savestate from the OSD ?
FPGAzumSpass
Core Developer
Posts: 380
Joined: Sat May 23, 2020 12:55 pm
Has thanked: 38 times
Been thanked: 383 times

Re: PlayStation Core Full Savestate Support

Unread post by FPGAzumSpass »

No bugreports for now please, but when the core gets more stable, providing a savestate BEFORE a bug happens (not when game is already crashed) is the prefered way.
User avatar
Caldor
Top Contributor
Posts: 930
Joined: Sat Jul 25, 2020 11:20 am
Has thanked: 112 times
Been thanked: 111 times

Re: PlayStation Core Full Savestate Support

Unread post by Caldor »

FPGAzumSpass wrote: Mon Dec 06, 2021 5:09 am
Caldor wrote: Sun Dec 05, 2021 9:33 pm The savestate feature alone uses 25% of the FPGA logic? I figured the PS1 core would on its own be close to using the full logic space of the DE10-Nano?

Wont the sound then be a problem when it comes to having audio and the savestate feature in the core at the same time? Unless you already reserved the required space for it or something. Although I do not know how much of the space you already used.
25% for the core, nothing for framework, so about 20% of the FPGA, yes.

There is still ~20% of the FPGA left.
Biggest modules currently are GPU with 14%, GTE with 10% and CPU with 9%.

So the chances are quiet good for SPU to fit in, as i would assume it to be around 10% max as well.
CD might need another 5% for supporting Audio CDs and Memory Cards another 1-2%.

Overall it's tight, but i have good hope that i don't need to remove the savestate feature to fit everything in.
That is great to hear :) Amazing work.
held
Posts: 209
Joined: Sun Sep 26, 2021 2:18 pm
Has thanked: 27 times
Been thanked: 31 times

Re: PlayStation Core Full Savestate Support

Unread post by held »

FPGAzumSpass wrote: Tue Dec 07, 2021 7:26 pm No bugreports for now please, but when the core gets more stable, providing a savestate BEFORE a bug happens (not when game is already crashed) is the prefered way.
Let me clarify what I mean, because I did poorly in retrospect.

Is it possible to add two functions to the core menu:
  • update option, to download and install the latest version of the core
  • bug-report option, to file a bug report *in the future*
This means for us users:
  • we can easily send a bug report *in the future* from the couch
  • we don't have to think about what information to provide to the dev except maybe a comment such as: "open left door", "pick up 5 stars", "when the timer expires" and the like.
  • this prevents mistakes being made on our side
and this means for the dev:
  • dev has 100% control over when bugs are reported (by dis/enabling the bug-report option)
  • dev has 100% control over how bugs are reported, what information is send/needed (bios version,disc serial,core version, snapshot)
  • bug-reports are always made with the latest core (at the time) or the bug-report option is disabled
  • chance of reproducibility is high because users are less likely to make mistakes
User avatar
Cebion
Posts: 115
Joined: Sun May 24, 2020 7:30 pm
Has thanked: 1 time
Been thanked: 3 times

Re: PlayStation Core Full Savestate Support

Unread post by Cebion »

Why would such a thing ever be in a core?!
User avatar
RealLarry
Top Contributor
Posts: 767
Joined: Mon May 25, 2020 4:04 am
Location: San Junipero/DE/Earth
Has thanked: 85 times
Been thanked: 328 times

Re: PlayStation Core Full Savestate Support

Unread post by RealLarry »

held wrote: Wed Dec 08, 2021 9:23 amIs it possible to add two functions to the core menu:
  • update option, to download and install the latest version of the core
  • bug-report option, to file a bug report *in the future*
[sarcasm] Great idea! Additionally a full blown KDE setup and Thunderbird for mailing the bug reports! And a flux capacitor!1!! [/sarcasm]
No, boy, nice intention, but nothing for a core.
Contributor of tty2oled, author of tty2tft, tty2rpi and update_tty2xxx
MostroW
Posts: 323
Joined: Tue Aug 18, 2020 3:32 pm
Has thanked: 140 times
Been thanked: 43 times

Re: PlayStation Core Full Savestate Support

Unread post by MostroW »

i'm guessing if Robert (or any other core devs) really want some user input about core features they will query people for this?
User avatar
Caldor
Top Contributor
Posts: 930
Joined: Sat Jul 25, 2020 11:20 am
Has thanked: 112 times
Been thanked: 111 times

Re: PlayStation Core Full Savestate Support

Unread post by Caldor »

RealLarry wrote: Wed Dec 08, 2021 12:01 pm
held wrote: Wed Dec 08, 2021 9:23 amIs it possible to add two functions to the core menu:
  • update option, to download and install the latest version of the core
  • bug-report option, to file a bug report *in the future*
[sarcasm] Great idea! Additionally a full blown KDE setup and Thunderbird for mailing the bug reports! And a flux capacitor!1!! [/sarcasm]
No, boy, nice intention, but nothing for a core.
Also, once the core has been released it will become part of the normal "update" flow for the MiSTer that updates all the cores. Well, that is how it usually goes at least. As long as a core is beta, we have to be happy if we can get core updates without having to build it ourselves, and this core now has nightly builds, on the official Discord MiSTer server group. I think there is a link on the forums somewhere, or maybe in the FB group.

When it comes to bug reports, that will probably be done on the GIT and there will probably be some guidelines on how to do bug reports.
FPGAzumSpass
Core Developer
Posts: 380
Joined: Sat May 23, 2020 12:55 pm
Has thanked: 38 times
Been thanked: 383 times

Re: PlayStation Core Full Savestate Support

Unread post by FPGAzumSpass »

A proper bug report requires a thinking user, that's nothing that can just be automated.

For this year, i have enough open bugs. When i start a new bugfix phase next year, I will open a thread and ask for specific bugs and tell what info is required.

It's great the community wants to help, and I'm more than willing to take this help to make the core stable and working as fast as possible.
held
Posts: 209
Joined: Sun Sep 26, 2021 2:18 pm
Has thanked: 27 times
Been thanked: 31 times

Re: PlayStation Core Full Savestate Support

Unread post by held »

@RealLarry:
Who said anything about email?
I like constructive feedback, but if you start answering your own made up sarcastic hypothetical question by not reading properly, and then start insulting people by calling them boy: "you are not helping anyone".

@Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.

@FPGAzumSpass:
Thinking users?, I did not know the requirement was that steep ;) I still think it was a great idea, but there is hardly a point in arguing with a dev.
Neocaron
Posts: 341
Joined: Sun Sep 27, 2020 10:16 am
Has thanked: 187 times
Been thanked: 66 times

Re: PlayStation Core Full Savestate Support

Unread post by Neocaron »

held wrote: Thu Dec 09, 2021 7:36 am @RealLarry:
Who said anything about email?
I like constructive feedback, but if you start answering your own made up sarcastic hypothetical question by not reading properly, and then start insulting people by calling them boy: "you are not helping anyone".

@Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.

@FPGAzumSpass:
Thinking users?, I did not know the requirement was that steep ;) I still think it was a great idea, but there is hardly a point in arguing with a dev.
I think you overthink it. Up until this point this was not necessary for any core, and it never stopped the bugs to be squish. So don't worry too much about it. Once the core is almost done, people will leave feedback the way they always do and the core will be nice and shinny. The thought was still good, but it's just not applicable for the Mister project in general. Cheers!
User avatar
RealLarry
Top Contributor
Posts: 767
Joined: Mon May 25, 2020 4:04 am
Location: San Junipero/DE/Earth
Has thanked: 85 times
Been thanked: 328 times

Re: PlayStation Core Full Savestate Support

Unread post by RealLarry »

Neocaron wrote: Thu Dec 09, 2021 10:01 am
held wrote: Thu Dec 09, 2021 7:36 am @RealLarry:
Who said anything about email?
I like constructive feedback, but if you start answering your own made up sarcastic hypothetical question by not reading properly, and then start insulting people by calling them boy: "you are not helping anyone".

@Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.

@FPGAzumSpass:
Thinking users?, I did not know the requirement was that steep ;) I still think it was a great idea, but there is hardly a point in arguing with a dev.
I think you overthink it. Up until this point this was not necessary for any core, and it never stopped the bugs to be squish. So don't worry too much about it. Once the core is almost done, people will leave feedback the way they always do and the core will be nice and shinny. The thought was still good, but it's just not applicable for the Mister project in general. Cheers!
Copy that, thanks for these wise words and sorry to @held if I've pissed you off - but there were these [sarcastic] tags for that, meaning Don't really take my words seriously.
I'm co-authoring and co-developing hard- and software for the MiSTer project and we love and need feedback from users, but there are "the usual channels" like this forum and github repos for that, therefore no need for Extrawurst :)
Contributor of tty2oled, author of tty2tft, tty2rpi and update_tty2xxx
User avatar
Caldor
Top Contributor
Posts: 930
Joined: Sat Jul 25, 2020 11:20 am
Has thanked: 112 times
Been thanked: 111 times

Re: PlayStation Core Full Savestate Support

Unread post by Caldor »

held wrote: Thu Dec 09, 2021 7:36 am @Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.
Why through a terminal? I download it on my computer and copy it to my MiSTer through FTP.
held
Posts: 209
Joined: Sun Sep 26, 2021 2:18 pm
Has thanked: 27 times
Been thanked: 31 times

Re: PlayStation Core Full Savestate Support

Unread post by held »

Neocaron wrote: Thu Dec 09, 2021 10:01 am I think you overthink it. Up until this point this was not necessary for any core, and it never stopped the bugs to be squish. So don't worry too much about it. Once the core is almost done, people will leave feedback the way they always do and the core will be nice and shinny. The thought was still good, but it's just not applicable for the Mister project in general. Cheers!
I did not mind the idea being turned down, only the sarcasm that came with it.
RealLarry wrote: Thu Dec 09, 2021 10:19 am .. and sorry to @held if I've pissed you off - but there were these [sarcastic] tags for that, meaning Don't really take my words seriously.
I'm co-authoring and co-developing hard- and software for the MiSTer project and we love and need feedback from users, but there are "the usual channels" like this forum and github repos for that, therefore no need for Extrawurst :)
Accepted

I figured an external call from a core to a local git client would be a practical solution.
Preventing all the manual labor and errors that come with that.

Moving on :D
held
Posts: 209
Joined: Sun Sep 26, 2021 2:18 pm
Has thanked: 27 times
Been thanked: 31 times

Re: PlayStation Core Full Savestate Support

Unread post by held »

Caldor wrote: Thu Dec 09, 2021 10:51 pm
held wrote: Thu Dec 09, 2021 7:36 am @Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.
Why through a terminal? I download it on my computer and copy it to my MiSTer through FTP.
That's fine too. I download it directly to the place where it needs to be by using the terminal. If there was a link to the latest version, it could be added to the user script. There is hardly a point in downloading it every day manually (although I really appreciate that its pre-compiled). I just want to turn on the MiSTer and it grabbing the latest test core, so I don't have to.
User avatar
Caldor
Top Contributor
Posts: 930
Joined: Sat Jul 25, 2020 11:20 am
Has thanked: 112 times
Been thanked: 111 times

Re: PlayStation Core Full Savestate Support

Unread post by Caldor »

held wrote: Sat Dec 11, 2021 12:39 pm
Caldor wrote: Thu Dec 09, 2021 10:51 pm
held wrote: Thu Dec 09, 2021 7:36 am @Caldor:
I am familiar with the update_all script, but the daily's aren't included. And to download it every day through a terminal, I though to be kind of cumbersome. So I thought: automation. Oh well.
Why through a terminal? I download it on my computer and copy it to my MiSTer through FTP.
That's fine too. I download it directly to the place where it needs to be by using the terminal. If there was a link to the latest version, it could be added to the user script. There is hardly a point in downloading it every day manually (although I really appreciate that its pre-compiled). I just want to turn on the MiSTer and it grabbing the latest test core, so I don't have to.
I think you might be able to use this to download updates by script, but you need to make the script yourself I guess:
https://github.com/MiSTer-unstable-nigh ... older.json

Its a json file with a list of all nightly releases and Peips MiSTer PS1 core was added to this afaik.
image_2021-12-11_155518.png
image_2021-12-11_155518.png (24.83 KiB) Viewed 16319 times
Post Reply