diff options
author | Joost Peters | 2009-09-23 16:11:23 +0000 |
---|---|---|
committer | Joost Peters | 2009-09-23 16:11:23 +0000 |
commit | 6a5bd81599fbafe8edc92add8316eba74504e49d (patch) | |
tree | 57de3e785bdbe0dfa83ab38131f546a3a2f2a89d /backends/platform/psp | |
parent | 227e4e65df7093b2be9b45106b529761a13068dd (diff) | |
download | scummvm-rg350-6a5bd81599fbafe8edc92add8316eba74504e49d.tar.gz scummvm-rg350-6a5bd81599fbafe8edc92add8316eba74504e49d.tar.bz2 scummvm-rg350-6a5bd81599fbafe8edc92add8316eba74504e49d.zip |
PSP: improved suspend/resume support [patch from bluddy]
svn-id: r44276
Diffstat (limited to 'backends/platform/psp')
-rw-r--r-- | backends/platform/psp/osys_psp.cpp | 8 | ||||
-rw-r--r-- | backends/platform/psp/powerman.cpp | 242 | ||||
-rw-r--r-- | backends/platform/psp/powerman.h | 82 | ||||
-rw-r--r-- | backends/platform/psp/trace.cpp | 10 | ||||
-rw-r--r-- | backends/platform/psp/trace.h | 13 |
5 files changed, 290 insertions, 65 deletions
diff --git a/backends/platform/psp/osys_psp.cpp b/backends/platform/psp/osys_psp.cpp index 78081db384..79804b3e18 100644 --- a/backends/platform/psp/osys_psp.cpp +++ b/backends/platform/psp/osys_psp.cpp @@ -35,6 +35,7 @@ #include "osys_psp.h" #include "trace.h" +#include "powerman.h" #include "backends/saves/psp/psp-saves.h" #include "backends/timer/default/default-timer.h" @@ -927,6 +928,13 @@ bool OSystem_PSP::processInput(Common::Event &event) { bool OSystem_PSP::pollEvent(Common::Event &event) { float nub_angle = -1; int x, y; + + // If we're polling for events, we should check for pausing the engine + // Pausing the engine is a necessary fix for games that use the timer for music synchronization + // recovering many hours later causes the game to crash. We're polling without mutexes since it's not critical to + // get it right now. + + PowerMan.pollPauseEngine(); sceCtrlSetSamplingCycle(0); sceCtrlSetSamplingMode(1); diff --git a/backends/platform/psp/powerman.cpp b/backends/platform/psp/powerman.cpp index c553669fc3..678f3697fb 100644 --- a/backends/platform/psp/powerman.cpp +++ b/backends/platform/psp/powerman.cpp @@ -23,17 +23,33 @@ * */ +#include <psppower.h> +#include <pspthreadman.h> + #include "./powerman.h" #include "./trace.h" +#include "engine.h" DECLARE_SINGLETON(PowerManager); +#ifdef __PSP_DEBUG_SUSPEND__ +void PowerManager::debugPM() { + PSPDebugTrace("PM status is %d. Listcount is %d. CriticalCount is %d. ThreadId is %x. Error = %d\n", _PMStatus, _listCounter, + _criticalCounter, sceKernelGetThreadId(), _error); +} +#else + #define debugPM() + #define PMStatusSet(x) +#endif /* __PSP_DEBUG_SUSPEND__ */ + + /******************************************* * * Constructor * ********************************************/ PowerManager::PowerManager() { + _flagMutex = NULL; /* Init mutex handle */ _listMutex = NULL; /* Init mutex handle */ _condSuspendable = NULL; /* Init condition variable */ @@ -41,26 +57,33 @@ PowerManager::PowerManager() { _condSuspendable = SDL_CreateCond(); if (_condSuspendable <= 0) { - PSPDebugTrace("PowerManager::PowerManager(): Couldn't create condSuspendable\n"); + PSPDebugSuspend("PowerManager::PowerManager(): Couldn't create condSuspendable\n"); } _condPM = SDL_CreateCond(); if (_condPM <= 0) { - PSPDebugTrace("PowerManager::PowerManager(): Couldn't create condPM\n"); + PSPDebugSuspend("PowerManager::PowerManager(): Couldn't create condPM\n"); } _flagMutex = SDL_CreateMutex(); if (_flagMutex <= 0) { - PSPDebugTrace("PowerManager::PowerManager(): Couldn't create flagMutex\n"); + PSPDebugSuspend("PowerManager::PowerManager(): Couldn't create flagMutex\n"); } _listMutex = SDL_CreateMutex(); if (_listMutex <= 0) { - PSPDebugTrace("PowerManager::PowerManager(): Couldn't create listMutex\n"); + PSPDebugSuspend("PowerManager::PowerManager(): Couldn't create listMutex\n"); } _suspendFlag = false; _criticalCounter = 0; + _pauseFlag = 0; _pauseFlagOld = 0; _pauseClientState = 0; + +#ifdef __PSP_DEBUG_SUSPEND__ + _listCounter = 0; + PMStatusSet(kInitDone); + _error = 0; +#endif } /******************************************* @@ -70,20 +93,25 @@ PowerManager::PowerManager() { ********************************************/ int PowerManager::registerSuspend(Suspendable *item) { // Register in list - PSPDebugTrace("In registerSuspend\n"); + PSPDebugSuspend("In registerSuspend\n"); + debugPM(); if (SDL_mutexP(_listMutex) != 0) { PSPDebugTrace("PowerManager::registerSuspend(): Couldn't lock _listMutex %d\n", _listMutex); } _suspendList.push_front(item); +#ifdef __PSP_DEBUG_SUSPEND__ + _listCounter++; +#endif if (SDL_mutexV(_listMutex) != 0) { PSPDebugTrace("PowerManager::registerSuspend(): Couldn't unlock _listMutex %d\n", _listMutex); } - PSPDebugTrace("Out of registerSuspend\n"); - + PSPDebugSuspend("Out of registerSuspend\n"); + debugPM(); + return 0; } @@ -94,7 +122,8 @@ int PowerManager::registerSuspend(Suspendable *item) { ********************************************/ int PowerManager::unregisterSuspend(Suspendable *item) { - PSPDebugTrace("In unregisterSuspend\n"); + PSPDebugSuspend("In unregisterSuspend\n"); + debugPM(); // Unregister from stream list if (SDL_mutexP(_listMutex) != 0) { @@ -102,13 +131,17 @@ int PowerManager::unregisterSuspend(Suspendable *item) { } _suspendList.remove(item); - +#ifdef __PSP_DEBUG_SUSPEND__ + _listCounter--; +#endif + if (SDL_mutexV(_listMutex) != 0) { PSPDebugTrace("PowerManager::unregisterSuspend(): Couldn't unlock _listMutex %d\n", _listMutex); } - PSPDebugTrace("Out of unregisterSuspend\n"); - + PSPDebugSuspend("Out of unregisterSuspend\n"); + debugPM(); + return 0; } @@ -118,6 +151,11 @@ int PowerManager::unregisterSuspend(Suspendable *item) { * ********************************************/ PowerManager::~PowerManager() { + +#ifdef __PSP_DEBUG_SUSPEND__ + PMStatusSet(kDestroyPM); +#endif + SDL_DestroyCond(_condSuspendable); _condSuspendable = 0; @@ -130,48 +168,84 @@ int PowerManager::unregisterSuspend(Suspendable *item) { SDL_DestroyMutex(_listMutex); _listMutex = 0; } + +/******************************************* +* +* Unsafe function to poll for a pause event (first stage of suspending) +* Only for pausing the engine, which doesn't need high synchronization ie. we don't care if it misreads +* the flag a couple of times since there is NO mutex protection (for performance reasons). +* Polling the engine happens regularly. +* On the other hand, we don't know if there will be ANY polling which prevents us from using proper events. +* +********************************************/ +void PowerManager::pollPauseEngine() { + + bool pause = _pauseFlag; // We copy so as not to have multiple values + + if ((pause != _pauseFlagOld) && g_engine) { // Check to see if we have an engine + if (pause && _pauseClientState == PowerManager::Unpaused) { + _pauseClientState = PowerManager::Pausing; // Tell PM we're in the middle of pausing + g_engine->pauseEngine(true); + PSPDebugSuspend("Pausing engine in PowerManager::pollPauseEngine()\n"); + _pauseClientState = PowerManager::Paused; // Tell PM we're done pausing + } + else if (!pause && _pauseClientState == PowerManager::Paused) { + g_engine->pauseEngine(false); + PSPDebugSuspend("Unpausing for resume in PowerManager::pollPauseEngine()\n"); + _pauseClientState = PowerManager::Unpaused; // Tell PM we're in the middle of pausing + } + + _pauseFlagOld = pause; + } +} - - /******************************************* +/******************************************* * * Function to be called by threads wanting to block on the PSP entering suspend +* Use this for small critical sections where you can easily restore the previous state. * ********************************************/ int PowerManager::blockOnSuspend() { return beginCriticalSection(true); } - /* - * Function to block on a suspend, then start a non-suspendable critical section - */ -int PowerManager::beginCriticalSection(bool justBlock) { - int ret = PowerManager::NotBlocked; +/******************************************* +* +* Function to block on a suspend, then start a non-suspendable critical section +* Use this for large or REALLY critical critical-sections. +* Make sure to call endCriticalSection or the PSP won't suspend. +********************************************/ + + int PowerManager::beginCriticalSection(bool justBlock) { + int ret = NotBlocked; if (SDL_mutexP(_flagMutex) != 0) { PSPDebugTrace("PowerManager::blockOnSuspend(): Couldn't lock flagMutex %d\n", _flagMutex); - ret = PowerManager::Error; + ret = Error; } // Check the access flag if (_suspendFlag == true) { - PSPDebugTrace("Blocking!!\n"); - ret = PowerManager::Blocked; + PSPDebugSuspend("We're being blocked!\n"); + debugPM(); + ret = Blocked; // If it's true, we wait for a signal to continue - if( SDL_CondWait(_condSuspendable, _flagMutex) != 0) { + if (SDL_CondWait(_condSuspendable, _flagMutex) != 0) { PSPDebugTrace("PowerManager::blockOnSuspend(): Couldn't wait on cond %d\n", _condSuspendable); } - PSPDebugTrace("We got blocked!!\n"); + PSPDebugSuspend("We got blocked!!\n"); + debugPM(); } - // Now put the pm to sleep + // Now prevent the PM from suspending until we're done if (justBlock == false) _criticalCounter++; if (SDL_mutexV(_flagMutex) != 0) { PSPDebugTrace("PowerManager::blockOnSuspend(): Couldn't unlock flagMutex %d\n", _flagMutex); - ret = PowerManager::Error; + ret = Error; } return ret; @@ -182,25 +256,32 @@ int PowerManager::endCriticalSection() { if (SDL_mutexP(_flagMutex) != 0) { PSPDebugTrace("PowerManager::endCriticalSection(): Couldn't lock flagMutex %d\n", _flagMutex); - ret = PowerManager::Error; + ret = Error; } // We're done with our critical section _criticalCounter--; if (_criticalCounter <= 0) { - if(_suspendFlag == true) PSPDebugTrace("Waking up the PM and suspendFlag is true\n"); - - SDL_CondBroadcast(_condPM); + if (_suspendFlag == true) { // If the PM is sleeping, this flag must be set + PSPDebugSuspend("Unblocked thread waking up the PM.\n"); + debugPM(); + + SDL_CondBroadcast(_condPM); + + PSPDebugSuspend("Woke up the PM\n"); + debugPM(); + } - if (_criticalCounter < 0) { + if (_criticalCounter < 0) { // Check for bad usage of critical sections PSPDebugTrace("PowerManager::endCriticalSection(): Error! Critical counter is %d\n", _criticalCounter); + debugPM(); } } if (SDL_mutexV(_flagMutex) != 0) { PSPDebugTrace("PowerManager::endCriticalSection(): Couldn't unlock flagMutex %d\n", _flagMutex); - ret = PowerManager::Error; + ret = Error; } return ret; @@ -213,40 +294,86 @@ int PowerManager::endCriticalSection() { ********************************************/ int PowerManager::suspend() { int ret = 0; + + if (_pauseFlag) return ret; // Very important - make sure we only suspend once - // First we set the suspend flag to true + scePowerLock(0); // Critical to make sure PSP doesn't suspend before we're done + + // The first stage of suspend is pausing the engine if possible. We don't want to cause files + // to block, or we might not get the engine to pause. On the other hand, we might wait for polling + // and it'll never happen. We also want to do this w/o mutexes (for speed) which is ok in this case. + _pauseFlag = true; + + PMStatusSet(kWaitForClientPause); + + // Now we wait, giving the engine thread some time to find our flag. + for (int i = 0; i < 10 && _pauseClientState == Unpaused; i++) + sceKernelDelayThread(50000); // We wait 50 msec x 10 times = 0.5 seconds + + if (_pauseClientState == Pausing) { // Our event has been acknowledged. Let's wait until the client is done. + PMStatusSet(kWaitForClientToFinishPausing); + + while (_pauseClientState != Paused) + sceKernelDelayThread(50000); // We wait 50 msec at a time + } + + // It's possible that the polling thread missed our pause event, but there's nothing we can do about that. + // We can't know if there's polling going on or not. It's usually not a critical thing anyway. + + PMStatusSet(kGettingFlagMutexSuspend); + + // Now we set the suspend flag to true to cause reading threads to block + if (SDL_mutexP(_flagMutex) != 0) { PSPDebugTrace("PowerManager::suspend(): Couldn't lock flagMutex %d\n", _flagMutex); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kGotFlagMutexSuspend); + _suspendFlag = true; - if (_criticalCounter > 0) + // Check if anyone is in a critical section. If so, we'll wait for them + if (_criticalCounter > 0) { + PMStatusSet(kWaitCritSectionSuspend); SDL_CondWait(_condPM, _flagMutex); + PMStatusSet(kDoneWaitingCritSectionSuspend); + } if (SDL_mutexV(_flagMutex) != 0) { PSPDebugTrace("PowerManager::suspend(): Couldn't unlock flagMutex %d\n", _flagMutex); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kGettingListMutexSuspend); + // Loop over list, calling suspend() if (SDL_mutexP(_listMutex) != 0) { PSPDebugTrace("PowerManager::suspend(): Couldn't lock listMutex %d\n", _listMutex); - ret = -1; + _error = Error; + ret = Error; } - + PMStatusSet(kIteratingListSuspend); + // Iterate Common::List<Suspendable *>::iterator i = _suspendList.begin(); for (; i != _suspendList.end(); i++) { (*i)->suspend(); } + + PMStatusSet(kDoneIteratingListSuspend); if (SDL_mutexV(_listMutex) != 0) { PSPDebugTrace("PowerManager::suspend(): Couldn't unlock listMutex %d\n", _listMutex); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kDoneSuspend); + scePowerUnlock(0); // Allow the PSP to go to sleep now + return ret; } @@ -258,40 +385,67 @@ int PowerManager::suspend() { int PowerManager::resume() { int ret = 0; + // Make sure we can't get another suspend + scePowerLock(0); + + if (!_pauseFlag) return ret; // Make sure we can only resume once + + PMStatusSet(kGettingListMutexResume); + // First we notify our Suspendables. Loop over list, calling resume() if (SDL_mutexP(_listMutex) != 0) { PSPDebugTrace("PowerManager::resume(): Couldn't lock listMutex %d\n", _listMutex); - ret = -1; + _error = Error; + ret = Error; } - + PMStatusSet(kIteratingListResume); + // Iterate Common::List<Suspendable *>::iterator i = _suspendList.begin(); for (; i != _suspendList.end(); i++) { (*i)->resume(); } + + PMStatusSet(kDoneIteratingListResume); if (SDL_mutexV(_listMutex) != 0) { PSPDebugTrace("PowerManager::resume(): Couldn't unlock listMutex %d\n", _listMutex); - ret = -1; + _error = Error; + ret = Error; } + + PMStatusSet(kGettingFlagMutexResume); // Now we set the suspend flag to false if (SDL_mutexP(_flagMutex) != 0) { PSPDebugTrace("PowerManager::resume(): Couldn't lock flagMutex %d\n", _flagMutex); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kGotFlagMutexResume); + _suspendFlag = false; + PMStatusSet(kSignalSuspendedThreadsResume); + // Signal the other threads to wake up if (SDL_CondBroadcast(_condSuspendable) != 0) { PSPDebugTrace("PowerManager::resume(): Couldn't broadcast condition %d\n", _condSuspendable); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kDoneSignallingSuspendedThreadsResume); if (SDL_mutexV(_flagMutex) != 0) { PSPDebugTrace("PowerManager::resume(): Couldn't unlock flagMutex %d\n", _flagMutex); - ret = -1; + _error = Error; + ret = Error; } + PMStatusSet(kDoneResume); + + _pauseFlag = false; // Signal engine to unpause + scePowerUnlock(0); // Allow new suspends + return ret; } diff --git a/backends/platform/psp/powerman.h b/backends/platform/psp/powerman.h index 0a5f7a2361..43fb55459b 100644 --- a/backends/platform/psp/powerman.h +++ b/backends/platform/psp/powerman.h @@ -50,19 +50,6 @@ * *******************************************************************************************************/ class PowerManager: public Common::Singleton<PowerManager> { -private: - friend class Common::Singleton<PowerManager>; - PowerManager(); - ~PowerManager(); - - Common::List<Suspendable *> _suspendList; /* list to register in */ - - bool _suspendFlag; /* protected variable */ - SDL_mutex *_flagMutex; /* mutex to access access flag */ - SDL_mutex *_listMutex; /* mutex to access Suspendable list */ - SDL_cond *_condSuspendable; /* signal to synchronize accessing threads */ - SDL_cond *_condPM; /* signal to wake up the PM from a critical section */ - int _criticalCounter; /* Counter of how many threads are in a critical section */ public: int blockOnSuspend(); /* block if suspending */ @@ -72,13 +59,78 @@ public: int unregisterSuspend(Suspendable *item); /* remove from suspend/resume list */ int suspend(); /* callback to have all items in list suspend */ int resume(); /* callback to have all items in list resume */ + // Functions for pausing the engine + void pollPauseEngine(); /* Poll whether the engine should be paused */ enum { Error = -1, NotBlocked = 0, - Blocked = 1 + Blocked = 1 + }; + + enum PauseState { + Unpaused = 0, + PauseEvent, + UnpauseEvent, + Pausing, + Paused }; - + + private: + friend class Common::Singleton<PowerManager>; + PowerManager(); + ~PowerManager(); + + Common::List<Suspendable *> _suspendList; /* list to register in */ + + volatile bool _pauseFlag; /* For pausing, which is before suspending */ + volatile bool _pauseFlagOld; /* Save the last state of the flag while polling */ + volatile int _pauseClientState; /* Pause state of the target */ + + volatile bool _suspendFlag; /* protected variable */ + SDL_mutex *_flagMutex; /* mutex to access access flag */ + SDL_mutex *_listMutex; /* mutex to access Suspendable list */ + SDL_cond *_condSuspendable; /* signal to synchronize accessing threads */ + SDL_cond *_condPM; /* signal to wake up the PM from a critical section */ + volatile int _criticalCounter; /* Counter of how many threads are in a critical section */ + int _error; /* error code - PM can't talk to us. For debugging */ + + // States for PM to be in (used for debugging) + enum PMState { + kInitDone = 1 , + kDestroyPM, + kWaitForClientPause, + kWaitForClientToFinishPausing, + kGettingFlagMutexSuspend, + kGotFlagMutexSuspend, + kWaitCritSectionSuspend, + kDoneWaitingCritSectionSuspend, + kGettingListMutexSuspend, + kIteratingListSuspend, + kDoneIteratingListSuspend, + kDoneSuspend, + kGettingListMutexResume, + kIteratingListResume, + kDoneIteratingListResume, + kGettingFlagMutexResume, + kGotFlagMutexResume, + kSignalSuspendedThreadsResume, + kDoneSignallingSuspendedThreadsResume, + kDoneResume + }; +#ifdef __PSP_DEBUG_SUSPEND__ + + volatile int _listCounter; /* How many people are in the list - just for debugging */ + + void debugPM(); /* print info about the PM */ + void PMStatusSet(PMState s) { _PMStatus = s; } + volatile int _PMStatus; /* What the PM is doing */ + + public: + int getPMStatus() { return _PMStatus; } + +#endif /* __PSP_DEBUG_SUSPEND__ */ + }; // For easy access diff --git a/backends/platform/psp/trace.cpp b/backends/platform/psp/trace.cpp index 00d19b582c..752c0b8306 100644 --- a/backends/platform/psp/trace.cpp +++ b/backends/platform/psp/trace.cpp @@ -27,10 +27,8 @@ #include "./trace.h" -//#define __DEBUG__ - void PSPDebugTrace (const char *format, ...) { -#ifdef __DEBUG__ +#ifdef __PSP_DEBUG__ va_list opt; char buff[2048]; int bufsz, fd; @@ -46,11 +44,11 @@ void PSPDebugTrace (const char *format, ...) { sceIoWrite(fd, (const void*)buff, bufsz); sceIoClose(fd); -#endif +#endif /* __PSP_DEBUG__ */ } void PSPDebugTrace (const char * filename, const char *format, ...) { -#ifdef __DEBUG__ +#ifdef __PSP_DEBUG__ va_list opt; char buff[2048]; int bufsz, fd; @@ -66,5 +64,5 @@ void PSPDebugTrace (const char * filename, const char *format, ...) { sceIoWrite(fd, (const void*)buff, bufsz); sceIoClose(fd); -#endif +#endif /* __PSP_DEBUG__ */ } diff --git a/backends/platform/psp/trace.h b/backends/platform/psp/trace.h index 80afdbc4fe..ef8992ceb2 100644 --- a/backends/platform/psp/trace.h +++ b/backends/platform/psp/trace.h @@ -33,8 +33,21 @@ #include <stdarg.h> #include <pspdebug.h> +// Use these defines for debugging + +//#define __PSP_DEBUG__ +//#define __PSP_DEBUG_SUSPEND__ + void PSPDebugTrace (const char *filename, const char *format, ...); void PSPDebugTrace (const char *format, ...); +#ifdef __PSP_DEBUG_SUSPEND__ +#define PSPDebugSuspend(format,...) PSPDebugTrace(format, ## __VA_ARGS__) +#else +#define PSPDegbugSuspend(x) +#define PSPDebugSuspend(format,...) +#endif /* __PSP_DEBUG_SUSPEND__ */ + + #endif // TRACE_H |