summaryrefslogtreecommitdiffstats
path: root/src/core/hle/kernel/svc.cpp (follow)
Commit message (Collapse)AuthorAgeFilesLines
* kernel: Eliminate kernel global stateLioncash2018-08-291-25/+55
| | | | | | | | | | | | | | | | | | | | | | 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.
* svc: Return process title ID if queried in GetInfo()Lioncash2018-08-251-2/+1
| | | | | | We already have the variable itself set up to perform this task, so we can just return its value from the currently executing process instead of always stubbing it to zero.
* kernel/svc: Log svcBreak parametersLioncash2018-08-151-2/+5
| | | | | | Given if we hit here all is lost, we should probably be logging the break reason code and associated information to distinguish between the causes.
* Merge pull request #1043 from Subv/timingbunnei2018-08-131-1/+0
|\ | | | | Use an approximated amortized amount of ticks when advancing timing.
| * Kernel/SVC: Don't reschedule the current core when creating a new thread.Subv2018-08-131-1/+0
| | | | | | | | The current core may have nothing to do with the core where the new thread was scheduled to run. In case it's the same core, then the following PrepareReshedule call will take care of that.
* | Kernel/Mutex: Don't duplicate threads in the mutex waiter list.Subv2018-08-121-2/+1
|/ | | | | | | | | | Exit from AddMutexWaiter early if the thread is already waiting for a mutex owned by the owner thread. This accounts for the possibility of a thread that is waiting on a condition variable being awakened twice in a row. Also added more validation asserts. This should fix one of the random crashes in Breath Of The Wild.
* kernel/vm_manager: Use const where applicableLioncash2018-08-021-1/+1
| | | | Makes our immutable state explicit.
* 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/+3
|/ | | | | Removes unnecessary direct dependencies in some headers and also gets rid of indirect dependencies that were being relied on to be included.
* Merge pull request #804 from lioncash/logMat M2018-07-251-1/+3
|\ | | | | svc: Log parameters in SetMemoryAttribute()
| * svc: Log parameters in SetMemoryAttribute()Lioncash2018-07-241-1/+3
| | | | | | | | Provides slightly more context than only logging out the address value.
* | svc: Resolve sign comparison warnings in WaitSynchronization()Lioncash2018-07-241-4/+7
|/ | | | | The loop's induction variable was signed, but we were comparing against an unsigned variable.
* Kernel/SVC: Perform atomic accesses in SignalProcessWideKey as per the real kernel.Subv2018-07-221-7/+31
|
* thread: Convert ThreadStatus into an enum classLioncash2018-07-201-10/+10
| | | | | Makes the thread status strongly typed, so implicit conversions can't happen. It also makes it easier to catch mistakes at compile time.
* svc: Correct always true assertion case in SetThreadCoreMaskLioncash2018-07-191-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason this would never be true is that ideal_processor is a u8 and THREADPROCESSORID_DEFAULT is an s32. In this case, it boils down to how arithmetic conversions are performed before performing the comparison. If an unsigned value has a lesser conversion rank (aka smaller size) than the signed type being compared, then the unsigned value is promoted to the signed value (i.e. u8 -> s32 happens before the comparison). No sign-extension occurs here either. An alternative phrasing: Say we have a variable named core and it's given a value of -2. u8 core = -2; This becomes 254 due to the lack of sign. During integral promotion to the signed type, this still remains as 254, and therefore the condition will always be true, because no matter what value the u8 is given it will never be -2 in terms of 32 bits. Now, if one type was a s32 and one was a u32, this would be entirely different, since they have the same bit width (and the signed type would be converted to unsigned instead of the other way around) but would still have its representation preserved in terms of bits, allowing the comparison to be false in some cases, as opposed to being true all the time. --- We also get rid of two signed/unsigned comparison warnings while we're at it.
* Update clang formatJames Rowe2018-07-031-28/+25
|
* Rename logging macro back to LOG_*James Rowe2018-07-031-47/+47
|
* Kernel/Arbiters: Fix casts, cleanup comments/magic numbersMichael Scire2018-06-221-4/+4
|
* Add additional missing format.Michael Scire2018-06-221-19/+21
|
* Kernel/Arbiters: Initialize arb_wait_address in thread struct.Michael Scire2018-06-211-0/+4
|
* Kernel/Arbiters: Mostly implement SignalToAddressMichael Scire2018-06-211-1/+1
|
* Kernel/Arbiters: Add stubs for 4.x SignalToAddress/WaitForAddres SVCs.Michael Scire2018-06-211-3/+50
|
* Build: Fixed some MSVC warnings in various parts of the code.Subv2018-06-201-2/+2
|
* Merge pull request #572 from Armada651/user-except-stubbunnei2018-06-181-0/+5
|\ | | | | svc: Add a stub for UserExceptionContextAddr.
| * svc: Add a stub for UserExceptionContextAddr.Jules Blok2018-06-181-0/+5
| |
* | Kernel/SVC: Support special core values -2 and -3 in svcSetThreadCoreMask.Subv2018-05-311-1/+26
| | | | | | | | Also added some proper error handling.
* | Kernel/SVC: Signal the highest priority threads first in svcSignalProcessWideKey.Subv2018-05-191-51/+68
| |
* | Kernel/Threads: Reschedule the proper core when operating on that core's threads.Subv2018-05-191-2/+6
| |
* | SVC: Removed unused WaitSynchronization1 functionSubv2018-05-191-30/+0
| |
* | thread: Rename mask to affinity_masks.bunnei2018-05-111-1/+1
| |
* | threading: Reschedule only on cores that are necessary.bunnei2018-05-111-2/+2
| |
* | svc: Implement GetThreadCoreMask and SetThreadCoreMask.bunnei2018-05-111-7/+22
| |
* | svc: SignalProcessWideKey should apply to all cores.bunnei2018-05-111-43/+50
| |
* | svc: Implement GetCurrentProcessorNumber.bunnei2018-05-111-2/+2
| |
* | core: Implement multicore support.bunnei2018-05-111-21/+22
| |
* | general: Make formatting of logged hex values more straightforwardLioncash2018-05-021-38/+39
| | | | | | | | | | | | This makes the formatting expectations more obvious (e.g. any zero padding specified is padding that's entirely dedicated to the value being printed, not any pretty-printing that also gets tacked on).
* | string_util: Remove StringFromFormat() and related functionsLioncash2018-04-301-1/+1
| | | | | | | | Given we utilize fmt, we don't need to provide our own functions for formatting anymore
* | general: Convert assertion macros over to be fmt-compatibleLioncash2018-04-271-1/+1
| |
* | kernel: Migrate logging macros to fmt-compatible onesLioncash2018-04-261-69/+69
| |
* | Merge pull request #370 from Subv/sync_primitivesbunnei2018-04-231-107/+55
|\ \ | | | | | | Kernel: Reworked the new kernel synchronization primitives.
| * | Kernel: Implemented mutex priority inheritance.Subv2018-04-231-0/+9
| | | | | | | | | | | | | | | | | | | | | 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 unused ConditionVariable class.Subv2018-04-211-6/+0
| | |
| * | Kernel: Properly implemented svcWaitProcessWideKey and svcSignalProcessWideKeySubv2018-04-211-83/+46
| | | | | | | | | | | | They work in tandem with guest code to provide synchronization primitives along with svcArbitrateLock/Unlock
| * | Kernel: Corrected the implementation of svcArbitrateLock and svcArbitrateUnlock.Subv2018-04-211-20/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | resource_limit: Make ResourceTypes an enum classLioncash2018-04-211-2/+2
|/ / | | | | | | Prevents enum identifiers from leaking into the surrounding scope.
* | common_funcs: Remove ARRAY_SIZE macroLioncash2018-04-201-1/+2
| | | | | | | | C++17 has non-member size() which we can just call where necessary.
* | Various service name fixes - part 2 (rebased) (#322)Hexagon122018-04-171-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Updated ACC with more service names * Updated SVC with more service names * Updated set with more service names * Updated sockets with more service names * Updated SPL with more service names * Updated time with more service names * Updated vi with more service names
* | svc: Stub out SetThreadActivity, GetThreadContext.bunnei2018-04-031-2/+14
| |
* | svc: Stub GetThreadCoreMask.bunnei2018-03-301-3/+11
| |
* | More Warning cleanupsN00byKing2018-03-191-1/+1
| |
* | Clean Warnings (?)N00byKing2018-03-191-1/+1
| |
* | svc: Use more correct values for GetInfo MapRegion and NewMapRegion.bunnei2018-03-161-5/+5
| |
* | MemoryState: Add additional memory states and improve naming.bunnei2018-03-161-1/+1
| |
* | core: Move process creation out of global state.bunnei2018-03-141-18/+20
|/
* Merge pull request #215 from N00byKing/umapsharedmmrybunnei2018-02-261-1/+12
|\ | | | | UnmapSharedMemory
| * (Hopefully) Fix MinGW BuildN00byKing2018-02-251-1/+1
| |
| * Add UnmapSharedMemoryN00byKing2018-02-251-1/+12
| | | | | | | | | | | | | | | | C++11 requires spaces on the Identifier Add inttypes include clang
* | Stub more functionsmailwl2018-02-221-1/+11
|/
* kernel: Use Scheduler class for threading.bunnei2018-02-181-2/+2
|
* GetInfo: Implement IsCurrentProcessBeingDebugged.bunnei2018-02-041-0/+3
|
* WaitProcessWideKeyAtomic: Handle case where condition variable was already created.bunnei2018-02-041-8/+15
|
* svc: SharedMemory size should be 64-bits and cleanup.bunnei2018-02-031-8/+8
|
* ArbitrateLock: Assert that requesting_thread is current_thread.bunnei2018-02-031-0/+1
|
* hle: Remove Domain and SyncObject kernel objects.bunnei2018-01-251-2/+1
|
* Format: Run the new clang format on everythingJames Rowe2018-01-211-4/+6
|
* Added CreateSharedMemory & UNIMPLEMENTED() for non existent services. (#113)David2018-01-201-1/+13
| | | | | | | | | | | | * Added svcCreateSharedMemory * Services which are not implemented now throw UNIMPLEMENTED() * clang-format * changed perms to u32 * removed camelcase
* svc: Fix svcGetInfo MapRegionBaseAddr.bunnei2018-01-191-1/+1
|
* svc: Rename some entries to match their analogue on SwitchBrewLioncash2018-01-181-7/+7
| | | | Makes the codebase a little more consistent with regards to available documentation. Also amends the duplicate case where there was a similar entry at 0x72 named ConnectToPort.
* svc: Add CreateJitMemory and MapJitMemory svc stringsLioncash2018-01-181-2/+2
| | | | Makes the table match SwitchBrew for these entries
* svc: Clang-format fix.bunnei2018-01-171-6/+4
|
* SVC: Correct some return values in svcGetInfo and added TitleId and PrivilegedProcessId stubs.Subv2018-01-171-6/+21
| | | | | # Conflicts: # src/core/hle/kernel/svc.cpp
* Merge pull request #52 from ogniK5377/fspbunnei2018-01-171-1/+18
|\ | | | | added more svcGetInfo pairs for 3.0.0+ support, Changed HEAP_SIZE and TLS_AREA_VADDR. changed mem usage & heap usage stub added, ISelfController, IApplication function stubs. Added SetThreadCoreMask
| * SetThreadCoreMask stub, time to implement fspDavid Marcec2018-01-161-1/+6
| |
| * Added more svcGetInfo pairsDavid Marcec2018-01-161-0/+12
| |
* | clang-formatMerryMage2018-01-161-6/+4
|/
* svc: Implement svcMapSharedMemory.bunnei2018-01-141-1/+33
|
* yuzu: Update license text to be consistent across project.bunnei2018-01-131-1/+1
|
* core: Include <algorithm> where used.bunnei2018-01-121-0/+2
|
* svc: Implement GetSystemTick.bunnei2018-01-121-2/+13
|
* svc: Stub ResetSignal and CreateTransferMemorySubv2018-01-111-3/+20
|
* svc: Stub SetMemoryAttributeSubv2018-01-111-0/+5
|
* Threads: Added enum values for the Switch's 4 cpu cores and implemented svcGetInfo(AllowedCpuIdBitmask)Subv2018-01-101-6/+11
|
* SVC: Fixed WaitSynchronization with multiple handles when none is immediately ready.Subv2018-01-091-7/+18
|
* SVC: Implemented CancelSynchronization.Subv2018-01-091-1/+17
|
* SVC: Fixed WaitSynchronization with multiple handles when at least one of them is ready.Subv2018-01-091-1/+24
|
* kernel: Rename Semaphore to ConditionVariable.bunnei2018-01-091-25/+33
|
* Kernel: Actually wake up the requested number of threads in Semaphore::Release.Subv2018-01-091-1/+1
| | | | | | Also properly keep track of data in guest memory, this fixes managing the semaphore from userland. It was found that Semaphores are actually Condition Variables, with Release(1) and Release(-1) being equivalent to notify_one and notify_all. We should change the name of the class to reflect this.
* Kernel: Properly keep track of mutex lock data in the guest memory. This fixes userland locking/unlocking.Subv2018-01-091-1/+4
|
* Kernel: Allow chaining WaitSynchronization calls inside a wakeup callback.Subv2018-01-091-15/+41
|
* svc: Implement svcSignalProcessWideKey.bunnei2018-01-071-2/+21
|
* svc: Implement svcWaitProcessWideKeyAtomic.bunnei2018-01-061-1/+49
|
* svc: Implement WaitSynchronization for a single handle.bunnei2018-01-061-4/+24
|
* svc: Refactor LockMutex code to use WaitSynchronization1.bunnei2018-01-061-13/+45
|
* svc: Add missing string_util include.bunnei2018-01-051-0/+1
|
* arm: Remove SkyEye/Dyncom code that is ARMv6-only.bunnei2018-01-031-19/+9
|
* svc: Remove unnecessary "svc" prefix to naming scheme.bunnei2018-01-031-106/+106
|
* hle: Move SVC code to kernel namespace.bunnei2018-01-031-0/+612