| Summary: | Choosing "Document Colors" from font/background palette takes 2-3 minutes with 100% CPU use | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Ron Johnson <ronljohnsonjr> |
| Component: | Calc | Assignee: | ursache |
| Status: | VERIFIED FIXED | ||
| Severity: | minor | CC: | jbfaure, jmadero.dev, markus.mohrhard, miguelangelrv, xiscofauli |
| Priority: | high | Keywords: | perf |
| Version: | 4.4.0.3 release | ||
| Hardware: | All | ||
| OS: | Linux (All) | ||
| Whiteboard: | target:4.5.0 target:4.4.2 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 107380 | ||
| Attachments: | ODS that demonstrates the bug. | ||
|
Description
Ron Johnson
2015-02-01 00:49:37 UTC
Please attach a document that we can see this 3 minute wait. I just did a test on a spreadsheet with about 10 cells with different colors and saw no delay. Ubuntu 14.04 x64 LibreOffice 4.4.0.2 rc Marking as NEEDIFNO. Please attach the document and let us know what your system is running (OS + Distro at least). Once you do this mark the bug as UNCONFIRMED. Thanks It has sensitive financial data. I'll have to see if I can create a sanitized version of it. Created attachment 113011 [details]
ODS that demonstrates the bug.
This is the sanitized version of the document. Note how long it takes to choose the Document color palette.
I'm not able to reproduce. Win7x64 Version: 4.4.0.3 Build ID: de093506bcdc5fafd9023ee680b8c60e3e0645d7 Please could you try resetting the user profile, sometimes solves strange issues. https://wiki.documentfoundation.org/UserProfile "try resetting the user profile, sometimes solves strange issues." Maybe that helped some, since it only took 1:25 this time. Linux 64 bit Version: 4.4.0.3 Build ID: de093506bcdc5fafd9023ee680b8c60e3e0645d7 Locale: en_US Is the same when selecting font color or line color? (In reply to m.a.riosv from comment #6) > Is the same when selecting font color or line color? Font color: yes. Line color: ?? Format Cells->Borders->Color has a drop-down list. Please verify if options in Menu/Tools/LibreOffice/View - Graphics Output, has some influence. Not reproducible for me too with my own build of version 4.4.1.0.0+ under Ubuntu 14.10 x86-64. Best regards. JBF (In reply to m.a.riosv from comment #8) > Please verify if options in Menu/Tools/LibreOffice/View - Graphics Output, > has some influence. The bug is a CPU-bound scan through the document, not an issue of display. Are there -dbg versions of LO 4.4? I tried to run strace, but after a bit of startup, it sat at:
shutdown(3, SHUT_RDWR) = 0
close(3) = 0
futex(0x1bcf0a4, FUTEX_WAIT_BITSET_PRIVATE|FUTEX_CLOCK_REALTIME, 7, {1422999234, 628689000}, ffffffff) = 0
I'm setting this to new as I can confirm the issue with the test document: Ubuntu 14.04 x64 LibreOffice 4.4.0.3 Priority on this one is pretty hard but for now I'm sticking to the routine: Minor - it can slow down professional quality work but will not completely prevent it. High - It makes the particular feature almost unuseable in documents like this (2-3 minute delay). What makes it particularly troublesome is that it happens every time you use the font color drop down (the color pallete isn't stored at all so every time it regenerates the list which takes another 2-3 minutes) As for a debug, I'll try to get something together. (In reply to Joel Madero from comment #12) > I'm setting this to new as I can confirm the issue with the test document: > Ubuntu 14.04 x64 > LibreOffice 4.4.0.3 Thanks, because with the greatly expanded palette size, Document Colors is a great idea. Ursache Vladimir committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6a348503ad504a82778f86d35325737b712c5352 tdf#89004 improve performance of document data collection It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Ursache Vladimir committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=1e475fef47fe6bd9dba6d830aaf0b6c12dc88881 related tdf#89004 improve performance of document data collection It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Ursache Vladimir committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e7672e9633d0fe23917a2a348e77808a40ae0942&h=libreoffice-4-4 tdf#89004 improve performance of document data collection It will be available in 4.4.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Ursache Vladimir committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5fb88fddb856f53e0b187a3cf2ca98c9b83e874c&h=libreoffice-4-4 related tdf#89004 improve performance of document data collection It will be available in 4.4.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Ursache Vladimir committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=50a36041e18dc2a595259b5414ded409687dc47b related tdf#89004 move to std:set instead of std::vector It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Verified on 4.5 master built today. Thanks for the quick fix! Works perfect in v5.0.0.5. Thanks. |