Bug 151109 - SVG cache size is insufficient with a large screen
Summary: SVG cache size is insufficient with a large screen
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-20 20:44 UTC by Jay
Modified: 2008-01-11 06:36 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
grows the cache up to 1% of memory (8.39 KB, patch)
2007-12-24 11:48 UTC, Jason Stubbs
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jay 2007-10-20 20:44:50 UTC
Version:            (using KDE KDE 3.94.0)
Installed from:    SuSE RPMs
OS:                Linux

I tried KDE 4 beta 3 on a 1920x1200 screen and it was horribly slow, because apparently the background svg was rerendered all the time. When I ran plasma from console it constantly reported something akin to this:

plasma::svg::findInCache(): pixmap cache too small to hold <wallpapernamehere.svg> with 1920x1200.
Comment 1 Aaron J. Seigo 2007-10-20 20:53:36 UTC
thanks for reporting this. we are already aware of the issue, however, and will be fixing it before release. i'll close this bug when the issue has been properly addressed in svn. =)
Comment 2 Luka Renko 2007-10-22 02:40:22 UTC
It looks like this is also the problem on my laptop (HP nw8240 with 1920x1200 display). Would be really great if this can be addressed soon as current version is not useful at all.
Comment 3 Aaron J. Seigo 2007-10-22 03:43:31 UTC
in the meantime you can quite easily work around this by opening up ~/.kde/share/config/plasmarc and adding something like this:

[General]
wallpaper=foo

you'll get no wallpaper, but it should not be usable ;)
Comment 4 Aaron J. Seigo 2007-10-25 13:22:13 UTC
ok, so i've disabled cached painting for the wallpaper which should help already substantially. i still don't consider this fully/properly fixed as we can still run into caching issues through too many applets. we've discussed this on the plasma list and should be able to come up with a proper solution.

but for now, at least, wallpapers won't kill you ;)
Comment 5 Luka Renko 2007-10-31 21:52:10 UTC
Aaron, just to confirm - your workaround works great for me (no CPU load anymore). Thanks a lot.
Comment 6 Pino Toscano 2007-12-05 11:09:41 UTC
Re-unconfirm (owner changing confirmed the bug).
Comment 7 Jason Stubbs 2007-12-24 11:48:20 UTC
Created attachment 22671 [details]
grows the cache up to 1% of memory

A 1920x1200 pixmap will take approximately 9mb. This patch tries to increase
the cache size to double but limits the cache size to 1% of memory. Thus on a
1gb system, you'll end up with a cache size of ~10mb allowing it to fit, and a
cache size of up to ~18mb on 2gb+ systems.
Comment 8 Jason Stubbs 2008-01-02 11:46:30 UTC
SVN commit 755892 by jstubbs:

Dynamically set the pixmap cache size to the greater of 1.1 times the screen
resolution and 1% of physical system memory. This should be enough to carry
over until there's many SVG based applets available for people to run and
overload their systems with. ;)

CCBUG: 151109


 M  +25 -6     plasmaapp.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=755892
Comment 9 Aaron J. Seigo 2008-01-11 06:36:08 UTC
ok, this seems to be working decently on high resolutions. there are other problems there, but this doesn't seem to be one of them anymore =)