Fixed incorrect types for svcBreak

Passing an invalid nmap handle to Remap should throw an error

thread: Remove unnecessary memset from ResetThreadContext()

Added error codes for nvmap

Regular value initialization is adequate here for zeroing out data. It
also has the benefit of not invoking undefined behavior if a non-trivial
type is ever added to the struct for whatever reason.

HwOpus, Implemented DecodeInterleavedWithPerformance

svc: Add missing address range sanitizing checks to MapMemory/UnmapMemory

nvhost_as_gpu: Flush/invalidate CPU VAddr on UnmapBuffer.

gl_shader_decompiler: Implement VMAD

Added error for invalid nmap handles

Used by sonic ages

Fixed block height settings for RenderTargets and Depth Buffers

This adds the missing address range checking that the service functions
do before attempting to map or unmap memory. Given that both service
functions perform the same set of checks in the same order, we can wrap
these into a function and just call it from both functions, which
deduplicates a little bit of code.

Implemented Scissor Testing

gl_shader_decompiler: Implement geometry shaders

kernel/thread: Use a regular pointer for the owner/current process

There's no real need to use a shared pointer in these cases, and only
makes object management more fragile in terms of how easy it would be to
introduce cycles. Instead, just do the simple thing of using a regular
pointer. Much of this is just a hold-over from citra anyways.

It also doesn't make sense from a behavioral point of view for a
process' thread to prolong the lifetime of the process itself (the
process is supposed to own the thread, not the other way around).

ips_layer: Silence truncation and conversion warnings

patch_manager: Return a std::unique_ptr from ParseControlNCA() and GetControlMetadata() instead of a std::shared_ptr

signal_debugger seems like a more fitting name

svcBreak reason should be a u32, not a u64.

Fixed block height settings for RenderTargets and Depth Buffers, and added block width and block depth

gl_shader_decompiler: Remove unused variables in TMML's implementation

Port citra-emu/citra#4310: "Handle touch input"

svcBreak, Signalling to the debugger should not kill execution

telemetry_session: Minor miscellaneous changes

ips_layer: Minor miscellaneous changes

Fixed smo softlock due to incorrect effect state updating

Given "y" isn't always used, but "x" is, we can rearrange this to avoid
unused variable warnings by changing the names of op_a and op_b

There's no "func" parameter, so this can just be removed.

Prevents potential compilation issues in the future by including missing
headers for certain functions and types.

This isn't implemented anywhere, so it can just be removed.