Many of these functions are carried over from Dolphin (where they aren't
used anymore). Given these have no use (and we really shouldn't be
screwing around with OS-specific thread scheduler handling from the
emulator, these can be removed.

The function for setting the thread name is left, however, since it can
have debugging utility usages.

kernel/vm_manager: Unify heap allocation/freeing functions

video_core: Add missing override specifiers

video_core/gpu: Amend typo in GPU member variable name

Ensures that the signatures will always match with the base class.

Also silences a few compilation warnings.

smaphore -> semaphore

kernel/process: Remove unused AddressMapping struct

core/core_timing: Make callback parameters consistent

kernel/kernel: Remove unnecessary forward declaration

core/cheat_engine: Make MemoryReadImpl and MemoryWriteImpl internally linked

These don't need to be visible outside of the translation unit, so they
can be enclosed within an anonymous namespace.

In some cases, our callbacks were using s64 as a parameter, and in other
cases, they were using an int, which is inconsistent.

To make all callbacks consistent, we can just use an s64 as the type for
late cycles, given it gets rid of the need to cast internally.

While we're at it, also resolve some signed/unsigned conversions that
were occurring related to the callback registration.

This is no longer necessary, as ResultVal isn't used anywhere in the
header.

Another leftover from citra that's now no longer necessary.

One behavior that we weren't handling properly in our heap allocation
process was the ability for the heap to be shrunk down in size if a
larger size was previously requested.

This adds the basic behavior to do so and also gets rid of HeapFree, as
it's no longer necessary now that we have allocations and deallocations
going through the same API function.

While we're at it, fully document the behavior that this function
performs.

Makes it more obvious that this function is intending to stand in for
the actual supervisor call itself, and not acting as a general heap
allocation function.

Also the following change will merge the freeing behavior of HeapFree
into this function, so leaving it as HeapAllocate would be misleading.

In cases where HeapAllocate is called with the same size of the current
heap, we can simply do nothing and return successfully.

This avoids doing work where we otherwise don't have to. This is also
what the kernel itself does in this scenario.

Over time these have fallen out of use due to refactoring, so these can
be removed.

This isn't required anymore, as all the kernel ever queries is the size
of the current heap, not the total usage of it.

Another holdover from citra that can be tossed out is the notion of the
heap needing to be allocated in different addresses. On the switch, the
base address of the heap will always be managed by the memory allocator
in the kernel, so this doesn't need to be specified in the function's
interface itself.

The heap on the switch is always allocated with read/write permissions,
so we don't need to add specifying the memory permissions as part of the
heap allocation itself either.

This also corrects the error code returned from within the function.
If the size of the heap is larger than the entire heap region, then the
kernel will report an out of memory condition.

 core/hle/kernel: Split transfer memory handling out into its own class

set_sys: Implement GetFirmwareVersion(2) for libnx hosversion

Migrate off directly modifying CMAKE_* compilation-related flags directly

loader/nso: Minor refactoring

file_sys/cheat_engine: Remove use of global system accessors

gpu: Rewrite MemoryManager based on the VMManager implementation.

Revert "Devirtualize Register/Unregister and use a wrapper instead."

Makes it impossible to indirectly violate the ODR in some other
translation unit due to these existing.

Instead, pass in the core timing instance and make the dependency
explicit in the interface.

Now that the NSO header has the proper size, we can just use sizeof on
it instead of having magic constants.

This source file was utilizing its own version of the NSO header.
Instead of keeping this around, we can have the patch manager also use
the version of the header that we have defined in loader/nso.h

The total struct itself is 0x100 (256) bytes in size, so we should be
providing that amount of data.

Without the data, this can result in omitted data from the final loaded
NSO file.

These correspond to the NSOBuildHeader.

core/hle/kernel: Make Mutex a per-process class.

core/memory: Remove unnecessary includes

- Fixes graphical issues from transitions in Super Mario Odyssey.

kernel/vm_manager: Amend flag value for code data

service/am: Add function table for IDebugFunctions

file_sys: Implement parser and interpreter for game memory cheats