scawful/yaze
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Reduce Logging Limits to Prevent Crashes During Emulator Execution

Browse Source 
- Decreased logging limits in Snes, Apu, and Spc700 classes to prevent overflow crashes during execution.
- Updated comments in the APU boot ROM for clarity regarding the hardware dump and critical fixes.
- Enhanced logging to provide better insights while maintaining system stability.
This commit is contained in:
scawful
2025-10-06 16:14:32 -04:00
parent a09d7d10c8
commit a5d4722d13
3 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/app/emu/audio/apu.cc
View File

@@ -16,15 +16,15 @@ namespace emu {
static const double apuCyclesPerMaster = (32040 * 32) / (1364 * 262 * 60.0);
static const double apuCyclesPerMasterPal = (32040 * 32) / (1364 * 312 * 50.0);
// SNES IPL ROM with counter acknowledgments - WORKING VERSION
// Counter echo at $FFE6 proven functional with ALTTP
// SNES IPL ROM - Anomie's official hardware dump (64 bytes)
// With our critical fixes: CMP Z flag, multi-step bstep, address preservation
static const uint8_t bootRom[0x40] = {
0xcd, 0xef, 0xbd, 0xe8, 0x00, 0xc6, 0x1d, 0xd0, 0xfc, 0x8f, 0xaa,
0xf4, 0x8f, 0xbb, 0xf5, 0xe4, 0xf4, 0x68, 0xcc, 0xd0, 0xfa, 0x2f,
0x19, 0xeb, 0xf4, 0xd0, 0xfc, 0x7e, 0xf4, 0xd0, 0x0b, 0xe4, 0xf5,
0xcb, 0xf4, 0xd7, 0x00, 0xfc, 0xcb, 0xf4, 0xd0, 0xf3, 0xab, 0x01,
0x10, 0xef, 0x7e, 0xf4, 0x10, 0xeb, 0xba, 0xf6, 0xda, 0x00, 0xba,
0xf4, 0xc4, 0xf4, 0xdd, 0x5d, 0xd0, 0xdb, 0x1f, 0xc0, 0xff};
0xf4, 0x8f, 0xbb, 0xf5, 0x78, 0xcc, 0xf4, 0xd0, 0xfb, 0x2f, 0x19,
0xeb, 0xf4, 0xd0, 0xfc, 0x7e, 0xf4, 0xd0, 0x0b, 0xe4, 0xf5, 0xcb,
0xf4, 0xd7, 0x00, 0xfc, 0xd0, 0xf3, 0xab, 0x01, 0x10, 0xef, 0x7e,
0xf4, 0x10, 0xeb, 0xba, 0xf6, 0xda, 0x00, 0xba, 0xf4, 0xc4, 0xf4,
0xdd, 0x5d, 0xd0, 0xdb, 0x1f, 0x00, 0x00, 0xc0, 0xff};
// Helper to reset the cycle tracking on emulator reset
static uint64_t g_last_master_cycles = 0;
@@ -160,7 +160,7 @@ uint8_t Apu::Read(uint16_t adr) {
case 0xf7: {
uint8_t val = in_ports_[adr - 0xf4];
port_read_count++;
if (port_read_count < 100) { // Increased limit to see full handshake
if (port_read_count < 10) { // Reduced to prevent logging overflow crash
LOG_INFO("APU", "SPC read port $%04X (F%d) = $%02X at PC=$%04X",
adr, adr - 0xf4 + 4, val, spc700_.PC);
}
@@ -231,7 +231,7 @@ void Apu::Write(uint16_t adr, uint8_t val) {
case 0xf7: {
out_ports_[adr - 0xf4] = val;
port_write_count++;
if (port_write_count < 100) { // Increased limit to see full handshake
if (port_write_count < 10) { // Reduced to prevent logging overflow crash
LOG_INFO("APU", "SPC wrote port $%04X (F%d) = $%02X at PC=$%04X [APU_cycles=%llu]",
adr, adr - 0xf4 + 4, val, spc700_.PC, cycles_);
}

4
src/app/emu/audio/spc700.cc
View File

@@ -59,10 +59,10 @@ void Spc700::RunOpcode() {
bool in_critical_range = (PC >= 0xFFCF && PC <= 0xFFFF);
bool is_transfer_loop = (PC >= 0xFFDB && PC <= 0xFFEB);
if (in_critical_range && spc_exec_count++ < 200) {
if (in_critical_range && spc_exec_count++ < 10) {
LOG_INFO("SPC", "Execute: PC=$%04X step=0 bstep=%d Y=%02X A=%02X", PC, bstep, Y, A);
}
if (is_transfer_loop && spc_exec_count < 500) {
if (is_transfer_loop && spc_exec_count < 20) {
// Read ports to log transfer state
uint8_t f4_val = callbacks_.read(0xF4);
uint8_t f5_val = callbacks_.read(0xF5);

2
src/app/emu/snes.cc
View File

@@ -420,7 +420,7 @@ void Snes::WriteBBus(uint8_t adr, uint8_t val) {
CatchUpApu(); // catch up the apu before writing
apu_.in_ports_[adr & 0x3] = val;
static int cpu_port_write_count = 0;
if (cpu_port_write_count++ < 200) { // Increased to see full boot sequence
if (cpu_port_write_count++ < 10) { // Reduced to prevent crash
LOG_INFO("SNES", "CPU wrote APU port $21%02X (F%d) = $%02X at PC=$%02X:%04X",
0x40 + (adr & 0x3), (adr & 0x3) + 4, val, cpu_.PB, cpu_.PC);
}