| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| |\
| |
| | |
shader_cache: Permit a Null Shader in case of a bad host_ptr.
|
| | | |
|
| |\ \
| | |
| | | |
maxwell_3d: Reduce severity of ProcessSyncPoint
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
shader_ir: Implement AOFFI for TEX and TLD4
|
| | | | | |
|
| | | | |
| | | |
| | | | |
Co-Authored-By: ReinUsesLisp <reinuseslisp@airmail.cc>
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
core/memory: Minor simplifications to page table management
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Now that nothing actually touches the internal page table aside from the
memory subsystem itself, we can remove the accessor to it.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Given the page table will always be guaranteed to be that of whatever
the current process is, we no longer need to keep this around.
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | | |
Centralizes the page table switching to one spot, rather than making
calling code deal with it everywhere.
|
| |\ \ \ \
| | | | |
| | | | | |
gl_state: Rework to enable individual applies
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | | |
gl_shader_disk_cache: Use Zstandard for compression
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
decompression instead.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | | |
kernel/{server_port, server_session}: Return pairs instead of tuples from pair creation functions
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Keeps the return type consistent with the function name. While we're at
it, we can also reduce the amount of boilerplate involved with handling
these by using structured bindings.
|
| | | |_|/ / /
| |/| | | |
| | | | | |
| | | | | | |
Returns the same type that the function name describes.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | | |
core/memory: Remove unused enum constants
|
| | |/ / / / /
| | | | | |
| | | | | |
| | | | | | |
These are holdovers from Citra and can be removed.
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
memory_manager: Improved implementation of read/write/copy block.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Fixes graphical issues with Chocobo's Mystery Dungeon EVERY BUDDY!
- Fixes a crash with Mario Tennis Aces
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Implement SyncPoint Register in the GPU.
|
| | | |_|_|/ /
| |/| | | | |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | | |
kernel/server_session: Provide a GetName() override
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Given server sessions can be given a name, we should allow retrieving
it instead of using the default implementation of GetName(), which would
just return "[UNKNOWN KERNEL OBJECT]".
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
common/multi_level_queue: Silence truncation warnings
|
| | | | | | | | | |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
return types
Makes the return type consistently uniform (like the intrinsics we're
wrapping). This also conveniently silences a truncation warning within
the kernel multi_level_queue.
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | | |
Port citra-emu/citra#4651: "gdbstub: Fix some bugs in IsMemoryBreak() and ServeBreak. Add workaround to let watchpoints break into GDB."
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
let watchpoints break into GDB. (#4651)
* gdbstub: fix IsMemoryBreak() returning false while connected to client
As a result, the only existing codepath for a memory watchpoint hit to break into GDB (InterpeterMainLoop, GDB_BP_CHECK, ARMul_State::RecordBreak) is finally taken,
which exposes incorrect logic* in both RecordBreak and ServeBreak.
* a blank BreakpointAddress structure is passed, which sets r15 (PC) to NULL
* gdbstub: DynCom: default-initialize two members/vars used in conditionals
* gdbstub: DynCom: don't record memory watchpoint hits via RecordBreak()
For now, instead check for GDBStub::IsMemoryBreak() in InterpreterMainLoop and ServeBreak.
Fixes PC being set to a stale/unhit breakpoint address (often zero) when a memory watchpoint (rwatch, watch, awatch) is handled in ServeBreak() and generates a GDB trap.
Reasons for removing a call to RecordBreak() for memory watchpoints:
* The``breakpoint_data`` we pass is typed Execute or None. It describes the predicted next code breakpoint hit relative to PC;
* GDBStub::IsMemoryBreak() returns true if a recent Read/Write operation hit a watchpoint. It doesn't specify which in return, nor does it trace it anywhere. Thus, the only data we could give RecordBreak() is a placeholder BreakpointAddress at offset NULL and type Access. I found the idea silly, compared to simply relying on GDBStub::IsMemoryBreak().
There is currently no measure in the code that remembers the addresses (and types) of any watchpoints that were hit by an instruction, in order to send them to GDB as "extended stop information."
I'm considering an implementation for this.
* gdbstub: Change an ASSERT to DEBUG_ASSERT
I have never seen the (Reg[15] == last_bkpt.address) assert fail in practice, even after several weeks of (locally) developping various branches around GDB. Only leave it inside Debug builds.
|
| |\ \ \ \ \ \ \ \ \
| |_|_|_|_|/ / / /
|/| | | | | | | | |
video_core/engines: Remove unnecessary inclusions where applicable
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
These aren't used externally by anything, so they can be made private
data members.
|
| | | |/ / / / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Replaces header inclusions with forward declarations where applicable
and also removes unused headers within the cpp file. This reduces a few
more dependencies on core/memory.h
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | | |
video_core/memory_manager: Mark a few member functions with the const qualifier
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Given this doesn't actually alter internal state, this can be made a
const member function.
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Now, since we have a const qualified variant of GetPointer(), we can put
it to use in ReadBlock() to retrieve the source pointer that is passed
into memcpy.
Now block reading may be done from a const context.
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Allows retrieving read-only pointers from a const context externally.
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
This doesn't modify internal state, so it can be made a const member
function.
|
| | |/ / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This doesn't modify any internal state, so it can be made a const member
function to allow its use in const contexts.
|