summaryrefslogtreecommitdiffstats
path: root/src/core/hle/kernel/mutex.cpp (follow)
Commit message (Collapse)AuthorAgeFilesLines
* kernel: Remove unnecessary includesLioncash2019-12-081-0/+1
| | | | | | Over the course of the changes to the kernel code, a few includes are no longer necessary, particularly with the change over to std::shared_ptr from Boost's intrusive_ptr.
* core/memory: Migrate over Write{8, 16, 32, 64, Block} to the Memory classLioncash2019-11-271-2/+2
| | | | | | | | | The Write functions are used slightly less than the Read functions, which make these a bit nicer to move over. The only adjustments we really need to make here are to Dynarmic's exclusive monitor instance. We need to keep a reference to the currently active memory instance to perform exclusive read/write operations.
* core/memory: Migrate over Read{8, 16, 32, 64, Block} to the Memory classLioncash2019-11-271-1/+1
| | | | | | | | | | | | | | With all of the trivial parts of the memory interface moved over, we can get right into moving over the bits that are used. Note that this does require the use of GetInstance from the global system instance to be used within hle_ipc.cpp and the gdbstub. This is fine for the time being, as they both already rely on the global system instance in other functions. These will be removed in a change directed at both of these respectively. For now, it's sufficient, as it still accomplishes the goal of de-globalizing the memory code.
* kernel: Replace usage of boost::intrusive_ptr with std::shared_ptr for kernel objects. (#3154)bunnei2019-11-251-10/+12
| | | | | | * kernel: Replace usage of boost::intrusive_ptr with std::shared_ptr for kernel objects. - See https://github.com/citra-emu/citra/pull/4710 for details.
* Kernel: Correct Results in Condition Variables and MutexesFernando Sahmkow2019-10-151-0/+1
|
* Kernel: Style and CorrectionsFernando Sahmkow2019-10-151-1/+1
|
* Add PrepareReschedule where required.Fernando Sahmkow2019-10-151-0/+2
|
* core/hle/kernel/mutex: Remove usages of global system accessorsLioncash2019-03-151-11/+15
| | | | | Removes the use of global system accessors, and instead uses the explicit interface provided.
* core/hle/kernel: Make Mutex a per-process class.Lioncash2019-03-151-2/+7
| | | | | | | Makes it an instantiable class like it is in the actual kernel. This will also allow removing reliance on global accessors in a following change, now that we can encapsulate a reference to the system instance in the class.
* general: Remove unused boost inclusions where applicableLioncash2018-10-301-2/+0
| | | | Cleans up unused includes and trims off some dependencies on externals.
* kernel/mutex: Amend behavior of TransferMutexOwnership()Lioncash2018-10-061-1/+1
| | | | | | | | | This was the result of a typo accidentally introduced in e51d715700a35a8f14e5b804b6f7553c9a40888b. This restores the previous correct behavior. The behavior with the reference was incorrect and would cause some games to fail to boot.
* kernel/thread: Make all instance variables privateLioncash2018-10-041-17/+17
| | | | | | | | | | | | | | | | | | | | Many of the member variables of the thread class aren't even used outside of the class itself, so there's no need to make those variables public. This change follows in the steps of the previous changes that made other kernel types' members private. The main motivation behind this is that the Thread class will likely change in the future as emulation becomes more accurate, and letting random bits of the emulator access data members of the Thread class directly makes it a pain to shuffle around and/or modify internals. Having all data members public like this also makes it difficult to reason about certain bits of behavior without first verifying what parts of the core actually use them. Everything being public also generally follows the tendency for changes to be introduced in completely different translation units that would otherwise be better introduced as an addition to the Thread class' public interface.
* arm_interface: Replace kernel vm_manager include with a forward declarationLioncash2018-09-211-0/+1
| | | | | | Avoids an unnecessary inclusion and also uncovers three places where indirect inclusions were relied upon, which allows us to also resolve those.
* kernel/mutex: Replace ResultCode construction for invalid addresses with the named variantLioncash2018-09-181-2/+2
| | | | | | We already have a ResultCode constant for the case of an invalid address, so we can just use it instead of re-rolling that ResultCode type.
* kernel: Eliminate kernel global stateLioncash2018-08-291-3/+3
| | | | | | | | | | | | | | | | | | | | | | As means to pave the way for getting rid of global state within core, This eliminates kernel global state by removing all globals. Instead this introduces a KernelCore class which acts as a kernel instance. This instance lives in the System class, which keeps its lifetime contained to the lifetime of the System class. This also forces the kernel types to actually interact with the main kernel instance itself instead of having transient kernel state placed all over several translation units, keeping everything together. It also has a nice consequence of making dependencies much more explicit. This also makes our initialization a tad bit more correct. Previously we were creating a kernel process before the actual kernel was initialized, which doesn't really make much sense. The KernelCore class itself follows the PImpl idiom, which allows keeping all the implementation details sealed away from everything else, which forces the use of the exposed API and allows us to avoid any unnecessary inclusions within the main kernel header.
* kernel: Move object class to its own source filesLioncash2018-08-021-1/+1
| | | | | | General moving to keep kernel object types separate from the direct kernel code. Also essentially a preliminary cleanup before eliminating global kernel state in the kernel code.
* Merge pull request #877 from lioncash/removebunnei2018-08-011-1/+0
|\ | | | | kernel: Remove unused object_address_table.cpp/.h
| * kernel: Remove unused object_address_table.cpp/.hLioncash2018-07-311-1/+0
| | | | | | | | | | | | These source files were entirely unused throughout the rest of the codebase. This also has the benefit of getting rid of a global variable as well.
* | kernel: Remove unnecessary includesLioncash2018-07-311-0/+4
|/ | | | | Removes unnecessary direct dependencies in some headers and also gets rid of indirect dependencies that were being relied on to be included.
* mutex: Pass SharedPtr to GetHighestPriorityMutexWaitingThread() by referenceLioncash2018-07-241-1/+1
| | | | | The pointed to thread's members are simply observed in this case, so we don't need to copy it here.
* thread: Convert ThreadStatus into an enum classLioncash2018-07-201-3/+3
| | | | | Makes the thread status strongly typed, so implicit conversions can't happen. It also makes it easier to catch mistakes at compile time.
* Kernel/Arbiters: Add stubs for 4.x SignalToAddress/WaitForAddres SVCs.Michael Scire2018-06-211-2/+2
|
* Mutex: Do not assert when the mutex waiting threads list isn't empty on mutex release.Subv2018-05-201-1/+0
| | | | A thread may own multiple mutexes at the same time, and only release one of them while other threads are waiting for the other mutexes.
* Kernel: Implemented mutex priority inheritance.Subv2018-04-231-8/+31
| | | | | | | Verified with a hwtest and implemented based on reverse engineering. Thread A's priority will get bumped to the highest priority among all the threads that are waiting for a mutex that A holds. Once A releases the mutex and ownership is transferred to B, A's priority will return to normal and B's priority will be bumped.
* Kernel: Remove old and unused Mutex code.Subv2018-04-211-120/+0
|
* Kernel: Corrected the implementation of svcArbitrateLock and svcArbitrateUnlock.Subv2018-04-211-0/+94
| | | | | | | | | | Switch mutexes are no longer kernel objects, they are managed in userland and only use the kernel to handle the contention case. Mutex addresses store a special flag value (0x40000000) to notify the guest code that there are still some threads waiting for the mutex to be released. This flag is updated when a thread calls ArbitrateUnlock. TODO: * Fix svcWaitProcessWideKey * Fix svcSignalProcessWideKey * Remove the Mutex class.
* mutex: Update hasWaiters on release.bunnei2018-02-061-0/+1
|
* mutex: Remove unused call to VerifyGuestState.bunnei2018-01-091-3/+0
|
* Kernel: Properly keep track of mutex lock data in the guest memory. This fixes userland locking/unlocking.Subv2018-01-091-54/+43
|
* svc: Implement svcLockMutex.bunnei2018-01-011-18/+63
|
* Merge remote-tracking branch 'upstream/master' into nxbunnei2017-10-101-2/+3
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # src/core/CMakeLists.txt # src/core/arm/dynarmic/arm_dynarmic.cpp # src/core/arm/dyncom/arm_dyncom.cpp # src/core/hle/kernel/process.cpp # src/core/hle/kernel/thread.cpp # src/core/hle/kernel/thread.h # src/core/hle/kernel/vm_manager.cpp # src/core/loader/3dsx.cpp # src/core/loader/elf.cpp # src/core/loader/ncch.cpp # src/core/memory.cpp # src/core/memory.h # src/core/memory_setup.h
| * Fixed type conversion ambiguityHuw Pascoe2017-09-301-1/+1
|/
* Kernel/Mutex: Propagate thread priority changes to other threads inheriting the priority via mutexesSubv2017-01-041-38/+22
|
* Kernel/Mutex: Update a mutex priority when a thread stops waiting on it.Subv2017-01-041-13/+22
|
* Kernel/Mutex: Implemented priority inheritance.Subv2017-01-041-13/+45
| | | | | | | The implementation is based on reverse engineering of the 3DS's kernel. A mutex holder's priority will be temporarily boosted to the best priority among any threads that want to acquire any of its held mutexes. When the holder releases the mutex, it's priority will be boosted to the best priority among the threads that want to acquire any of its remaining held mutexes.
* Kernel: Object ShouldWait and Acquire calls now take a thread as a parameter.Subv2017-01-041-18/+6
| | | | This will be useful when implementing mutex priority inheritance.
* Kernel/Synch: Do not attempt a reschedule on every syscall.Subv2017-01-041-0/+1
| | | | Not all syscalls should cause reschedules, this commit attempts to remedy that, however, it still does not cover all cases.
* Use negative priorities to avoid special-casing the self-includeYuri Kunde Schlesner2016-09-211-1/+1
|
* Remove empty newlines in #include blocks.Emmanuel Gil Peyrot2016-09-211-4/+1
| | | | | | | This makes clang-format useful on those. Also add a bunch of forgotten transitive includes, which otherwise prevented compilation.
* Manually tweak source formatting and then re-run clang-formatYuri Kunde Schlesner2016-09-191-4/+2
|
* Sources: Run clang-format on everything.Emmanuel Gil Peyrot2016-09-181-2/+4
|
* kernel: Fix svcWaitSynch to always acquire requested wait objects.bunnei2015-06-171-8/+1
|
* Kernel: Move reschedules from SVCs to actual mechanisms that reschedule.bunnei2015-05-211-0/+2
|
* Common: Remove common.hYuri Kunde Schlesner2015-05-071-1/+1
|
* Kernel: Implemented priority inheritance for mutexes.bunnei2015-04-101-1/+9
|
* Asserts: break/crash program, fit to style guide; log.h->assert.harchshift2015-02-111-1/+1
| | | | | | | Involves making asserts use printf instead of the log functions (log functions are asynchronous and, as such, the log won't be printed in time) As such, the log type argument was removed (printf obviously can't use it, and it's made obsolete by the file and line printing) Also removed some GEKKO cruft.
* Mutex: Locks should be recursive.bunnei2015-02-101-14/+19
|
* Kernel: Stop creating useless Handles during object creationYuri Kunde Schlesner2015-02-021-4/+2
| | | | | They're finally unnecessary, and will stop cluttering the application's handle table.
* Kernel: Make WaitObjects share ownership of Threads waiting on themYuri Kunde Schlesner2015-02-021-2/+2
| | | | | | | | | | | | | | | | During normal operation, a thread waiting on an WaitObject and the object hold mutual references to each other for the duration of the wait. If a process is forcefully terminated (The CTR kernel has a SVC to do this, TerminateProcess, though no equivalent exists for threads.) its threads would also be stopped and destroyed, leaving dangling pointers in the WaitObjects. The solution is to simply have the Thread remove itself from WaitObjects when it is stopped. The vector of Threads in WaitObject has also been changed to hold SharedPtrs, just in case. (Better to have a reference cycle than a crash.)
* Explicitly instantiate constructors/destructors for Kernel objectsYuri Kunde Schlesner2015-02-021-0/+3
| | | | | | This should speed up compile times a bit, as well as enable more liberal use of forward declarations. (Due to SharedPtr not trying to emit the destructor anymore.)
* Mutex: Replace g_mutex_held_locks with a set inside ThreadYuri Kunde Schlesner2015-02-021-21/+7
|
* Kernel: Convert Mutex to not use HandlesYuri Kunde Schlesner2015-01-301-88/+43
|
* Kernel: Remove useless/duplicated comments; mark functions staticYuri Kunde Schlesner2015-01-301-14/+4
|
* Mutex: Cleanup and remove redundant code.bunnei2015-01-221-45/+27
|
* Kernel: Renamed some functions for clarity.bunnei2015-01-221-1/+1
| | | | | - ReleaseNextThread->WakeupNextThread - ReleaseAllWaitingThreads->WakeupAllWaitingThreads.
* Kernel: Changed "ShouldWait" to return bool and "Acquire" to return void.bunnei2015-01-221-15/+9
|
* WaitObject: Renamed "Wait" to "ShouldWait", made "ShouldWait" and "Acquire" pure virtual.bunnei2015-01-221-2/+2
|
* Mutex: Fix a bug where the thread should not wait if it already has the mutex.bunnei2015-01-221-1/+4
|
* Kernel: Get rid of WaitTypes and simplify lots of code, removing hacks.bunnei2015-01-221-7/+2
|
* WaitSynchronizationN: Refactor to fix several bugsbunnei2015-01-221-4/+4
| | | | | | - Separate wait checking from waiting the current thread - Resume thread when wait_all=true only if all objects are available at once - Set output to correct wait object index when there are duplicate handles
* Kernel: Separate WaitSynchronization into Wait and Acquire methods.bunnei2015-01-221-5/+14
|
* WaitSynchronizationN: Implement return valuesbunnei2015-01-221-4/+4
|
* Kernel: Added WaitObject and changed "waitable" objects inherit from it.bunnei2015-01-221-23/+6
|
* Kernel: Start using boost::intrusive_ptr for lifetime managementYuri Kunde Schlesner2015-01-091-3/+3
|
* Thread: Reduce use of Handles and move some funcs to inside the class.Yuri Kunde Schlesner2015-01-091-6/+13
|
* Mutex: Add the calling thread to the waiting list when neededSubv2015-01-041-2/+2
| | | | This will happen when the mutex is already owned by another thread. Should fix some issues with games being stuck due to waiting threads not being awoken.
* Kernel: New handle managerYuri Kunde Schlesner2014-12-281-2/+3
| | | | | | | | | | | This handle manager more closely mirrors the behaviour of the CTR-OS one. In addition object ref-counts and support for DuplicateHandle have been added. Note that support for DuplicateHandle is still experimental, since parts of the kernel still use Handles internally, which will likely cause troubles if two different handles to the same object are used to e.g. wait on a synchronization primitive.
* Kernel: Replace GetStaticHandleType by HANDLE_TYPE constantsYuri Kunde Schlesner2014-12-281-2/+2
|
* Rename ObjectPool to HandleTableYuri Kunde Schlesner2014-12-281-3/+3
|
* License changepurpasmart962014-12-211-1/+1
|
* Mutex: Remove some forward declarationsSubv2014-12-071-16/+15
| | | | Moved Mutex::WaitSynchronization to the end of the file.
* Mutex: Release all held mutexes when a thread exits.Subv2014-12-071-22/+46
|
* Mutex: Properly lock the mutex when a thread enters itSubv2014-12-061-12/+9
| | | | Also resume only the next immediate thread waiting for the mutex when it is released, instead of resuming them all.
* Mutex: Changed behavior to always release mutex for all threads.bunnei2014-11-261-8/+7
|
* Remove duplicated docs/update them for changed parameters.Yuri Kunde Schlesner2014-11-241-10/+0
|
* HLE: Revamp error handling throrough the HLE codeYuri Kunde Schlesner2014-11-241-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | All service calls in the CTR OS return result codes indicating the success or failure of the call. Previous to this commit, Citra's HLE emulation of services and the kernel universally either ignored errors or returned dummy -1 error codes. This commit makes an initial effort to provide an infrastructure for error reporting and propagation which can be use going forward to make HLE calls accurately return errors as the original system. A few parts of the code have been updated to use the new system where applicable. One part of this effort is the definition of the `ResultCode` type, which provides facilities for constructing and parsing error codes in the structured format used by the CTR. The `ResultVal` type builds on `ResultCode` by providing a container for values returned by function that can report errors. It enforces that correct error checking will be done on function returns by preventing the use of the return value if the function returned an error code. Currently this change is mostly internal since errors are still suppressed on the ARM<->HLE border, as a temporary compatibility hack. As functionality is implemented and tested this hack can be eventually removed.
* Remove trailing spaces in every file but the ones imported from SkyEye, AOSP or generatedEmmanuel Gil Peyrot2014-11-191-2/+2
|
* Add `override` keyword through the code.Yuri Kunde Schlesner2014-10-261-5/+5
| | | | This was automated using `clang-modernize`.
* Core: Alter the kernel string functions to use std::string instead of const char*.Lioncash2014-08-181-2/+2
| | | | Most functions already operate on std::strings. This also removes the need to manually null terminate thread names.
* Kernel: Updated Event and Mutex to specify handle that they are blocking for.bunnei2014-08-061-1/+1
|
* HLE: Updated all uses of NULL to nullptr (to be C++11 compliant)bunnei2014-06-131-1/+1
|
* Kernel: Updated various kernel function "name" arguments to be const references.bunnei2014-06-131-2/+2
|
* HLE: Updated various handle debug assertions to be more clear.bunnei2014-06-131-1/+1
|
* Mutex: Moved ReleaseMutex iterator declaration to be inside while loop.bunnei2014-06-131-2/+1
|
* Kernel: Updated several member functions to be constbunnei2014-06-131-3/+3
|
* Kernel: Added real support for thread and event blockingbunnei2014-06-131-0/+5
| | | | | | | | | | | | | - SVC: Added ExitThread support - SVC: Added SignalEvent support - Thread: Added WAITTYPE_EVENT for waiting threads for event signals - Thread: Added support for blocking on other threads to finish (e.g. Thread::Join) - Thread: Added debug function for printing current threads ready for execution - Thread: Removed hack/broken thread ready state code from Kernel::Reschedule - Mutex: Moved WaitCurrentThread from SVC to Mutex::WaitSynchronization - Event: Added support for blocking threads on event signalling Kernel: Added missing algorithm #include for use of std::find on non-Windows platforms.
* svc: added optional name field to Event and Mutex (used for debugging)bunnei2014-06-031-3/+9
|
* mutex: fixed typo in ReleaseMutexbunnei2014-05-301-1/+3
|
* mutex: added preliminary SyncRequest/WaitSynchronization, added some comments/assertionsbunnei2014-05-281-0/+6
|
* kernel: added WaitSynchronization method to Kernel::Objectbunnei2014-05-271-0/+11
|
* kernel: updated SyncRequest to take boolean thread wait result as a parameterbunnei2014-05-271-2/+6
|
* kernel: add a SyncRequest method to KernelObject for use with svcSendSyncRequestbunnei2014-05-271-0/+5
|
* kernel: refactored function naming to remove "__" prefixbunnei2014-05-231-14/+14
|
* mutex: refactored the interface to code to return a Mutex* handlebunnei2014-05-211-2/+12
|
* mutex: initial commit of HLE modulebunnei2014-05-211-0/+122