summaryrefslogtreecommitdiffstats
path: root/src/core/hle/kernel (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #1847 from ogniK5377/backtrace-breakbunnei2018-12-301-0/+3
|\ | | | | Print backtrace on svcBreak
| * Moved backtrace to ArmInterfaceDavid Marcec2018-12-193-11/+3
| |
| * Moved backtrace to ArmInterfaceDavid Marcec2018-12-031-14/+1
| | | | | | | | Added to both dynarmic and unicorn
| * Print backtrace on svcBreakDavid Marcec2018-12-033-0/+24
| | | | | | | | When we get an svcBreak we get a backtrace now
* | kernel/process: Remove most allocation functions from Process' interfaceLioncash2018-12-283-38/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In all cases that these functions are needed, the VMManager can just be retrieved and used instead of providing the same functions in Process' interface. This also makes it a little nicer dependency-wise, since it gets rid of cases where the VMManager interface was being used, and then switched over to using the interface for a Process instance. Instead, it makes all accesses uniform and uses the VMManager instance for all necessary tasks. All the basic memory mapping functions did was forward to the Process' VMManager instance anyways.
* | Merge pull request #1928 from lioncash/capsbunnei2018-12-276-123/+642
|\ \ | | | | | | kernel: Handle kernel capability descriptors
| * | kernel/process: Hook up the process capability parser to the process itselfLioncash2018-12-212-120/+18
| | | | | | | | | | | | | | | While we're at it, we can also toss out the leftover capability parsing from Citra.
| * | kernel/process_capability: Handle debug capability flagsLioncash2018-12-212-1/+18
| | |
| * | kernel/process_capability: Handle handle table capability flagsLioncash2018-12-212-1/+11
| | | | | | | | | | | | | | | This just specifies the handle table size. There's also a section of reserved bits that are checked against.
| * | kernel/process_capability: Handle kernel version capability flagsLioncash2018-12-212-1/+18
| | |
| * | kernel/process_capability: Handle program capability flagsLioncash2018-12-213-2/+29
| | |
| * | kernel/process_capability: Handle interrupt capability flagsLioncash2018-12-211-1/+21
| | | | | | | | | | | | | | | | | | Similar to the service capability flags, however, we currently don't emulate the GIC, so this currently handles all interrupts as being valid for the time being.
| * | kernel/process_capability: Handle syscall capability flagsLioncash2018-12-212-1/+29
| | |
| * | kernel/process_capability: Handle the priority mask and core mask flagsLioncash2018-12-212-1/+40
| | | | | | | | | | | | | | | | | | Handles the priority mask and core mask flags to allow building up the masks to determine the usable thread priorities and cores for a kernel process instance.
| * | kernel/process: Introduce process capability parsing skeletonLioncash2018-12-214-3/+466
| | | | | | | | | | | | | | | | | | | | | We've had the old kernel capability parser from Citra, however, this is unused code and doesn't actually map to how the kernel on the Switch does it. This introduces the basic functional skeleton for parsing process capabilities.
* | | kernel/vm_manager: Reset region attributes when unmapping a VMALioncash2018-12-271-0/+1
| | | | | | | | | | | | | | | Like the other members related to memory regions, the attributes need to be reset back to their defaults as well.
* | | Merge pull request #1849 from encounter/svcSetThreadActivitybunnei2018-12-264-6/+72
|\ \ \ | | | | | | | | svc: Implement SetThreadActivity (thread suspension)
| * | | svc: Implement SetThreadActivity (thread suspension)Luke Street2018-12-044-6/+72
| | | |
* | | | Merge pull request #1921 from ogniK5377/no-unitbunnei2018-12-211-0/+1
|\ \ \ \ | | | | | | | | | | Fixed uninitialized memory due to missing returns in canary
| * | | | Fixed uninitialized memory due to missing returns in canaryDavid Marcec2018-12-191-0/+1
| | | | | | | | | | | | | | | | | | | | Functions which are suppose to crash on non canary builds usually don't return anything which lead to uninitialized memory being used.
* | | | | Merge pull request #1925 from lioncash/pidbunnei2018-12-216-26/+57
|\ \ \ \ \ | |_|_|/ / |/| | | | kernel/{process, thread}: Amend behavior related to IDs
| * | | | kernel/svc: Handle thread handles within GetProcessIdLioncash2018-12-191-10/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a thread handle is passed to svcGetProcessId, the kernel attempts to access the process ID via the thread's instance's owning process. Technically, this function should also be handling the kernel debug objects as well, however we currently don't handle those kernel objects yet, so I've left a note via a comment about it to remind myself when implementing it in the future.
| * | | | kernel/kernel: Use correct initial PID for userland Process instancesLioncash2018-12-192-4/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Starts the process ID counter off at 81, which is what the kernel itself checks against internally when creating processes. It's actually supposed to panic if the PID is less than 81 for a userland process.
| * | | | kernel/svc: Correct output parameter for svcGetThreadIdLioncash2018-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | The service call uses a 64-bit value, just like svcGetProcessId. This amends the function signature accordingly.
| * | | | kernel/thread: Make thread_id a 64-bit valueLioncash2018-12-193-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | The kernel uses a 64-bit value for the thread ID, so we shouldn't be using a 32-bit value.
| * | | | kernel/svc: Correct output parameter for svcGetProcessIdLioncash2018-12-192-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | svcGetProcessId's out parameter is a pointer to a 64-bit value, not a 32-bit one.
| * | | | kernel/process: Make process_id a 64-bit valueLioncash2018-12-193-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | In the actual kernel, this is a 64-bit value, so we shouldn't be using a 32-bit type to handle it.
* | | | | Merge pull request #1907 from lioncash/attributebunnei2018-12-193-14/+279
|\ \ \ \ \ | |_|/ / / |/| | | | kernel/svc: Implement svcSetMemoryAttribute
| * | | | svc: Implement svcSetMemoryAttributeLioncash2018-12-191-5/+46
| | | | | | | | | | | | | | | | | | | | | | | | | With all the basic backing functionality implemented, we can now unstub svcSetMemoryAttribute.
| * | | | vm_manager: Add member function for setting memory attributes across an address rangeLioncash2018-12-192-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | This puts the backing functionality for svcSetMemoryAttribute in place, which will be utilized in a following change.
| * | | | vm_manager: Add member function for checking a memory range adheres to certain attributes, permissions and statesLioncash2018-12-192-0/+100
| | | | |
| * | | | vm_manager: Rename meminfo_state to stateLioncash2018-12-162-10/+9
| | | | | | | | | | | | | | | | | | | | | | | | | This is shorter and more concise. This also removes the now-innaccurate comment, as it's not returned wholesale to svcQueryMemory anymore.
| * | | | vm_manager: Add backing functionality for memory attributesLioncash2018-12-162-1/+85
| |/ / / | | | | | | | | | | | | | | | | | | | | Adds the barebones enumeration constants and functions in place to handle memory attributes, while also essentially leaving the attribute itself non-functional.
* / / / kernel/thread: Set default fpcrMerryMage2018-12-181-0/+3
|/ / /
* | | Merge pull request #1732 from DarkLordZach/yield-typesbunnei2018-12-154-9/+165
|\ \ \ | | | | | | | | svc: Implement yield types 0 and -1
| * | | svc: Avoid incorrect fast yield conditionZach Hilman2018-12-051-6/+1
| | | |
| * | | scheduler: Avoid manual Reschedule callZach Hilman2018-12-042-11/+11
| | | | | | | | | | | | This will automatically occur anyway when PrepareReschedule is called
| * | | scheduler: Only work steal higher priority threads from other coresZach Hilman2018-12-033-35/+24
| | | |
| * | | svc: Avoid performance-degrading unnecessary rescheduleZach Hilman2018-12-022-8/+6
| | | |
| * | | scheduler: Add explanations for YieldWith and WithoutLoadBalancingZach Hilman2018-11-225-77/+139
| | | |
| * | | svc: Implement yield types 0 and -1Zach Hilman2018-11-195-2/+114
| | | |
* | | | Merge pull request #1899 from lioncash/statebunnei2018-12-147-84/+188
|\ \ \ \ | | | | | | | | | | vm_manager/svc: Modify MemoryState enum, and correct error handling for svcQueryMemory
| * | | | svc: Enable svcQueryProcessMemoryLioncash2018-12-122-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | svcQueryProcessMemory is trivial to implement, given all the behavior necessary for it is present, it just needs a handler for it.
| * | | | svc: Write out the complete MemoryInfo structure in QueryProcessMemoryLioncash2018-12-121-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the previous change, the memory writing was moved into the service function itself, however it still had a problem, in that the entire MemoryInfo structure wasn't being written out, only the first 32 bytes of it were being written out. We still need to write out the trailing two reference count members and zero out the padding bits. Not doing this can result in wrong behavior in userland code in the following scenario: MemoryInfo info; // Put on the stack, not quaranteed to be zeroed out. svcQueryMemory(&info, ...); if (info.device_refcount == ...) // Whoops, uninitialized read. This can also cause the wrong thing to happen if the user code uses std::memcmp to compare the struct, with another one (questionable, but allowed), as the padding bits are not guaranteed to be a deterministic value. Note that the kernel itself also fully zeroes out the structure before writing it out including the padding bits.
| * | | | svc: Handle memory writing explicitly within QueryProcessMemoryLioncash2018-12-122-26/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moves the memory writes directly into QueryProcessMemory instead of letting the wrapper function do it. It would be inaccurate to allow the handler to do it because there's cases where memory shouldn't even be written to. For example, if the given process handle is invalid. HOWEVER, if the memory writing is within the wrapper, then we have no control over if these memory writes occur, meaning in an error case, 68 bytes of memory randomly get trashed with zeroes, 64 of those being written to wherever the memory info address points to, and the remaining 4 being written wherever the page info address points to. One solution in this case would be to just conditionally check within the handler itself, but this is kind of smelly, given the handler shouldn't be performing conditional behavior itself, it's a behavior of the managed function. In other words, if you remove the handler from the equation entirely, does the function still retain its proper behavior? In this case, no. Now, we don't potentially trash memory from this function if an invalid query is performed.
| * | | | vm_manager: Correct ordering of last two struct members of MemoryInfoLioncash2018-12-121-2/+2
| | | | | | | | | | | | | | | | | | | | These should be swapped.
| * | | | vm_manager: Amend the returned values for invalid memory queries in QueryMemory()Lioncash2018-12-122-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kernel returns a memory info instance with the base address set to the end of the address space, and the size of said block as 0 - address_space_end, it doesn't set both of said members to zero.
| * | | | vm_manager: Migrate memory querying to the VMManager interfaceLioncash2018-12-124-18/+33
| | | | | | | | | | | | | | | | | | | | | | | | | Gets rid of the need to directly access the managed VMAs outside of the memory manager itself just for querying memory.
| * | | | vm_manager: Migrate MemoryInfo and PageInfo to vm_manager.hLioncash2018-12-123-17/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gets the two structures out of an unrelated header and places them with the rest of the memory management code. This also corrects the structures. PageInfo appears to only contain a 32-bit flags member, and the extra padding word in MemoryInfo isn't necessary.
| * | | | vm_manager: Amend MemoryState enum membersLioncash2018-12-125-28/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Amends the MemoryState enum to use the same values like the actual kernel does. Also provides the necessary operators to operate on them. This will be necessary in the future for implementing svcSetMemoryAttribute, as memory block state is checked before applying the attribute.
* | | | | Merge pull request #1900 from lioncash/wrapperbunnei2018-12-141-1/+1
|\ \ \ \ \ | | | | | | | | | | | | svc_wrap: Correct register index for a wrapper specialization
| * | | | | svc_wrap: Correct register index for a wrapper specializationLioncash2018-12-121-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This would result in svcSetMemoryAttribute getting the wrong value for its third parameter. This is currently fine, given the service function is stubbed, however this will be unstubbed in a future change, so this needs to change.
* / / / / Fix Process object leak on emulation stopJens Schmer2018-12-123-13/+12
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Process object kept itself alive indefinitely because its handle_table contains a SharedMemory object which owns a reference to the same Process object, creating a circular ownership scenario. Break that up by storing only a non-owning pointer in the SharedMemory object.
* | | | Merge pull request #1872 from lioncash/proc-infoHexagon122018-12-101-0/+1
|\ \ \ \ | | | | | | | | | | kernel/process: Set ideal core from metadata
| * | | | kernel/process: Set ideal core from metadataLioncash2018-12-051-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | A very trivial change. If metadata is available, the process should use it to retrieve the desired core for the process to run on.
* | | | | Merge pull request #1876 from lioncash/vmabunnei2018-12-104-22/+36
|\ \ \ \ \ | | | | | | | | | | | | vm_manager: Make vma_map private
| * | | | | vm_manager: Make vma_map privateLioncash2018-12-064-22/+36
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was only ever public so that code could check whether or not a handle was valid or not. Instead of exposing the object directly and allowing external code to potentially mess with the map contents, we just provide a member function that allows checking whether or not a handle is valid. This makes all member variables of the VMManager class private except for the page table.
* | | | | kernel/svc: Correct behavior of svcResetSignal()Lioncash2018-12-051-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While partially correct, this service call allows the retrieved event to be null, as it also uses the same handle to check if it was referring to a Process instance. The previous two changes put the necessary machinery in place to allow for this, so we can simply call those member functions here and be done with it.
* | | | | kernel/process: Make Process a WaitObjectLioncash2018-12-053-6/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Process instances can be waited upon for state changes. This is also utilized by svcResetSignal, which will be modified in an upcoming change. This simply puts all of the WaitObject related machinery in place.
* | | | | kernel/readable_event: Add member function for enforcing a strict reset contractLioncash2018-12-052-1/+22
|/ / / / | | | | | | | | | | | | | | | | | | | | svcResetSignal relies on the event instance to have already been signaled before attempting to reset it. If this isn't the case, then an error code has to be returned.
* | | | kernel/svc: Remove unused header inclusionLioncash2018-12-041-1/+0
| | | |
* | | | kernel/svc: Implement svcSignalEvent()Lioncash2018-12-041-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function simply does a handle table lookup for a writable event instance identified by the given handle value. If a writable event cannot be found for the given handle, then an invalid handle error is returned. If a writable event is found, then it simply signals the event, as one would expect.
* | | | kernel/svc: Implement svcCreateEvent()Lioncash2018-12-042-1/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | svcCreateEvent operates by creating both a readable and writable event and then attempts to add both to the current process' handle table. If adding either of the events to the handle table fails, then the relevant error from the handle table is returned. If adding the readable event after the writable event to the table fails, then the writable event is removed from the handle table and the relevant error from the handle table is returned. Note that since we do not currently test resource limits, we don't check the resource limit table yet.
* | | | Merge pull request #1853 from lioncash/eventbunnei2018-12-045-10/+19
|\ \ \ \ | | | | | | | | | | kernel/object: Amend handle types to distinguish between readable and writable events
| * | | | kernel/object: Amend handle types to distinguish between readable and writable eventsLioncash2018-12-045-10/+19
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two kernel object should absolutely never have the same handle ID type. This can cause incorrect behavior when it comes to retrieving object types from the handle table. In this case it allows converting a WritableEvent into a ReadableEvent and vice-versa, which is undefined behavior, since the object types are not the same. This also corrects ClearEvent() to check both kernel types like the kernel itself does.
* | | | kernel/handle_table: Amend reference to CTR-OS in Create()Lioncash2018-12-041-2/+3
| | | | | | | | | | | | | | | | Another hold-over from Citra.
* | | | kernel/svc: Implement the resource limit svcGetInfo optionLioncash2018-12-044-9/+34
|/ / / | | | | | | | | | | | | Allows a process to register the resource limit as part of its handle table.
* | | [Kernel::CreateThread] Match format specifiers to LOG_TRACE's argumentsV.Kalyuzhny2018-12-041-1/+1
| | |
* | | Merge pull request #1840 from lioncash/infobunnei2018-12-041-50/+100
|\ \ \ | | | | | | | | svc: Reorganize svcGetInfo, handle more error cases for existing implemented info categories
| * | | svc: Use the current process' handle table for retrieving the process instance to act uponLioncash2018-12-021-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kernel uses the handle table of the current process to retrieve the process that should be used to retrieve certain information. To someone not familiar with the kernel, this might raise the question of "Ok, sounds nice, but doesn't this make it impossible to retrieve information about the current process?". No, it doesn't, because HandleTable instances in the kernel have the notion of a "pseudo-handle", where certain values allow the kernel to lookup objects outside of a given handle table. Currently, there's only a pseudo-handle for the current process (0xFFFF8001) and a pseudo-handle for the current thread (0xFFFF8000), so to retrieve the current process, one would just pass 0xFFFF8001 into svcGetInfo. The lookup itself in the handle table would be something like: template <typename T> T* Lookup(Handle handle) { if (handle == PSEUDO_HANDLE_CURRENT_PROCESS) { return CurrentProcess(); } if (handle == PSUEDO_HANDLE_CURRENT_THREAD) { return CurrentThread(); } return static_cast<T*>(&objects[handle]); } which, as is shown, allows accessing the current process or current thread, even if those two objects aren't actually within the HandleTable instance.
| * | | svc: Reorganize svcGetInfo, handle more error cases for existing implemented info categoriesLioncash2018-12-021-50/+99
| | |/ | |/| | | | | | | | | | | | | Our implementation of svcGetInfo was slightly incorrect in that we weren't doing proper error checking everywhere. Instead, reorganize it to be similar to how the kernel seems to do it.
* | | Merge pull request #1803 from DarkLordZach/k-able-eventbunnei2018-12-038-58/+163
|\ \ \ | |/ / |/| | kernel: Divide Event into ReadableEvent and WritableEvent
| * | hle_ipc: Refactor SleepClientThread to avoid ReadableEventZach Hilman2018-11-295-8/+10
| | |
| * | kernel/event: Reference ReadableEvent from WritableEventZach Hilman2018-11-298-125/+47
| | |
| * | core: Port all current usages of Event to Readable/WritableEventZach Hilman2018-11-291-5/+6
| | |
| * | hle_ipc: Use event pair for SleepClientThreadZach Hilman2018-11-292-19/+22
| | |
| * | kernel: Add named event tableZach Hilman2018-11-292-0/+30
| | | | | | | | | | | | Used to store ReadableEvents of all events on the system.
| * | kernel: Divide Event into ReadableEvent and WritableEventZach Hilman2018-11-295-59/+206
| | | | | | | | | | | | More hardware accurate. On the actual system, there is a differentiation between the signaler and signalee, they form a client/server relationship much like ServerPort and ClientPort.
| * | kernel/object: Add descriptions to ResetTypesZach Hilman2018-11-291-3/+3
| | |
* | | Fix debug buildLioncash2018-12-011-1/+1
|/ / | | | | | | | | A non-existent parameter was left in some formatting calls (the logging macro for which only does anything meaningful on debug builds)
* | Merge pull request #1801 from ogniK5377/log-before-executebunnei2018-11-291-24/+206
|\ \ | | | | | | Changed logging to be "Log before execution", Added more error logging, all services/svc should now log on some level
| * | Added comment on Main memory size for more clarityDavid Marcec2018-11-271-0/+1
| | |
| * | Made svcSetHeapSize and svcCreateSharedMemory more readableDavid Marcec2018-11-271-4/+4
| | |
| * | Reworked svcs slightly, improved error messages in AM and fsp_srvDavid Marcec2018-11-271-12/+20
| | |
| * | Improved error messages for SVCsDavid Marcec2018-11-261-76/+170
| | |
| * | Changed logging to be "Log before execution", Added more error logging, all services should now log on some levelDavid Marcec2018-11-261-11/+90
| | |
* | | svc: Implement svcSetResourceLimitLimitValue()Lioncash2018-11-271-1/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The opposite of the getter functions, this function sets the limit value for a particular ResourceLimit resource category, with the restriction that the new limit value must be equal to or greater than the current resource value. If this is violated, then ERR_INVALID_STATE is returned. e.g. Assume: current[Events] = 10; limit[Events] = 20; a call to this service function lowering the limit value to 10 would be fine, however, attempting to lower it to 9 in this case would cause an invalid state error.
* | | svc: Implement svcGetResourceLimitCurrentValue()Lioncash2018-11-271-16/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This kernel service function is essentially the exact same as svcGetResourceLimitLimitValue(), with the only difference being that it retrieves the current value for a given resource category using the provided resource limit handle, rather than retrieving the limiting value of that resource limit instance. Given these are exactly the same and only differ on returned values, we can extract the existing code for svcGetResourceLimitLimitValue() to handle both values.
* | | svc: Implement svcGetResourceLimitLimitValue()Lioncash2018-11-272-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This kernel service function retrieves the maximum allowable value for a provided resource category for a given resource limit instance. Given we already have the functionality added to the resource limit instance itself, it's sufficient to just hook it up. The error scenarios for this are: 1. If an invalid resource category type is provided, then ERR_INVALID_ENUM is returned. 2. If an invalid handle is provided, then ERR_INVALID_HANDLE is returned (bad thing goes in, bad thing goes out, as one would expect). If neither of the above error cases occur, then the out parameter is provided with the maximum limit value for the given category and success is returned.
* | | svc: Implement svcCreateResourceLimit()Lioncash2018-11-272-1/+27
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function simply creates a ResourceLimit instance and attempts to create a handle for it within the current process' handle table. If the kernal fails to either create the ResourceLimit instance or create a handle for the ResourceLimit instance, it returns a failure code (OUT_OF_RESOURCE, and HANDLE_TABLE_FULL respectively). Finally, it exits by providing the output parameter with the handle value for the ResourceLimit instance and returning that it was successful. Note: We do not return OUT_OF_RESOURCE because, if yuzu runs out of available memory, then new will currently throw. We *could* allocate the kernel instance with std::nothrow, however this would be inconsistent with how all other kernel objects are currently allocated.
* | svc: Return ERR_INVALID_ENUM_VALUE from svcGetInfoLuke Street2018-11-251-1/+2
| |
* | kernel/handle_table: Move private static functions into the cpp fileLioncash2018-11-222-7/+9
| | | | | | | | | | These don't depend on class state, and are effectively implementation details, so they can go into the cpp file .
* | kernel/handle_table: Restrict handle table size to 1024 entriesLioncash2018-11-221-5/+2
| | | | | | | | | | | | The previous handle table size is a holdover from Citra. The actual handle table construct on Horizon only allows for a maximum of 1024 entries.
* | kernel/handle_table: Default destructor in the cpp fileLioncash2018-11-222-0/+3
| | | | | | | | | | We don't need to potentially inline the teardown logic of all of the handle instances.
* | Merge pull request #1734 from lioncash/sharedbunnei2018-11-213-29/+45
|\ \ | | | | | | kernel/shared_memory: Make data members private, plus minor interface changes
| * | kernel/shared_memory: Make Map() and Unmap() take the target process by reference rather than as a pointerLioncash2018-11-193-12/+12
| | | | | | | | | | | | | | | | | | Both member functions assume the passed in target process will not be null. Instead of making this assumption implicit, we can change the functions to be references and enforce this at the type-system level.
| * | kernel/shared_memory: Add a const qualified member function overload for GetPointer()Lioncash2018-11-192-1/+12
| | | | | | | | | | | | | | | Given this doesn't mutate instance state, we can provide a const-qualified variant as well.
| * | kernel/shared_memory: Use 64-bit types for offset and size in CreateForAppletLioncash2018-11-192-2/+2
| | | | | | | | | | | | Keeps the interface consistent with the regular Create() function.
| * | kernel/shared_memory: Make GetPointer() take a std::size_t instead of a u32Lioncash2018-11-192-2/+2
| | | | | | | | | | | | | | | | | | | | | Makes the interface nicer to use in terms of 64-bit code, as it makes it less likely for one to get truncation warnings (and also makes sense in the context of the rest of the interface where 64-bit types are used for sizes and offsets
| * | kernel/shared_memory: Make data members privateLioncash2018-11-191-12/+17
| |/ | | | | | | | | | | Rather than allow unfettered access to the class internals, we hide all members by default and create and API that other code can operate against.
* | kernel/process: Move <random> include to the cpp fileLioncash2018-11-202-1/+1
| | | | | | | | | | | | <random> isn't necesary directly within the header and can be placed in the cpp file where its needed. Avoids propagating random generation utilities via a header file.
* | Merge pull request #1667 from DarkLordZach/swkbdbunnei2018-11-201-3/+33
|\ \ | | | | | | am: Implement HLE software keyboard applet
| * | am: Deglobalize software keyboard appletZach Hilman2018-11-181-4/+4
| | |
| * | svc: Implement svcCreateTransferMemoryZach Hilman2018-11-181-3/+33
| | | | | | | | | Seems to be used and created identically to SharedMemory, so just reuse that.
* | | kernel/resource_limit: Clean up interfaceLioncash2018-11-206-190/+81
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | Cleans out the citra/3DS-specific implementation details that don't apply to the Switch. Sets the stage for implementing ResourceLimit instances properly. While we're at it, remove the erroneous checks within CreateThread() and SetThreadPriority(). While these are indeed checked in some capacity, they are not checked via a ResourceLimit instance. In the process of moving out Citra-specifics, this also replaces the system ResourceLimit instance's values with ones from the Switch.
* | Merge pull request #1620 from DarkLordZach/ldr-robunnei2018-11-194-7/+9
|\ \ | | | | | | ldr_ro: Complete LDR:RO implementation
| * | ldr_ro: Add error check for memory allocation failureZach Hilman2018-11-183-6/+7
| | |
| * | process: Make MirrorMemory take state to map new memory asZach Hilman2018-11-151-1/+2
| | | | | | | | | | | | Credits to Subv
* | | Merge pull request #1728 from FearlessTobi/reset-signalMat M2018-11-181-1/+1
|\ \ \ | |_|/ |/| | svc: ResetSignal is not stubbed
| * | svc: ResetSignal is not stubbedTobias2018-11-181-1/+1
| | | | | | | | | https://user-images.githubusercontent.com/20753089/48677874-b8e01c80-eb7b-11e8-8043-b99faa29022c.PNG
* | | kernel/errors: Clean up error codesLioncash2018-11-162-62/+32
| | | | | | | | | | | | | | | | | | | | | | | | Similar to PR 1706, which cleans up the error codes for the filesystem code, but done for the kernel error codes. This removes the ErrCodes namespace and specifies the errors directly. This also fixes up any straggling lines of code that weren't using the named error codes where applicable.
* | | Merge pull request #1638 from FreddyFunk/SetMemoryPermission-StubbedMat M2018-11-162-1/+48
|\ \ \ | | | | | | | | Implement SetMemoryPermission
| * | | Implement SetMemoryPermissionFrederic Laing2018-11-061-3/+39
| | | |
| * | | Stubbed SetMemoryPermissionFrederic Laing2018-11-032-1/+12
| | | |
* | | | Merge pull request #1687 from lioncash/deduplicationbunnei2018-11-152-37/+13
|\ \ \ \ | |_|_|/ |/| | | kernel/thread: Deduplicate scheduler switching code
| * | | kernel/thread: Deduplicate scheduler switching codeLioncash2018-11-142-37/+13
| | | | | | | | | | | | | | | | | | | | The code in both places was the same verbatim, so we can extract it to a function to deduplicate the logic.
* | | | Merge pull request #1679 from DarkLordZach/deterministic-rng-2bunnei2018-11-143-1/+27
|\ \ \ \ | | | | | | | | | | svc: Use proper random entropy generation algorithm
| * | | | svc: Use proper random entropy generation algorithmZach Hilman2018-11-133-1/+27
| |/ / /
* | | | vm_manager: Unstub GetTotalHeapUsage()Lioncash2018-11-131-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Now that we've moved all of the heap-related stuff to the VMManager class, we can unstub this function, as the necessary members are visible now.
* | | | kernel/process: Migrate heap-related memory management out of the process class and into the vm managerLioncash2018-11-134-84/+97
|/ / / | | | | | | | | | | | | Avoids a breach of responsibilities in the interface and keeps the direct code for memory management within the VMManager class.
* | | svc: Return random seed for svcGetInfo RandomEntropyZach Hilman2018-11-131-1/+2
| | |
* | | svcBreak now dumps information from the debug buffer passed (#1646)David2018-11-081-0/+28
|/ / | | | | | | | | | | | | | | * svcBreak now dumps information from the debug buffer passed info1 and info2 seem to somtimes hold an address to a buffer, this is usually 4 bytes or the size of the int and contains an error code. There's other circumstances where it can be something different so we hexdump these to examine them at a later date. * Addressed comments
* | general: Remove unused boost inclusions where applicableLioncash2018-10-301-2/+0
| | | | | | | | Cleans up unused includes and trims off some dependencies on externals.
* | global: Use std::optional instead of boost::optional (#1578)Frederic L2018-10-301-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * get rid of boost::optional * Remove optional references * Use std::reference_wrapper for optional references * Fix clang format * Fix clang format part 2 * Adressed feedback * Fix clang format and MacOS build
* | Merge pull request #1621 from lioncash/ipcbunnei2018-10-302-4/+8
|\ \ | | | | | | hle_ipc: Make GetDomainMessageHeader return a regular pointer
| * | hle_ipc: Add member function for querying the existence of a domain headerLioncash2018-10-302-1/+5
| | | | | | | | | | | | Gets rid of the need to call the getter and then check for null.
| * | hle_ipc: Make GetDomainMessageHeader return a regular pointerLioncash2018-10-302-3/+3
| |/ | | | | | | | | Nothing requires the shared owner ship here, so we can just return a plain pointer.
* / core: Make System references const where applicableLioncash2018-10-282-3/+3
|/
* svc: Localize the GetInfo enum class to the function itselfLioncash2018-10-262-32/+31
| | | | Nothing from this enum is intended to be used outside of this function.
* svc: Implement svcGetInfo command 0xF0000002Lioncash2018-10-266-4/+98
| | | | | | | | | | This retrieves: if (curr_thread == handle_thread) { result = total_thread_ticks + (hardware_tick_count - last_context_switch_ticks); } else if (curr_thread == handle_thread && sub_id == current_core_index) { result = hardware_tick_count - last_context_switch_ticks; }
* process: LoadModule should clear JIT instruction cache.bunnei2018-10-261-0/+6
|
* Kernel/Memory: Added a function to first a suitable guest address at which to allocate a region of a given size.bunnei2018-10-262-0/+28
|
* kernel/errors: Remove now-unused, unnecessary, error codesLioncash2018-10-241-8/+0
| | | | | | Now that we've gotten the innaccurate error codes out of the way, we can finally toss away a bunch of these, trimming down the error codes to ones that are actually used and knocking out two TODO comments.
* kernel/shared_memory: Return ERR_INVALID_MEMORY_PERMISSIONS instead of ERR_INVALID_COMBINATIONLioncash2018-10-241-4/+3
| | | | This is more consistent with what the kernel does.
* kernel/server_port: Simplify emptiness check within ShouldWait()Lioncash2018-10-241-1/+1
|
* kernel/server_port: Change error case return value in Accept() to ERR_NOT_FOUNDLioncash2018-10-242-3/+1
| | | | This is what the kernel does in this instance.
* kernel/error: Remove leftover 3DS error codesLioncash2018-10-241-5/+0
| | | | These are now entirely unused and can be removed.
* kernel/svc: Amend returned error code for invalid priorities in CreateThreadLioncash2018-10-241-1/+1
| | | | | Like with the previous change, the kernel doesn't return NOT_AUTHORIZED here. It returns INVALID_THREAD_PRIORITY.
* kernel/svc: Move and correct returned error code for invalid thread priorities in SetThreadPriority()Lioncash2018-10-241-5/+6
| | | | | | All priority checks are supposed to occur before checking the validity of the thread handle, we're also not supposed to return ERR_NOT_AUTHORIZED here.
* kernel/error: Add error code for invalid pointersLioncash2018-10-241-1/+1
| | | | | The kernel appears to return 0xE601 for this situation. Particularly in svcWaitSynchronization, svcReplyAndReceive, and svcGetThreadContext
* kernel/error: Add error code for closed sessionsLioncash2018-10-241-1/+3
| | | | The kernel appears to return 0xF601 for this case.
* Merge pull request #1551 from ogniK5377/improved-svcbreakbunnei2018-10-241-5/+51
|\ | | | | Added break types to svcBreak
| * Added assertion failed, reworked logging levelsDavid Marcec2018-10-231-16/+24
| |
| * Added break types to svcBreakDavid Marcec2018-10-231-4/+42
| | | | | | | | There seems to be more such as type 1, and 2. Unsure what these currently are but when a game hits them we can investigate and add the rest
* | Merge pull request #1540 from lioncash/handlebunnei2018-10-248-98/+95
|\ \ | |/ |/| kernel/process: Make the handle table per-process
| * kernel/process: Make the handle table per-processLioncash2018-10-208-98/+95
| | | | | | | | | | | | | | | | In the kernel, there isn't a singular handle table that everything gets tossed into or used, rather, each process gets its own handle table that it uses. This currently isn't an issue for us, since we only execute one process at the moment, but we may as well get this out of the way so it's not a headache later on.
* | svc: Fix vma boundary check in svcQueryMemoryLioncash2018-10-201-1/+1
|/ | | | | | This should be comparing against the queried process' vma_map, not the current process'. The only reason this hasn't become an issue yet is we currently only handle one process being active at any time.
* Merge pull request #1520 from lioncash/sanbunnei2018-10-203-3/+50
|\ | | | | svc: Add missing sanitizing checks for MapSharedMemory/UnmapSharedMemory
| * svc: Add missing sanitizing checks for MapSharedMemory/UnmapSharedMemoryLioncash2018-10-183-3/+50
| | | | | | | | | | | | | | | | Now that the changes clarifying the address spaces has been merged, we can wrap the checks that the kernel performs when mapping shared memory (and other forms of memory) into its own helper function and then use those within MapSharedMemory and UnmapSharedMemory to complete the sanitizing checks that are supposed to be done.
* | svc: Check for word alignment of addresses within svcArbitrateLock/svcArbitrateUnlockLioncash2018-10-181-0/+8
| | | | | | | | | | The kernel itself checks whether or not the provided addresses are word aligned before continuing, so we should be doing the same.
* | common: Move Is4KBAligned() to alignment.hLioncash2018-10-181-9/+7
|/ | | | | Aligning on 4KB pages isn't a Switch-specific thing, so this can be moved to common so it can be used with other things as well.
* Merge pull request #1498 from lioncash/aslrbunnei2018-10-184-28/+44
|\ | | | | svc: Clarify enum values for AddressSpaceBaseAddr and AddressSpaceSize in svcGetInfo()
| * svc: Clarify enum values for AddressSpaceBaseAddr and AddressSpaceSize in svcGetInfo()Lioncash2018-10-154-28/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | So, one thing that's puzzled me is why the kernel seemed to *not* use the direct code address ranges in some cases for some service functions. For example, in svcMapMemory, the full address space width is compared against for validity, but for svcMapSharedMemory, it compares against 0xFFE00000, 0xFF8000000, and 0x7FF8000000 as upper bounds, and uses either 0x200000 or 0x8000000 as the lower-bounds as the beginning of the compared range. Coincidentally, these exact same values are also used in svcGetInfo, and also when initializing the user address space, so this is actually retrieving the ASLR extents, not the extents of the address space in general.
* | core_cpu: Make Cpu scheduler instances unique_ptrs instead of shared_ptrsLioncash2018-10-154-15/+15
|/
* Merge pull request #1492 from lioncash/procbunnei2018-10-143-4/+50
|\ | | | | svc: Implement svcGetProcessInfo
| * svc: Implement svcGetProcessInfoLioncash2018-10-133-4/+50
| | | | | | | | | | | | | | A fairly basic service function, which only appears to currently support retrieving the process state. This also alters the ProcessStatus enum to contain all of the values that a kernel process seems to be able of reporting with regards to state.
* | Stop all threads on svcBreakDavid Marcec2018-10-141-0/+6
|/ | | | This should help diagnose crashes easier and prevent many users thinking that a game is still running when in fact it's just an audio thread still running(this is typically not killed when svcBreak is hit since the game expects us to do this)
* Merge pull request #1483 from lioncash/codesetbunnei2018-10-134-40/+14
|\ | | | | kernel/process: Make CodeSet a regular non-inherited object
| * kernel/process: Make CodeSet a regular non-inherited objectLioncash2018-10-124-40/+14
| | | | | | | | | | | | | | | | These only exist to ferry data into a Process instance and end up going out of scope quite early. Because of this, we can just make it a plain struct for holding things and just std::move it into the relevant function. There's no need to make this inherit from the kernel's Object type.
* | Merge pull request #1481 from lioncash/typobunnei2018-10-131-3/+3
|\ \ | | | | | | svc: Fix typos in sanitizing checks for MapMemory/UnmapMemory
| * | svc: Fix typos in sanitizing checks for MapMemory/UnmapMemoryLioncash2018-10-121-3/+3
| |/
* | Merge pull request #1467 from ogniK5377/svcbreak-type-fixbunnei2018-10-122-28/+36
|\ \ | | | | | | Fixed incorrect types for svcBreak
| * | Changed all casts in svc_wrap.h to be static_cast insteadDavid Marcec2018-10-101-25/+28
| | |
| * | Use a better name than "dont_kill_application"David Marcec2018-10-101-2/+2
| | | | | | | | | | | | signal_debugger seems like a more fitting name
| * | Fixed incorrect types for svcBreakDavid Marcec2018-10-102-3/+8
| | | | | | | | | | | | svcBreak reason should be a u32, not a u64.
* | | thread: Remove unnecessary memset from ResetThreadContext()Lioncash2018-10-121-4/+1
| |/ |/| | | | | | | | | Regular value initialization is adequate here for zeroing out data. It also has the benefit of not invoking undefined behavior if a non-trivial type is ever added to the struct for whatever reason.
* | svc: Add missing address range sanitizing checks to MapMemory/UnmapMemoryLioncash2018-10-112-12/+81
| | | | | | | | | | | | | | | | This adds the missing address range checking that the service functions do before attempting to map or unmap memory. Given that both service functions perform the same set of checks in the same order, we can wrap these into a function and just call it from both functions, which deduplicates a little bit of code.
* | kernel/thread: Use a regular pointer for the owner/current processLioncash2018-10-106-29/+29
|/ | | | | | | | | | | There's no real need to use a shared pointer in these cases, and only makes object management more fragile in terms of how easy it would be to introduce cycles. Instead, just do the simple thing of using a regular pointer. Much of this is just a hold-over from citra anyways. It also doesn't make sense from a behavioral point of view for a process' thread to prolong the lifetime of the process itself (the process is supposed to own the thread, not the other way around).
* Added bitfield instead of manually checking if the bit is setDavid Marcec2018-10-091-4/+12
|
* Actual kill execution when the bit isn't set, not the other way aroundDavid Marcec2018-10-091-1/+1
|
* svcBreak, Signalling to the debugger should not kill executionDavid Marcec2018-10-091-5/+12
| | | | When loading NROs, svcBreak is called to signal to the debugger that a new "module" is loaded. As no debugger is technically attached we shouldn't be killing the programs execution.
* Added forward define for ServerPortDavid Marcec2018-10-062-4/+6
|
* Ported #4296 from citraDavid Marcec2018-10-062-1/+6
| | | | This will allow us to easily remove the use of "NFC" in "System"
* 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.
* thread: Make the scheduler pointer a regular pointerbalika0112018-10-052-4/+4
| | | | | | | | | | | | | | Conceptually, it doesn't make sense for a thread to be able to persist the lifetime of a scheduler. A scheduler should be taking care of the threads; the threads should not be taking care of the scheduler. If the threads outlive the scheduler (or we simply don't actually terminate/shutdown the threads), then it should be considered a bug that we need to fix. Attributing this to balika011, as they opened #1317 to attempt to fix this in a similar way, but my refactoring of the kernel code caused quite a few conflicts.
* kernel/thread: Make all instance variables privateLioncash2018-10-0411-187/+363
| | | | | | | | | | | | | | | | | | | | 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.
* kernel/svc: Implement svcGetThreadContext()Lioncash2018-09-303-2/+37
| | | | | | | | | Now that we have all of the rearranging and proper structure sizes in place, it's fairly trivial to implement svcGetThreadContext(). In the 64-bit case we can more or less just write out the context as is, minus some minor value sanitizing. In the 32-bit case we'll need to clear out the registers that wouldn't normally be accessible from a 32-bit AArch32 exectuable (or process).
* kernel/process: Add a data member to determine if a process is 64-bit or not.Lioncash2018-09-302-0/+11
| | | | | | | | | | | This will be necessary for the implementation of svcGetThreadContext(), as the kernel checks whether or not the process that owns the thread that has it context being retrieved is a 64-bit or 32-bit process. If the process is 32-bit, then the upper 15 general-purpose registers and upper 16 vector registers are cleared to zero (as AArch32 only has 15 GPRs and 16 128-bit vector registers. not 31 general-purpose registers and 32 128-bit vector registers like AArch64).
* kernel/process: Make data member variables privateLioncash2018-09-305-51/+96
| | | | | | | Makes the public interface consistent in terms of how accesses are done on a process object. It also makes it slightly nicer to reason about the logic of the process class, as we don't want to expose everything to external code.
* Merge pull request #1412 from lioncash/movebunnei2018-09-292-3/+2
|\ | | | | kernel/object: Remove unnecessary std::move from DynamicObjectCast()
| * kernel/object: Remove unnecessary std::move from DynamicObjectCast()Lioncash2018-09-282-3/+2
| | | | | | | | | | | | | | | | | | boost::static_pointer_cast for boost::intrusive_ptr (what SharedPtr is), takes its parameter by const reference. Given that, it means that this std::move doesn't actually do anything other than obscure what the function's actual behavior is, so we can remove this. To clarify, this would only do something if the parameter was either taking its argument by value, by non-const ref, or by rvalue-reference.
* | Merge pull request #1395 from lioncash/vmbunnei2018-09-297-53/+319
|\ \ | |/ |/| process/vm_manager: Initial modifications to load NPDM metadata
| * memory: Dehardcode the use of fixed memory range constantsLioncash2018-09-254-13/+17
| | | | | | | | | | | | | | | | The locations of these can actually vary depending on the address space layout, so we shouldn't be using these when determining where to map memory or be using them as offsets for calculations. This keeps all the memory ranges flexible and malleable based off of the virtual memory manager instance state.
| * svc: Report correct memory-related values within some of the cases in svcGetInfo()Lioncash2018-09-253-28/+41
| | | | | | | | | | | | Previously, these were reporting hardcoded values, but given the regions can change depending on the requested address spaces, these need to report the values that the memory manager contains.
| * memory: Dehardcode the use of a 36-bit address spaceLioncash2018-09-252-5/+16
| | | | | | | | | | Given games can also request a 32-bit or 39-bit address space, we shouldn't be hardcoding the address space range as 36-bit.
| * process/vm_manager: Amend API to allow reading parameters from NPDM metadataLioncash2018-09-244-10/+248
| | | | | | | | | | | | | | | | | | Rather than hard-code the address range to be 36-bit, we can derive the parameters from supplied NPDM metadata if the supplied exectuable supports it. This is the bare minimum necessary for this to be possible. The following commits will rework the memory code further to adjust to this.
* | Merge pull request #1399 from lioncash/schedbunnei2018-09-262-9/+9
|\ \ | | | | | | kernel/scheduler: Take ARM_Interface instances by reference
| * | kernel/scheduler: Take ARM_Interface instance by reference in the constructorLioncash2018-09-252-9/+9
| |/ | | | | | | | | It doesn't make sense to allow a scheduler to be constructed around a null pointer.
* | Merge pull request #1393 from tech4me/svcbunnei2018-09-251-7/+7
|\ \ | |/ |/| svc: Updated svc names
| * svc: Updated svc namestech4me2018-09-241-7/+7
| |
* | svc: Move most process termination code to its own function within ProcessLioncash2018-09-213-32/+56
| | | | | | | | | | Reduces the use of Process class members externally and keeps most code related to tearing down a process with the rest of the process code.
* | thread/process: Move TLS slot marking/freeing to the process classLioncash2018-09-214-68/+89
| | | | | | | | | | | | Allows making several members of the process class private, it also avoids going through Core::CurrentProcess() just to retrieve the owning process.
* | Merge pull request #1372 from lioncash/threadbunnei2018-09-213-5/+5
|\ \ | | | | | | kernel/thread: Use owner_process when setting the page table in SetupMainThread()
| * | kernel/thread: Use owner_process when setting the page table in SetupMainThread()Lioncash2018-09-213-5/+5
| |/ | | | | | | | | | | | | The owning process of a thread is required to exist before the thread, so we can enforce this API-wise by using a reference. We can also avoid the reliance on the system instance by using that parameter to access the page table that needs to be set.
* / 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.
* Merge pull request #1346 from lioncash/svcbunnei2018-09-191-37/+36
|\ | | | | svc_wrap: Convert the PARAM macro into a function
| * svc_wrap: Convert the PARAM macro into a functionLioncash2018-09-181-37/+36
| | | | | | | | | | | | This can just be a regular function, getting rid of the need to also explicitly undef the define at the end of the file. Given FuncReturn() was already converted into a function, it's #undef can also be removed.
* | Merge pull request #1343 from lioncash/mutexbunnei2018-09-182-2/+10
|\ \ | | | | | | kernel/svc: Handle invalid address cases within svcArbitrateLock() and svcArbitrateUnlock()
| * | 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/svc: Handle error cases for svcArbitrateLock() and svcArbitrateUnlock()Lioncash2018-09-181-0/+8
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kernel does the equivalent of the following check before proceeding: if (address + 0x8000000000 < 0x7FFFE00000) { return ERR_INVALID_MEMORY_STATE; } which is essentially what our IsKernelVirtualAddress() function does. So we should also be checking for this. The kernel also checks if the given input addresses are 4-byte aligned, however our Mutex::TryAcquire() and Mutex::Release() functions already handle this, so we don't need to add code for this case.
* / arm_interface: Remove ARM11-isms from the CPU interfaceLioncash2018-09-181-2/+2
|/ | | | | | | | | This modifies the CPU interface to more accurately match an AArch64-supporting CPU as opposed to an ARM11 one. Two of the methods don't even make sense to keep around for this interface, as Adv Simd is used, rather than the VFP in the primary execution state. This is essentially a modernization change that should have occurred from the get-go.
* Merge pull request #1313 from lioncash/errorbunnei2018-09-171-1/+2
|\ | | | | kernel/errors: Amend error code for ERR_NOT_FOUND
| * kernel/errors: Amend error code for ERR_NOT_FOUNDLioncash2018-09-131-1/+2
| | | | | | | | | | This is the value returned by the kernel for svcConnectToNamedPort() if the named port cannot be found.
* | Merge pull request #1315 from lioncash/sizebunnei2018-09-172-19/+74
|\ \ | | | | | | kernel/svc: Handle a few error cases within memory-related functions
| * | kernel/svc: Sanitize creation of shared memory via svcCreateSharedMemory()Lioncash2018-09-141-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kernel caps the size limit of shared memory to 8589930496 bytes (or (1GB - 512 bytes) * 8), so approximately 8GB, where every GB has a 512 byte sector taken off of it. It also ensures the shared memory is created with either read or read/write permissions for both permission types passed in, allowing the remote permissions to also be set as "don't care".
| * | kernel/svc: Sanitize addresses, permissions, and sizes within svcMapSharedMemory() and svcUnmapSharedMemory()Lioncash2018-09-141-17/+25
| | | | | | | | | | | | | | | | | | | | | | | | Part of the checking done by the kernel is to check if the given address and size are 4KB aligned, as well as checking if the size isn't zero. It also only allows mapping shared memory as readable or read/write, but nothing else, and so we shouldn't allow mapping as anything else either.
| * | kernel/svc: Sanitize addresses and sizes within svcMapMemory() and svcUnmapMemory()Lioncash2018-09-141-0/+23
| | | | | | | | | | | | | | | The kernel checks if the addresses and given size is 4KB aligned before continuing onwards to map the memory.
| * | kernel/svc: Sanitize heap sizes within svcSetHeapSize()Lioncash2018-09-142-0/+8
| |/ | | | | | | | | The kernel checks if the given size is a multiple of 2MB and <= to 4GB before going ahead and attempting to allocate that much memory.
* | Merge pull request #1328 from FearlessTobi/port-4192bunnei2018-09-171-1/+1
|\ \ | | | | | | Port #4192 from Citra: "svc: change unknown to thread in CreateThread"
| * | Port # #4192 from Citra: "svc: change unknown to thread in CreateThread"Valentin Vanelslande2018-09-151-1/+1
| | |
* | | Port #4182 from Citra: "Prefix all size_t with std::"fearlessTobi2018-09-1514-62/+65
|/ /
* / kernel/thread: Include thread-related enums within the kernel namespaceLioncash2018-09-131-6/+6
|/ | | | | | Previously, these were sitting outside of the Kernel namespace, which doesn't really make sense, given they're related to the Thread class which is within the Kernel namespace.
* Merge pull request #1303 from lioncash/errorbunnei2018-09-123-9/+11
|\ | | | | kernel/errors: Amend invalid thread priority and invalid processor ID error codes
| * svc: Return ERR_INVALID_PROCESSOR_ID in CreateThread() if an invalid processor ID is givenLioncash2018-09-121-2/+2
| | | | | | | | This is what the kernel does for an out-of-range processor ID.
| * kernel/errors: Correct error codes for invalid thread priority and invalid processor IDLioncash2018-09-123-7/+9
| |
* | svc: Do nothing if svcOutputDebugString() is given a length of zeroLioncash2018-09-121-0/+4
| | | | | | | | | | | | While unlikely, it does avoid constructing a std::string and unnecessarily calling into the memory code if a game or executable decides to be really silly about their logging.
* | svc: Correct parameter type for OutputDebugString()Lioncash2018-09-122-3/+3
|/ | | | This should be a u64 to represent size.
* core: Migrate current_process pointer to the kernelLioncash2018-09-072-0/+23
| | | | | | | | | | Given we now have the kernel as a class, it doesn't make sense to keep the current process pointer within the System class, as processes are related to the kernel. This also gets rid of a subtle case where memory wouldn't be freed on core shutdown, as the current_process pointer would never be reset, causing the pointed to contents to continue to live.
* service: Migrate global named port map to the KernelCore classLioncash2018-09-023-5/+49
| | | | | | Now that we have a class representing the kernel in some capacity, we now have a place to put the named port map, so we move it over and get rid of another piece of global state within the core.
* core/core: Replace includes with forward declarations where applicableLioncash2018-08-315-4/+12
| | | | | | | | | | | The follow-up to e2457418dae19b889b2ad85255bb95d4cd0e4bff, which replaces most of the includes in the core header with forward declarations. This makes it so that if any of the headers the core header was previously including change, then no one will need to rebuild the bulk of the core, due to core.h being quite a prevalent inclusion. This should make turnaround for changes much faster for developers.
* kernel: Eliminate kernel global stateLioncash2018-08-2932-405/+574
| | | | | | | | | | | | | | | | | | | | | | 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.
* Merge pull request #1177 from lioncash/errbunnei2018-08-284-12/+15
|\ | | | | kernel/error: Amend several error codes
| * kernel/error: Amend error code for ERR_MAX_CONNECTIONS_REACHEDLioncash2018-08-251-2/+4
| | | | | | | | | | | | We can make this error code an alias of the resource limit exceeded error code, allowing us to get rid of the lingering 3DS error code of the same type.
| * kernel/error: Amend error code for ERR_PORT_NAME_TOO_LONGLioncash2018-08-251-2/+1
| | | | | | | | | | We can treat this as an alias of TooLarge for documentation purposes. This also lets us get rid of another lingering 3DS-related error code.
| * kernel/error: Add error code for the handle table being fullLioncash2018-08-253-4/+4
| | | | | | | | | | This replaces the lingering 3DS constant with the proper one, and utilizes it within HandleTable's Create() member function.
| * kernel/error: Add error code for invalid memory permissionsLioncash2018-08-252-3/+4
| |
| * kernel/error: Correct kernel error code for invalid combinationLioncash2018-08-251-1/+2
| |
* | Merge pull request #1175 from lioncash/nsbunnei2018-08-284-6/+8
|\ \ | | | | | | core: Namespace all code in the arm subdirectory under the Core namespace
| * | core: Namespace all code in the arm subdirectory under the Core namespaceLioncash2018-08-254-6/+8
| |/ | | | | | | Gets all of these types and interfaces out of the global namespace.
* / 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.
* Merge pull request #1051 from B3n30/UnscheduleEventThreadsafebunnei2018-08-161-1/+1
|\ | | | | Core::CoreTiming: add UnscheduleEventThreadsafe
| * Core::CoreTiming: add UnscheduleEventThreadsafeB3n302018-08-131-1/+1
| |
* | kernel/server_session: Add IsSession() member functionLioncash2018-08-152-2/+7
| | | | | | | | | | | | Allows querying the inverse of IsDomain() to make things more readable. This will likely also be usable in the event of implementing ConvertDomainToSession().
* | 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.
* kernel/object: Tighten object against data racesLioncash2018-08-132-8/+9
| | | | | | | | | | Despite being covered by a global mutex, we should still ensure that the class handles its reference counts properly. This avoids potential shenanigans when it comes to data races. Given this is the root object that drives quite a bit of the kernel object hierarchy, ensuring we always have the correct behavior (and no races) is a good thing.
* 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.
* | Merge pull request #1036 from lioncash/threadbunnei2018-08-132-2/+2
|\ \ | | | | | | scheduler: Make HaveReadyThreads() a const member function
| * | scheduler: Make HaveReadyThreads() a const member functionLioncash2018-08-122-2/+2
| |/ | | | | | | | | This function doesn't modify instance state, so the const qualifier can be added to it.
* | Merge pull request #1042 from Subv/racesbunnei2018-08-131-2/+9
|\ \ | | | | | | Fixed a bunch of race conditions when running in multicore mode.
| * | Kernel/Threads: Lock the HLE mutex when executing the wakeup callback.Subv2018-08-131-0/+5
| | | | | | | | | | | | Another thread might be in the middle of a reschedule, thus altering the state of the schedulers.
| * | Kernel/Thread: Always use the threadsafe option when scheduling wakeups.Subv2018-08-131-2/+4
| |/ | | | | | | | | | | WakeAfterDelay might be called from any host thread, so err on the side of caution and use the thread-safe CoreTiming::ScheduleEventThreadsafe. Note that CoreTiming is still far from thread-safe, there may be more things we have to work on for it to be up to par with what we want.
* / Kernel/Mutex: Don't duplicate threads in the mutex waiter list.Subv2018-08-122-2/+22
|/ | | | | | | | | | 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.
* server_session: Provide more useful information and don't crash on bad IPC request.bunnei2018-08-121-0/+8
|
* hle_ipc: Make WriteToOutgoingCommandBuffer()'s reference parameter constLioncash2018-08-092-2/+2
| | | | | This function doesn't modify anything within the reference Thread instance.
* client_port: Make all data members privateLioncash2018-08-073-7/+21
| | | | | | | | These members don't need to be entirely exposed, we can instead expose an API to operate on them without directly needing to mutate them We can also guard against overflow/API misuse this way as well, given active_sessions is an unsigned value.
* kernel/event: Make data members privateLioncash2018-08-061-4/+8
| | | | | Instead we can simply provide accessors to the required data instead of giving external read/write access to the variables directly.
* address_arbiter: Return by value from GetThreadsWaitingOnAddress()Lioncash2018-08-051-15/+15
| | | | | In all cases the vector being supplied is empty, so we can just return by value in these instances.
* Merge pull request #914 from lioncash/codesetbunnei2018-08-042-15/+36
|\ | | | | kernel/process: Use accessors instead of class members for referencing segment array
| * kernel/process: Use std::array where applicableLioncash2018-08-031-1/+2
| |
| * kernel/process: Use accessors instead of class members for referencing segment arrayLioncash2018-08-032-15/+35
| | | | | | | | | | | | | | Using member variables for referencing the segments array increases the size of the class in memory for little benefit. The same behavior can be achieved through the use of accessors that just return the relevant segment.
* | kernel/thread: Fix potential crashes introduced in 26de4bb521b1ace7af76eff4f6956cb23ac0d58cLioncash2018-08-043-13/+38
|/ | | | | | This amends cases where crashes can occur that were missed due to the odd way the previous code was set up (using 3DS memory regions that don't exist).
* core/memory: Get rid of 3DS leftoversLioncash2018-08-039-302/+24
| | | | Removes leftover code from citra that isn't needed.
* Merge pull request #894 from lioncash/objectbunnei2018-08-0329-136/+164
|\ | | | | kernel: Move object class to its own source files
| * kernel: Move object class to its own source filesLioncash2018-08-0229-136/+164
| | | | | | | | | | | | 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 #904 from lioncash/staticbunnei2018-08-031-8/+6
|\ \ | | | | | | kernel/thread: Minor changes
| * | kernel/thread: Make GetFreeThreadLocalSlot()'s loop indices size_tLioncash2018-08-021-8/+5
| | | | | | | | | | | | | | | | | | Avoids using a u32 to compare against a range of size_t, which can be a source of warnings. While we're at it, compress a std::tie into a structured binding.
| * | kernel/thread: Make GetFreeThreadLocalSlot() reference parameter a const referenceLioncash2018-08-021-1/+2
| | | | | | | | | | | | | | | This function only reads the data being referenced, it doesn't modify it, so we can turn the reference into a const reference.
| * | kernel/thread: Make GetFreeThreadLocalSlot() internally linkedLioncash2018-08-021-1/+1
| |/ | | | | | | | | This function isn't used outside of this translation unit, so we can make it internally linked.
* | kernel/vm_manager: Convert loop into std::any_of()Lioncash2018-08-021-4/+4
| |
* | kernel/vm_manager: Use const where applicableLioncash2018-08-023-19/+19
| | | | | | | | Makes our immutable state explicit.
* | kernel/vm_manager: Use the VAddr type alias in CarveVMA()Lioncash2018-08-021-2/+2
|/ | | | These two variables correspond to address ranges.
* kernel/thread: Remove unimplemented function prototypeLioncash2018-08-011-6/+0
| | | | | Given there's no implementation, we may as well remove the code entirely.
* Merge pull request #877 from lioncash/removebunnei2018-08-015-102/+0
|\ | | | | kernel: Remove unused object_address_table.cpp/.h
| * kernel: Remove unused object_address_table.cpp/.hLioncash2018-07-315-102/+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-3122-27/+46
|/ | | | | Removes unnecessary direct dependencies in some headers and also gets rid of indirect dependencies that were being relied on to be included.
* kernel/timer: Make data members private where applicableLioncash2018-07-261-5/+17
| | | | | Instead, we can just expose functions that return the queryable state instead of letting anything modify it.
* 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.
* | Merge pull request #803 from MerryMage/core_timing_utilbunnei2018-07-252-0/+2
|\ \ | | | | | | core_timing: Split off utility functions into core_timing_util
| * | core_timing: Split off utility functions into core_timing_utilMerryMage2018-07-242-0/+2
| | |
* | | 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.
* | Merge pull request #794 from lioncash/refbunnei2018-07-241-1/+1
|\ \ | |/ |/| mutex: Pass SharedPtr to GetHighestPriorityMutexWaitingThread() by reference
| * 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.
* | hle_ipc: Make constructors explicit where applicableLioncash2018-07-241-1/+1
|/
* hle: Remove config_mem.h/.cppLioncash2018-07-233-13/+0
| | | | | This is just an unused hold-over from citra, so we can get rid of this to trim off an exposed global, among other things.
* hle: Remove shared_page.h/.cppLioncash2018-07-233-4/+0
| | | | This is a holdover from citra that's essentially unused.
* Kernel/SVC: Perform atomic accesses in SignalProcessWideKey as per the real kernel.Subv2018-07-221-7/+31
|
* Merge pull request #751 from Subv/tpidr_el0bunnei2018-07-213-0/+13
|\ | | | | CPU: Save and restore the TPIDR_EL0 system register on every context switch
| * CPU: Save and restore the TPIDR_EL0 system register on every context switch.Subv2018-07-213-0/+13
| | | | | | | | Note that there's currently a dynarmic bug preventing this register from being written.
* | thread: Convert ThreadStatus into an enum classLioncash2018-07-209-71/+70
|/ | | | | Makes the thread status strongly typed, so implicit conversions can't happen. It also makes it easier to catch mistakes at compile time.
* Merge pull request #726 from lioncash/overloadbunnei2018-07-202-6/+21
|\ | | | | hle_ipc: Introduce generic WriteBuffer overload for multiple container types
| * hle_ipc: Introduce generic WriteBuffer overload for multiple container typesLioncash2018-07-192-6/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces a slightly more generic variant of WriteBuffer(). Notably, this variant doesn't constrain the arguments to only accepting std::vector instances. It accepts whatever adheres to the ContiguousContainer concept in the C++ standard library. This essentially means, std::array, std::string, and std::vector can be used directly with this interface. The interface no longer forces you to solely use containers that dynamically allocate. To ensure our overloads play nice with one another, we only enable the container-based WriteBuffer if the argument is not a pointer, otherwise we fall back to the pointer-based one.
* | Merge pull request #721 from lioncash/svcbunnei2018-07-201-3/+4
|\ \ | |/ |/| svc: Correct always true assertion case in SetThreadCoreMask
| * 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.
* | hle_ipc: Amend usage of buffer_index within one of HLERequestContext's WriteBuffer() overloadsLioncash2018-07-191-1/+1
|/ | | | | | | | | Previously, the buffer_index parameter was unused, causing all writes to use the buffer index of zero, which is not necessarily what is wanted all the time. Thankfully, all current usages don't use a buffer index other than zero, so this just prevents a bug before it has a chance to spring.
* Merge pull request #692 from lioncash/assignbunnei2018-07-191-1/+1
|\ | | | | address_arbiter: Correct assignment within an assertion statement in WakeThreads()
| * address_arbiter: Correct assignment within an assertion statement in WakeThreads()Lioncash2018-07-191-1/+1
| | | | | | | | | | | | This was introduced within 4f81bc4e1bd12e4df7410c6790ba818d8dbba9c0, and considering there's no comment indicating that this is intentional, this is very likely a bug.
* | Merge pull request #690 from lioncash/movebunnei2018-07-198-13/+21
|\ \ | | | | | | core/memory, core/hle/kernel: Use std::move where applicable
| * | core/memory, core/hle/kernel: Use std::move where applicableLioncash2018-07-198-13/+21
| |/ | | | | | | Avoids pointless copies
* | Merge pull request #688 from lioncash/commabunnei2018-07-191-22/+12
|\ \ | | | | | | vm_manager: Add missing commas to string literal array elements in GetMemoryStateName()
| * | vm_manager: Add missing commas to string literal array elements in GetMemoryStateName()Lioncash2018-07-191-22/+12
| |/ | | | | | | | | Without these, this would perform concatenation, which is definitely not what we want here.
* / core: Don't construct instance of Core::System, just to access its live instanceLioncash2018-07-191-6/+6
|/ | | | | | | | | This would result in a lot of allocations and related object construction, just to toss it all away immediately after the call. These are definitely not intentional, and it was intended that all of these should have been accessing the static function GetInstance() through the name itself, not constructed instances.
* Merge pull request #671 from MerryMage/clear-exclusive-statebunnei2018-07-171-0/+1
|\ | | | | scheduler: Clear exclusive state when switching contexts
| * scheduler: Clear exclusive state when switching contextsMerryMage2018-07-161-0/+1
| |
* | Merge pull request #672 from SciresM/to_address_fixbunnei2018-07-171-2/+4
|\ \ | | | | | | svc:: Fix bug in svcWaitForAddress
| * | Kernel/Arbiter: Fix bug in WaitIfLessThanMichael Scire2018-07-171-2/+4
| |/
* / nvflinger: Fix for BufferQueue event handling.bunnei2018-07-172-3/+11
|/
* No need to use ASSERT_MSG with an empty messageDavid Marcec2018-07-141-2/+2
|
* Update clang formatJames Rowe2018-07-036-38/+35
|
* Rename logging macro back to LOG_*James Rowe2018-07-0311-79/+79
|
* Merge pull request #579 from SciresM/masterbunnei2018-06-228-9/+295
|\ | | | | svc: Fully implement svcSignalToAddress and svcWaitForAddress
| * Kernel/Arbiters: Fix casts, cleanup comments/magic numbersMichael Scire2018-06-222-17/+16
| |
| * Add additional missing format.Michael Scire2018-06-222-21/+27
| |
| * Run clang-format on PR.Michael Scire2018-06-223-180/+181
| |
| * Kernel/Arbiters: HLE is atomic, adjust code to reflect that.Michael Scire2018-06-222-37/+13
| |
| * Kernel/Arbiters: Initialize arb_wait_address in thread struct.Michael Scire2018-06-212-1/+5
| |
| * Kernel/Arbiters: Clear WaitAddress in SignalToAddressMichael Scire2018-06-211-0/+1
| |
| * Kernel/Arbiters: Mostly implement SignalToAddressMichael Scire2018-06-214-10/+110
| |
| * Kernel/Arbiters: Implement WaitForAddressMichael Scire2018-06-214-6/+67
| |
| * Kernel/Arbiters: Add stubs for 4.x SignalToAddress/WaitForAddres SVCs.Michael Scire2018-06-217-9/+147
| |
* | IPC: skip empty buffer writemailwl2018-06-221-0/+5
|/ | | | prevent yuzu crash, if games, like Axiom Verge, trying to read 0 bytes from file
* 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/Threads: A thread waking up by timeout from a WaitProcessWideKey may already have an assigned lock owner.Subv2018-06-021-2/+5
| | | | | | | | | | | | | | | | | | | | This situation may happen like so: Thread 1 with low priority calls WaitProcessWideKey with timeout. Thread 2 with high priority calls WaitProcessWideKey without timeout. Thread 3 calls SignalProcessWideKey - Thread 2 acquires the lock and awakens. - Thread 1 can't acquire the lock and is put to sleep with the lock owner being Thread 2. Thread 1's timeout expires, with the lock owner still being set to Thread 2.
* | Kernel/Thread: Corrected a typo that caused the affinity mask to never be changed.Subv2018-05-311-2/+2
| |
* | Kernel/SVC: Support special core values -2 and -3 in svcSetThreadCoreMask.Subv2018-05-312-1/+28
| | | | | | | | Also added some proper error handling.
* | Kernel/Thread: Corrected a typo in an assert about the processor id.Subv2018-05-301-1/+1
| |
* | Add & correct miscellaneous things (#470)greggameplayer2018-05-261-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | * add some InfoType * correct OpenApplicationProxy cmd number * add IDisplayController functions * fix clang-format * add more system languages
* | Merge pull request #454 from Subv/signal_processwidebunnei2018-05-231-83/+74
|\ \ | | | | | | Kernel/SVC: Signal the highest priority threads first in svcSignalProcessWideKey
| * | 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
| | |
* | | Merge pull request #457 from Subv/mutex_waitersbunnei2018-05-211-1/+0
|\ \ \ | | | | | | | | Mutex: Do not assert when the mutex waiting threads list isn't empty on mutex release.
| * | | 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.
* / / Added RequestWithContext & ControlWithContextDavid Marcec2018-05-171-1/+3
|/ /
* | thread: Rename mask to affinity_masks.bunnei2018-05-113-4/+4
| |
* | thread: Support core change on ResumeFromWait and improve ChangeCore.bunnei2018-05-111-37/+68
| |
* | scheduler: Protect scheduling functions with a global mutex.bunnei2018-05-112-0/+18
| |
* | thread: Initialize ideal_core and mask members.bunnei2018-05-111-0/+2
| |
* | threading: Reschedule only on cores that are necessary.bunnei2018-05-112-3/+3
| |
* | svc: Implement GetThreadCoreMask and SetThreadCoreMask.bunnei2018-05-111-7/+22
| |
* | thread: Implement ChangeCore function.bunnei2018-05-112-1/+58
| |
* | 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-115-45/+65
| |
* | Merge pull request #431 from lioncash/fmtbunnei2018-05-026-49/+50
|\ \ | | | | | | general: Make formatting of logged hex values more straightforward
| * | general: Make formatting of logged hex values more straightforwardLioncash2018-05-026-49/+50
| | | | | | | | | | | | | | | | | | 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).
* | | ipc: Add support for PopIpcInterface() method.bunnei2018-05-022-0/+15
|/ / | | | | | | - This can be used for domain objects as inputs to service functions.
* | GetSharedFontInOrderOfPriority (#381)David2018-05-012-23/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * GetSharedFontInOrderOfPriority * Update pl_u.cpp * Ability to use ReadBuffer and WriteBuffer with different buffer indexes, fixed up GetSharedFontInOrderOfPriority * switched to NGLOG * Update pl_u.cpp * Update pl_u.cpp * language_code is actually language code and not index * u32->u64 * final cleanups
* | core_timing: Namespace all functions and constants in core_timing's headerLioncash2018-04-302-3/+5
| | | | | | | | All of these variables and functions are related to timings and should be within the namespace.
* | 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
* | core: Replace usages of LOG_GENERIC with new fmt-capable equivalentsLioncash2018-04-273-6/+4
| |
* | general: Convert assertion macros over to be fmt-compatibleLioncash2018-04-274-9/+9
| |
* | kernel/shared_memory: Remove unnecessary semicolon at end of ConvertPermissions()Lioncash2018-04-261-1/+1
| | | | | | | | Functions don't need to be terminated by semicolons.
* | kernel: Migrate logging macros to fmt-compatible onesLioncash2018-04-2611-106/+109
| |
* | Merge pull request #370 from Subv/sync_primitivesbunnei2018-04-239-435/+228
|\ \ | | | | | | Kernel: Reworked the new kernel synchronization primitives.
| * | Kernel: Implemented mutex priority inheritance.Subv2018-04-234-10/+94
| | | | | | | | | | | | | | | | | | | | | 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: Use 0x2C as default main thread priority for homebrew and lone NRO/NSOsSubv2018-04-211-1/+1
| | |
| * | Qt: Update the WaitTree widget to show info about the current mutex of each thread.Subv2018-04-211-4/+0
| | |
| * | Kernel: Remove unused ConditionVariable class.Subv2018-04-215-148/+0
| | |
| * | Kernel: Remove old and unused Mutex code.Subv2018-04-214-209/+3
| | |
| * | 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-216-22/+126
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-213-38/+38
|/ / | | | | | | 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
* | vm_manager: Increase GetTotalMemoryUsage value.bunnei2018-04-151-1/+1
| | | | | | | | - Gets Binding of Isaac running.
* | svc: Stub out SetThreadActivity, GetThreadContext.bunnei2018-04-032-2/+19
| |
* | shared_memory: Remove incorrect 3ds-specific check.bunnei2018-04-031-12/+0
| |
* | hle_ipc, fsp_srv: Cleanup logging.bunnei2018-04-011-1/+1
| |
* | hle_ipc: Do not ensure write buffer size.bunnei2018-03-311-2/+5
| |
* | memory: Fix stack region.bunnei2018-03-312-3/+4
| |
* | svc: Stub GetThreadCoreMask.bunnei2018-03-302-3/+26
| |
* | Clang FixesN00byKing2018-03-192-5/+5
| |
* | More Warning cleanupsN00byKing2018-03-192-2/+2
| |
* | Clean Warnings (?)N00byKing2018-03-195-8/+8
| |
* | hle_ipc: Add SleepClientThread to block current thread within HLE routines.bunnei2018-03-192-0/+47
| |
* | hle_ipc: Use shared_ptr instead of unique_ptr to allow copies.bunnei2018-03-192-9/+9
| |
* | hle_ipc: Remove GetPointer(..) usage with WriteToOutgoingCommandBuffer.bunnei2018-03-192-5/+13
| |
* | thread: Add THREADSTATUS_WAIT_HLE_EVENT, remove THREADSTATUS_WAIT_ARB.bunnei2018-03-193-20/+6
| |
* | process: MirrorMemory should use MemoryState::Mapped.bunnei2018-03-171-1/+1
| |
* | process: Unmap previously allocated heap.bunnei2018-03-161-1/+3
| |
* | arm_interface: Support unmapping previously mapped memory.bunnei2018-03-161-0/+3
| |
* | svc: Use more correct values for GetInfo MapRegion and NewMapRegion.bunnei2018-03-163-29/+5
| |
* | kernel: Move stack region outside of application heap.bunnei2018-03-163-8/+3
| |
* | process: Fix stack memory state.bunnei2018-03-161-2/+4
| |
* | MemoryState: Add additional memory states and improve naming.bunnei2018-03-165-18/+45
| |
* | core: Move process creation out of global state.bunnei2018-03-149-33/+36
| |
* | Merge pull request #216 from Subv/savedatabunnei2018-03-022-9/+8
|\ \ | | | | | | Implemented the SaveData archive and MountSaveData.
| * | Kernel: Store the program id in the Process class instead of the CodeSet class.Subv2018-03-022-9/+8
| |/ | | | | | | There may be many CodeSets per Process, so it's wasteful and overcomplicated to store the program id in each of them.
* / thread: Clear the process list on shutdown.Jules Blok2018-02-271-1/+3
|/
* Merge pull request #207 from mailwl/duplicatesessionbunnei2018-02-272-5/+10
|\ | | | | IPC: add domain header to response if only it exists in request
| * Add warning if Domain request has no domain message headermailwl2018-02-201-0/+3
| |
| * Fix: change check for domain order and existance of domain message headermailwl2018-02-202-2/+2
| |
| * IPC: add domain header to response if only it exists in requestmailwl2018-02-202-5/+7
| |
* | Merge pull request #215 from N00byKing/umapsharedmmrybunnei2018-02-262-1/+17
|\ \ | | | | | | UnmapSharedMemory
| * | (Hopefully) Fix MinGW BuildN00byKing2018-02-251-1/+1
| | |
| * | Add UnmapSharedMemoryN00byKing2018-02-252-1/+17
| |/ | | | | | | | | | | | | | | C++11 requires spaces on the Identifier Add inttypes include clang
* / Stub more functionsmailwl2018-02-221-1/+11
|/
* scheduler: Cleanup based on PR feedback.bunnei2018-02-192-4/+3
|
* kernel: Use Scheduler class for threading.bunnei2018-02-183-172/+16
|
* kernel: Add Scheduler, which encapsulates the scheduling loading from Thread module.bunnei2018-02-182-0/+208
|
* kernel: Remove unused address_arbiter code.bunnei2018-02-184-197/+0
|
* Kernel/IPC: Add a small delay after each SyncRequest to prevent thread starvation.Subv2018-02-184-50/+63
| | | | | | | | Ported from citra PR #3091 The delay specified here is from a Nintendo 3DS, and should be measured in a Nintendo Switch. This change is enough to prevent Puyo Puyo Tetris's main thread starvation.
* Service/hid: stub some functionsmailwl2018-02-162-0/+53
|
* shared_memory: Remove some checks.bunnei2018-02-151-13/+0
|
* Merge pull request #188 from bunnei/refactor-buffer-descriptorbunnei2018-02-152-0/+59
|\ | | | | Refactor IPC buffer descriptor interface
| * hle_ipc: Remove const from WriteBuffer size.bunnei2018-02-142-2/+2
| |
| * hle_ipc: Add GetReadBufferSize and check write buffer size.bunnei2018-02-142-0/+10
| |
| * hle_ipc: Add helper functions for reading and writing buffers.bunnei2018-02-141-4/+2
| |
| * hle_ipc: Add helper functions for reading and writing buffers.bunnei2018-02-142-0/+51
| |
* | thread: Silence formatting specifier warningsLioncash2018-02-141-2/+3
| |
* | vm_manager: Silence formatting specifier warningsLioncash2018-02-141-5/+7
|/
* mutex: Update hasWaiters on release.bunnei2018-02-061-0/+1
|
* GetInfo: Implement IsCurrentProcessBeingDebugged.bunnei2018-02-041-0/+3
|
* WaitProcessWideKeyAtomic: Handle case where condition variable was already created.bunnei2018-02-043-13/+17
|
* svc: SharedMemory size should be 64-bits and cleanup.bunnei2018-02-033-11/+11
|
* ArbitrateLock: Assert that requesting_thread is current_thread.bunnei2018-02-031-0/+1
|
* shared_memory: Only mark addresses as invalid if they are within the heapshinyquagsire232018-01-301-1/+2
|
* Merge pull request #148 from MerryMage/feature/special-memorybunnei2018-01-272-6/+6
|\ | | | | memory: Replace all memory hooking with Special regions
| * memory: Replace all memory hooking with Special regionsMerryMage2018-01-272-6/+6
| |
* | server_session: Fix scenario where all domain handlers are closed.bunnei2018-01-251-3/+3
| |
* | hle: Rename RequestBuilder to ResponseBuilder.bunnei2018-01-251-1/+1
| |
* | ipc_helpers: Make interface domain agnostic and add header validation.bunnei2018-01-251-1/+12
| |
* | hle: Integrate Domain handling into ServerSession.bunnei2018-01-255-28/+66
| |
* | hle: Remove Domain and SyncObject kernel objects.bunnei2018-01-257-164/+2
| |
* | handle_table: Remove ConvertSessionToDomain.bunnei2018-01-252-17/+0
|/
* Format: Run the new clang format on everythingJames Rowe2018-01-2122-31/+32
|
* Added CreateSharedMemory & UNIMPLEMENTED() for non existent services. (#113)David2018-01-202-1/+22
| | | | | | | | | | | | * Added svcCreateSharedMemory * Services which are not implemented now throw UNIMPLEMENTED() * clang-format * changed perms to u32 * removed camelcase
* Fixes some cast warnings, partial port of citra #3064 (#106)River City Ransomware2018-01-202-2/+2
| | | | | | | | * Fixes some cast warnings, partially fixes citra #3064 * Converted casts to uint32_t to u32 * Ran clang-format
* svc: Fix svcGetInfo MapRegionBaseAddr.bunnei2018-01-193-1/+9
|
* svc: Add additional fields to MemoryInfo struct.bunnei2018-01-191-0/+4
|
* Stub PopLaunchParameter and implement Buffer C Descriptors reading on hle_ipc (#96)gdkchan2018-01-182-7/+33
| | | | | | | | | | * Stub PopLaunchParameter and implement Buffer C Descriptors reading * Address PR feedback * Ensure we push a u64 not a size_t * Fix formatting
* 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
|
* hle_ipc: Clang format.bunnei2018-01-171-2/+3
|
* ipc: Implement domain command CloseVirtualHandle.bunnei2018-01-172-2/+28
|
* 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
* SVC: Add 4.0.0+ comment to GetInfoType enum values.Subv2018-01-171-0/+1
|
* IPC: Push domain objects as move handles when not in a domain.Subv2018-01-171-0/+8
|
* Merge pull request #52 from ogniK5377/fspbunnei2018-01-174-3/+35
|\ | | | | 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-164-2/+29
| |
* | clang-formatMerryMage2018-01-166-18/+14
|/
* shared_memory: Minor fixes and cleanup.bunnei2018-01-141-6/+6
|
* svc: Implement svcMapSharedMemory.bunnei2018-01-142-1/+38
|
* kernel: Increase default stack size to 64K.bunnei2018-01-141-1/+1
|
* yuzu: Update license text to be consistent across project.bunnei2018-01-1314-14/+14
|
* core: Include <algorithm> where used.bunnei2018-01-121-0/+2
|
* core: Fix recent GCC build breaks.bunnei2018-01-121-2/+2
|
* svc: Implement GetSystemTick.bunnei2018-01-122-2/+21
|
* IPC: Corrected some definitions for the buffer C descriptor flags.Subv2018-01-112-1/+8
|
* svc: Stub ResetSignal and CreateTransferMemorySubv2018-01-112-3/+28
|
* svc: Stub SetMemoryAttributeSubv2018-01-112-0/+11
|
* Threads: Added enum values for the Switch's 4 cpu cores and implemented svcGetInfo(AllowedCpuIdBitmask)Subv2018-01-104-10/+25
|
* SVC: Fixed WaitSynchronization with multiple handles when none is immediately ready.Subv2018-01-091-7/+18
|
* SVC: Implemented CancelSynchronization.Subv2018-01-092-1/+19
|
* ErrorCodes: Updated the InvalidHandle and Timeout kernel error codes.Subv2018-01-091-2/+7
|
* SVC: Fixed WaitSynchronization with multiple handles when at least one of them is ready.Subv2018-01-092-3/+29
|
* kernel: Rename Semaphore to ConditionVariable.bunnei2018-01-098-159/+167
|
* mutex: Remove unused call to VerifyGuestState.bunnei2018-01-091-3/+0
|
* Kernel: Actually wake up the requested number of threads in Semaphore::Release.Subv2018-01-093-18/+16
| | | | | | 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-093-63/+60
|
* Kernel: Allow chaining WaitSynchronization calls inside a wakeup callback.Subv2018-01-094-30/+78
|
* CoreTiming: Reworked CoreTiming (cherry-picked from Citra #3119)B3n302018-01-092-9/+5
| | | | * CoreTiming: New CoreTiming; Add Test for CoreTiming
* IPC: Add functions to read the input move/copy objects from an IPC request.Subv2018-01-072-2/+26
|
* IPC: Don't attempt to read the command buffer if it holds a Close request.Subv2018-01-071-0/+5
|
* IPC Cleanup: Remove 3DS-specific code and translate copy, move and domain objects in IPC requests.Subv2018-01-072-74/+72
| | | | Popping objects from the buffer is still not implemented.
* IPC: Skip the entire u64 of the command id when receiving an IPC request.Subv2018-01-071-1/+2
| | | | Service code now doesn't have to deal with this.
* IPC: Use the correct size when pushing raw data to the command buffer and fixed pushing domain objects.Subv2018-01-072-3/+7
| | | | Domain object ids are always stored immediately after the raw data.
* svc: Implement svcSignalProcessWideKey.bunnei2018-01-072-4/+23
|
* semaphore: More changes for Switch.bunnei2018-01-072-11/+17
|
* wait_object: Refactor to allow waking up a single thread.bunnei2018-01-072-15/+28
|
* svc: Implement svcWaitProcessWideKeyAtomic.bunnei2018-01-062-1/+54
|
* semaphore: Updates for Switch.bunnei2018-01-062-21/+31
|
* 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-032-23/+11
|
* vm_manager: Use a more reasonable MAX_ADDRESS size.bunnei2018-01-031-5/+4
|
* svc: Remove unnecessary "svc" prefix to naming scheme.bunnei2018-01-031-106/+106
|
* hle: Move SVC code to kernel namespace.bunnei2018-01-033-0/+805
|
* vm_manager: Stub out a bunch of interfaces used by svcGetInfo.bunnei2018-01-012-1/+51
|
* core/video_core: Fix a bunch of u64 -> u32 warnings.bunnei2018-01-011-2/+2
|
* svc: Implement svcExitProcess.bunnei2018-01-012-7/+38
|
* svc: Implement svcLockMutex.bunnei2018-01-012-23/+95
|
* kernel: Add ObjectAddressTable class.bunnei2018-01-013-2/+101
|
* thread: Keep track of the initially created handle.bunnei2017-12-312-1/+5
| | | | This is kinda crufty, but we need it for now to update guest state variables.
* errors: Define missing kernel error codes.bunnei2017-12-311-0/+3
|
* svc: Implement svcUnmapMemory.bunnei2017-12-312-0/+7
|
* thread: Main thread should set thread handle to reg 1.bunnei2017-12-301-1/+4
|
* thread: Remove THUMB mode flag.bunnei2017-12-301-1/+1
|
* thread: Main thread should be ready by default, all others dormant.bunnei2017-12-301-4/+3
|
* kernel: Various 64-bit fixes in memory/process/threadbunnei2017-12-295-14/+14
|
* kernel: Fix implementation of ConvertSessionToDomain.bunnei2017-12-296-30/+67
|
* kernel: Add basic support for Domain object.bunnei2017-12-294-4/+110
|
* kernel: Add SyncObject primitive, use it for ClientSession.bunnei2017-12-292-8/+38
|
* process: Add method to mirror a memory region.bunnei2017-12-292-0/+27
|
* service: Return proper result code for IPC::CommandType::Close.bunnei2017-11-012-6/+7
|
* hle: Use Switch formatted result codes.bunnei2017-11-012-77/+22
|
* hle: Fix QueryMemory response for MemoryInfo.bunnei2017-10-204-115/+14
|
* hle_ipc: Only copy necessary fields for outgoing command buffer.bunnei2017-10-191-1/+1
|
* hle_ipc: Parse out buffer X/A/B/B descriptors from incoming command buffer.bunnei2017-10-192-14/+19
|
* hle: Implement ConvertSessionToDomain, various cleanups.bunnei2017-10-153-5/+31
|
* hle: Initial implementation of NX service framework and IPC.bunnei2017-10-152-70/+107
|
* Merge remote-tracking branch 'upstream/master' into nxbunnei2017-10-1018-103/+143
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # 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
| * Merge pull request #2977 from Subv/shmem_createbunnei2017-10-031-15/+12
| |\ | | | | | | SharedMemory: Don't take over and unmap the source memory block when creating a shared memory, just reference it
| | * Kernel/SharedMemory: Don't take over and unmap the source memory block when creating a shared memory, just reference it.Subv2017-10-021-15/+12
| | | | | | | | | | | | Also reference the right offset into the backing block for the requested address.
| * | Merge pull request #2971 from Subv/per_process_memopsSebastian Valle2017-10-012-0/+12
| |\ \ | | |/ | |/| Memory: Add overloads for ReadBlock and WriteBlock that operate on a specific process.
| | * Kernel/Thread: Added a helper function to get a thread's command buffer VAddr.Subv2017-10-012-0/+12
| | |
| * | Merge pull request #2967 from Subv/thread_wakeup_callbacksSebastian Valle2017-09-303-13/+26
| |\ \ | | |/ | |/| Kernel/Threads: When putting a thread to wait, specify a function to execute when it is awoken
| | * Kernel/Threads: When putting a thread to wait, specify a function to execute when it is awoken.Subv2017-09-283-13/+26
| | | | | | | | | | | | | | | | | | This change makes for a clearer (less confusing) path of execution in the scheduler, now the code to execute when a thread awakes is closer to the code that puts the thread to sleep (WaitSynch1, WaitSynchN). It also allows us to implement the special wake up behavior of ReplyAndReceive without hacking up WaitObject::WakeupAllWaitingThreads. If savestates are desired in the future, we can change this implementation to one similar to the CoreTiming event system, where we first register the callback functions at startup and assign their identifiers to the Thread callback variable instead of directly assigning a lambda to the wake up callback variable.
| * | Fixed type conversion ambiguityHuw Pascoe2017-09-309-23/+24
| | |
| * | Kernel/Thread: Allow specifying which process a thread belongs to when creating it.Subv2017-09-273-15/+19
| |/ | | | | | | Don't automatically assume that Thread::Create will only be called when the parent process is currently scheduled. This assumption will be broken when applets or system modules are loaded.
| * memory: Add GetCurrentPageTable/SetCurrentPageTableMerryMage2017-09-241-7/+4
| | | | | | | | Don't expose Memory::current_page_table as a global.
| * Merge pull request #2842 from Subv/switchable_page_tableB3n302017-09-155-30/+33
| |\ | | | | | | Kernel/Memory: Give each process its own page table and allow switching the current page table upon reschedule
| | * Kernel/Threads: Don't clear the CPU instruction cache when performing a context switch from an idle thread into a thread in the same process.Subv2017-09-151-1/+3
| | | | | | | | | | | | We were unnecessarily clearing the cache when going from Process A -> Idle -> Process A, this caused extreme performance regressions.
| | * Kernel/Memory: Changed GetPhysicalPointer so that it doesn't go through the current process' page table to obtain a pointer.Subv2017-09-152-25/+7
| | |
| | * Kernel/Memory: Switch the current page table when a new process is scheduled.Subv2017-09-101-0/+10
| | |
| | * Kernel/Memory: Give each Process its own page table.Subv2017-09-102-5/+14
| | | | | | | | | | | | The loader is in charge of setting the newly created process's page table as the main one during the loading process.
| * | Merge pull request #2839 from Subv/global_kernel_lockJames Rowe2017-08-241-1/+1
| |\ \ | | | | | | | | Kernel/HLE: Use a mutex to synchronize access to the HLE kernel state between the cpu thread and any other possible threads that might touch the kernel (network thread, etc).
| | * | Kernel/HLE: Use a mutex to synchronize access to the HLE kernel state between the cpu thread and any other possible threads that might touch the kernel (network thread, etc).Subv2017-08-221-1/+1
| | | | | | | | | | | | | | | | This mutex is acquired in SVC::CallSVC, ie, as soon as the guest application enters the HLE kernel, and should be acquired by the aforementioned threads before modifying kernel structures.
| * | | Merge pull request #2893 from Subv/not_schedule_main_threadbunnei2017-08-221-5/+1
| |\ \ \ | | | | | | | | | | Kernel/Threads: Don't immediately switch to the new main thread when loading a new process.
| | * | | Kernel/Threads: Don't immediately switch to the new main thread when loading a new process.Subv2017-08-221-5/+1
| | | |/ | | |/| | | | | | | | | | | | | This is necessary for loading multiple processes at the same time. The main thread will be automatically scheduled when necessary once the scheduler runs.
| * / | Warnings: Add UNREACHABLE macros to switches that contemplate all possible values.Subv2017-08-211-0/+3
| |/ /
* | | loader: Various improvements for NSO/NRO loaders.bunnei2017-10-102-4/+4
| | |
* | | nso: Refactor and allocate .bss section.bunnei2017-09-302-8/+10
| | |
* | | process: Support loading multiple codesets.bunnei2017-09-302-20/+27
| | |
* | | kernel: Various threading fixes to support 64-bit addressing.bunnei2017-09-302-8/+8
| | |
* | | core: Various changes to support 64-bit addressing.bunnei2017-09-302-21/+21
|/ /
* | Merge pull request #2799 from yuriks/virtual-cached-range-flushWeiyi Wang2017-07-221-1/+1
|\ \ | |/ |/| Add address conversion functions returning optional, Add function to flush virtual region from rasterizer cache
| * Memory: Make PhysicalToVirtualAddress return a boost::optionalYuri Kunde Schlesner2017-06-221-1/+1
| | | | | | | | And fix a few places in the code to take advantage of that.
* | Merge pull request #2793 from Subv/replyandreceiveSebastian Valle2017-06-304-11/+39
|\ \ | | | | | | Kernel/SVC: Partially implemented svcReplyAndReceive
| * | Kernel/SVC: Pass the current thread as a parameter to ClientSession::SendSyncRequest.Subv2017-06-292-3/+6
| | |
| * | Kernel/Sessions: Clean up the list of pending request threads of a session when the client endpoint is closed.Subv2017-06-261-0/+5
| | |
| * | Kernel/ServerSession: Keep track of which threads have issued sync requests.Subv2017-06-253-9/+29
| | |
* | | Kernel: Implement AcceptSession SVCYuri Kunde Schlesner2017-06-233-2/+26
| | |
* | | Kernel/IPC: Support translation of null handlesYuri Kunde Schlesner2017-06-211-7/+12
| |/ |/| | | | | | | Missed this in my first implementation. Thanks to @wwylele for pointing out that this was missing.
* | Merge pull request #2789 from yuriks/misc-kernelWeiyi Wang2017-06-211-0/+2
|\ \ | | | | | | Trivial no-op additions
| * | Kernel: Add comment about the extended linear heap areaYuri Kunde Schlesner2017-06-191-0/+2
| |/
* | Merge pull request #2790 from yuriks/remove-movefromYuri Kunde Schlesner2017-06-214-7/+7
|\ \ | | | | | | Remove ResultVal::MoveFrom
| * | ResultVal: Remove MoveFrom()Yuri Kunde Schlesner2017-06-194-7/+7
| |/ | | | | | | | | Replace it with std::move(result_val).Unwrap(), or Foo().Unwrap() in case you already have an rvalue.
* / Kernel/IPC: Make HLERequestContext usable from outside kernelYuri Kunde Schlesner2017-06-192-3/+9
|/
* Kernel/IPC: Use boost::small_vector for HLE context objectsYuri Kunde Schlesner2017-06-121-1/+3
|
* Kernel: Allow clearing request_objects to re-use buffer spaceYuri Kunde Schlesner2017-06-112-0/+11
| | | | | Reduces the necessary allocation to max(in_handles, out_handles) rather than (in_handles + out_handles).
* Kernel: Basic support for IPC translation for HLE servicesYuri Kunde Schlesner2017-06-112-6/+125
|
* Kernel: Add methods in HLERequestContext abstracting handle creationYuri Kunde Schlesner2017-06-112-0/+12
|
* ServiceFramework: Use separate copy of command bufferYuri Kunde Schlesner2017-06-111-3/+6
| | | | | | Copy the IPC command buffer to/from the request context before/after the handler is invoked. This is part of a move away from using global data for handling IPC requests.
* Merge pull request #2756 from yuriks/service-frameworkYuri Kunde Schlesner2017-06-092-6/+42
|\ | | | | New service framework
| * Service: Add new ServiceFramework framework for writing HLE servicesYuri Kunde Schlesner2017-06-082-2/+36
| | | | | | | | | | | | | | | | | | | | | | | | The old "Interface" class had a few problems such as using free functions (Which didn't allow you to write the service handler as if it were a regular class.) which weren't very extensible. (Only received one parameter with a pointer to the Interface object.) The new ServiceFramework aims to solve these problems by working with member functions and passing a generic context struct as parameter. This struct can be extended in the future without having to update all existing service implementations.
| * Kernel: Remove some unnecessary namespace qualificationsYuri Kunde Schlesner2017-06-061-4/+6
| |
* | Session: Remove/add some forward declarationsYuri Kunde Schlesner2017-06-082-1/+2
| |
* | Kernel: Ensure objects are kept alive during ClientSession disconnectionYuri Kunde Schlesner2017-06-081-7/+13
|/ | | | Fixes #2760
* Kernel: Add a dedicated SetHleHandler method to ServerPort/ServerSessionYuri Kunde Schlesner2017-06-067-33/+36
| | | | | | This allows attaching a HLE handle to a ServerPort at any point after it is created, allowing port/session creation to be generic between HLE and regular services.
* HLE: Move SessionRequestHandler from Service:: to Kernel::Yuri Kunde Schlesner2017-06-068-15/+90
| | | | | Most of the code that works with this is or will be in the kernel, so it's a more appropriate place for it to be.
* Kernel: Move HandleTable to a separate fileYuri Kunde Schlesner2017-05-3013-202/+234
|
* Kernel: Move WaitObject to a separate fileYuri Kunde Schlesner2017-05-3011-132/+173
| | | | | Now that HandleTable doesn't directly depend on WaitObject anymore, this can be separated from the main kernel.h header.
* Kernel: Removed HandleTable::GetWaitObjectYuri Kunde Schlesner2017-05-301-9/+0
| | | | | This isn't necessary anymore since plain Get works correctly for WaitObjects.
* Kernel: Extract dynamic Object pointer cast into its own functionYuri Kunde Schlesner2017-05-291-11/+24
|
* Kernel: Centralize error definitions in errors.hYuri Kunde Schlesner2017-05-2513-46/+133
|
* Merge pull request #2406 from Subv/session_disconnectYuri Kunde Schlesner2017-05-226-51/+82
|\ | | | | Kernel: Properly update port counters on session disconnection.
| * Kernel/Sessions: Remove the ClientSession::Create function.Subv2017-05-223-16/+3
| | | | | | | | It is not meant to be used by anything other than CreateSessionPair.
| * Kernel: Remove a now unused enum and variable regarding a session's status.Subv2017-05-152-8/+0
| |
| * Kernel: Use a Session object to keep track of the status of a Client/Server session pair.Subv2017-05-156-32/+84
| | | | | | | | Reduce the associated port's connection count when a ServerSession is destroyed.
* | Kernel: Map special regions according to ExHeaderYuri Kunde Schlesner2017-05-104-50/+102
| | | | | | | | | | | | | | This replaces the hardcoded VRAM/DSP mappings with ones made based on the ExHeader ARM11 Kernel caps list. While this has no visible effect for most applications (since they use a standard set of mappings) it does improve support for system modules and n3DS exclusives.
* | DSP: Create backing memory for entire DSP RAMYuri Kunde Schlesner2017-05-101-1/+6
|/ | | | Also move address space mapping out of video_core.
* Timer: restore missing signaled=true from #2421wwylele2017-02-271-0/+2
|
* Fix log entry in timer::signal (#2600)B3n302017-02-271-1/+1
|
* Doxygen: Amend minor issues (#2593)Mat M2017-02-271-1/+2
| | | | | | | | | Corrects a few issues with regards to Doxygen documentation, for example: - Incorrect parameter referencing. - Missing @param tags. - Typos in @param tags. and a few minor other issues.
* Merge pull request #2587 from yuriks/status-barYuri Kunde Schlesner2017-02-272-1/+1
|\ | | | | Replace built-in Profiler with indicators in status bar
| * Core: Remove unnecessary include in thread.hYuri Kunde Schlesner2017-02-272-1/+1
| |
* | Timers: Immediately signal the timer if it was started with an initial value of 0.Subv2017-02-222-16/+31
|/
* Threads: Check the process' resource limit for the max allowed priority when creating a thread and remove the priority clamping code.Subv2017-01-111-8/+2
|
* Thread: Added priority range checking to svcSetThreadPriority and removed priority clamping code from Thread::SetPriority.Subv2017-01-112-18/+4
|
* Merge pull request #2397 from Subv/pulsebunnei2017-01-105-13/+20
|\ | | | | Kernel: Implemented Pulse event and timers.
| * Kernel: Implemented Pulse event and timers.Subv2017-01-055-13/+20
| | | | | | | | Closes #1904
* | Merge pull request #2410 from Subv/sleepthreadbunnei2017-01-072-0/+9
|\ \ | | | | | | Don't yield execution in SleepThread(0) if there are no available threads to run
| * | Kernel: Don't attempt to yield execution in SleepThread(0) if there are no available threads to run.Subv2017-01-062-0/+9
| | | | | | | | | | | | With this we avoid an useless temporary deschedule of the current thread.
* | | Merge pull request #2396 from Subv/sema_acquirebunnei2017-01-071-1/+2
|\ \ \ | | | | | | | | Kernel/Semaphore: Fixed a regression in semaphore waits.
| * | | Kernel/Semaphore: Fixed a regression in semaphore waits.Subv2017-01-051-1/+2
| |/ / | | | | | | | | | | | | | | | The regression was caused by a missing check in #2260. The new behavior is consistent with the real kernel.
* | | Merge pull request #2408 from Subv/priority_boostingbunnei2017-01-061-27/+0
|\ \ \ | | | | | | | | Kernel: Removed the priority boost code for starved threads.
| * | | Kernel: Removed the priority boost code for starved threads.Subv2017-01-051-27/+0
| |/ / | | | | | | | | | | | | | | | After hwtesting and reverse engineering the kernel, it was found that the CTROS scheduler performs no priority boosting for threads like this, although some other forms of scheduling priority-starved threads might take place. For example, it was found that hardware interrupts might cause low-priority threads to run if the CPU is preempted in the middle of an SVC handler that deschedules the current (high priority) thread before scheduling it again.
* / / Kernel: Remove some unused functions.Subv2017-01-052-32/+0
|/ /
* | Kernel: Add some asserts to enforce the invariants in the scheduler.Subv2017-01-052-2/+13
| |
* | Kernel: Remove a thread from all of its waiting objects' waiting_threads list when it is awoken.Subv2017-01-051-18/+4
| | | | | | | | This fixes a potential bug where threads would not get removed from said list if they awoke after waiting with WaitSynchronizationN with wait_all = false
* | Kernel: Remove Thread::wait_objects_index and use wait_objects to hold all the objects that a thread is waiting on.Subv2017-01-053-10/+19
| |
* | Kernel: Use different thread statuses when a thread calls WaitSynchronization1 and WaitSynchronizationN with wait_all = true.Subv2017-01-042-13/+17
| | | | | | | | | | | | | | | | This commit removes the overly general THREADSTATUS_WAIT_SYNCH and replaces it with two more granular statuses: THREADSTATUS_WAIT_SYNCH_ANY when a thread waits on objects via WaitSynchronization1 or WaitSynchronizationN with wait_all = false. THREADSTATUS_WAIT_SYNCH_ALL when a thread waits on objects via WaitSynchronizationN with wait_all = true.
* | Kernel/Mutex: Propagate thread priority changes to other threads inheriting the priority via mutexesSubv2017-01-044-42/+54
| |
* | Kernel/Mutex: Update a mutex priority when a thread stops waiting on it.Subv2017-01-044-22/+39
| |
* | Kernel/Mutex: Implemented priority inheritance.Subv2017-01-044-22/+51
| | | | | | | | | | | | | | 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-0416-62/+50
| | | | | | | | 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.
* ThreadContext: Move from "core" to "arm_interface".bunnei2016-12-222-4/+5
|
* core: Replace "AppCore" nomenclature with just "CPU".bunnei2016-12-221-3/+3
|
* core: Remove HLE module, consolidate code & various cleanups.bunnei2016-12-224-7/+3
|
* core: Consolidate core and system state, remove system module & cleanups.bunnei2016-12-221-3/+3
|
* Thread: remove the thread from the thread list when exitingwwylele2016-12-172-2/+14
|
* Kernel: remove object's waiting thread if it is deadwwylele2016-12-161-1/+2
|
* Merge pull request #2260 from Subv/schedulingbunnei2016-12-166-115/+102
|\ | | | | Threading: Reworked the way our scheduler works.
| * Fixed the codestyle to match our clang-format rules.Subv2016-12-142-8/+11
| |
| * Properly remove a thread from its wait_objects' waitlist when it is awoken by a timeout.Subv2016-12-102-1/+10
| |
| * WaitSynch: Removed unused variables and reduced SharedPtr copies.Subv2016-12-093-13/+11
| | | | | | | | | | | | Define a variable with the value of the sync timeout error code. Use a boost::flat_map instead of an unordered_map to hold the equivalence of objects and wait indices in a WaitSynchN call.
| * Use boost remove_erase_if instead of the erase-remove idiomSubv2016-12-071-2/+3
| |
| * Improved the algorithm for GetHighestPriorityReadyThread.Subv2016-12-071-14/+13
| |
| * Threading: Added some utility functions and const correctness.Subv2016-12-042-10/+22
| |
| * Threading: Reworked the way our scheduler works.Subv2016-12-046-111/+76
| | | | | | | | | | | | | | | | | | | | Threads will now be awakened when the objects they're waiting on are signaled, instead of repeating the WaitSynchronization call every now and then. The scheduler is now called once after every SVC call, and once after a thread is awakened from sleep by its timeout callback. This new implementation is based off reverse-engineering of the real kernel. See https://gist.github.com/Subv/02f29bd9f1e5deb7aceea1e8f019c8f4 for a more detailed description of how the real kernel handles rescheduling.
* | Fixed the codestyle to match our clang-format rules.Subv2016-12-147-33/+51
| |
* | Moved the HLE command buffer translation task to ServerSession instead of the HLE handler superclass.Subv2016-12-092-2/+23
| |
* | Added a framework for partially handling Session disconnections.Subv2016-12-084-9/+35
| | | | | | | | | | | | Further implementation will happen in a future commit. Fixes a regression.
* | Use std::move where appropriate.Subv2016-12-085-7/+13
| |
* | Return an error code when connecting to a saturated port.Subv2016-12-052-4/+11
| | | | | | | | The error code was taken from the 3DS kernel.
* | Split SessionRequestHandler::HandleSyncRequest into HandleSyncRequest, TranslateRequest and HandleSyncRequestImpl.Subv2016-12-052-0/+2
| | | | | | | | HandleSyncRequest now takes care of calling the command buffer translate function before actually invoking the command handler for HLE services.
* | Kernel: Remove the Redirection handle type.Subv2016-12-051-2/+0
| |
* | KServerPorts now have an HLE handler "template", which is inherited by all ServerSessions created from it.Subv2016-12-058-35/+60
| |
* | Declare empty ServerSession and ClientSession constructors as default.Subv2016-12-032-4/+4
| |
* | Fixed the rebase mistakes.Subv2016-12-017-51/+46
| |
* | A bit of a redesign.Subv2016-12-016-215/+24
| | | | | | | | | | | | | | Sessions and Ports are now detached from each other. HLE services are handled by means of a SessionRequestHandler class, Interface now inherits from this class. The File and Directory classes are no longer kernel objects, but SessionRequestHandlers instead, bound to a ServerSession when requested. File::OpenLinkFile now creates a new session pair and binds the File instance to it.
* | IPC/HLE: Associate the ClientSessions with their parent port's HLE interface if it exists.Subv2016-12-014-22/+15
| | | | | | | | Pass the triggering ServerSession to the HLE command handler to differentiate which session caused the request.
* | Kernel/HLE: Service::Interface no longer inherits from any Kernel object, and is now its own standalone class.Subv2016-12-012-8/+44
| | | | | | | | Interface is now used by aggregation in ClientPort, to forward service commands to their HLE implementation if needed.
* | fixup! Kernel/IPC: Use Ports and Sessions as the fundamental building block of Inter Process Communication.Subv2016-12-013-4/+5
| |
* | Kernel/IPC: Use Ports and Sessions as the fundamental building block of Inter Process Communication.Subv2016-12-017-56/+233
|/ | | | | | | All handles obtained via srv::GetServiceHandle or svcConnectToPort are references to ClientSessions. Service modules will wait on the counterpart of those ClientSessions (Called ServerSessions) using svcReplyAndReceive or svcWaitSynchronization[1|N], and will be awoken when a SyncRequest is performed. HLE Interfaces are now ClientPorts which override the HandleSyncRequest virtual member function to perform command handling immediately.
* Merge pull request #2196 from Subv/system_modeYuri Kunde Schlesner2016-11-282-6/+4
|\ | | | | Kernel/Loader: Grab the system mode from the NCCH ExHeader.
| * Kernel/Loader: Grab the system mode from the NCCH ExHeader.Subv2016-11-202-6/+4
| | | | | | | | | | | | | | 3dsx and elf files default to system mode 2 (96MB allocated to the application). This allows Home Menu to boot without modifications. Closes #1849
* | Kernel/Events: Log an error when trying to create Pulse events and timers.Subv2016-11-192-0/+10
|/ | | | Related to #1904
* Fix typosRicardo de Almeida Gonzaga2016-10-202-2/+2
|
* move ResetType to kernel.hwwylele2016-09-223-7/+6
|
* implement wait tree widgetwwylele2016-09-224-0/+16
|
* Use negative priorities to avoid special-casing the self-includeYuri Kunde Schlesner2016-09-2114-14/+14
|
* Remove empty newlines in #include blocks.Emmanuel Gil Peyrot2016-09-2129-74/+16
| | | | | | | 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-1915-64/+36
|
* Sources: Run clang-format on everything.Emmanuel Gil Peyrot2016-09-1830-419/+616
|
* arm: ResetContext shouldn't be part of ARM_Interface.bunnei2016-09-151-1/+17
|
* fix #1942 and adds a few IPC functions for descriptorsLectem2016-08-021-15/+103
|
* Merge pull request #1869 from wwylele/dont-be-lazyYuri Kunde Schlesner2016-06-291-2/+6
|\ | | | | Switch context to the same thread if necessary
| * Switch context on the same thread if necessarywwylele2016-05-301-2/+6
| |
* | Merge pull request #1867 from mailwl/srv-updatebunnei2016-06-291-0/+4
|\ \ | | | | | | srv: Update according 3dbrew
| * | Fix parameter name in EnableNotificationmailwl2016-05-311-0/+4
| |/
* | Merge pull request #1877 from wwylele/wait-fix-timeoutbunnei2016-06-181-0/+49
|\ \ | | | | | | Thread: update timeout when reruning WaitSynch
| * | Thread: update timeout when rerunning WaitSynchwwylele2016-06-041-0/+49
| |/
* | Kernel/SVC: Implemented svcCreatePort.Subv2016-06-114-2/+11
| |
* | Kernel: Added ClientPort and ServerPort classes.Subv2016-06-055-2/+135
|/ | | | This is part of an ongoing effort to implement support for multiple processes.
* Merge pull request #1692 from Subv/rm_getpointer2bunnei2016-05-301-1/+1
|\ | | | | Memory: Remove most usages of GetPointer
| * Kernel/Thread: Remove use of Memory::GetPointerMerryMage2016-05-211-1/+1
| |
* | Memory: Added necessary headers and removed unnecessary headerMerryMage2016-05-261-0/+1
|/
* Merge pull request #1800 from JayFoxRox/set-fpscrbunnei2016-05-181-0/+2
|\ | | | | Set fpscr for new threads
| * Set fpscr for new threadsJannik Vogel2016-05-171-0/+2
| |
* | Memory: Fixed a regression caused by #1695 and #1689.Subv2016-05-141-0/+3
|/ | | | | | Reserve enough space in the vector that holds the linear heap memory to prevent relocations of the backing memory when growing too much. Closes #1790
* Merge pull request #1689 from Subv/shmembunnei2016-05-134-67/+161
|\ | | | | Kernel: Implemented shared memory.
| * HLE/Applets: Give each applet its own block of heap memory, and use that when creating the framebuffer shared memory block.Subv2016-05-132-1/+30
| |
| * Kernel: Account for automatically-allocated shared memories in the amount of used linear heap memory.Subv2016-05-131-0/+5
| |
| * Kernel/SharedMemory: Log an error when Map fails.Subv2016-05-131-1/+10
| |
| * Kernel: Implemented shared memory permissions.Subv2016-05-132-9/+47
| |
| * Kernel/Memory: Remove the Shared Memory region from the legacy memory map.Subv2016-05-131-1/+0
| |
| * Kernel/SharedMemory: Properly implemented shared memory support.Subv2016-05-132-65/+79
| | | | | | | | | | | | | | Applications can request the kernel to allocate a piece of the linear heap for them when creating a shared memory object. Shared memory areas are now properly mapped into the target processes when calling svcMapMemoryBlock. Removed the APT Shared Font hack as it is no longer needed.
| * Kernel/SVC: Fixed the register order for svcCreateMemoryBlock.Subv2016-05-131-1/+1
| | | | | | | | R0 is used as the last parameter instead of R4.
* | Merge pull request #1695 from Subv/tls_allocbunnei2016-05-134-22/+74
|\ \ | |/ |/| Kernel/Threads: Dynamically allocate the TLS region for threads.
| * Kernel/Threads: Dynamically allocate the TLS region for threads in the BASE region of the linear heap.Subv2016-05-074-22/+74
| | | | | | | | | | | | Each thread gets a 0x200-byte area from the 0x1000-sized page, when all 8 thread slots in a single page are used up, the kernel allocates a new page to hold another 8 entries. This is consistent with what the real kernel does.
* | Merge pull request #1766 from Subv/log_cpubunnei2016-05-081-0/+2
|\ \ | | | | | | Kernel/Threading: Warn when a thread can be scheduled in the Syscore (Core 1)
| * | Kernel/Threading: Warn when a thread can be scheduled in the Syscore (Core 1).Subv2016-05-071-0/+2
| |/ | | | | | | We do not currently implement any cores other than the AppCore (Core 0).
* | Merge pull request #1762 from bunnei/globalbunnei2016-05-061-1/+2
|\ \ | |/ |/| hle: Get rid of direct global access to g_reschedule
| * hle: Get rid of global access to g_rescheduleLioncash2016-03-211-1/+2
| | | | | | | | | | This shouldn't be directly exposed if there's already a partial API that operates on it. We can just provide the rest of that API.
* | Merge pull request #1643 from MerryMage/make_uniqueMathew Maidment2016-04-061-1/+2
|\ \ | | | | | | Common: Remove Common::make_unique, use std::make_unique
| * | Common: Remove Common::make_unique, use std::make_uniqueMerryMage2016-04-051-1/+2
| |/
* / session: Make helper functions constexprLioncash2016-03-211-6/+6
|/
* svc: Move ResetType enum to the kernel event headerLioncash2016-03-132-2/+8
|
* svc: Make ResetType an enum classLioncash2016-03-122-2/+2
|
* Memory: Do correct Phys->Virt address translation for non-APP linheapYuri Kunde Schlesner2016-03-062-2/+5
|
* AudioCore: Skeleton ImplementationMerryMage2016-02-211-1/+4
| | | | | | | | | This commit: * Adds a new subproject, audio_core. * Defines structures that exist in DSP shared memory. * Hooks up various other parts of the emulator into audio core. This sets the foundation for a later HLE DSP implementation.
* BitField: Make trivially copyable and remove assignment operatorMerryMage2016-02-121-1/+1
|
* Memory: Implement MMIOMerryMage2016-01-302-4/+8
|
* HLE/SVC: Implement UnmapMemoryBlock.Subv2016-01-142-0/+28
| | | | This implementation will need to be (almost completely) changed when we implement multiprocess support.
* HLE/Timers: Reset OneShot timers when they are acquired instead of when they're triggered.Subv2015-12-301-3/+3
| | | | Closes #1139
* SVC: Fixed ArbitrateAddress to behave as it does on hardware.Subv2015-12-282-9/+18
| | | | This was verified with hwtests that i plan to upload later on.
* Kernel: Implement svcGetSystemInfoYuri Kunde Schlesner2015-12-014-0/+12
| | | | | This makes smealum/ctrulib@b96dd51d3349961189d4ab1bc2a5c45deff21c09 work with Citra.
* Silence -Wsign-compare warnings.Rohit Nirmal2015-10-071-1/+1
|
* general: Silence some warnings when using clangLioncash2015-09-161-2/+4
|
* General: Fix up doxygen commentsLioncash2015-09-103-6/+3
|
* Kernel: Fix wrong linear heap base on titles using newer kernelsYuri Kunde Schlesner2015-08-281-1/+1
| | | | Typo which sneaked in through review on #1025
* Kernel: Fix assertion failure when ControlMemory is called with size=0Yuri Kunde Schlesner2015-08-271-0/+8
|
* Core: Improve APT Shared Font hackYuri Kunde Schlesner2015-08-272-2/+27
| | | | Should fix invalid read loops in some games
* Kernel: Remove unused legacy heap MapBlock_* functionsYuri Kunde Schlesner2015-08-162-77/+0
|
* Kernel: Implement svcGetProcessInfo in a basic wayYuri Kunde Schlesner2015-08-163-1/+15
| | | | | This also adds some basic memory usage accounting. These two types are used by Super Smash Bros. during startup.
* Kernel: Add more infrastructure to support different memory layoutsYuri Kunde Schlesner2015-08-165-20/+138
| | | | | | This adds some structures necessary to support multiple memory regions in the future. It also adds support for different system memory types and the new linear heap mapping at 0x30000000.
* Move core/mem_map.{cpp,h} => core/hle/kernel/memory.{cpp,h}Yuri Kunde Schlesner2015-08-163-1/+160
|
* Memory: Move address type conversion routines to memory.cpp/hYuri Kunde Schlesner2015-08-161-1/+0
| | | | | These helpers aren't really part of the kernel, and mem_map.cpp/h is going to be moved there next.
* Process: Store kernel compatibility version during loadingYuri Kunde Schlesner2015-08-162-3/+7
|
* Kernel: Properly implement ControlMemory FREE and COMMITYuri Kunde Schlesner2015-08-164-23/+243
|
* VMManager: Introduce names for used ResultCodesYuri Kunde Schlesner2015-08-162-6/+11
|
* VMManager: Make LogLayout log level configurable as a parameterYuri Kunde Schlesner2015-08-163-5/+15
|
* VMManager: Change block offsets to size_tYuri Kunde Schlesner2015-08-162-3/+3
|
* dyncom: Rename armdefs.h to armstate.hLioncash2015-07-261-1/+1
|
* Kernel/Scheduling: Clean up a thread's wait_objects when its scheduled.Subv2015-07-211-0/+8
| | | | They'll be reset if needed during the next svcWaitSynchronization call (if there's any pending)
* Ensure all kernel objects are released during shutdownYuri Kunde Schlesner2015-07-171-7/+14
| | | | | | | | This commit fixes several kernel object leaks. The most severe of them was threads not being removed from the private handle table used for CoreTiming events. This resulted in Threads never being released, which in turn held references to Process, causing CodeSets to never be freed when loading other applications.
* Merge pull request #921 from linkmauve/fix-appletbunnei2015-07-122-0/+4
|\ | | | | Fix applet includes using iwyu
| * Core: Fix applet includes using iwyu.Emmanuel Gil Peyrot2015-07-122-0/+4
| |
* | Kernel: Add CodeSet case to Object::IsWaitableYuri Kunde Schlesner2015-07-121-0/+1
|/
* Core: Properly configure address space when loading a binaryYuri Kunde Schlesner2015-07-125-14/+88
| | | | | | The code now properly configures the process image to match the loaded binary segments (code, rodata, data) instead of just blindly allocating a large chunk of dummy memory.
* Kernel: Remove unused member from EventYuri Kunde Schlesner2015-07-122-2/+1
|
* Core: Cleanup file_sys includes.Emmanuel Gil Peyrot2015-06-281-1/+2
|
* Core: Cleanup core includes.Emmanuel Gil Peyrot2015-06-282-1/+2
|
* Common: Cleanup key_map includes.Emmanuel Gil Peyrot2015-06-284-6/+10
|
* Add helpers to create IPC command buffer headers and descriptorsYuri Kunde Schlesner2015-06-231-0/+34
|
* kernel: Fix svcWaitSynch to always acquire requested wait objects.bunnei2015-06-177-101/+37
|
* Merge pull request #810 from yuriks/memmapYuri Kunde Schlesner2015-05-302-0/+445
|\ | | | | Kernel: Add VMManager to manage process address spaces
| * Kernel: Add VMManager to manage process address spacesYuri Kunde Schlesner2015-05-272-0/+445
| | | | | | | | | | | | | | | | This enables more dynamic management of the process address space, compared to just directly configuring the page table for major areas. This will serve as the foundation upon which the rest of the Kernel memory management functions will be built.
* | Remove every trailing whitespace from the project (but externals).Emmanuel Gil Peyrot2015-05-295-11/+11
|/
* Kernel: Fix a warning introduced with ResourceLimit, and remove the fallback code to prevent it from happening again.Emmanuel Gil Peyrot2015-05-211-2/+1
|
* Kernel: Move reschedules from SVCs to actual mechanisms that reschedule.bunnei2015-05-216-0/+20
|
* Merge pull request #772 from lioncash/warnbunnei2015-05-181-3/+3
|\ | | | | core/video_core: Fix a few warnings when compiling on MSVC.
| * process: Get rid of warningsLioncash2015-05-141-3/+3
| | | | | | | | Sign mismatches and "forcing value to bool" warnings.
* | Core/ResourceLimits: Implemented the basic structure of ResourceLimits.Subv2015-05-156-1/+286
| | | | | | | | | | | | Implemented svcs GetResourceLimit, GetResourceLimitCurrentValues and GetResourceLimitLimitValues. Note that the resource limits do not currently keep track of used objects, since we have no way to distinguish between an object created by the application, and an object created by some HLE module once we're inside Kernel::T::Create.
* | Memmap: Re-organize memory function in two filesYuri Kunde Schlesner2015-05-156-6/+5
| | | | | | | | | | | | | | memory.cpp/h contains definitions related to acessing memory and configuring the address space mem_map.cpp/h contains higher-level definitions related to configuring the address space accoording to the kernel and allocating memory.
* | thread: Fix a conditional check in RescheduleLioncash2015-05-141-1/+1
|/
* Merge pull request #748 from Subv/tls_maxbunnei2015-05-123-7/+19
|\ | | | | Core/Memory: Add TLS support for creating up to 300 threads
| * Core/Memory: Add TLS support for creating up to 300 threadsSubv2015-05-123-7/+19
| |
* | Merge pull request #751 from yuriks/idle-threadbunnei2015-05-122-44/+19
|\ \ | | | | | | Thread: Remove the idle thread
| * | Thread: Remove the idle threadYuri Kunde Schlesner2015-05-122-44/+19
| | | | | | | | | | | | Instead just use nullptr to represent no thread is active.
* | | Merge pull request #757 from Subv/schedulingbunnei2015-05-121-0/+2
|\ \ \ | |_|/ |/| | Core/Scheduling: Prepare the new priority in the thread queue when svcSetPriority is called
| * | Core/Scheduling: Prepare the new priority in the thread queue when svcSetPriority is calledSubv2015-05-121-0/+2
| |/
* | Merge pull request #750 from Subv/process_svcYuri Kunde Schlesner2015-05-125-2/+15
|\ \ | |/ |/| Core/HLE: Implemented the SVCs GetProcessId and GetProcessIdOfThread
| * fixup!Subv2015-05-122-7/+3
| |
| * Core/HLE: Implemented the SVCs GetProcessId and GetProcessIdOfThreadSubv2015-05-115-2/+19
| |
* | Thread: Correctly set main thread initial stack positionYuri Kunde Schlesner2015-05-113-5/+4
|/
* Merge pull request #740 from yuriks/gsp-shmemarchshift2015-05-112-16/+37
|\ | | | | Fix crashes due to un-initialized GSP shared memory
| * Kernel: Zero-fill shared memory blocks when mappingYuri Kunde Schlesner2015-05-111-0/+8
| | | | | | | | | | | | This works around crashes related to GSP/HID/etc. shared memory blocks having garbage values. The proper fix requires proper management of mapped memory blocks in the process.
| * Kernel: Capture SharedMemory attributes at creation, not when mappingYuri Kunde Schlesner2015-05-112-16/+29
| |
* | fixup! Set the TLS address in the schedulerSubv2015-05-112-2/+7
| |
* | Core/Memory: Give every emulated thread it's own TLS area.Subv2015-05-113-4/+22
|/ | | | | The TLS area for thread T with id Ti is located at TLS_AREA_VADDR + (Ti - 1) * 0x200. This allows some games like Mario Kart 7 to continue further.
* Common: Remove the BIT macroYuri Kunde Schlesner2015-05-091-2/+2
| | | | | | | When the macro was introduced in 326ec51261299e48de97592631c02523da9c8118 it wasn't noticed that it conflicted in name with a heavily used macro inside of dyncom. This causes some compiler warnings. Since it's only lightly used, it was opted to simply remove the new macro.
* Memory: Re-organize and rename memory area address constantsYuri Kunde Schlesner2015-05-092-3/+4
|
* Kernel: Remove unused g_main_thread variableYuri Kunde Schlesner2015-05-093-5/+1
|
* Process: Rename StaticAddressMapping => AddressMappingYuri Kunde Schlesner2015-05-092-5/+5
|
* Process: Add more documentation to the class membersYuri Kunde Schlesner2015-05-091-2/+16
|
* Process: Use BitField to store process flagsYuri Kunde Schlesner2015-05-092-16/+24
|
* Process: Support parsing of exheader kernel capsYuri Kunde Schlesner2015-05-092-4/+72
|
* Kernel: Remove g_program_idYuri Kunde Schlesner2015-05-092-8/+0
| | | | This has been obsoleted by the field in Process.
* Kernel: Introduce skeleton Process class to hold process dataYuri Kunde Schlesner2015-05-094-19/+101
|
* Common: Remove common.hYuri Kunde Schlesner2015-05-079-8/+14
|
* Move typedefs from kernel.h to more appropriate placesYuri Kunde Schlesner2015-05-071-10/+1
|
* Kernel: Properly initialize and shutdown all modules.bunnei2015-05-024-9/+20
|
* Kernel: Use the correct format string for u64 hex.Emmanuel Gil Peyrot2015-04-141-1/+1
|
* SVC: Update various SVCs to cause a reschedule.bunnei2015-04-101-4/+0
| | | | - CreateMutex/ReleaseMutex/ReleaseSemaphore/SetTimer/CancelTimer/ArbitrateAddress
* Kernel: Implemented priority inheritance for mutexes.bunnei2015-04-103-4/+22
|
* Thread: Implement priority boost for starved threads.bunnei2015-04-102-22/+51
| | | | | | SVC: Return correct error code on invalid CreateThread processor ID. SVC: Assert when creating a thread with an invalid userland priority.
* Kernel: Fixed default thread priority.bunnei2015-04-102-5/+4
|
* Initialize base address to 0x0Gareth Higgins2015-04-091-0/+1
|
* Misc cleanup of common and related functionsarchshift2015-02-201-2/+3
|
* Build: Fixed some warningsSubv2015-02-123-4/+4
|
* Asserts: break/crash program, fit to style guide; log.h->assert.harchshift2015-02-117-15/+13
| | | | | | | 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.
* Scheduler refactor Pt. 1Kevin Hartman2015-02-103-223/+258
| | | | | | | | | | | | | * Simplifies scheduling logic, specifically regarding thread status. It should be much clearer which statuses are valid for a thread at any given point in the system. * Removes dead code from thread.cpp. * Moves the implementation of resetting a ThreadContext to the corresponding core's implementation. Other changes: * Fixed comments in arm interfaces. * Updated comments in thread.cpp * Removed confusing, useless, functions like MakeReady() and ChangeStatus() from thread.cpp. * Removed stack_size from Thread. In the CTR kernel, the thread's stack would be allocated before thread creation.
* Mutex: Locks should be recursive.bunnei2015-02-102-16/+20
|
* core: Fix some warnings on OSXLioncash2015-02-031-1/+0
|
* Kernel: Stop creating useless Handles during object creationYuri Kunde Schlesner2015-02-0212-36/+17
| | | | | 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-026-12/+17
| | | | | | | | | | | | | | | | 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-0216-8/+50
| | | | | | 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-023-23/+18
|
* Kernel: Fix bug in HandleTable::CloseYuri Kunde Schlesner2015-02-021-1/+1
|
* Kernel: Remove Object::GetHandle (it's not used anymore :D)Yuri Kunde Schlesner2015-02-022-9/+1
|
* Kernel: Introduce unique Object ids for debuggingYuri Kunde Schlesner2015-02-024-8/+16
|
* Kernel: Use separate Handle tables for CoreTiming userdataYuri Kunde Schlesner2015-02-024-18/+25
| | | | This is to support the removal of GetHandle soon
* Kernel: Remove previous scheduled event when a Timer is re-SetYuri Kunde Schlesner2015-02-021-0/+3
|
* Thread: Modernize two functions that slipped through previous rebasesYuri Kunde Schlesner2015-02-023-15/+13
|
* arm: Clean up ARMul_StateLioncash2015-02-011-1/+1
| | | | Remove unnecessary/unused struct variables.
* shared_memory: Fix assignments in SharedMemory::MapLioncash2015-01-302-4/+4
|
* Kernel: Mark all appropriate kernel objects as "final"Yuri Kunde Schlesner2015-01-307-8/+7
|
* Remove result.h InvalidHandleYuri Kunde Schlesner2015-01-301-1/+2
| | | | | It was only being used in two places, where it was replaced by a local constant.
* Kernel: Convert Event to not use HandlesYuri Kunde Schlesner2015-01-302-83/+51
|
* Kernel: Convert Timer to (mostly) not use HandlesYuri Kunde Schlesner2015-01-302-104/+72
|
* Kernel: Convert Mutex to not use HandlesYuri Kunde Schlesner2015-01-302-101/+82
|
* Kernel: Convert AddressArbiter to not use HandlesYuri Kunde Schlesner2015-01-302-32/+31
|
* Kernel: Convert Semaphore to not use HandlesYuri Kunde Schlesner2015-01-302-61/+61
|
* Kernel: Convert SharedMemory to not use HandlesYuri Kunde Schlesner2015-01-302-71/+54
|
* Move VAddr/PAddr typedefs to kernel.hYuri Kunde Schlesner2015-01-301-0/+5
|
* Kernel: Remove useless/duplicated comments; mark functions staticYuri Kunde Schlesner2015-01-306-32/+8
|
* Thread: Fix WaitSynchronization1 to not set register 1 on thread wakeup.bunnei2015-01-222-22/+42
|
* Thread: Use std::find in CheckWait_WaitObject.bunnei2015-01-221-4/+5
|
* Mutex: Cleanup and remove redundant code.bunnei2015-01-223-47/+29
|
* Kernel: Renamed some functions for clarity.bunnei2015-01-227-10/+10
| | | | | - ReleaseNextThread->WakeupNextThread - ReleaseAllWaitingThreads->WakeupAllWaitingThreads.
* Kernel: Changed "ShouldWait" to return bool and "Acquire" to return void.bunnei2015-01-228-64/+39
|
* WaitObject: Renamed "Wait" to "ShouldWait", made "ShouldWait" and "Acquire" pure virtual.bunnei2015-01-228-21/+20
|
* Event: Fix implementation of "non-sticky" events.bunnei2015-01-221-0/+4
|
* Session: Change to a WaitObject.bunnei2015-01-223-2/+9
|
* Kernel: Reschedule on SignalEvent and SendSyncRequest, fix some bugs.bunnei2015-01-221-1/+1
|
* Mutex: Fix a bug where the thread should not wait if it already has the mutex.bunnei2015-01-221-1/+4
|
* Kernel: Moved Wait and Acquire to WaitObject, added way to retrieve a WaitObject safely.bunnei2015-01-223-18/+57
|
* AddressArbiter: Changed to Kernel::Object, big cleanup, removed code that made no sense.bunnei2015-01-224-35/+42
|
* Kernel: Get rid of WaitTypes and simplify lots of code, removing hacks.bunnei2015-01-228-112/+43
|
* WaitSynchronizationN: Refactor to fix several bugsbunnei2015-01-227-54/+49
| | | | | | - 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-227-14/+54
|
* WaitSynchronizationN: Implement return valuesbunnei2015-01-229-56/+139
|
* Event: Fixed some bugs and cleanup (Subv)bunnei2015-01-222-54/+13
|
* Thread: Keep track of multiple wait objects.bunnei2015-01-223-16/+30
|
* Event: Get rid of permanent_lock hack.bunnei2015-01-222-36/+8
|
* WaitObject: Added RemoveWaitingThread, fixed a bug, and cleanup.bunnei2015-01-222-4/+17
|
* Kernel: Added WaitObject and changed "waitable" objects inherit from it.bunnei2015-01-228-71/+73
|
* core: Fix a few docstringsLioncash2015-01-202-2/+2
|
* AddrArbiter: Implement arbitration types 3 and 4.Subv2015-01-132-3/+20
|
* Merge pull request #466 from Subv/wakebunnei2015-01-111-0/+3
|\ | | | | Thread: Prevent waking a thread multiple times.
| * Thread: Prevent waking a thread multiple times.Subv2015-01-111-0/+3
| | | | | | | | If a thread was woken up by something, cancel the wakeup timeout.
* | Kernel: Start using boost::intrusive_ptr for lifetime managementYuri Kunde Schlesner2015-01-0910-76/+75
| |
* | Kernel: Don't re-assign object's handle when duplicating oneYuri Kunde Schlesner2015-01-092-2/+3
|/
* Thread: Fix nullptr access in a logging functionYuri Kunde Schlesner2015-01-091-1/+2
|
* Thread: Rename thread_queue => thread_listYuri Kunde Schlesner2015-01-091-6/+6
|
* Thread: Reduce use of Handles and move some funcs to inside the class.Yuri Kunde Schlesner2015-01-099-281/+190
|
* Kernel: Move Thread's definition to the header fileYuri Kunde Schlesner2015-01-093-53/+67
|
* Move ThreadContext to core/core.h and deal with the falloutYuri Kunde Schlesner2015-01-092-4/+6
|
* Merge pull request #255 from Subv/cbranch_3bunnei2015-01-095-1/+194
|\ | | | | Implemented timers
| * SVC: Implemented the Timer service calls.Subv2015-01-095-1/+194
| |
* | SVC: Fixed SleepThread.Subv2015-01-092-8/+39
| | | | | | | | It will now properly wait the specified number of nanoseconds and then wake up the thread.
* | Threads: Use a dummy idle thread when no other are ready.Subv2015-01-083-1/+35
| | | | | | | | This thread will not actually execute instructions, it will only advance the timing/events and try to yield immediately to the next ready thread, if there aren't any ready threads then it will be rescheduled and start its job again.
* | Common: Clean up ThreadQueueListYuri Kunde Schlesner2015-01-071-1/+1
|/ | | | | | | | Replace all the C-style complicated buffer management with a std::deque. In addition to making the code easier to understand it also adds support for non-POD IdTypes. Also clean the rest of the code to follow our code style.
* Merge pull request #407 from Subv/arbiterbunnei2015-01-051-0/+11
|\ | | | | AddressArbiter: Ported arbitration type 2 from 3dmoo.
| * AddressArbiter: Ported arbitration type 2 from 3dmoo.Subv2015-01-031-0/+11
| | | | | | | | (Thanks 3dmoo!)
* | 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-289-142/+189
| | | | | | | | | | | 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-288-15/+15
|
* Rename ObjectPool to HandleTableYuri Kunde Schlesner2014-12-288-41/+41
|
* Merge pull request #291 from purpasmart96/licensebunnei2014-12-2115-16/+16
|\ | | | | License change
| * License changepurpasmart962014-12-2115-16/+16
| |
* | Thread: Wait current thread on svc_SleepThreadbunnei2014-12-212-21/+33
| | | | | | | | | | | | - Removed unused VBLANK sleep mode - Added error log for bad context switch - Renamed VerifyWait to CheckWaitType to be more clear
* | Merge pull request #316 from yuriks/thread-handlebunnei2014-12-203-2/+16
|\ \ | | | | | | Kernel: Implement support for current thread pseudo-handle
| * | Kernel: Implement support for current thread pseudo-handleYuri Kunde Schlesner2014-12-203-2/+16
| | | | | | | | | | | | This boots a few (mostly Nintendo 1st party) games further.
* | | Clean up some warningsChin2014-12-202-5/+5
| |/ |/|
* | Merge pull request #185 from purpasmart96/mem_permbunnei2014-12-181-5/+9
|\ \ | | | | | | Kernel: Add missing permissions
| * | Kernel:Add missing permissions in shared memory & svcpurpasmart962014-11-191-5/+9
| | |
* | | Filesystem/Archives: Implemented the SaveData archiveSubv2014-12-182-0/+7
| |/ |/| | | | | | | | | | | | | | | | | The savedata for each game is stored in /savedata/<ProgramID> for NCCH files. ELF files and 3DSX files use the folder 0 because they have no ID information Got rid of the code duplication in File and Directory Files that deal with the host machine's file system now live in DiskFile, similarly for directories and DiskDirectory and archives with DiskArchive. FS_U: Use the correct error code when a file wasn't found
* | HLE: Rename namespaces to match move & fix initialization orderYuri Kunde Schlesner2014-12-161-5/+0
| |
* | HLE: Move kernel/archive.* to service/fs/Yuri Kunde Schlesner2014-12-163-534/+1
| |
* | Remove SyncRequest from K::Object and create a new K::Session typeYuri Kunde Schlesner2014-12-153-38/+75
| | | | | | | | | | | | | | This is a first step at fixing the conceptual insanity that is our handling of service and IPC calls. For now, interfaces still directly derived from Session because we don't have the infrastructure to do it properly. (That is, Processes and scheduling them.)
* | Kernel/Semaphore: Small style changeSubv2014-12-131-1/+1
| |
* | Kernel/Semaphores: Invert the available count checking.Subv2014-12-131-11/+9
| | | | | | | | Same semantics, idea by @yuriks
* | Kernel/Semaphores: Addressed some issues.Subv2014-12-132-32/+18
| |
* | Semaphore: Removed an unneeded functionSubv2014-12-131-5/+0
| |
* | Semaphores: Addressed some style issuesSubv2014-12-131-6/+5
| |
* | Semaphore: Implemented the initial_count parameter.Subv2014-12-132-5/+7
| |
* | SVC: Implemented ReleaseSemaphore.Subv2014-12-132-16/+64
| | | | | | | | This behavior was tested on hardware, however i'm still not sure what use the "initial_count" parameter has
* | SVC: Implemented svcCreateSemaphoreSubv2014-12-132-0/+98
| | | | | | | | | | ToDo: Implement svcReleaseSemaphore * Some testing against hardware needed
* | kernel: Remove unused log argumentsLioncash2014-12-131-3/+3
| |
* | Convert old logging calls to new logging macrosYuri Kunde Schlesner2014-12-136-72/+45
| |
* | Merge pull request #256 from Subv/mutexbunnei2014-12-113-37/+67
|\ \ | | | | | | Kernel/Mutex: Properly lock the mutex when a thread enters it
| * | 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-073-22/+56
| | |
| * | 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.
* | | Thread: Fixed to wait on address when in arbitration.bunnei2014-12-093-11/+31
| | |
* | | Make OpenDirectory fail if the directory doesn't existarchshift2014-12-071-0/+5
|/ / | | | | | | | | | | | | This is in line with what the hardware itself does. It does this by splitting the initial directory opening into Directory.Open(), which will return false if a stat fails. Then, Archive::OpenDirectory will return nullptr, and archive.cpp will return an error code .
* | Merge pull request #250 from Subv/cbranch_2bunnei2014-12-052-0/+26
|\ \ | | | | | | SVC: Implemented GetThreadId.
| * | Threads: Remove a redundant function.Subv2014-12-041-9/+1
| | | | | | | | | | | | Use the next_thread_id variable directly.
| * | Threads: Implemented a sequential thread idSubv2014-12-042-4/+19
| | |
| * | SVC: Implemented GetThreadId.Subv2014-12-042-0/+19
| | | | | | | | | | | | For now threads are using their Handle value as their Id, it should not really cause any problems because Handle values are unique in Citra, but it should be changed. I left a ToDo there because this is not correct behavior as per hardware.
* | | Merge pull request #222 from archshift/renamexyzbunnei2014-12-052-33/+74
|\ \ \ | | | | | | | | Implemented RenameFile and RenameDirectory in FS:USER
| * | | Updated archive.cpp functions for proper error handlingarchshift2014-12-042-65/+36
| | | |
| * | | Implemented RenameDirectory in FS:USERarchshift2014-11-252-0/+35
| | | |
| * | | Implemented RenameFile in FS:USERarchshift2014-11-252-0/+35
| | | |
* | | | kernel: Shorten GetCountLioncash2014-12-041-6/+3
| | | |
* | | | kernel: Make some functions constLioncash2014-12-042-4/+4
| |/ / |/| |
* | | Merge pull request #225 from bunnei/fix-release-mutexbunnei2014-11-301-8/+7
|\ \ \ | | | | | | | | Mutex: Changed behavior to always release mutex for all threads.
| * | | Mutex: Changed behavior to always release mutex for all threads.bunnei2014-11-261-8/+7
| |/ /
* / / Thread: Check that thread is actually in "wait state" when verifying wait.bunnei2014-11-261-1/+1
|/ /
* | Use pointers instead of passing handles around in some functions.Yuri Kunde Schlesner2014-11-241-19/+15
| |
* | Remove duplicated docs/update them for changed parameters.Yuri Kunde Schlesner2014-11-249-78/+0
| |
* | HLE: Revamp error handling throrough the HLE codeYuri Kunde Schlesner2014-11-2413-201/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | Merge pull request #191 from archshift/deletexyzbunnei2014-11-242-1/+47
|\ \ | | | | | | Added DeleteFile and DeleteDirectory functions to FS:USER and the archives.
| * | Added DeleteFile and DeleteDirectory functions to FS:USER and the archives.archshift2014-11-232-1/+47
| | |
* | | Merge pull request #211 from linkmauve/masterbunnei2014-11-1912-42/+42
|\ \ \ | | | | | | | | Remove trailing spaces from the entire project
| * | | Remove trailing spaces in every file but the ones imported from SkyEye, AOSP or generatedEmmanuel Gil Peyrot2014-11-1912-42/+42
| | | |
* | | | Merge pull request #208 from lioncash/staticsbunnei2014-11-191-22/+22
|\ \ \ \ | |/ / / |/| | | Add static to some variables
| * | | Add static to some variablesLioncash2014-11-191-22/+22
| |/ /
* / / Remove extraneous semicolonsLioncash2014-11-181-1/+1
|/ /
* | Archive: Fixed to not destroy archive handle on close.bunnei2014-11-181-3/+3
| |
* | Archive: Fixed close archive before freeing.bunnei2014-11-181-1/+1
| |
* | FS_User: Support FileSye::Path in a more generic way.bunnei2014-11-181-0/+11
| | | | | | | | added a todo to kernel archive
* | FileSys: Updated backend code to use FileSys::Path instead of string for paths.bunnei2014-11-182-10/+10
| |
* | Fix two format strings.Lioncash2014-11-141-1/+1
|/
* Added CreateDirectory function to service/fs.cpp, and in Archive.archshift2014-11-022-2/+25
|
* Fix some warningsSean2014-10-301-3/+3
|
* Add `override` keyword through the code.Yuri Kunde Schlesner2014-10-266-35/+35
| | | | This was automated using `clang-modernize`.
* FileSys: split the constructor into an Open method, in order to notify the opener something went wrong.Emmanuel Gil Peyrot2014-10-061-0/+3
| | | | Kernel: Return an invalid handle to OpenFile when it failed to open.
* FileSys/Kernel: Implement SetSize service call for File objects.Emmanuel Gil Peyrot2014-10-061-0/+8
|
* Use the citra user path for the sdmc directoryarchshift2014-09-211-2/+1
|
* Kernel: Implement the Close command for Archive, File and Directory.Emmanuel Gil Peyrot2014-09-172-0/+43
|
* Kernel: Add a Directory object and a getter for it from an Archive object.Emmanuel Gil Peyrot2014-09-173-0/+91
|
* Kernel: Add a File object and a getter for it from an Archive object.Emmanuel Gil Peyrot2014-09-172-0/+118
|
* Core: Get rid of unnecessary switch statement in KernelLioncash2014-09-151-41/+2
|
* core: Prune redundant includesarchshift2014-09-093-6/+0
|
* Threading: Fix thread starting to execute first instruction correctly.bunnei2014-08-281-0/+5
|
* Added FS functions to Archive and Archive_RomFSarchshift2014-08-231-3/+31
|
* Core: Use std::array for managing kernel object spaceLioncash2014-08-192-5/+5
| | | | These avoid relying on memset for clearing the arrays.
* Core: Alter the kernel string functions to use std::string instead of const char*.Lioncash2014-08-188-25/+22
| | | | Most functions already operate on std::strings. This also removes the need to manually null terminate thread names.
* Thread: Added more descriptive comment to WaitCurrentThread.bunnei2014-08-072-2/+10
|
* AddressArbiter: Removed unnecessary HLE::Reschedule.bunnei2014-08-061-1/+0
|
* AddressArbiter: Fixed bug with break statements missing from case statements.bunnei2014-08-061-0/+2
|
* Kernel: Updated Event and Mutex to specify handle that they are blocking for.bunnei2014-08-062-2/+2
|
* Kernel: Added preliminary support for address arbiters.bunnei2014-07-093-1/+124
| | | | | | | | AddressArbiter: Added documentation comment, fixed whitespace issue. AddressArbiter: Fixed incorrect comment, reordered if-statement to be more clear. SVC: Removed trailing whitespace.
* Thread: Added functions to resume threads from address arbitration.bunnei2014-07-092-0/+44
| | | | | | Thread: Cleaned up arbitrate address functions. Thread: Cleaned up ArbitrateAllThreads function.
* SharedMemory: Updated MapSharedMemory to use an enum for permissions.bunnei2014-07-052-6/+27
| | | | - Also added some safety checks to MapSharedMemory.
* Kernel: Added support for shared memory objects.bunnei2014-07-052-0/+132
| | | | SharedMemory: Added optional name field for tracking known objects.
* Archive: Added Init/Shutdown methods to reset kernel archive state.bunnei2014-07-053-0/+19
|
* FileSys: Added preliminary support for applications reading the RomFS archive.bunnei2014-07-052-10/+105
| | | | | | | | | | Archive: Fixed brace ugliness for neobrain :) FS: Commented out unused local variables to prevent warnings. ...But keeping them here for future use. archive_romfs: Removed unused #include.
* Kernel: Added stubbed code to support creation of kernel Archive objects.bunnei2014-06-273-0/+85
|
* Kernel: Removed unnecessary "#pragma once".bunnei2014-06-131-2/+0
|
* Kernel: Added freeing of kernel objects on emulator shutdown.bunnei2014-06-132-0/+10
|
* Event: Updated several log messages to be assertions.bunnei2014-06-131-16/+8
|
* Thread: Renamed occurrences of "t" to "thread" to improve readability.bunnei2014-06-131-48/+45
|
* Thread: Cleaned up VerifyWait, fixed issue where nullptr msg could unnecessarily be logged.bunnei2014-06-131-9/+7
|
* HLE: Removed usnused EatCycles function.bunnei2014-06-131-9/+0
|
* Thread: Moved position of * in arguments.bunnei2014-06-131-2/+2
|
* Thread: Updated VerifyWait to be more readable (but functionally the same).bunnei2014-06-131-4/+3
|
* HLE: Updated all uses of NULL to nullptr (to be C++11 compliant)bunnei2014-06-133-9/+9
|
* Kernel: Updated various kernel function "name" arguments to be const references.bunnei2014-06-134-6/+6
|
* HLE: Updated various handle debug assertions to be more clear.bunnei2014-06-132-3/+3
|
* Mutex: Moved ReleaseMutex iterator declaration to be inside while loop.bunnei2014-06-131-2/+1
|
* Kernel: Updated several member functions to be constbunnei2014-06-134-11/+11
|
* Thread: Fixed bug with ResetThread where cpu_registers[15] was being incorrectly setbunnei2014-06-131-1/+1
|
* Kernel: Made SyncRequest not pure virtual, with a default implementation of error (as this is not required for all kernel objects)bunnei2014-06-133-23/+4
|
* Kernel: Added real support for thread and event blockingbunnei2014-06-135-48/+165
| | | | | | | | | | | | | - 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.
* kernel: changed current default thread priority back to 0x30 - I think this is more correctbunnei2014-06-051-1/+1
|
* svc: added optional name field to Event and Mutex (used for debugging)bunnei2014-06-034-8/+21
|
* kernel: moved position of * for GetTypeName and GetNamebunnei2014-06-031-2/+2
|
* svc: added GetThreadPriority and SetThreadPriority, added (incomplete) DuplicateHandle supportbunnei2014-06-022-0/+51
|
* kernel: changed main thread priority to default, updated Kernel::Reschedule to use PrepareReschedulebunnei2014-06-023-4/+6
|
* thread: updated Reschedule to sit at a synchronization barrier when no other threads are ready for executionbunnei2014-06-011-0/+18
|
* event: added a hackish ability to set an event as "locked" to its current state, cleaned up some commentsbunnei2014-06-012-4/+32
|
* mutex: fixed typo in ReleaseMutexbunnei2014-05-301-1/+3
|
* event: added support for ClearEvent, fixed a bug with CreateEvent, fixed some commentsbunnei2014-05-302-9/+14
|
* mutex: added preliminary SyncRequest/WaitSynchronization, added some comments/assertionsbunnei2014-05-281-0/+6
|
* event: fixed typos and updated CMakeListsbunnei2014-05-281-1/+1
|
* event: added SetEventLocked method to change status an events lockbunnei2014-05-282-0/+18
|
* kernel: added event module to support creation of CTR "Event" objectsbunnei2014-05-282-0/+119
|
* mutex: removed docstring comment that is no longer relevantbunnei2014-05-271-1/+0
|
* mutex: added additional docstringsbunnei2014-05-271-0/+2
|
* kernel: added WaitSynchronization method to Kernel::Objectbunnei2014-05-273-0/+29
|
* kernel: updated SyncRequest to take boolean thread wait result as a parameterbunnei2014-05-273-5/+20
|
* kernel: added enum for known CurrentThread and CurrentProcess handlesbunnei2014-05-271-0/+5
|
* kernel: add a SyncRequest method to KernelObject for use with svcSendSyncRequestbunnei2014-05-273-0/+11
|
* thread: renamed "WaitCurThread" to "WaitCurrentThread", removed unused "reason" argumentbunnei2014-05-232-4/+4
|
* thread: removed unused SwitchContext/Reschedule reason field, added missing arg parameter to SVC CreateThreadbunnei2014-05-232-4/+4
|
* kernel: refactored function naming to remove "__" prefixbunnei2014-05-235-62/+72
|
* thread: moved ThreadStatus/WaitType to header, added support for arg on CreateThread, added correct CPSR resetbunnei2014-05-232-35/+40
|
* thread: fixed bug where result of __NextThread was not being properly checked when NULLbunnei2014-05-221-1/+1
|
* mutex: refactored the interface to code to return a Mutex* handlebunnei2014-05-212-3/+13
|
* mutex: initial commit of HLE modulebunnei2014-05-212-0/+148
|
* kernel: fixed include, in general include "common.h" not "common_types.h"bunnei2014-05-211-1/+1
|
* thread: added correct lowest thread priority, added a thread priority check, and added some commentsbunnei2014-05-212-6/+10
|
* thread: exposed ResumeThreadFromWait function for use in other kernel modulesbunnei2014-05-212-8/+11
|
* thread: moved threading calls to the Kernel namespacebunnei2014-05-213-101/+115
|
* ARM_Interface: added SaveContext and LoadContext functions for HLE thread switchingbunnei2014-05-211-36/+2
|
* renamed "syscall" module to "svc" (more accurate naming)bunnei2014-05-211-1/+1
|
* thread: whitespace change - fixed * and & placementbunnei2014-05-212-27/+27
|
* - created a Kernel namespacebunnei2014-05-214-67/+79
| | | | | - cleaned up Kernel code a bit (moved stuff into namespace, fixed whitespace issues) - added handle types for all different CTROS handles
* thread: added declaration for __KernelReschedule to be used by syscall modulebunnei2014-05-201-0/+3
|
* - updated service(s) to be KernelObject'sbunnei2014-05-191-4/+5
| | | | - various cleanups
* - moved Handle/Result definitions to kernel.hbunnei2014-05-192-3/+2
| | | | - added ResetType enum
* changed a commentbunnei2014-05-172-2/+2
|
* - added enum ThreadProcessorIdbunnei2014-05-172-53/+107
| | | | | - reorganized some kernel thread functions - added placeholder __KernelWaitThread_Synchronization function
* - replaced KERNELOBJECT_MAX_NAME_LENGTH with KERNEL_MAX_NAME_LENGTHbunnei2014-05-173-8/+12
| | | | - added KERNEL_DEFAULT_STACK_SIZE definition (0x4000)
* completely gutted/refactored threading code to be simplerbunnei2014-05-162-658/+230
|
* changed "UID" to "Handle" to be a little more consistent with CTR namingbunnei2014-05-162-18/+21
|
* - added helper function for __KernelCreateThreadbunnei2014-05-152-4/+76
| | | | | - added __KernelSwitchToThread for enabling a thread - added __KernelRotateThreadReadyQueue
* changed primary thread priority to 0x30 - this is typical, not 0x31bunnei2014-05-151-1/+2
|
* fixed thread reset to not set stack addressbunnei2014-05-141-1/+1
|
* various cleanups / remove unused codebunnei2014-05-142-65/+29
|
* added a bunch of threading code, recycled from PPSSPP, with lots of hacks in for 3DS... doesn't really do much yet. Just a jumping off pointbunnei2014-05-142-70/+543
|
* - added __KernelLoadExec functionbunnei2014-05-142-13/+35
| | | | - fixed some logging
* added initial kernel/thread modulesbunnei2014-05-104-0/+527
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-23 00:15:12 +0100