Skip to content

update error message formatting and rom / ram documentation#311

Open
nn357 wants to merge 1 commit intoblkerby:mainfrom
nn357:update_selfcheck
Open

update error message formatting and rom / ram documentation#311
nn357 wants to merge 1 commit intoblkerby:mainfrom
nn357:update_selfcheck

Conversation

@nn357
Copy link
Contributor

@nn357 nn357 commented Feb 21, 2026

move checksum self check variables to a place they are not likely to get clobbered by the unsafe behaviour of sd2snes/fxpak menus/ingame hooks / savestates

(this feature might overwrite other variables that were around where the old checksum variables used to be) but having a hard crashing routine highlighted this problem.

move and re-write common boot bank 7e clear routine to not wipe our variable.. this uses dma now which is 66% faster than the original part unrolled stz routine (0336-033f) are preserved.

542298 master clock cycles vs 1.53m clock cycles.

move checksum self check variables to a place they are not likely to get clobbered

update bank / ram usage notes

update ram clearing method for more performance.

change ram clear method

cannot math

clear ram via dma

(leave our variables in place)
@blkerby
Copy link
Owner

blkerby commented Feb 26, 2026

Holding off on merging this until we can get more information on how this affects the FXPAK, e.g. if it makes the in-game hooks work properly without still causing other forms of corruption. If the alternative is some other, more subtle type of corruption, the concern is that it could be worse than just getting an error. We also may want to look at what happens with the RTA timer, since the RTA timer starts counting early in bootup process, and I forget what if anything we did to avoid it being affected the RAM clearing.

@nn357
Copy link
Contributor Author

nn357 commented Feb 27, 2026

i'll look into the RTA timer, might be worth asking in the general discord for those who play with fxpak /s2snes if they ..

a. use ingame hooks.
b. don't use ingame hooks.
c. dont' know what they are (please check if they are on /off)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants