summaryrefslogtreecommitdiff
path: root/Source/Core/VideoCommon/OpcodeDecoding.cpp
diff options
context:
space:
mode:
authordegasus <wickmarkus@web.de>2017-01-27 09:44:31 +0100
committerdegasus <wickmarkus@web.de>2017-01-27 19:04:56 +0100
commit3787af9ee9ca3279f437e17b1a343f6aa7a091ef (patch)
treede21c34d03dbb1a2ae4fef37485c81497a850a75 /Source/Core/VideoCommon/OpcodeDecoding.cpp
parent017e8050b64ca86ce6ca91092b07bc078bb0ba75 (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.cpp43
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;