- Used by Final Fantasy X/X-2 HD Remaster.

Should finally resolve building with clang.

Recent changes to the build system that made more warnings be flagged as
errors caused building via clang to break.

Fixes #4795

We were performing an int < size_t comparison. We can just correct the
type of the induction variable.

It's not needed to have it in its previous position anymore

According to Atmosphére (https://github.com/Atmosphere-NX/Atmosphere/blob/c7026b90940a1d88f9c10a6d98263bf22e654fa5/libraries/libstratosphere/include/stratosphere/ro/ro_types.hpp), nrr_kind (Atmosphére calls it "type") is 7.0.0+

This was based on Switchbrew pages:

https://switchbrew.org/wiki/NRR

https://switchbrew.org/wiki/NRO

Keeps the service function tables up to date.

Updated based off information on SwitchBrew.

- Should be no functional changes.

- Includes removing some service impls. that are untested.

- Fixes #3541, used by Final Fantasy VIII Remastered.

With all of the trivial parts of the memory interface moved over, we can
get right into moving over the bits that are used.

Note that this does require the use of GetInstance from the global
system instance to be used within hle_ipc.cpp and the gdbstub. This is
fine for the time being, as they both already rely on the global system
instance in other functions. These will be removed in a change directed
at both of these respectively.

For now, it's sufficient, as it still accomplishes the goal of
de-globalizing the memory code.

These functions are marked for deprecation and it's recommended that the
*_ret variants be used instead.

Migrates the HLE service code off the use of directly accessing the
global system instance where trivially able to do so.

This removes all usages of Core::CurrentProcess from the service code,
only 8 occurrences of this function exist elsewhere. There's still quite
a bit of "System::GetInstance()" being used, however this was able to
replace a few instances.

Prior to PR, Yuzu did not restore memory to RW-
on unmap of mirrored memory or unloading of NRO.

(In fact, in the NRO case, the memory was unmapped
instead of reprotected to --- on Load, so it was
actually lost entirely...)

This PR addresses that, and restores memory to RW-
as it should.

This fixes a crash in Super Smash Bros when creating
a World of Light save for the first time, and possibly
other games/circumstances.

These can be generified together by using a concept type to designate
them. This also has the benefit of not making copies of potentially very
large arrays.

Updates function tables based off information from SwitchBrew.

This doesn't modify instance state, so it can be made const.

The initial two words indicate a process ID. Also UnloadNro only
specifies one address, not two.

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.

In all cases that these functions are needed, the VMManager can just be
retrieved and used instead of providing the same functions in Process'
interface.

This also makes it a little nicer dependency-wise, since it gets rid of
cases where the VMManager interface was being used, and then switched
over to using the interface for a Process instance. Instead, it makes
all accesses uniform and uses the VMManager instance for all necessary
tasks.

All the basic memory mapping functions did was forward to the Process'
VMManager instance anyways.

We don't need to specify all of the ARM interfaces explicitly.

The first word is just a padding byte, it's not an actual entry
instruction. Also renames the rest of the entries according to
SwitchBrew.

The padding after the magic signature value should be 12 bytes rather
than 28 bytes. The other 16 should be placed after the title ID pattern.

Changed logging to be "Log before execution", Added more error logging, all services should now log on some level

The separate enum isn't particularly necessary here, and the values can
just be directly put into the ResultCode instances, given the names are
also self-documenting here.

Includes actual unmapping and address error checking.