SedrynTyros wrote:

Awesome! Thank you very much. 😀

EDIT:

Patch seems to work for me in the few tests I ran. This is great! I've been hoping for this for a long time.

Your post on ZenJu's "Save States - Proof of concept" thread was also one of the reasons for this journey. Though ZenJu and tikalat have done a really wonderful job, there are random crashes occasionally. Troubleshooting such issues is simply above me.

Now that the diff is uploaded, don't expect anything more from this thread on release of new SVN's 🤣 , especially at the rate Qbix is releasing one after the other 🤣 . Thankfully no significant changes in the core dosbox code has helped this patch to stay alive with only minor modifications. So, anyone is welcome to take the diff and apply it to future SVN's themselves. It is not that hard, only tedious, to manually copy the savestate code to 70 odd files, once you have the diff as reference. 'Code Compare' helped a lot in this case for me. However keep in mind that this may not work in case of significant changes to the code.

I'm using Linux build but some games have a vsync problem. For example with Pinball Fantasies, Aladdin and Pinball Illusion. Is there a patch for vsync?

Do you mean to say that the vsync issue only happens after loading a savestate in the games you mentioned?

Edit: On second thoughts, the above case is rather doubtful. Somehow the combination of pinball games and savestates seems odd i.e. unless you are trying to break some world record score 🤣 . Assuming that the vsync problem you are facing is more general in nature and not particular to this dosbox+savestate build, please see this thread for possible solution to your problem(though it is for windows build). Topics abound if you search for 'vsync' in this forum. Please go through them and if your issue is still not resolved, it would be better to open a new thread in the appropriate sub-forum with full details.

I'm sorry, it's not about savestate but only at dosbox linux binary which I used. I'll change thread.

@gandhig, I'm having trouble patching DOSBox, it freezes or takes a long time.
What's your patching command? "patch -p0 < patch,diff" from the HOME (~) directory right?

Also the patch calls for having "r4019_plain" "r4019_sav" directories, wouldn't it better to have them from withing these directories?
Then you don't actually have to have a "r4019_sav" directory to patch.

For example to make a patch from within "r4019_sav" directory:
diff -Nurb ~/r4019_plain . > ~/nameofpatch.patch

@James-F, as you might be aware, there are many different options for generating a diff / applying a patch(as per user's preferences).

In my case(to keep it simple),

1) copy patch file to your dosbox home directory (containing autogen.sh(no intention to demean 😀 , just to ensure we are on the same page))
2) cd to dosbox home directory in mingw/linux
3) execute command 'patch -p1 < nameofpatch.patch'

Thanks it worked, I've been using -p0 with my patches and that included the first directory, -p1 ignores the first directory.
But compilation did not.

Am I missing a library?
They are all related to ZIP and Compression.

