common/thread: Remove unused functions

video_core: Amend constructor initializer list order where applicable

video_core/{gl_rasterizer, gpu_thread}: Remove unused class variables where applicable

kernel/scheduler: Minor tidying up

service/fatal: Name FatalInfo structure members

This was made unused in b404fcdf, but
the parameter itself wasn't removed.

Since C++17, we no longer need to explicitly specify the type of the
mutex within the lock_guard. The type system can now deduce these with
deduction guides.

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.

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: 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

The pusher instance is only ever used in the constructor of the
ThreadManager for creating the thread that the ThreadManager instance
contains. Aside from that, the member is unused, so it can be removed.

This member variable is no longer being used, so it can be removed,
removing a dependency on EmuWindow from the rasterizer's interface"

Specifies the members in the same order that initialization would take
place in.

This also silences -Wreorder warnings.

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.