diff options
| author | degasus <wickmarkus@web.de> | 2017-01-27 09:44:31 +0100 |
|---|---|---|
| committer | degasus <wickmarkus@web.de> | 2017-01-27 19:04:56 +0100 |
| commit | 3787af9ee9ca3279f437e17b1a343f6aa7a091ef (patch) | |
| tree | de21c34d03dbb1a2ae4fef37485c81497a850a75 /Source/Core/VideoCommon/OpcodeDecoding.cpp | |
| parent | 017e8050b64ca86ce6ca91092b07bc078bb0ba75 (diff) | |
CommandProcessor: Limit scope of ugly SCPFifoStruct.
It's only used as an interface between two classes. So no need to declare
it in the backend export header.
Diffstat (limited to 'Source/Core/VideoCommon/OpcodeDecoding.cpp')
| -rw-r--r-- | Source/Core/VideoCommon/OpcodeDecoding.cpp | 43 |
1 files changed, 1 insertions, 42 deletions
diff --git a/Source/Core/VideoCommon/OpcodeDecoding.cpp b/Source/Core/VideoCommon/OpcodeDecoding.cpp index a0b73effaf..5299401f68 100644 --- a/Source/Core/VideoCommon/OpcodeDecoding.cpp +++ b/Source/Core/VideoCommon/OpcodeDecoding.cpp @@ -75,47 +75,6 @@ static void InterpretDisplayListPreprocess(u32 address, u32 size) } } -static void UnknownOpcode(u8 cmd_byte, void* buffer, bool preprocess) -{ - // TODO(Omega): Maybe dump FIFO to file on this error - PanicAlertT("GFX FIFO: Unknown Opcode (0x%02x @ %p, %s).\n" - "This means one of the following:\n" - "* The emulated GPU got desynced, disabling dual core can help\n" - "* Command stream corrupted by some spurious memory bug\n" - "* This really is an unknown opcode (unlikely)\n" - "* Some other sort of bug\n\n" - "Further errors will be sent to the Video Backend log and\n" - "Dolphin will now likely crash or hang. Enjoy.", - cmd_byte, buffer, preprocess ? "preprocess=true" : "preprocess=false"); - - { - SCPFifoStruct& fifo = CommandProcessor::fifo; - - PanicAlert("Illegal command %02x\n" - "CPBase: 0x%08x\n" - "CPEnd: 0x%08x\n" - "CPHiWatermark: 0x%08x\n" - "CPLoWatermark: 0x%08x\n" - "CPReadWriteDistance: 0x%08x\n" - "CPWritePointer: 0x%08x\n" - "CPReadPointer: 0x%08x\n" - "CPBreakpoint: 0x%08x\n" - "bFF_GPReadEnable: %s\n" - "bFF_BPEnable: %s\n" - "bFF_BPInt: %s\n" - "bFF_Breakpoint: %s\n" - "bFF_GPLinkEnable: %s\n" - "bFF_HiWatermarkInt: %s\n" - "bFF_LoWatermarkInt: %s\n", - cmd_byte, fifo.CPBase, fifo.CPEnd, fifo.CPHiWatermark, fifo.CPLoWatermark, - fifo.CPReadWriteDistance, fifo.CPWritePointer, fifo.CPReadPointer, fifo.CPBreakpoint, - fifo.bFF_GPReadEnable ? "true" : "false", fifo.bFF_BPEnable ? "true" : "false", - fifo.bFF_BPInt ? "true" : "false", fifo.bFF_Breakpoint ? "true" : "false", - fifo.bFF_GPLinkEnable ? "true" : "false", fifo.bFF_HiWatermarkInt ? "true" : "false", - fifo.bFF_LoWatermarkInt ? "true" : "false"); - } -} - void Init() { s_bFifoErrorSeen = false; @@ -277,7 +236,7 @@ u8* Run(DataReader src, u32* cycles, bool in_display_list) else { if (!s_bFifoErrorSeen) - UnknownOpcode(cmd_byte, opcodeStart, is_preprocess); + CommandProcessor::HandleUnknownOpcode(cmd_byte, opcodeStart, is_preprocess); ERROR_LOG(VIDEO, "FIFO: Unknown Opcode(0x%02x @ %p, preprocessing = %s)", cmd_byte, opcodeStart, is_preprocess ? "yes" : "no"); s_bFifoErrorSeen = true; |