1C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/save_state.cpp:67: undefined referen
2ce to `compressBound'
3C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/save_state.cpp:73: undefined referen
4ce to `compress2'
5save_state.o: In function `ZN4Util10decompressERKNSt7__cxx1112basic_stringIcSt11
6char_traitsIcESaIcEEE':
7C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/save_state.cpp:99: undefined referen
8ce to `uncompress'
9minizip.o: In function `getFileCrc':
10C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/minizip.c:180: undefined reference t
11o `crc32'
12C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/minizip.c:180: undefined reference t
13o `crc32'
14unzip.o: In function `unzReadCurrentFile':
15C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/unzip.c:1784: undefined reference to
16 `crc32'
17C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/unzip.c:1854: undefined reference to
18 `inflate'
19C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/unzip.c:1865: undefined reference to
20 `crc32'
21unzip.o: In function `unzCloseCurrentFile':
22C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/unzip.c:2030: undefined reference to
23 `inflateEnd'
24unzip.o: In function `unzOpenCurrentFile3':
25C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/unzip.c:1584: undefined reference to
26 `inflateInit2_'
27zip.o: In function `zipWriteInFileInZip':
28C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1413: undefined reference to `
29crc32'
30C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1472: undefined reference to `
31deflate'
32zip.o: In function `zipCloseFileInZipRaw64':
33C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1542: undefined reference to `
34deflate'
35C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1584: undefined reference to `
36deflateEnd'
37zip.o: In function `zipOpenNewFileInZip4_64':
38C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1248: undefined reference to `
39get_crc_table'
40C:\MinGW\msys\1.0\home\TESTER\r4019_sav\src/zip.c:1220: undefined reference to `
41deflateInit2_'
42collect2.exe: error: ld returned 1 exit status
43make[3]: *** [dosbox.exe] Error 1
44make[3]: Leaving directory `/home/TESTER/r4019_sav/src'
45make[2]: *** [all-recursive] Error 1
46make[2]: Leaving directory `/home/TESTER/r4019_sav/src'
47make[1]: *** [all-recursive] Error 1
48make[1]: Leaving directory `/home/TESTER/r4019_sav'

These 'undefined references' point to the libraries, 'lpng' & its dependency, 'zlib'. Have they been installed in your mingw environment? If yes, please check their versions too. For your ready reference, the details of the dependencies and its versions in my mingw win32 build are reproduced from one of the earlier posts:

Win32 Build Summary:

11) Followed Building DOSBox wiki
2
32) Works in both WinXP and Win7 unlike the previous VS2017 build(WinXP - not a valid win32 application error)
4
53) Dependencies compiled: 
6   a) SDL (1.2.15) - library directly added
7   b) Direct Draw - libraries directly added
8   c) libpng (1.6.29) & its dependency zlib( 1.2.8 )
9   d) SDL_net (1.2.8)
10   e) libogg (1.3.2)
11   f) libvorbis (1.3.5)
12   g) SDL_sound (1.0.3)

Hi, I updated save states, some things were not saved, so I took missing stuff from daum build and I fixed some errors by myself, I don't understand why these errors didn't appear in daum... I use more enhanced version, so I merged changes to r4019, hope without errors. Sorry, I don't know how to create diff. With these changes increases significantly range of games which supports save/load states, but still not perfect. Dynrec needs more mork, it wont load everything correctly. Seems, it needs reset, like has _x86 version.

Great job bruenor41. It would be helpful if you could list out the games that got fixed or performed better w.r.t savesate loads.

C&C1 and Red Alert - fixed segmentation faults
Fallout 1 - fixed crash when loading states through different areas
Earth 2140 - fixed crash
Wolfenstein 3d + SWOLF - fixes graphical glitches
Lands of Lore 1 - loads now correctly when loading through different areas, dynrec does not work here
etc

Games what I tested for a longer time and seems to be OK:
Warcraft 1,2
Albion - from beginning to the end I used states and works at 100%
Fallout 1
C&C1
Doom
Might and Magic 1
KKND
Ignition
Wacky wheels
Dangerous Dave
Bio Menace

Shortly tested :
Settlers 2
Realms of Arkania 1,2 (3rd part was not tested)
Monkey Island 1
Ultima 7 - states wont load during intro and menu, but in game it works
Duke 3D
Syndicate
Willy Beamish

Games which I cant get work :
Daggerfall

Hi people, here is newest version of save/load states. Because my knowledge of cpu is not so good, I asked M-HT for a help. He implemented missing dynrec cache reset. I send him big thanks. I merged that fix to version posted above. As far as I tried, all games tested by me were loaded correctly now. Please, if you find game what does not work correctly with save/load states, post here it's name and your dosbox configuration. I'll try look at this.

EDIT : Save states checks which exe file is currently loaded. Might happen that you try load save stated during intro, but intro might be another exe file loaded by main game file. In this case, states are loaded when intro finishes and is loaded correct exe file.

Compiled version would be great. 😀

bruenor41 wrote:

Hi people, here is newest version of save/load states.

Thanks again bruenor41.

Firtasik wrote:

Compiled version would be great. 😀

Sorry for the delay...work 😵 . Bruneor's latest builds(lin & win) attached in the first post.

There's no need to apologise. Thanks! 😁

Alien Carnage is still crashing.

Yes, right. I am now on vacation, but I can look at this next week. Thanks for info

I checked Alien Carnage. Sorry, currently I don't know how to fix it. Problem is that this game creates temporary files like HARRY14.$$$, but temporary files are not stored in save states. And if game does not find them or makes decision that are invalid, then it crashes. Problem is that I don't know which file is temporary, game can create many files, like save files or config files which cant be included in save state. I need some game identifier, then I can make game specific fixes.

One way is to set somewhere (in config?) that game which you run is really alien carnage, but best would be to let dosbox recognize game automatically, but I dont know how

Does it work in daum?

It doesn't work in Daum, but gandhig has made this:
Dosbox SVN r4019 + savestates Build (Alpha)

will check it

It was more of a workaround than a fix actually and nothing brainy. As bruenor41 said, the issue was due to temporary files created by the game. The save/load code for dosfiles was just commented out for the Alien carnage specific build, as an attempt for an actual fix failed on my part due to insufficient knowledge. Hence the advice to use the save/load feature of the game itself between levels and to use savestate feature within a level.