| Age | Commit message (Collapse) | Author |
|---|
|
vfs: Use sanitized paths within MoveFile() and MoveDirectory()
|
|
card_image: Simplify return statement of GetSubdirectories()
|
|
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.
|
|
Use an approximated amortized amount of ticks when advancing timing.
|
|
scheduler: Make HaveReadyThreads() a const member function
|
|
Fixed a bunch of race conditions when running in multicore mode.
|
|
Kernel/Mutex: Don't duplicate threads in the mutex waiter list.
|
|
We divide the number of ticks to add by the number of cores (4) to obtain a more or less rough estimate of the actual number of ticks added. This assumes that all 4 cores are doing similar work. Previously we were adding ~4 times the number of ticks, thus making the games think that time was going way too fast.
This lets us bypass certain hangs in some games like Breath of the Wild.
We should modify our CoreTiming to support multiple cores (both running in a single thread, and in multiple host threads).
|
|
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.
|
|
Another thread may write to this variable while the core in question is in the middle of checking for a reschedule request.
|
|
Another thread might be in the middle of an SVC, thus altering the state of the schedulers.
|
|
Another thread might be in the middle of a reschedule, thus altering the state of the schedulers.
|
|
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.
|
|
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.
|
|
Makes it consistent with the other VFS interfaces and prevents implicit
construction.
|
|
struct should be used when the data type is very simple or otherwise has
no invariants associated with it. Given these are used to form a
hierarchy, class should be used instead.
|
|
Fixed invalid cast in loader
|
|
GetAudioDeviceServiceWithRevisionInfo (Used by Bloodstained: Curse of the Moon)
|
|
Added GetAudioRendererSampleRate, GetAudioRendererSampleCount & GetAudioRendererMixBufferCount
|
|
This function doesn't modify instance state, so the const qualifier can
be added to it.
|
|
As we're not handling any anything about the revision data for GetAudioDeviceServiceWithRevisionInfo, it's currently marked as stubbed. However for games this shouldn't affect the result. Proper revision info would be more for homebrew.
|
|
Prevents clang-format from butchering them.
|
|
This is required by ARMS.
|
|
Previously these were being unused (or partially unused). While we're at
it, use better naming to make it visibly obvious which variant of the
path is being used.
|
|
We have the aliases, so we may as well use 'em.
|
|
We don't need to write out the construction long-form, we can just let
the language itself work it out off the return type.
|
|
|
|
|
|
GetAudioRendererMixBufferCount
GetAudioRendererSampleRate is set as a "STUB" as a game could check if the sample rate it sent and the sample rate it wants don't match. Just a thought of something which could happen so keeping it as stub for the mean time
|
|
Needed for Retro City Rampage to go in game
|
|
GetMessageForResultStatus takes a u16, not a size_t.
|
|
Several Friend service fixes
|
|
- Used by Splatoon 2.
|
|
|
|
request.
|
|
Gets the class out of the global namespace.
|
|
loader: Add more descriptive errors
|
|
Instead, we make a struct for renderer settings and allow the renderer
to update all of these settings, getting rid of the need for
global-scoped variables.
This also uncovered a few indirect inclusions for certain headers, which
this commit also fixes.
|
|
Full list of new errors and descriptions in core/loader/loader.h
|
|
core: Make function reference parameters const where applicable
|
|
fsp_srv: Emplace entries first when building index instead of emplacing last
|
|
vfs: Add VfsFilesystem and fix RealVfs* implementations
|
|
This is simply copied by value, so there's no need to make it a
modifiable reference.
While we're at it, make the names of the parameters match its
definition.
|
|
This function doesn't modify anything within the reference Thread
instance.
|
|
|
|
Service/Account: stub LoadImage function
|
|
|
|
|
|
|
|
|
