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.

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.

- Fixes graphical issues from transitions in Super Mario Odyssey.

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.

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

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

- Avoid a crash in Octopath Traveler.

- Avoid a crash in Xenoblade Chronicles 2.

- Fixes a crash in Puyo Puyo Tetris.

When #2247 was created, thread_queue_list.h was the only user of
boost-related code, however #2252 moved the page table struct into
common, which makes use of Boost.ICL, so we need to add the dependency
to the common library's link interface again.

The segment itself isn't actually modified.

Given this is utilized by the loaders, this allows avoiding inclusion of
the kernel process definitions where avoidable.

This also keeps the loading format for all executable data separate from
the kernel objects.

Given the class is now currently unused, it can be removed.

Neither the NRO or NSO loaders actually make use of the functions or
members provided by the Linker interface, so we can just remove the
inheritance altogether.

While we're at it, give each entry some documentation.

In both cases, we weren't actually returning anything, which is
undefined behavior.

Just makes the definitions a little bit more tidy.

This just acts as a basic setter for a given PID value and performs no
further checking, so we can just store the passed in value.