Bug 159605 - CPU cycles on file-open increased with factor 4
Summary: CPU cycles on file-open increased with factor 4
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.4.0.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, perf, regression
Depends on:
Blocks: File-Opening Performance
  Show dependency tree / graph
 
Reported: 2024-02-06 18:48 UTC by Telesto
Modified: 2024-12-13 19:25 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2024-02-06 18:48:05 UTC 
Description:
Takes 55 seconds before the file becomes responsive (old CPU!). Previously 15 seconds

Steps to Reproduce:
1. Open process monitor tool
2. Open attachment 145863 [details] (bug 128280 and bug 120735) (automatic spell-check disabled)



Actual Results:
55 seconds CPU time spend on file-open (old CPU; modern mid-end CPU would probably do it in 25-35 seconds)

Expected Results:
15 seconds or so


Reproducible: Always


User Profile Reset: No

Additional Info:
Found in
Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 4d381b54d1c598c181b4a21a8bf0db86eb4668d1
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL threaded

but not in 
Version: 7.0.7.0.0+ (x64)
Build ID: 626ea4e62a3e5005fe9825923a1c0c5bdb61cc08
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 1 Telesto 2024-02-06 19:12:43 UTC 
Bisected to:
author	Luboš Luňák <l.lunak@collabora.com>	2022-05-01 19:24:26 +0200
committer	Luboš Luňák <l.lunak@collabora.com>	2022-05-02 15:06:43 +0200
commit 4bb730be0909d9cf55b7a44d7e916aa5de16b9f7 (patch)
tree 3668a9e5cb0c258955e90e87072dd8addc38b21b
parent 8e6ab6502278702499e7468404d1010069176578 (diff)
better cache size limit for vcl::text::TextLayoutCache
This is not as important as SalLayoutGlyphsCache, as these should
be smaller and less needed, but still, make sure to limit the memory
the cache may use.

https://gerrit.libreoffice.org/c/core/+/133674
Comment 2 m_a_riosv 2024-02-07 00:30:34 UTC 
Open instantly for me with
Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 4d381b54d1c598c181b4a21a8bf0db86eb4668d1
CPU threads: 16; OS: Windows 10.0 Build 22631; UI render: Skia/Vulkan; VCL: win
Locale: es-ES (es_ES); UI: es-ES
Calc: CL threaded
Comment 3 Telesto 2024-02-07 03:29:03 UTC 
FWIW: the file shows but is unresponsive for editing until the loading finishes
Comment 4 m_a_riosv 2024-02-07 15:19:10 UTC 
(In reply to Telesto from comment #3)
> FWIW: the file shows but is unresponsive for editing until the loading
> finishes

Sorry, but not for me.
Comment 5 Telesto 2024-02-08 09:34:37 UTC 
Also with (40 sec CPU spike)
Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: ba8f4bff6015013013df652efbfaf4d9ae10c881
CPU threads: 8; OS: macOS 14.3; UI render: Skia/Metal; VCL: osx
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded
Comment 6 Buovjaga 2024-02-28 16:42:18 UTC 
I got the same 55 secs with stopwatch when observing the CPU use for LibreOffice in Task Manager (mostly above 50%).

I confirm the bibisect result.