Bug 113564 - EDITING: Memory is released slowly when copying a large dataset first and small afterwards
Summary: EDITING: Memory is released slowly when copying a large dataset first and sma...
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.1.0.3 release
Hardware: All Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, perf
Depends on:
Blocks:
 
Reported: 2017-11-01 08:58 UTC by Telesto
Modified: 2017-11-23 14:47 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
Bibisect log (2.88 KB, text/plain)
2017-11-01 08:59 UTC, Telesto
Details
Screencast LibO6 (924.83 KB, video/mp4)
2017-11-01 12:42 UTC, Telesto
Details
Screencast LibO5.0.0.0alpha1+ (1.38 MB, video/mp4)
2017-11-01 12:43 UTC, Telesto
Details
Bibisect log (2.59 KB, text/plain)
2017-11-02 14:47 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2017-11-01 08:58:26 UTC
Description:
Memory is released slowly when copying a large dataset first and small afterwards

Steps to Reproduce:
1. Open the file found here: https://yadi.sk/i/rM9QctDym5y3M 
2. Open a the task manager and monitor the memory usage
3. Copy columns A-C
4. Copy cell A1. 

Actual Results:  
LibO5.0 releases all the memory instantly after copying the small dataset (cell A1). I takes a while with LibO5.1

Expected Results:
Should be as as fast as in LibO5 in my opinion. This probably will create issues when running multiple instances of Calc and copying back and forwards 


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 6.0.0.0.alpha1+
Build ID: 13c5dd1d98a480cb01ca8f24242c80e326e4ade8
CPU threads: 4; OS: Windows 6.3; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-10-30_23:52:07
Locale: nl-NL (nl_NL); Calc: CL


User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
Comment 1 Telesto 2017-11-01 08:59:32 UTC Comment hidden (obsolete)
Comment 2 Telesto 2017-11-01 09:03:34 UTC
Wrong bibisect results; commit is revert.
Comment 3 Telesto 2017-11-01 12:42:27 UTC
Created attachment 137418 [details]
Screencast LibO6
Comment 4 Telesto 2017-11-01 12:43:07 UTC
Created attachment 137419 [details]
Screencast LibO5.0.0.0alpha1+
Comment 5 Telesto 2017-11-02 14:47:34 UTC
Created attachment 137466 [details]
Bibisect log

Bisected to:
author	Eike Rathke <erack@redhat.com>	2015-09-08 11:59:57 (GMT)
committer	Eike Rathke <erack@redhat.com>	2015-09-08 12:02:22 (GMT)
commit 37a36671e0abfa0e1dfd2cfdd6470a3f2805b199 (patch)
tree 191ff35cdf8521c8903c983ad01125c3e3cd9b97
parent 7df729da30f89ad20cd5705e3a1a91866ac71898 (diff)
reactivate fixed mempool for ScFormulaCell
... it got lost with 8b252f30267d043522ff2cbf2bf42275bb7a6ec6
Comment 6 Buovjaga 2017-11-02 16:32:39 UTC
I confirm, but I will leave it to Eike to comment on whether this is a problem or not.

Version: 6.0.0.0.alpha1+ (x64)
Build ID: 7e03c4eed72452fdfb87341214a21956c08ba969
CPU threads: 4; OS: Windows 10.0; UI render: default; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2017-10-26_00:58:29
Locale: fi-FI (fi_FI); Calc: group
Comment 7 Telesto 2017-11-23 13:28:31 UTC
No repro with
Version: 6.0.0.0.alpha1+
Build ID: c24c32bf71b8e64bd0d36e511f554e1f6c015842
CPU threads: 4; OS: Windows 6.3; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-11-22_23:15:41
Locale: nl-NL (nl_NL); Calc: CL
Comment 8 Buovjaga 2017-11-23 14:47:24 UTC
(In reply to Telesto from comment #7)
> No repro with
> Version: 6.0.0.0.alpha1+
> Build ID: c24c32bf71b8e64bd0d36e511f554e1f6c015842
> CPU threads: 4; OS: Windows 6.3; UI render: default; 
> TinderBox: Win-x86@42, Branch:master, Time: 2017-11-22_23:15:41
> Locale: nl-NL (nl_NL); Calc: CL

True for me as well. I think we should close and be happy.

Version: 6.0.0.0.alpha1+ (x64)
Build ID: a0ebba3d8855fee0bcec04a10137ae3a4f9f0e77
CPU threads: 4; OS: Windows 10.0; UI render: default; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2017-11-22_05:59:17
Locale: fi-FI (fi_FI); Calc: group threaded