Previously we were overwriting the error case with a success code
further down (which is definitely not what we should be doing here).

We already have the thread instance that was created under the current
process, so we can just pass the handle table of it along to retrieve
the owner of the mutex.

Removes a few unnecessary dependencies on core-related machinery, such
as the core.h and memory.h, which reduces the amount of rebuilding
necessary if those files change.

This also uncovered some indirect dependencies within other source
files. This also fixes those.

Places all error codes in an easily includable header.

This also corrects the unsupported error code (I accidentally used the
hex value when I meant to use the decimal one).

Allows getting rid of reliance on the global accessor functions and
instead operating on the provided system instance.

- Invert conditions into guard clases where applicable.
- Mark std::vector parameter of WakeThreads as const

Places all of the functions for address arbiter operation into a class.
This will be necessary for future deglobalizing efforts related to both
the memory and system itself.

This isn't needed anymore, according to Hexagon

This is a TODO carried over from Citra that doesn't apply here.

Significantly tidies up two guard conditionals.

Makes the interface uniform when it comes to checking various memory
regions.

Provides a bit of a more proper interface for these functions.

Reduces the potential amount of rebuilding necessary if any headers
change. In particular, we were including a header from the core library
when we don't even link the core library to the web_service library, so
this also gets rid of an indirect dependency.

Moves local global state into the Impl class itself and initializes it
at the creation of the instance instead of in the function.

This makes it nicer for weakly-ordered architectures, given the
CreateEntry() class won't need to have atomic loads executed for each
individual call to the CreateEntry class.

This function is only ever used within this source file and makes it
easier to remove static state in the following change.

MSVC does not seem to like using constexpr values in a lambda that were declared outside of it.
Previously on MSVC build the hotkeys to inc-/decrease the speed limit were not working correctly because in the lambda the SPEED_LIMIT_STEP had garbage values.
After googling around a bit I found: https://github.com/codeplaysoftware/computecpp-sdk/issues/95 which seems to be a similar issue.
Trying the suggested fix to make the variable static constexpr also fixes the bug here.

The comment already invalidates itself: neither MMIO nor rasterizer cache belongsHLE kernel state. This mutex has a too large scope if MMIO or cache is included, which is prone to dead lock when multiple thread acquires these resource at the same time. If necessary, each MMIO component or rasterizer should have their own lock.

This buffer cache is just like OpenGL's buffer cache with some minor
style changes. It uses VKStreamBuffer.

This currently has the same behavior as the regular
OpenAudioRenderer API function, so we can just move the code within
OpenAudioRenderer to an internal function that both service functions
call.

This service function appears to do nothing noteworthy on the switch.
All it does at the moment is either return an error code or abort the
system. Given we obviously don't want to kill the system, we just opt
for always returning the error code.

Unbinding was required when OpenGL delete operations didn't unbind a
resource if it was bound. This is no longer needed and can be removed.

rasterizer_cache: mark reinterpreted surfaces and add ability to reload marked surfaces on next use.

rasterizer_cache: Expose FlushObject to Child classes and allow redefining of Register and Unregister

- Memory::MapPages total samplecount was reduced from 4.6% to 1.06%.
- From main menu into the game from 1.03% to 0.35%

cubeb now requires that COM explicitly be initialized on the thread
prior to calling cubeb_init.

The SM namespace is within the Service namespace, so this was forward
declaring a type that didn't exist.