2015-05-23 22:55:12 -06:00
|
|
|
// Copyright 2008 Dolphin Emulator Project
|
2021-07-04 19:22:19 -06:00
|
|
|
// SPDX-License-Identifier: GPL-2.0-or-later
|
2008-12-07 22:25:12 -07:00
|
|
|
|
|
|
|
// DL facts:
|
|
|
|
// Ikaruga uses (nearly) NO display lists!
|
|
|
|
// Zelda WW uses TONS of display lists
|
|
|
|
// Zelda TP uses almost 100% display lists except menus (we like this!)
|
2009-08-09 05:03:58 -06:00
|
|
|
// Super Mario Galaxy has nearly all geometry and more than half of the state in DLs (great!)
|
2008-12-07 22:25:12 -07:00
|
|
|
|
|
|
|
// Note that it IS NOT GENERALLY POSSIBLE to precompile display lists! You can compile them as they
|
2019-12-05 06:18:12 -07:00
|
|
|
// are while interpreting them, and hope that the vertex format doesn't change, though, if you do
|
|
|
|
// it right when they are called. The reason is that the vertex format affects the sizes of the
|
|
|
|
// vertices.
|
2008-12-07 22:25:12 -07:00
|
|
|
|
2014-02-17 03:18:15 -07:00
|
|
|
#include "VideoCommon/OpcodeDecoding.h"
|
2019-12-05 06:47:22 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
#include "Common/Assert.h"
|
2016-01-17 14:54:31 -07:00
|
|
|
#include "Common/Logging/Log.h"
|
2014-02-17 03:18:15 -07:00
|
|
|
#include "Core/FifoPlayer/FifoRecorder.h"
|
|
|
|
#include "Core/HW/Memmap.h"
|
|
|
|
#include "VideoCommon/BPMemory.h"
|
2014-02-18 18:27:20 -07:00
|
|
|
#include "VideoCommon/CPMemory.h"
|
2014-02-17 03:18:15 -07:00
|
|
|
#include "VideoCommon/CommandProcessor.h"
|
|
|
|
#include "VideoCommon/DataReader.h"
|
|
|
|
#include "VideoCommon/Fifo.h"
|
|
|
|
#include "VideoCommon/Statistics.h"
|
2021-04-22 21:57:56 -06:00
|
|
|
#include "VideoCommon/VertexLoaderBase.h"
|
2014-02-17 03:18:15 -07:00
|
|
|
#include "VideoCommon/VertexLoaderManager.h"
|
2021-04-22 21:57:56 -06:00
|
|
|
#include "VideoCommon/VertexShaderManager.h"
|
2014-02-18 18:27:20 -07:00
|
|
|
#include "VideoCommon/XFMemory.h"
|
2021-04-22 21:57:56 -06:00
|
|
|
#include "VideoCommon/XFStructs.h"
|
2014-02-18 18:27:20 -07:00
|
|
|
|
2016-01-23 23:29:44 -07:00
|
|
|
namespace OpcodeDecoder
|
|
|
|
{
|
2019-12-05 06:11:52 -07:00
|
|
|
bool g_record_fifo_data = false;
|
|
|
|
|
2014-11-27 15:53:11 -07:00
|
|
|
template <bool is_preprocess>
|
2021-04-22 21:57:56 -06:00
|
|
|
class RunCallback final : public Callback
|
2008-12-07 22:25:12 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
public:
|
|
|
|
OPCODE_CALLBACK(void OnXF(u16 address, u8 count, const u8* data))
|
|
|
|
{
|
|
|
|
m_cycles += 18 + 6 * count;
|
2019-12-05 06:24:47 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if constexpr (!is_preprocess)
|
2019-12-05 06:24:47 -07:00
|
|
|
{
|
2021-05-06 18:22:31 -06:00
|
|
|
LoadXFReg(address, count, data);
|
2019-12-05 06:24:47 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
INCSTAT(g_stats.this_frame.num_xf_loads);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnCP(u8 command, u32 value))
|
2008-12-07 22:25:12 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
m_cycles += 12;
|
2021-05-13 17:05:31 -06:00
|
|
|
const u8 sub_command = command & CP_COMMAND_MASK;
|
2021-04-22 21:57:56 -06:00
|
|
|
if constexpr (!is_preprocess)
|
2008-12-07 22:25:12 -07:00
|
|
|
{
|
2021-05-13 17:05:31 -06:00
|
|
|
if (sub_command == MATINDEX_A)
|
2021-04-22 21:57:56 -06:00
|
|
|
VertexShaderManager::SetTexMatrixChangedA(value);
|
2021-05-13 17:05:31 -06:00
|
|
|
else if (sub_command == MATINDEX_B)
|
2021-04-22 21:57:56 -06:00
|
|
|
VertexShaderManager::SetTexMatrixChangedB(value);
|
2021-05-13 17:05:31 -06:00
|
|
|
else if (sub_command == VCD_LO || sub_command == VCD_HI)
|
|
|
|
{
|
|
|
|
VertexLoaderManager::g_main_vat_dirty = BitSet8::AllTrue(CP_NUM_VAT_REG);
|
|
|
|
VertexLoaderManager::g_bases_dirty = true;
|
|
|
|
}
|
|
|
|
else if (sub_command == CP_VAT_REG_A || sub_command == CP_VAT_REG_B ||
|
|
|
|
sub_command == CP_VAT_REG_C)
|
|
|
|
{
|
|
|
|
VertexLoaderManager::g_main_vat_dirty[command & CP_VAT_MASK] = true;
|
|
|
|
}
|
|
|
|
else if (sub_command == ARRAY_BASE)
|
|
|
|
{
|
|
|
|
VertexLoaderManager::g_bases_dirty = true;
|
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
|
|
|
|
INCSTAT(g_stats.this_frame.num_cp_loads);
|
|
|
|
}
|
2021-05-13 17:05:31 -06:00
|
|
|
else if constexpr (is_preprocess)
|
|
|
|
{
|
|
|
|
if (sub_command == VCD_LO || sub_command == VCD_HI)
|
|
|
|
{
|
|
|
|
VertexLoaderManager::g_preprocess_vat_dirty = BitSet8::AllTrue(CP_NUM_VAT_REG);
|
|
|
|
}
|
|
|
|
else if (sub_command == CP_VAT_REG_A || sub_command == CP_VAT_REG_B ||
|
|
|
|
sub_command == CP_VAT_REG_C)
|
|
|
|
{
|
|
|
|
VertexLoaderManager::g_preprocess_vat_dirty[command & CP_VAT_MASK] = true;
|
|
|
|
}
|
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
GetCPState().LoadCPReg(command, value);
|
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnBP(u8 command, u32 value))
|
|
|
|
{
|
|
|
|
m_cycles += 12;
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if constexpr (is_preprocess)
|
Add the 'desynced GPU thread' mode.
It's a relatively big commit (less big with -w), but it's hard to test
any of this separately...
The basic problem is that in netplay or movies, the state of the CPU
must be deterministic, including when the game receives notification
that the GPU has processed FIFO data. Dual core mode notifies the game
whenever the GPU thread actually gets around to doing the work, so it
isn't deterministic. Single core mode is because it notifies the game
'instantly' (after processing the data synchronously), but it's too slow
for many systems and games.
My old dc-netplay branch worked as follows: everything worked as normal
except the state of the CP registers was a lie, and the CPU thread only
delivered results when idle detection triggered (waiting for the GPU if
they weren't ready at that point). Usually, a game is idle iff all the
work for the frame has been done, except for a small amount of work
depending on the GPU result, so neither the CPU or the GPU waiting on
the other affected performance much. However, it's possible that the
game could be waiting for some earlier interrupt, and any of several
games which, for whatever reason, never went into a detectable idle
(even when I tried to improve the detection) would never receive results
at all. (The current method should have better compatibility, but it
also has slightly higher overhead and breaks some other things, so I
want to reimplement this, hopefully with less impact on the code, in the
future.)
With this commit, the basic idea is that the CPU thread acts as if the
work has been done instantly, like single core mode, but actually hands
it off asynchronously to the GPU thread (after backing up some data that
the game might change in memory before it's actually done). Since the
work isn't done, any feedback from the GPU to the CPU, such as real
XFB/EFB copies (virtual are OK), EFB pokes, performance queries, etc. is
broken; but most games work with these options disabled, and there is no
need to try to detect what the CPU thread is doing.
Technically: when the flag g_use_deterministic_gpu_thread (currently
stuck on) is on, the CPU thread calls RunGpu like in single core mode.
This function synchronously copies the data from the FIFO to the
internal video buffer and updates the CP registers, interrupts, etc.
However, instead of the regular ReadDataFromFifo followed by running the
opcode decoder, it runs ReadDataFromFifoOnCPU ->
OpcodeDecoder_Preprocess, which relatively quickly scans through the
FIFO data, detects SetFinish calls etc., which are immediately fired,
and saves certain associated data from memory (e.g. display lists) in
AuxBuffers (a parallel stream to the main FIFO, which is a bit slow at
the moment), before handing the data off to the GPU thread to actually
render. That makes up the bulk of this commit.
In various circumstances, including the aforementioned EFB pokes and
performance queries as well as swap requests (i.e. the end of a frame -
we don't want the CPU potentially pumping out frames too quickly and the
GPU falling behind*), SyncGPU is called to wait for actual completion.
The overhead mainly comes from OpcodeDecoder_Preprocess (which is,
again, synchronous), as well as the actual copying.
Currently, display lists and such are escrowed from main memory even
though they usually won't change over the course of a frame, and
textures are not even though they might, resulting in a small chance of
graphical glitches. When the texture locking (i.e. fault on write) code
lands, I can make this all correct and maybe a little faster.
* This suggests an alternate determinism method of just delaying results
until a short time before the end of each frame. For all I know this
might mostly work - I haven't tried it - but if any significant work
hinges on the competion of render to texture etc., the frame will be
missed.
2014-08-27 20:56:19 -06:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
LoadBPRegPreprocess(command, value, m_cycles);
|
Add the 'desynced GPU thread' mode.
It's a relatively big commit (less big with -w), but it's hard to test
any of this separately...
The basic problem is that in netplay or movies, the state of the CPU
must be deterministic, including when the game receives notification
that the GPU has processed FIFO data. Dual core mode notifies the game
whenever the GPU thread actually gets around to doing the work, so it
isn't deterministic. Single core mode is because it notifies the game
'instantly' (after processing the data synchronously), but it's too slow
for many systems and games.
My old dc-netplay branch worked as follows: everything worked as normal
except the state of the CP registers was a lie, and the CPU thread only
delivered results when idle detection triggered (waiting for the GPU if
they weren't ready at that point). Usually, a game is idle iff all the
work for the frame has been done, except for a small amount of work
depending on the GPU result, so neither the CPU or the GPU waiting on
the other affected performance much. However, it's possible that the
game could be waiting for some earlier interrupt, and any of several
games which, for whatever reason, never went into a detectable idle
(even when I tried to improve the detection) would never receive results
at all. (The current method should have better compatibility, but it
also has slightly higher overhead and breaks some other things, so I
want to reimplement this, hopefully with less impact on the code, in the
future.)
With this commit, the basic idea is that the CPU thread acts as if the
work has been done instantly, like single core mode, but actually hands
it off asynchronously to the GPU thread (after backing up some data that
the game might change in memory before it's actually done). Since the
work isn't done, any feedback from the GPU to the CPU, such as real
XFB/EFB copies (virtual are OK), EFB pokes, performance queries, etc. is
broken; but most games work with these options disabled, and there is no
need to try to detect what the CPU thread is doing.
Technically: when the flag g_use_deterministic_gpu_thread (currently
stuck on) is on, the CPU thread calls RunGpu like in single core mode.
This function synchronously copies the data from the FIFO to the
internal video buffer and updates the CP registers, interrupts, etc.
However, instead of the regular ReadDataFromFifo followed by running the
opcode decoder, it runs ReadDataFromFifoOnCPU ->
OpcodeDecoder_Preprocess, which relatively quickly scans through the
FIFO data, detects SetFinish calls etc., which are immediately fired,
and saves certain associated data from memory (e.g. display lists) in
AuxBuffers (a parallel stream to the main FIFO, which is a bit slow at
the moment), before handing the data off to the GPU thread to actually
render. That makes up the bulk of this commit.
In various circumstances, including the aforementioned EFB pokes and
performance queries as well as swap requests (i.e. the end of a frame -
we don't want the CPU potentially pumping out frames too quickly and the
GPU falling behind*), SyncGPU is called to wait for actual completion.
The overhead mainly comes from OpcodeDecoder_Preprocess (which is,
again, synchronous), as well as the actual copying.
Currently, display lists and such are escrowed from main memory even
though they usually won't change over the course of a frame, and
textures are not even though they might, resulting in a small chance of
graphical glitches. When the texture locking (i.e. fault on write) code
lands, I can make this all correct and maybe a little faster.
* This suggests an alternate determinism method of just delaying results
until a short time before the end of each frame. For all I know this
might mostly work - I haven't tried it - but if any significant work
hinges on the competion of render to texture etc., the frame will be
missed.
2014-08-27 20:56:19 -06:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
else
|
2014-11-15 08:24:06 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
LoadBPReg(command, value, m_cycles);
|
|
|
|
INCSTAT(g_stats.this_frame.num_bp_loads);
|
2014-11-15 08:24:06 -07:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnIndexedLoad(CPArray array, u32 index, u16 address, u8 size))
|
|
|
|
{
|
|
|
|
m_cycles += 6;
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if constexpr (is_preprocess)
|
|
|
|
PreprocessIndexedXF(array, index, address, size);
|
|
|
|
else
|
|
|
|
LoadIndexedXF(array, index, address, size);
|
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnPrimitiveCommand(OpcodeDecoder::Primitive primitive, u8 vat,
|
|
|
|
u32 vertex_size, u16 num_vertices, const u8* vertex_data))
|
|
|
|
{
|
|
|
|
// load vertices
|
|
|
|
const u32 size = vertex_size * num_vertices;
|
2019-12-05 06:24:47 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
// HACK
|
|
|
|
DataReader src{const_cast<u8*>(vertex_data), const_cast<u8*>(vertex_data) + size};
|
|
|
|
const u32 bytes =
|
|
|
|
VertexLoaderManager::RunVertices(vat, primitive, num_vertices, src, is_preprocess);
|
2019-12-05 06:24:47 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
ASSERT(bytes == size);
|
2019-12-05 06:24:47 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
// 4 GPU ticks per vertex, 3 CPU ticks per GPU tick
|
|
|
|
m_cycles += num_vertices * 4 * 3 + 6;
|
|
|
|
}
|
|
|
|
// This can't be inlined since it calls Run, which makes it recursive
|
|
|
|
// m_in_display_list prevents it from actually recursing infinitely, but there's no real benefit
|
|
|
|
// to inlining Run for the display list directly.
|
|
|
|
OPCODE_CALLBACK_NOINLINE(void OnDisplayList(u32 address, u32 size))
|
|
|
|
{
|
|
|
|
m_cycles += 6;
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if (m_in_display_list)
|
2014-11-15 08:24:06 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
WARN_LOG_FMT(VIDEO, "recursive display list detected");
|
2016-06-24 02:43:46 -06:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
else
|
|
|
|
{
|
|
|
|
m_in_display_list = true;
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if constexpr (is_preprocess)
|
Add the 'desynced GPU thread' mode.
It's a relatively big commit (less big with -w), but it's hard to test
any of this separately...
The basic problem is that in netplay or movies, the state of the CPU
must be deterministic, including when the game receives notification
that the GPU has processed FIFO data. Dual core mode notifies the game
whenever the GPU thread actually gets around to doing the work, so it
isn't deterministic. Single core mode is because it notifies the game
'instantly' (after processing the data synchronously), but it's too slow
for many systems and games.
My old dc-netplay branch worked as follows: everything worked as normal
except the state of the CP registers was a lie, and the CPU thread only
delivered results when idle detection triggered (waiting for the GPU if
they weren't ready at that point). Usually, a game is idle iff all the
work for the frame has been done, except for a small amount of work
depending on the GPU result, so neither the CPU or the GPU waiting on
the other affected performance much. However, it's possible that the
game could be waiting for some earlier interrupt, and any of several
games which, for whatever reason, never went into a detectable idle
(even when I tried to improve the detection) would never receive results
at all. (The current method should have better compatibility, but it
also has slightly higher overhead and breaks some other things, so I
want to reimplement this, hopefully with less impact on the code, in the
future.)
With this commit, the basic idea is that the CPU thread acts as if the
work has been done instantly, like single core mode, but actually hands
it off asynchronously to the GPU thread (after backing up some data that
the game might change in memory before it's actually done). Since the
work isn't done, any feedback from the GPU to the CPU, such as real
XFB/EFB copies (virtual are OK), EFB pokes, performance queries, etc. is
broken; but most games work with these options disabled, and there is no
need to try to detect what the CPU thread is doing.
Technically: when the flag g_use_deterministic_gpu_thread (currently
stuck on) is on, the CPU thread calls RunGpu like in single core mode.
This function synchronously copies the data from the FIFO to the
internal video buffer and updates the CP registers, interrupts, etc.
However, instead of the regular ReadDataFromFifo followed by running the
opcode decoder, it runs ReadDataFromFifoOnCPU ->
OpcodeDecoder_Preprocess, which relatively quickly scans through the
FIFO data, detects SetFinish calls etc., which are immediately fired,
and saves certain associated data from memory (e.g. display lists) in
AuxBuffers (a parallel stream to the main FIFO, which is a bit slow at
the moment), before handing the data off to the GPU thread to actually
render. That makes up the bulk of this commit.
In various circumstances, including the aforementioned EFB pokes and
performance queries as well as swap requests (i.e. the end of a frame -
we don't want the CPU potentially pumping out frames too quickly and the
GPU falling behind*), SyncGPU is called to wait for actual completion.
The overhead mainly comes from OpcodeDecoder_Preprocess (which is,
again, synchronous), as well as the actual copying.
Currently, display lists and such are escrowed from main memory even
though they usually won't change over the course of a frame, and
textures are not even though they might, resulting in a small chance of
graphical glitches. When the texture locking (i.e. fault on write) code
lands, I can make this all correct and maybe a little faster.
* This suggests an alternate determinism method of just delaying results
until a short time before the end of each frame. For all I know this
might mostly work - I haven't tried it - but if any significant work
hinges on the competion of render to texture etc., the frame will be
missed.
2014-08-27 20:56:19 -06:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
const u8* const start_address = Memory::GetPointer(address);
|
2019-12-05 06:05:00 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
Fifo::PushFifoAuxBuffer(start_address, size);
|
2019-12-05 06:05:00 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if (start_address != nullptr)
|
2014-11-27 15:53:11 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
Run(start_address, size, *this);
|
Add the 'desynced GPU thread' mode.
It's a relatively big commit (less big with -w), but it's hard to test
any of this separately...
The basic problem is that in netplay or movies, the state of the CPU
must be deterministic, including when the game receives notification
that the GPU has processed FIFO data. Dual core mode notifies the game
whenever the GPU thread actually gets around to doing the work, so it
isn't deterministic. Single core mode is because it notifies the game
'instantly' (after processing the data synchronously), but it's too slow
for many systems and games.
My old dc-netplay branch worked as follows: everything worked as normal
except the state of the CP registers was a lie, and the CPU thread only
delivered results when idle detection triggered (waiting for the GPU if
they weren't ready at that point). Usually, a game is idle iff all the
work for the frame has been done, except for a small amount of work
depending on the GPU result, so neither the CPU or the GPU waiting on
the other affected performance much. However, it's possible that the
game could be waiting for some earlier interrupt, and any of several
games which, for whatever reason, never went into a detectable idle
(even when I tried to improve the detection) would never receive results
at all. (The current method should have better compatibility, but it
also has slightly higher overhead and breaks some other things, so I
want to reimplement this, hopefully with less impact on the code, in the
future.)
With this commit, the basic idea is that the CPU thread acts as if the
work has been done instantly, like single core mode, but actually hands
it off asynchronously to the GPU thread (after backing up some data that
the game might change in memory before it's actually done). Since the
work isn't done, any feedback from the GPU to the CPU, such as real
XFB/EFB copies (virtual are OK), EFB pokes, performance queries, etc. is
broken; but most games work with these options disabled, and there is no
need to try to detect what the CPU thread is doing.
Technically: when the flag g_use_deterministic_gpu_thread (currently
stuck on) is on, the CPU thread calls RunGpu like in single core mode.
This function synchronously copies the data from the FIFO to the
internal video buffer and updates the CP registers, interrupts, etc.
However, instead of the regular ReadDataFromFifo followed by running the
opcode decoder, it runs ReadDataFromFifoOnCPU ->
OpcodeDecoder_Preprocess, which relatively quickly scans through the
FIFO data, detects SetFinish calls etc., which are immediately fired,
and saves certain associated data from memory (e.g. display lists) in
AuxBuffers (a parallel stream to the main FIFO, which is a bit slow at
the moment), before handing the data off to the GPU thread to actually
render. That makes up the bulk of this commit.
In various circumstances, including the aforementioned EFB pokes and
performance queries as well as swap requests (i.e. the end of a frame -
we don't want the CPU potentially pumping out frames too quickly and the
GPU falling behind*), SyncGPU is called to wait for actual completion.
The overhead mainly comes from OpcodeDecoder_Preprocess (which is,
again, synchronous), as well as the actual copying.
Currently, display lists and such are escrowed from main memory even
though they usually won't change over the course of a frame, and
textures are not even though they might, resulting in a small chance of
graphical glitches. When the texture locking (i.e. fault on write) code
lands, I can make this all correct and maybe a little faster.
* This suggests an alternate determinism method of just delaying results
until a short time before the end of each frame. For all I know this
might mostly work - I haven't tried it - but if any significant work
hinges on the competion of render to texture etc., the frame will be
missed.
2014-08-27 20:56:19 -06:00
|
|
|
}
|
2016-06-24 02:43:46 -06:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
else
|
Refactor opcode decoding a bit to kill FifoCommandRunnable.
Separated out from my gpu-determinism branch by request. It's not a big
commit; I just like to write long commit messages.
The main reason to kill it is hopefully a slight performance improvement
from avoiding the double switch (especially in single core mode);
however, this also improves cycle calculation, as described below.
- FifoCommandRunnable is removed; in its stead, Decode returns the
number of cycles (which only matters for "sync" GPU mode), or 0 if there
was not enough data, and is also responsible for unknown opcode alerts.
Decode and DecodeSemiNop are almost identical, so the latter is replaced
with a skipped_frame parameter to Decode. Doesn't mean we can't improve
skipped_frame mode to do less work; if, at such a point, branching on it
has too much overhead (it certainly won't now), it can always be changed
to a template parameter.
- FifoCommandRunnable used a fixed, large cycle count for display lists,
regardless of the contents. Presumably the actual hardware's processing
time is mostly the processing time of whatever commands are in the list,
and with this change InterpretDisplayList can just return the list's
cycle count to be added to the total. (Since the calculation for this
is part of Decode, it didn't seem easy to split this change up.)
To facilitate this, Decode also gains an explicit 'end' parameter in
lieu of FifoCommandRunnable's call to GetVideoBufferEndPtr, which can
point to there or to the end of a display list (or elsewhere in
gpu-determinism, but that's another story). Also, as a small
optimization, InterpretDisplayList now calls OpcodeDecoder_Run rather
than having its own Decode loop, to allow Decode to be inlined (haven't
checked whether this actually happens though).
skipped_frame mode still does not traverse display lists and uses the
old fake value of 45 cycles. degasus has suggested that this hack is
not essential for performance and can be removed, but I want to separate
any potential performance impact of that from this commit.
2014-08-31 23:11:32 -06:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
const u8* start_address;
|
2019-12-05 06:05:00 -07:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
if (Fifo::UseDeterministicGPUThread())
|
|
|
|
start_address = static_cast<u8*>(Fifo::PopFifoAuxBuffer(size));
|
|
|
|
else
|
|
|
|
start_address = Memory::GetPointer(address);
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
// Avoid the crash if Memory::GetPointer failed ..
|
|
|
|
if (start_address != nullptr)
|
|
|
|
{
|
|
|
|
// temporarily swap dl and non-dl (small "hack" for the stats)
|
|
|
|
g_stats.SwapDL();
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
Run(start_address, size, *this);
|
|
|
|
INCSTAT(g_stats.this_frame.num_dlists_called);
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
// un-swap
|
|
|
|
g_stats.SwapDL();
|
|
|
|
}
|
Refactor opcode decoding a bit to kill FifoCommandRunnable.
Separated out from my gpu-determinism branch by request. It's not a big
commit; I just like to write long commit messages.
The main reason to kill it is hopefully a slight performance improvement
from avoiding the double switch (especially in single core mode);
however, this also improves cycle calculation, as described below.
- FifoCommandRunnable is removed; in its stead, Decode returns the
number of cycles (which only matters for "sync" GPU mode), or 0 if there
was not enough data, and is also responsible for unknown opcode alerts.
Decode and DecodeSemiNop are almost identical, so the latter is replaced
with a skipped_frame parameter to Decode. Doesn't mean we can't improve
skipped_frame mode to do less work; if, at such a point, branching on it
has too much overhead (it certainly won't now), it can always be changed
to a template parameter.
- FifoCommandRunnable used a fixed, large cycle count for display lists,
regardless of the contents. Presumably the actual hardware's processing
time is mostly the processing time of whatever commands are in the list,
and with this change InterpretDisplayList can just return the list's
cycle count to be added to the total. (Since the calculation for this
is part of Decode, it didn't seem easy to split this change up.)
To facilitate this, Decode also gains an explicit 'end' parameter in
lieu of FifoCommandRunnable's call to GetVideoBufferEndPtr, which can
point to there or to the end of a display list (or elsewhere in
gpu-determinism, but that's another story). Also, as a small
optimization, InterpretDisplayList now calls OpcodeDecoder_Run rather
than having its own Decode loop, to allow Decode to be inlined (haven't
checked whether this actually happens though).
skipped_frame mode still does not traverse display lists and uses the
old fake value of 45 cycles. degasus has suggested that this hack is
not essential for performance and can be removed, but I want to separate
any potential performance impact of that from this commit.
2014-08-31 23:11:32 -06:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
|
|
|
|
m_in_display_list = false;
|
2008-12-07 22:25:12 -07:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnNop(u32 count))
|
|
|
|
{
|
|
|
|
m_cycles += 6 * count; // Hm, this means that we scan over nop streams pretty slowly...
|
|
|
|
}
|
|
|
|
OPCODE_CALLBACK(void OnUnknown(u8 opcode, const u8* data))
|
|
|
|
{
|
2022-01-22 17:31:33 -07:00
|
|
|
if (static_cast<Opcode>(opcode) == Opcode::GX_CMD_UNKNOWN_METRICS)
|
2021-04-22 21:57:56 -06:00
|
|
|
{
|
|
|
|
// 'Zelda Four Swords' calls it and checks the metrics registers after that
|
|
|
|
m_cycles += 6;
|
|
|
|
DEBUG_LOG_FMT(VIDEO, "GX 0x44");
|
|
|
|
}
|
|
|
|
else if (static_cast<Opcode>(opcode) == Opcode::GX_CMD_INVL_VC)
|
|
|
|
{
|
|
|
|
// Invalidate Vertex Cache
|
|
|
|
m_cycles += 6;
|
|
|
|
DEBUG_LOG_FMT(VIDEO, "Invalidate (vertex cache?)");
|
|
|
|
}
|
|
|
|
else
|
|
|
|
{
|
2022-01-22 23:02:20 -07:00
|
|
|
CommandProcessor::HandleUnknownOpcode(opcode, data, is_preprocess);
|
2021-04-22 21:57:56 -06:00
|
|
|
m_cycles += 1;
|
|
|
|
}
|
|
|
|
}
|
2016-06-24 02:43:46 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
OPCODE_CALLBACK(void OnCommand(const u8* data, u32 size))
|
|
|
|
{
|
|
|
|
ASSERT(size >= 1);
|
2019-12-05 06:49:05 -07:00
|
|
|
if constexpr (!is_preprocess)
|
Refactor opcode decoding a bit to kill FifoCommandRunnable.
Separated out from my gpu-determinism branch by request. It's not a big
commit; I just like to write long commit messages.
The main reason to kill it is hopefully a slight performance improvement
from avoiding the double switch (especially in single core mode);
however, this also improves cycle calculation, as described below.
- FifoCommandRunnable is removed; in its stead, Decode returns the
number of cycles (which only matters for "sync" GPU mode), or 0 if there
was not enough data, and is also responsible for unknown opcode alerts.
Decode and DecodeSemiNop are almost identical, so the latter is replaced
with a skipped_frame parameter to Decode. Doesn't mean we can't improve
skipped_frame mode to do less work; if, at such a point, branching on it
has too much overhead (it certainly won't now), it can always be changed
to a template parameter.
- FifoCommandRunnable used a fixed, large cycle count for display lists,
regardless of the contents. Presumably the actual hardware's processing
time is mostly the processing time of whatever commands are in the list,
and with this change InterpretDisplayList can just return the list's
cycle count to be added to the total. (Since the calculation for this
is part of Decode, it didn't seem easy to split this change up.)
To facilitate this, Decode also gains an explicit 'end' parameter in
lieu of FifoCommandRunnable's call to GetVideoBufferEndPtr, which can
point to there or to the end of a display list (or elsewhere in
gpu-determinism, but that's another story). Also, as a small
optimization, InterpretDisplayList now calls OpcodeDecoder_Run rather
than having its own Decode loop, to allow Decode to be inlined (haven't
checked whether this actually happens though).
skipped_frame mode still does not traverse display lists and uses the
old fake value of 45 cycles. degasus has suggested that this hack is
not essential for performance and can be removed, but I want to separate
any potential performance impact of that from this commit.
2014-08-31 23:11:32 -06:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
// Display lists get added directly into the FIFO stream since this same callback is used to
|
|
|
|
// process them.
|
|
|
|
if (g_record_fifo_data && static_cast<Opcode>(data[0]) != Opcode::GX_CMD_CALL_DL)
|
2019-12-05 06:49:05 -07:00
|
|
|
{
|
2021-04-22 21:57:56 -06:00
|
|
|
FifoRecorder::GetInstance().WriteGPCommand(data, size);
|
2019-12-05 06:49:05 -07:00
|
|
|
}
|
Refactor opcode decoding a bit to kill FifoCommandRunnable.
Separated out from my gpu-determinism branch by request. It's not a big
commit; I just like to write long commit messages.
The main reason to kill it is hopefully a slight performance improvement
from avoiding the double switch (especially in single core mode);
however, this also improves cycle calculation, as described below.
- FifoCommandRunnable is removed; in its stead, Decode returns the
number of cycles (which only matters for "sync" GPU mode), or 0 if there
was not enough data, and is also responsible for unknown opcode alerts.
Decode and DecodeSemiNop are almost identical, so the latter is replaced
with a skipped_frame parameter to Decode. Doesn't mean we can't improve
skipped_frame mode to do less work; if, at such a point, branching on it
has too much overhead (it certainly won't now), it can always be changed
to a template parameter.
- FifoCommandRunnable used a fixed, large cycle count for display lists,
regardless of the contents. Presumably the actual hardware's processing
time is mostly the processing time of whatever commands are in the list,
and with this change InterpretDisplayList can just return the list's
cycle count to be added to the total. (Since the calculation for this
is part of Decode, it didn't seem easy to split this change up.)
To facilitate this, Decode also gains an explicit 'end' parameter in
lieu of FifoCommandRunnable's call to GetVideoBufferEndPtr, which can
point to there or to the end of a display list (or elsewhere in
gpu-determinism, but that's another story). Also, as a small
optimization, InterpretDisplayList now calls OpcodeDecoder_Run rather
than having its own Decode loop, to allow Decode to be inlined (haven't
checked whether this actually happens though).
skipped_frame mode still does not traverse display lists and uses the
old fake value of 45 cycles. degasus has suggested that this hack is
not essential for performance and can be removed, but I want to separate
any potential performance impact of that from this commit.
2014-08-31 23:11:32 -06:00
|
|
|
}
|
2014-11-26 14:12:54 -07:00
|
|
|
}
|
2021-04-22 21:57:56 -06:00
|
|
|
|
|
|
|
OPCODE_CALLBACK(CPState& GetCPState())
|
|
|
|
{
|
|
|
|
if constexpr (is_preprocess)
|
|
|
|
return g_preprocess_cp_state;
|
|
|
|
else
|
|
|
|
return g_main_cp_state;
|
|
|
|
}
|
|
|
|
|
|
|
|
u32 m_cycles = 0;
|
|
|
|
bool m_in_display_list = false;
|
|
|
|
};
|
|
|
|
|
|
|
|
template <bool is_preprocess>
|
|
|
|
u8* RunFifo(DataReader src, u32* cycles)
|
|
|
|
{
|
|
|
|
using CallbackT = RunCallback<is_preprocess>;
|
|
|
|
auto callback = CallbackT{};
|
|
|
|
u32 size = Run(src.GetPointer(), static_cast<u32>(src.size()), callback);
|
|
|
|
|
|
|
|
if (cycles != nullptr)
|
|
|
|
*cycles = callback.m_cycles;
|
|
|
|
|
|
|
|
src.Skip(size);
|
|
|
|
return src.GetPointer();
|
2009-11-15 15:26:39 -07:00
|
|
|
}
|
Add the 'desynced GPU thread' mode.
It's a relatively big commit (less big with -w), but it's hard to test
any of this separately...
The basic problem is that in netplay or movies, the state of the CPU
must be deterministic, including when the game receives notification
that the GPU has processed FIFO data. Dual core mode notifies the game
whenever the GPU thread actually gets around to doing the work, so it
isn't deterministic. Single core mode is because it notifies the game
'instantly' (after processing the data synchronously), but it's too slow
for many systems and games.
My old dc-netplay branch worked as follows: everything worked as normal
except the state of the CP registers was a lie, and the CPU thread only
delivered results when idle detection triggered (waiting for the GPU if
they weren't ready at that point). Usually, a game is idle iff all the
work for the frame has been done, except for a small amount of work
depending on the GPU result, so neither the CPU or the GPU waiting on
the other affected performance much. However, it's possible that the
game could be waiting for some earlier interrupt, and any of several
games which, for whatever reason, never went into a detectable idle
(even when I tried to improve the detection) would never receive results
at all. (The current method should have better compatibility, but it
also has slightly higher overhead and breaks some other things, so I
want to reimplement this, hopefully with less impact on the code, in the
future.)
With this commit, the basic idea is that the CPU thread acts as if the
work has been done instantly, like single core mode, but actually hands
it off asynchronously to the GPU thread (after backing up some data that
the game might change in memory before it's actually done). Since the
work isn't done, any feedback from the GPU to the CPU, such as real
XFB/EFB copies (virtual are OK), EFB pokes, performance queries, etc. is
broken; but most games work with these options disabled, and there is no
need to try to detect what the CPU thread is doing.
Technically: when the flag g_use_deterministic_gpu_thread (currently
stuck on) is on, the CPU thread calls RunGpu like in single core mode.
This function synchronously copies the data from the FIFO to the
internal video buffer and updates the CP registers, interrupts, etc.
However, instead of the regular ReadDataFromFifo followed by running the
opcode decoder, it runs ReadDataFromFifoOnCPU ->
OpcodeDecoder_Preprocess, which relatively quickly scans through the
FIFO data, detects SetFinish calls etc., which are immediately fired,
and saves certain associated data from memory (e.g. display lists) in
AuxBuffers (a parallel stream to the main FIFO, which is a bit slow at
the moment), before handing the data off to the GPU thread to actually
render. That makes up the bulk of this commit.
In various circumstances, including the aforementioned EFB pokes and
performance queries as well as swap requests (i.e. the end of a frame -
we don't want the CPU potentially pumping out frames too quickly and the
GPU falling behind*), SyncGPU is called to wait for actual completion.
The overhead mainly comes from OpcodeDecoder_Preprocess (which is,
again, synchronous), as well as the actual copying.
Currently, display lists and such are escrowed from main memory even
though they usually won't change over the course of a frame, and
textures are not even though they might, resulting in a small chance of
graphical glitches. When the texture locking (i.e. fault on write) code
lands, I can make this all correct and maybe a little faster.
* This suggests an alternate determinism method of just delaying results
until a short time before the end of each frame. For all I know this
might mostly work - I haven't tried it - but if any significant work
hinges on the competion of render to texture etc., the frame will be
missed.
2014-08-27 20:56:19 -06:00
|
|
|
|
2021-04-22 21:57:56 -06:00
|
|
|
template u8* RunFifo<true>(DataReader src, u32* cycles);
|
|
|
|
template u8* RunFifo<false>(DataReader src, u32* cycles);
|
2016-01-23 23:29:44 -07:00
|
|
|
|
|
|
|
} // namespace OpcodeDecoder
|