Bug 94792 - Fileopen: performance regression for xlsx with chart with >1.000 data labels; also dump (svllo!SfxBroadcaster::RemoveListener+16)
Summary: Fileopen: performance regression for xlsx with chart with >1.000 data labels;...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
3.4.0 release
Hardware: Other All
: medium minor
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisectRequest, perf, regression
Depends on:
Blocks:
 
Reported: 2015-10-05 15:33 UTC by c.kirbach
Modified: 2017-12-18 10:44 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
gdb information LO 4.4.5.2 (72.85 KB, text/x-log)
2015-10-05 15:33 UTC, c.kirbach
Details
minimum test case (2.15 MB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2015-10-18 08:47 UTC, c.kirbach
Details

Note You need to log in before you can comment on or make changes to this bug.
Description c.kirbach 2015-10-05 15:33:25 UTC
Created attachment 119312 [details]
gdb information LO 4.4.5.2

Infinite loop (100% CPU usage) trying to open Excel file

Cannot provide the Excel as it is confidential but may be able to reduce it to a minimum test case

attaching the gdb diagnose information I tool
Comment 1 Timur 2015-10-05 17:13:33 UTC
Please attach anonymized Excel file (xlsx, xls?)
Please try with the current LO master, for example http://dev-builds.libreoffice.org/daily/master/Linux-rpm_deb-x86_64@70-TDF/current/.
Comment 2 c.kirbach 2015-10-18 08:46:40 UTC

Infinite loop does not happen with lastest LO5 alpha build.
I reduced it to an anonymised minimum test case that I am attaching.
Comment 3 c.kirbach 2015-10-18 08:47:27 UTC
Created attachment 119712 [details]
minimum test case
Comment 4 Timur 2015-10-19 08:48:47 UTC
(In reply to c.kirbach from comment #2)
> Infinite loop does not happen with latest LO5 alpha build.
It doesn't happen with master, LO 5.1+, although it's rather slow to open.
It opens even with 5.0.2.2, but after a "very" long time, so I wouldn't say it's  infinite loop. It was some recent regression in terms of terribly slow opening.
Looks like something is fixed, and future releases 5.0.3 and 4.4.6 should be checked for backport.

I think this bug may be closed as "WorksForMe" but another one should be open for poor performance with the opening. It's a regression from LO 3.4. 
LO 5.1+ took 112 sec in Windows with i7 processor. OO 3.3 and LO 3.3.4 took "only" 27 seconds to open.
Or this bug may be changed to "Fileopen: performance regression for xlsx with chart"
Comment 5 c.kirbach 2015-10-20 15:27:03 UTC
Renaming as suggested by you.


It doesn't happen with master, LO 5.1+, although it's rather slow to open.
It opens even with 5.0.2.2, but after a "very" long time. It was some recent regression in terms of terribly slow opening.
Looks like something is fixed, and future releases 5.0.3 and 4.4.6 should be
checked for backport.
Comment 6 Markus Mohrhard 2016-04-08 02:08:58 UTC
Please don't forget to tag performance problems as they can be easily missed.
Comment 7 Markus Mohrhard 2016-04-18 23:52:43 UTC
(In reply to c.kirbach from comment #5)
> Renaming as suggested by you.
> 
> 
> It doesn't happen with master, LO 5.1+, although it's rather slow to open.
> It opens even with 5.0.2.2, but after a "very" long time. It was some recent
> regression in terms of terribly slow opening.
> Looks like something is fixed, and future releases 5.0.3 and 4.4.6 should be
> checked for backport.

Performance improvements are nearly never backported.
Comment 8 Xisco Faulí 2016-09-20 12:28:26 UTC
Adding keyword 'bibisectRequest'
Comment 9 Xisco Faulí 2017-09-29 08:48:14 UTC Comment hidden (obsolete)
Comment 10 c.kirbach 2017-12-18 10:10:03 UTC
I retested with LO 5.4.4 and the former behaviour was significantly improved. Opening the test file takes about 25 seconds.

I think we could do even better, but as the worst has been resolved. I will leave it for others to decide whether it is worth to keep this case open.

Thank you for your support in this issue.
Comment 11 Timur 2017-12-18 10:44:59 UTC
It takes 55 sec for me to open in LO 6.1+ and just 5 sec in MSO. 
Although it's also slow in MSO ang gives warning "Max number of data labels is 1.000. Some labels will be omitted from the chart". 

But, there also a dump on fileopen that can be seen with procdump:
SYMBOL_NAME:  svllo!SfxBroadcaster::RemoveListener+16
IMAGE_NAME:  svllo.dll
FAILURE_ID_HASH_STRING:  um:status_breakpoint_80000003_svllo.dll!sfxbroadcaster::removelistener

So I 'll set back to New. 
(BTW: if it worked it'd be WorksForMe and not FIXED which is used if fix commit is known)