Menu

#6983 SWORD25: long saving time

Broken Sword 2.5
closed-fixed
None
7
2016-03-28
2015-12-10
mim
No

It takes 15 second to save the game (from clicking on slot to completing the save). When I play the game outside of ScummVM, saving is without any delay.

ScummVM from December 9 16.20
BS English version
WinXP SP3, Core2DuoE6750

Discussion

  • Strangerke

    Strangerke - 2016-02-01
    • Priority: 5 --> 7
     
  • Strangerke

    Strangerke - 2016-02-01

    It would be nice to fix it before the release of 1.8.0. Raising priority.

     
  • Willem Jan Palenstijn

    Does this still happen for you?

     
  • Willem Jan Palenstijn

    And if it does, does it happen right from the start of the game? Or only after playing some time?

     
  • mim

    mim - 2016-02-04

    Yes, it still happens and right from the start of the game.

     
  • RichieSams

    RichieSams - 2016-02-04

    Can you confirm this still happens when using the latest daily build? What are the file sizes of the savegames?

    In my tests, saves are about 1-2 seconds. But I will do some profiling to find out more

     
  • mim

    mim - 2016-02-04

    Yes, it is the same in Feb 5 2016 9:53 build. The savegames size: 466kB, 489kB, 488kB. 13 seconds saving time now with stopwatch.

     

    Last edit: mim 2016-02-04
  • RichieSams

    RichieSams - 2016-02-04

    Thank you for the update. Can you attach one of the savegames?

     
  • mim

    mim - 2016-02-04

    Yes, the savegame attached.

     
  • Eugene Sandulenko

    • status: open --> closed-fixed
    • assigned_to: Eugene Sandulenko
     
  • Eugene Sandulenko

    Fixed. It was combination of poor reallocation strategy in ScummVM and poor memory management in Windows.

    During the save we were reallocating 15.5GB of memory, which was a single save growing from 3 bytes to 1.2MB.

    Now after fix we are reallocating 4.5MB, e.g. 300 times less.