Powerblock proper ES shutdown

lostless

@capeman Do a test for us all. Turn off auto save. Make a save in game. Then hit the a switch and see if it saves.

lostless

@capeman find a game that makes it obvious. Like one of the gba castlevania games. Save, kill one enemy. Check the xp before and after. Save and see what the xp is after the shutdown.

Capeman

@lostless Right, that was my plan. I'll test it after my wife watches her netflix, and report back.

lostless

@capeman i beat you to it. It does save! I will still keep auto on just in case of power outage. But it does solve then default delema of losing my save using a switch because I didn't to exit out of the emulator.

Capeman

You indeed beat me to it, but i just got done finishing testing. Appears to be working for me too. I read a bit more into it, apparently KILL when used with a properly running non-frozen application can be ignored by the application and treated as a termination call. Most applications will clean exit on a KILL command and not hard exit when not frozen. So yeah, this seem to work just fine.

cyperghost

@capeman It wasn't possible withouth the great work @meleu and his RegEx hacks.

petrockblog

You are doing awesome work here!
Do you think that the PowerBlock driver in the repository should be modified or extended in any way?

Capeman

@petrockblog I would vote yes on this! It works flawlessly with ES and retroarch. It is such a simple update to the .sh file and even if doesn't clean exit a non retroarch emulator, it will still exit the old fashioned way and clean exit ES for meta save. So either way it seems pretty harmless.

cyperghost

@petrockblog said in Powerblock proper ES shutdown:

You are doing awesome work here!

This one has several fathers.

Do you think that the PowerBlock driver in the repository should be modified or extended in any way?

Well ... I think the usecase must be cleared. I'm not sure if everybody will take case in the use of emulators. So I would rather prefer you take this sniplet and push it to your github and let the user get the choice to download/use it during driver installation.

@meleu What is your opinion?

meleu

@cyperghost

@meleu What is your opinion?

Well. I got my ego massage receiving a compliment from the RetroPie's creator. :-)

Oh... You're talking about modifying/extending the PowerBlock's driver in the repo... Well, what we made here is very specific for a RetroPie-EmulationStation setup. Some people use Attract Mode and at some point others will use Pegasus, and maybe there are PowerBlock owners that don't even run RetroPie.

Maybe you can put this code in the repo as an option, not as default.

That is my opinion.

petrockblog

@meleu I agree. I will put that script into the repository as alternative shutdown script. Thanks!

cyperghost

@petrockblog
There's a update to the code!

Version 1.59 --- I update forst posting!

Version 1.58 Introduced a method to kill child processes via PPID @cyperghost
Version 1.59 is a code polishing done by @meleu

SaturnX

@cyperghost @petrockblog

I know this topic a bit old - but I wanted to ask if this is still the best approach for a proper shutdown of a RetroPie/EmulationStation/PowerBlock combo?

@petrockblog you mentioned above that you were going to add this into your repo, but it looks like it's not there?

I just installed the PowerBlock on my Pi4 last night and it's working well, but I just wanted to add the proper shutdown of ES.

Thanks! Apologies for reviving an old thread.

lostless

@saturnx
@cyperghost has released a full power, exit script that works for many power options a while ago. It’s generic and just have to run certain tags from it to get the result you want.
https://github.com/crcerror/ES-generic-shutdown

SaturnX

@lostless Thanks!

I've seen that, but when using the PowerBlock, that script would be overkill since the PowerBlock driver already handles monitoring of the status of the switch and simply calls etc/powerblockswitchoff.sh

So really all that's necessary is ensuring the proper shutdown calls are made within that script to ensure the safe shutdown of ES, etc..

I'll dig through the generic script and compare - just wanted to check in to see there had been any updates/improvements or other best practices for shutting down ES gracefully.

lostless

@saturnx all you really have to do is, I think, is point to a Different file in that call. No need to edit the powerblockswitchoff.sh file.
Change the main script to be the SHUTDOWNSCRIPT= to
/home/pi/RetroPie/scripts/multi_switch.sh --es-poweroff
After you install the generic multi switch script. You don’t have to start it as boot up. Just install it where it says to install according to the get hub page and you can use it. No need to use the the scripts ability to sense a switch
Test it out and see if it works.

SaturnX

@lostless thanks!

Took a slightly different approach and called the multi-switch shutdown script from etc/powerblockswitchoff.sh So that I’d keep the main driver intact.

Thanks again! This was something so simple that I overlooked. Totally missed that the script could issue the necessary ES shutdown without tying it to a physical switch.