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.

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

We already have the service related stuff set up for this, however, it's
missing the function table.

Makes it more evident that one is for actual code and one is for actual
data. Mutable and static are less than ideal terms here, because
read-only data is technically not mutable, but we were mapping it with
that label.

This should actually be using the data flags, rather than the code
flags.

In 93da8e0a, the page table construct
was moved to the common library (which utilized these inclusions). Since
the move, nothing requires these headers to be included within the
memory header.

input_common/sdl: Correct return values within GetPollers implementations

Introduced as a result of #2090, we already define the copy constructor
further down below, so this isn't needed.

Port citra-emu/citra#4244 and citra-emu/citra#4599: Changes to BitField

file_sys/content_archive: Amend name of Data_Unknown5 enum entry

 common/uint128: Add missing header guard 

core/kernel: Migrate CodeSet to its own source files

- GPU will be released on shutdown, before pages are unmapped.
- On subsequent runs, current_page_table will be not nullptr, but GPU might not be valid yet.