MiSTer-devel / SNES_MiSTer

SNES for MiSTer
GNU General Public License v3.0
200 stars 79 forks source link

Super Mario RPG - Stray pixel noise/crash #53

Closed Hewhoisred closed 5 years ago

Hewhoisred commented 5 years ago

Latest 20190308 release, there are minor stray pixels around sprites. Also, the game crashes not long after you load the included save (see attached), usually after you go down the stairs.

These issues do not present on the 20190303 release.

Other user experience with the same sav: "actually after the dialogs something weird happens seems to sort of get stuck with mario able to jump but not move yeah, something weird there"

"sometimes freezes up at the opening "sky" screen a reset can fix it but loading in the save it will freeze up as Mario is falling in on the save block and Mario has some stray pixels around him"

Super_MarioRPG-_Legend_of_the_Seven_Stars_USA.zip

James-F2 commented 5 years ago

The compilation of 20190308 release is bad. I have compiled the core on March 4th (GSU + NMI fix) and it does not have any issues.

Even without this save SMRPG freezes in the intro sequence with 20190308 release. Kirsby Super Star also freezes in the intro demo sequence.

sorgelig commented 5 years ago

due to large size and many processors this core got stability issue...

James-F2 commented 5 years ago

I have tried to compile 3 times,, and now I have this issue too. Everything was okay on March 4th including the GSU + NMI fixes.

Maybe this one did it? https://github.com/MiSTer-devel/SNES_MiSTer/commit/2d889bf315577e903888f6eccdfb3c48edf729a5

If I'm not mistaken, SA-1 games use the SRAM on the cartridge not only for saves but as working ram too.

sorgelig commented 5 years ago

When battery is off or on first start the SRAM has garbage. If just clear to zero as it was before, then Baseball game won't work for example.

sorgelig commented 5 years ago

tried intros in Kirby and SMRPG - both work fine on both my DE10nanos with 0308 release. So it's definitely a metastability issue.

James-F2 commented 5 years ago

I have complied again with old snes.sv before ""//Thrash the BSRAM upon ROM loading"",, and it does not freeze now.

Did you watch the entire intro or just few seconds? It freezes/crashes when mario jumps on the koopa's during gameplay sequence.

sorgelig commented 5 years ago

Any change in code will produce other logic map, so it may fix the freezing or make it worse regardless which part of code you change. It has no relation to that commit. Which application relays on data from RAM on power up? It's nonsense!

sorgelig commented 5 years ago

one of my DE10 freezes on SMRPG demo if i turn off the fan. So i can test in this board.

James-F2 commented 5 years ago

SA1 game rely on the the battery powered sram on the cartridge for saves and as working ram,, so maybe they expect certain state?

sorgelig commented 5 years ago

so, if the battery will off you have to throw away the cartridge? ;)

Hewhoisred commented 5 years ago

I can confirm that even without a .sav file (or with a brand new one if the game auto generates one at first run), the game crashes on the third scene of the intro where Mario is jumping around avoiding monsters.

Hewhoisred commented 5 years ago

All issues appear to be fixed with the re-release. Intro finishes fine, no noise on sprites and no more crashes during gameplay. Thanks for the hard work.

James-F2 commented 5 years ago

Kirby Super Star still crashes for me and other person with the 0308 re-release.

I have compiled with default optimization settings in Quartus and it doesn't seem to freeze anymore in Kirby. I am no FPGA expert, but all the optimizations really necessary?

James-F2 commented 5 years ago

StarFox (GSU) freezes when switching to Turbo speed with the re-release.

Seems like something with the random ram commit broke the custom compiler optimization. Though as said in previous post, everything okay when using default compiler optimization settings.

kousaka commented 5 years ago

I'm getting graphical issues in multiple games with the original release and the re-release.

sorgelig commented 5 years ago

try this build: https://mega.nz/#!1cZwQS5K!5HLse-2HVkn1KXClYSonku1re1esoUWSzGnSDmHgrYw

James-F2 commented 5 years ago

For me this build fixed all the reported problems on this issue.

Hewhoisred commented 5 years ago

I tested SM RPG intro and Kirby Superstars intro with the mega link build. No crashing, no visual glitches.

sorgelig commented 5 years ago

good!

kousaka commented 5 years ago

I can confirm the new version has fixed everything also