Emulation/orbum
1
0
Fork 0
mirror of https://github.com/hch12907/orbum.git synced 2024-05-20 05:00:48 -04:00
Code Issues Packages Projects Releases Wiki Activity

GIF: Improve FIFO check

Browse source 
fingers crossed that it fixes #1 again
This commit is contained in:
hch12907 2018-11-07 15:55:35 +08:00
parent 620110cfa6
commit 8089b206dd
1 changed files with 18 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
liborbum/src/Controller/Ee/Gif/CGif.cpp
View file

@ -89,8 +89,8 @@ int CGif::time_step(const int ticks_available)
if (!fifo->has_read_available(NUMBER_BYTES_IN_QWORD))
continue;
// P3TAG holding something means that PATH3 was previously interrupted, and needs to recontinue
if (r.ee.gif.p3tag.read_uword())
// IP3 set to 1 indicates that PATH3 was previously interrupted, and needs to recontinue
if (r.ee.gif.stat.extract_field(GifRegister_Stat::IP3))
{
// Note: check endianness?
uqword p3data;
@ -157,6 +157,8 @@ int CGif::time_step(const int ticks_available)
uqword data;
fifo->read(reinterpret_cast<ubyte*>(&data), NUMBER_BYTES_IN_QWORD);
cycles_consumed = handle_data_image(data);
// In intermittent mode, the GIF checks for other requests from PATH1 & PATH2,
// and priotises them over PATH3 if there are.
// The check is done every 8 qwords (or 1 slice).
@ -185,8 +187,6 @@ int CGif::time_step(const int ticks_available)
}
}
cycles_consumed = handle_data_image(data);
break;
}
default:
@ -194,6 +194,20 @@ int CGif::time_step(const int ticks_available)
throw std::runtime_error("Unknown GIF data processing method");
}
}
// TODO: remove this hack.
// Hack: if there are active transfers, force FQC to 16, otherwise 0.
// This is because we do not have a reliable way of tracking the FIFO usage
if (fifo->has_read_available(NUMBER_BYTES_IN_QWORD))
{
BOOST_LOG(Core::get_logger()) << "GIF: FQC set to 16";
stat.insert_field(GifRegister_Stat::FQC, 16);
}
else
{
BOOST_LOG(Core::get_logger()) << "GIF: FQC set to 0";
stat.insert_field(GifRegister_Stat::FQC, 0);
}
}
// TODO: Confirm behaviour.
@ -211,19 +225,6 @@ int CGif::time_step(const int ticks_available)
}
}
// TODO: remove this hack.
// Hack: if there are active transfers, force FQC to 16, otherwise 0.
// This is because we do not have a reliable way of tracking the FIFO usage
if (fifo->has_read_available(NUMBER_BYTES_IN_QWORD))
{
BOOST_LOG(Core::get_logger()) << "GIF: FQC set to 16";
stat.insert_field(GifRegister_Stat::FQC, 16);
}
else
{
stat.insert_field(GifRegister_Stat::FQC, 0);
}
// Do not process other paths if at least one path was successfully processed.
if (cycles_consumed)
break;