Implement "lag frame" flag and counter (#949)

* Implement "lag frame" flag and counter, and expose flag and both frame counters. BizHawk wants these.

* Track frame count and lag frames while the system isn't running.
This commit is contained in:
SuuperW
2021-01-21 14:26:27 -06:00
committed by GitHub
parent 8a068c2294
commit d42ca1ec4b
3 changed files with 94 additions and 74 deletions

View File

@ -77,6 +77,8 @@ ARMv5* ARM9;
ARMv4* ARM7; ARMv4* ARM7;
u32 NumFrames; u32 NumFrames;
u32 NumLagFrames;
bool LagFrameFlag;
u64 LastSysClockCycles; u64 LastSysClockCycles;
u64 FrameStartTimestamp; u64 FrameStartTimestamp;
@ -765,6 +767,11 @@ bool DoSavestate(Savestate* file)
file->Var64(&LastSysClockCycles); file->Var64(&LastSysClockCycles);
file->Var64(&FrameStartTimestamp); file->Var64(&FrameStartTimestamp);
file->Var32(&NumFrames); file->Var32(&NumFrames);
if (file->IsAtleastVersion(7, 1))
{
file->Var32(&NumLagFrames);
file->Bool32(&LagFrameFlag);
}
// TODO: save KeyInput???? // TODO: save KeyInput????
file->Var16(&KeyCnt); file->Var16(&KeyCnt);
@ -910,99 +917,108 @@ u32 RunFrame()
{ {
FrameStartTimestamp = SysTimestamp; FrameStartTimestamp = SysTimestamp;
if (!Running) return 263; // dorp LagFrameFlag = true;
if (CPUStop & 0x40000000) return 263; bool runFrame = Running && !(CPUStop & 0x40000000);
if (runFrame)
GPU::StartFrame();
while (Running && GPU::TotalScanlines==0)
{ {
// TODO: give it some margin, so it can directly do 17 cycles instead of 16 then 1 GPU::StartFrame();
u64 target = NextTarget();
ARM9Target = target << ARM9ClockShift;
CurCPU = 0;
if (CPUStop & 0x80000000) while (Running && GPU::TotalScanlines==0)
{ {
// GXFIFO stall // TODO: give it some margin, so it can directly do 17 cycles instead of 16 then 1
s32 cycles = GPU3D::CyclesToRunFor(); u64 target = NextTarget();
ARM9Target = target << ARM9ClockShift;
CurCPU = 0;
ARM9Timestamp = std::min(ARM9Target, ARM9Timestamp+(cycles<<ARM9ClockShift)); if (CPUStop & 0x80000000)
}
else if (CPUStop & 0x0FFF)
{
DMAs[0]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[1]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[2]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[3]->Run<ConsoleType>();
if (ConsoleType == 1) DSi::RunNDMAs(0);
}
else
{
#ifdef JIT_ENABLED
if (EnableJIT)
ARM9->ExecuteJIT();
else
#endif
ARM9->Execute();
}
RunTimers(0);
GPU3D::Run();
target = ARM9Timestamp >> ARM9ClockShift;
CurCPU = 1;
while (ARM7Timestamp < target)
{
ARM7Target = target; // might be changed by a reschedule
if (CPUStop & 0x0FFF0000)
{ {
DMAs[4]->Run<ConsoleType>(); // GXFIFO stall
DMAs[5]->Run<ConsoleType>(); s32 cycles = GPU3D::CyclesToRunFor();
DMAs[6]->Run<ConsoleType>();
DMAs[7]->Run<ConsoleType>(); ARM9Timestamp = std::min(ARM9Target, ARM9Timestamp+(cycles<<ARM9ClockShift));
if (ConsoleType == 1) DSi::RunNDMAs(1); }
else if (CPUStop & 0x0FFF)
{
DMAs[0]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[1]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[2]->Run<ConsoleType>();
if (!(CPUStop & 0x80000000)) DMAs[3]->Run<ConsoleType>();
if (ConsoleType == 1) DSi::RunNDMAs(0);
} }
else else
{ {
#ifdef JIT_ENABLED #ifdef JIT_ENABLED
if (EnableJIT) if (EnableJIT)
ARM7->ExecuteJIT(); ARM9->ExecuteJIT();
else else
#endif #endif
ARM7->Execute(); ARM9->Execute();
} }
RunTimers(1); RunTimers(0);
} GPU3D::Run();
RunSystem(target); target = ARM9Timestamp >> ARM9ClockShift;
CurCPU = 1;
if (CPUStop & 0x40000000) while (ARM7Timestamp < target)
{ {
// checkme: when is sleep mode effective? ARM7Target = target; // might be changed by a reschedule
//CancelEvent(Event_LCD);
//GPU::TotalScanlines = 263; if (CPUStop & 0x0FFF0000)
break; {
DMAs[4]->Run<ConsoleType>();
DMAs[5]->Run<ConsoleType>();
DMAs[6]->Run<ConsoleType>();
DMAs[7]->Run<ConsoleType>();
if (ConsoleType == 1) DSi::RunNDMAs(1);
}
else
{
#ifdef JIT_ENABLED
if (EnableJIT)
ARM7->ExecuteJIT();
else
#endif
ARM7->Execute();
}
RunTimers(1);
}
RunSystem(target);
if (CPUStop & 0x40000000)
{
// checkme: when is sleep mode effective?
//CancelEvent(Event_LCD);
//GPU::TotalScanlines = 263;
break;
}
} }
}
#ifdef DEBUG_CHECK_DESYNC #ifdef DEBUG_CHECK_DESYNC
printf("[%08X%08X] ARM9=%ld, ARM7=%ld, GPU=%ld\n", printf("[%08X%08X] ARM9=%ld, ARM7=%ld, GPU=%ld\n",
(u32)(SysTimestamp>>32), (u32)SysTimestamp, (u32)(SysTimestamp>>32), (u32)SysTimestamp,
(ARM9Timestamp>>1)-SysTimestamp, (ARM9Timestamp>>1)-SysTimestamp,
ARM7Timestamp-SysTimestamp, ARM7Timestamp-SysTimestamp,
GPU3D::Timestamp-SysTimestamp); GPU3D::Timestamp-SysTimestamp);
#endif #endif
SPU::TransferOutput(); SPU::TransferOutput();
NDSCart::FlushSRAMFile(); NDSCart::FlushSRAMFile();
}
// In the context of TASes, frame count is traditionally the primary measure of emulated time,
// so it needs to be tracked even if NDS is powered off.
NumFrames++; NumFrames++;
if (LagFrameFlag)
NumLagFrames++;
return GPU::TotalScanlines; if (runFrame)
return GPU::TotalScanlines;
else
return 263;
} }
u32 RunFrame() u32 RunFrame()
@ -2787,8 +2803,8 @@ u8 ARM9IORead8(u32 addr)
{ {
switch (addr) switch (addr)
{ {
case 0x04000130: return KeyInput & 0xFF; case 0x04000130: LagFrameFlag = false; return KeyInput & 0xFF;
case 0x04000131: return (KeyInput >> 8) & 0xFF; case 0x04000131: LagFrameFlag = false; return (KeyInput >> 8) & 0xFF;
case 0x04000132: return KeyCnt & 0xFF; case 0x04000132: return KeyCnt & 0xFF;
case 0x04000133: return KeyCnt >> 8; case 0x04000133: return KeyCnt >> 8;
@ -2889,7 +2905,7 @@ u16 ARM9IORead16(u32 addr)
case 0x0400010C: return TimerGetCounter(3); case 0x0400010C: return TimerGetCounter(3);
case 0x0400010E: return Timers[3].Cnt; case 0x0400010E: return Timers[3].Cnt;
case 0x04000130: return KeyInput & 0xFFFF; case 0x04000130: LagFrameFlag = false; return KeyInput & 0xFFFF;
case 0x04000132: return KeyCnt; case 0x04000132: return KeyCnt;
case 0x04000180: return IPCSync9; case 0x04000180: return IPCSync9;
@ -3007,7 +3023,7 @@ u32 ARM9IORead32(u32 addr)
case 0x04000108: return TimerGetCounter(2) | (Timers[2].Cnt << 16); case 0x04000108: return TimerGetCounter(2) | (Timers[2].Cnt << 16);
case 0x0400010C: return TimerGetCounter(3) | (Timers[3].Cnt << 16); case 0x0400010C: return TimerGetCounter(3) | (Timers[3].Cnt << 16);
case 0x04000130: return (KeyInput & 0xFFFF) | (KeyCnt << 16); case 0x04000130: LagFrameFlag = false; return (KeyInput & 0xFFFF) | (KeyCnt << 16);
case 0x04000180: return IPCSync9; case 0x04000180: return IPCSync9;
case 0x04000184: return ARM9IORead16(addr); case 0x04000184: return ARM9IORead16(addr);

View File

@ -140,6 +140,10 @@ extern int CurCPU;
extern u8 ARM9MemTimings[0x40000][4]; extern u8 ARM9MemTimings[0x40000][4];
extern u8 ARM7MemTimings[0x20000][4]; extern u8 ARM7MemTimings[0x20000][4];
extern u32 NumFrames;
extern u32 NumLagFrames;
extern bool LagFrameFlag;
extern u64 ARM9Timestamp, ARM9Target; extern u64 ARM9Timestamp, ARM9Target;
extern u64 ARM7Timestamp, ARM7Target; extern u64 ARM7Timestamp, ARM7Target;
extern u32 ARM9ClockShift; extern u32 ARM9ClockShift;

View File

@ -23,7 +23,7 @@
#include "types.h" #include "types.h"
#define SAVESTATE_MAJOR 7 #define SAVESTATE_MAJOR 7
#define SAVESTATE_MINOR 0 #define SAVESTATE_MINOR 1
class Savestate class Savestate
{ {