Resolves a -Wextra-semi warning.

Based off RE, most of these structure members are register values, which
makes, sense given this service is used to convey fatal errors.

One member indicates the program entry point address, one is a set of
bit flags used to determine which registers to print, and one member
indicates the architecture type.

The only member that still isn't determined is the final member within
the data structure.

Kernel: Fixes to Arbitration and SignalProcessWideKey Management

Replace old Thread Queue for a new Multi Level Queue

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.