MiSTer-devel / C64_MiSTer

111 stars 57 forks source link

All Hallows' Eve Demo by Atlantis & Bonzai & Genesis Project #141

Closed francouai closed 1 year ago

francouai commented 1 year ago

Hello!

Just to let you know that the demo on the 2nd D64 disc is crashing. (ThePumpkins-Disk2.d64) I'm getting a full freeze when using the last core "C64_20221007". Can you have a quick check? Thanks!

https://www.pouet.net/prod.php?which=92727

paich64 commented 1 year ago

I second this :

francouai commented 1 year ago

yup.. works perfect on real C64 hardware.

sy2002 commented 1 year ago

Additional observation: While switching the CIA to 8521 (instead of using the default CIA 6526) fixes the bug in the XXX demo from Lethargy (as described here: https://github.com/MiSTer-devel/C64_MiSTer/issues/108), switching the CIA to 8521 does not fix the problem with All Hallows' Eve.

francouai commented 1 year ago

Fyi, C64_20221110.rbf tested and still crashing.

sy2002 commented 1 year ago

All, I've made an experimental build of the MEGA65 variant of the MiSTer core incorporating this fix from @gyurco: https://github.com/mist-devel/T65/commit/7027ad169553911b1d55ce6d220364e7c8595b94

🚀 All Hallows' Eve works now like a charme. 👍🏻

Someone might want to to a test-merge on MiSTer, too.

The big question of course is: Is this a "dangerous" fix - how much regression tests are needed until it can be merged to the release branch. As you can see in the above-mentioned commit: The IRQ handling was changed quite a bit...

francouai commented 1 year ago

I just tried C64_20221114.rbf and works great! very impatient to use MEGA 65 core on Mister.

sy2002 commented 1 year ago

@francouai Cool! I guess you can close the issue then :-)

francouai commented 1 year ago

until this core will be pushed out on the UDATE_ALL script from the Mister