uses too much memory

Bug #886633 reported by Scott Ringwelski
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Pantheon Wallpaper
Confirmed
Undecided
Unassigned

Bug Description

My pantheon-wallpaper is using 86.6 MB RAM... memory leak maybe?

Revision history for this message
Avi Romanoff (aroman) wrote :

 Scott, can you confirm this is still an issue with the latest granite/oneiric? Pantheon-wallpaper does take a decent amount of RAM, but I have found that even after changing wallpapers dozens of times it never goes above 50MB.

As Max explained it to me, it takes up much RAM because it needs to hold two buffers for the images in RAM simultaneously.

In any event, can you confirm whether your memory usage grows boundlessly, or if there's some cutoff point that would indicate that it isn't a bug and simply being something of an inherent ram hog?

Marking as incomplete for now.

Changed in pantheon-wallpaper:
status: New → Incomplete
Changed in pantheon-wallpaper:
milestone: none → 0.2-beta
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Here's a memory consumption graph

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

a script to generate it

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

and the raw log used for the graph

Changed in pantheon-wallpaper:
status: Incomplete → Confirmed
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

oh nvm, must be flawed somehow...

Changed in pantheon-wallpaper:
status: Confirmed → Incomplete
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

OK this works

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

no, it doesn't. derp.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

OK, THIS finally works.
The script monitors existing pantheon-wallpaper process if there's one, and spawns one if there isn't. It changes wallpaper lots of times, logs memory usage in kB after every change and finally feeds the log to gnuplot.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

log for "fresh start" (process spawned from the script)

Changed in pantheon-wallpaper:
status: Incomplete → Confirmed
Revision history for this message
Eduard Gotwig (gotwig) wrote :

It still uses around 30 MB RAM here. Tooo much, for such a program!

Can't we use the standard gnome one? I noted that when you kill pantheon wallpaper, and wait some time, somehow a desktop background gets drawn (without nautilus).

Without pantheon wallpaper RAM usage is around 200 MB. That is btw. really awesome.

Revision history for this message
Victor Martinez (victored) wrote :

Hmmm, I took a quick look at the code, and the best solution would be dropping the buffers. They're only used for smooth transitions between wallpapers AFAIK.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

We can't drop the buffers obviously, but that memory should be freed after every crossfade.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Here's the grapher script updated for settings location of latest trunk

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.