ppeccin / WebMSX

WebMSX - Online MSX Emulator
https://webmsx.org
399 stars 72 forks source link

Not all statefiles work correctly #55

Closed TFHFony closed 4 years ago

TFHFony commented 4 years ago

Not all statefiles work the same anymore. For example this one hangs the machine completely: https://www.file-hunter.com/MSX/games/File-Hunter.zip

The normal behaviour would be displaying my altered BASIC welcome (Just a bunch of print commands) followed by a reset (defusr=0:a=usr(0).

For the rest: Great Update!

ppeccin commented 4 years ago

Hum... I don't know why, no errors occur when I load the state, just machine hang.

What software do you load? What does it do? Does it work if you not use the savestate and load it from scratch? Can you send me the software?

Peccin

On Fri, Mar 13, 2020 at 9:11 AM TFHFony notifications@github.com wrote:

Not all statefiles work the same anymore. For example this one hangs the machine completely: https://www.file-hunter.com/MSX/games/File-Hunter.zip

The normal behaviour would be displaying my altered BASIC welcome (Just a bunch of print commands) followed by a reset (defusr=0:a=usr(0).

For the rest: Great Update!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIEJ2W7ES3JZZK6EHCTRHIPGTANCNFSM4LHBHE5A .

ppeccin commented 4 years ago

Also, how did you create this savestate? Is is too old? The contents of the state is very strange. It has several slots duplicated, like 2 BIOS Exstensions, 2 MSX-MUSIC and 2 DiskROMs.

Maybe on the previous release, you loaded a very old state, them saved it again? Something seems to have corrupted this savestate.

Paulo

On Fri, Mar 13, 2020 at 12:37 PM Paulo Peccin ppeccin@gmail.com wrote:

Hum... I don't know why, no errors occur when I load the state, just machine hang.

What software do you load? What does it do? Does it work if you not use the savestate and load it from scratch? Can you send me the software?

Peccin

On Fri, Mar 13, 2020 at 9:11 AM TFHFony notifications@github.com wrote:

Not all statefiles work the same anymore. For example this one hangs the machine completely: https://www.file-hunter.com/MSX/games/File-Hunter.zip

The normal behaviour would be displaying my altered BASIC welcome (Just a bunch of print commands) followed by a reset (defusr=0:a=usr(0).

For the rest: Great Update!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIEJ2W7ES3JZZK6EHCTRHIPGTANCNFSM4LHBHE5A .

ppeccin commented 4 years ago

Also, I tried to load this state with versions 5.4 and 5.3, and it also hangs or resets the machine. Are you sure this state you sent me is correct? On which version did it work for you???? Are you having problems with other states as well?

Paulo

On Fri, Mar 13, 2020 at 3:19 PM Paulo Peccin ppeccin@gmail.com wrote:

Also, how did you create this savestate? Is is too old? The contents of the state is very strange. It has several slots duplicated, like 2 BIOS Exstensions, 2 MSX-MUSIC and 2 DiskROMs.

Maybe on the previous release, you loaded a very old state, them saved it again? Something seems to have corrupted this savestate.

Paulo

On Fri, Mar 13, 2020 at 12:37 PM Paulo Peccin ppeccin@gmail.com wrote:

Hum... I don't know why, no errors occur when I load the state, just machine hang.

What software do you load? What does it do? Does it work if you not use the savestate and load it from scratch? Can you send me the software?

Peccin

On Fri, Mar 13, 2020 at 9:11 AM TFHFony notifications@github.com wrote:

Not all statefiles work the same anymore. For example this one hangs the machine completely: https://www.file-hunter.com/MSX/games/File-Hunter.zip

The normal behaviour would be displaying my altered BASIC welcome (Just a bunch of print commands) followed by a reset (defusr=0:a=usr(0).

For the rest: Great Update!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIEJ2W7ES3JZZK6EHCTRHIPGTANCNFSM4LHBHE5A .

ppeccin commented 4 years ago

Oh, just now I noticed you said the reset is expected... I'll continue investigating...

Paulo

On Fri, Mar 13, 2020 at 3:28 PM Paulo Peccin ppeccin@gmail.com wrote:

Also, I tried to load this state with versions 5.4 and 5.3, and it also hangs or resets the machine. Are you sure this state you sent me is correct? On which version did it work for you???? Are you having problems with other states as well?

Paulo

On Fri, Mar 13, 2020 at 3:19 PM Paulo Peccin ppeccin@gmail.com wrote:

Also, how did you create this savestate? Is is too old? The contents of the state is very strange. It has several slots duplicated, like 2 BIOS Exstensions, 2 MSX-MUSIC and 2 DiskROMs.

Maybe on the previous release, you loaded a very old state, them saved it again? Something seems to have corrupted this savestate.

Paulo

On Fri, Mar 13, 2020 at 12:37 PM Paulo Peccin ppeccin@gmail.com wrote:

Hum... I don't know why, no errors occur when I load the state, just machine hang.

What software do you load? What does it do? Does it work if you not use the savestate and load it from scratch? Can you send me the software?

Peccin

On Fri, Mar 13, 2020 at 9:11 AM TFHFony notifications@github.com wrote:

Not all statefiles work the same anymore. For example this one hangs the machine completely: https://www.file-hunter.com/MSX/games/File-Hunter.zip

The normal behaviour would be displaying my altered BASIC welcome (Just a bunch of print commands) followed by a reset (defusr=0:a=usr(0).

For the rest: Great Update!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIEJ2W7ES3JZZK6EHCTRHIPGTANCNFSM4LHBHE5A .

TFHFony commented 4 years ago

The reset is indeed expected. I made this statefile some time ago after you updated WebMSX with the function to create smaller savestates. The program it's running is actually very simple:

10 CLS 20 PRINT"FILE-HUNTER.COM version 4.0 30 PRINT"Copyright 1999 by TFH|Fony 40 PRINT"28815 Bytes free 50 PRINT"Ok 60 PRINT"10 ' 70 PRINT"20 ' WWW.FILE-HUNTER.COM RULEZ 80 PRINT"30 ' and ( . ) ( . ) too 90 PRINT"40 ' 100 PRINT"Ok" 110 PRINT CHR$(219) 120 FORA=0TO2500:NEXT 130 DEFUSR=0:PRINTUSR(0)

And I made the savestate right after running it. I've checked my other savestates. These all seem to work. I will create a new savestate for my site. If you want, you can investigate further. On the other hand: I don't think it's a big issue. I can create a new save-state easily :)

ppeccin commented 4 years ago

Yes, go on and fix it. Please send me the new savestate, ok?

I'll investigate a little more, because that is not supposed to happen, and can show a hidden bug...

Thanks!

On Fri, Mar 13, 2020 at 4:55 PM TFHFony notifications@github.com wrote:

The reset is indeed expected. I made this statefile some time ago after you updated WebMSX with the function to create smaller savestates. The program it's running is actually very simple:

10 CLS 20 PRINT"FILE-HUNTER.COM version 4.0 30 PRINT"Copyright 1999 by TFH|Fony 40 PRINT"28815 Bytes free 50 PRINT"Ok 60 PRINT"10 ' 70 PRINT"20 ' WWW.FILE-HUNTER.COM RULEZ 80 PRINT"30 ' and ( . ) ( . ) too 90 PRINT"40 ' 100 PRINT"Ok" 110 PRINT CHR$(219) 120 FORA=0TO2500:NEXT 130 DEFUSR=0:PRINTUSR(0)

And I made the savestate right after running it. I've checked my other savestates. These all seem to work. I will create a new savestate for my site. If you want, you can investigate further. On the other hand: I don't think it's a big issue. I can create a new save-state easily :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55#issuecomment-598887847, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIGNXWESPGXLJIW362DRHKFUHANCNFSM4LHBHE5A .

TFHFony commented 4 years ago

Well, the new one has been made and I've replaced the old one: https://www.file-hunter.com/MSX/games/File-Hunter.zip

ppeccin commented 4 years ago

Hi,

Just to let you know I release v6.0.1 with some fixes, one is important to you. Please update!

Paulo

On Fri, Mar 13, 2020 at 5:06 PM TFHFony notifications@github.com wrote:

Well, the new one has been made and I've replaced the old one: https://www.file-hunter.com/MSX/games/File-Hunter.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55#issuecomment-598891140, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFOLIEGE5RF47Z7UQ4DZJTRHKG3BANCNFSM4LHBHE5A .

TFHFony commented 4 years ago

Hi Paolo,

Thanks! I will download it and update it today. Thanks for your work!

Arnaud

On Mon, Mar 16, 2020 at 1:36 AM ppeccin notifications@github.com wrote:

Hi,

Just to let you know I release v6.0.1 with some fixes, one is important to you. Please update!

Paulo

On Fri, Mar 13, 2020 at 5:06 PM TFHFony notifications@github.com wrote:

Well, the new one has been made and I've replaced the old one: https://www.file-hunter.com/MSX/games/File-Hunter.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55#issuecomment-598891140, or unsubscribe < https://github.com/notifications/unsubscribe-auth/AAFOLIEGE5RF47Z7UQ4DZJTRHKG3BANCNFSM4LHBHE5A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ppeccin/WebMSX/issues/55#issuecomment-599288957, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGWANZGPNGGWNZJXNCPKVN3RHVYBNANCNFSM4LHBHE5A .