In-proc some applets to reduce memory

Bug #39155 reported by Ben Maurer
16
Affects Status Importance Assigned to Milestone
gnome-panel (Ubuntu)
Fix Released
Medium
Sebastien Bacher

Bug Description

By making the notification applet and wnck applet inprocess, about 4 mb of memory can be saved on the desktop. Trash-applet is also a good candidate for in-proc'ing.

See discuession and patch:

http://bmaurer.blogspot.com/2006/04/fighting-daemons.html
http://www.contrib.andrew.cmu.edu/~bmaurer/memory/13_inproc.patch

openSUSE has been applying a similar patch and it is on target for 10.1

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you Ben. Did you send those patches (they should be splitted since they changed different tarballs actually) upstream? Do you have the bugzilla bug numbers for them? I'll look at doing the changes before dapper but having them upstream too would be nice!

Changed in gnome-panel:
assignee: nobody → seb128
status: Unconfirmed → Confirmed
Revision history for this message
Michael Frank (msfrank) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Is there an easy way to verify if the patch works correctly? How do you get the usage for an applet?

Revision history for this message
Ben Maurer (bmaurer) wrote : Re: [Bug 39155] Re: In-proc some applets to reduce memory

On Sun, 2006-05-07 at 21:49 +0000, Sebastien Bacher wrote:
> Is there an easy way to verify if the patch works correctly? How do you
> get the usage for an applet?

Use the script here:

http://www.contrib.andrew.cmu.edu/~bmaurer/memory/smem.pl

You want to check that the non-shared rss for {gnome-panel after patch}
< Sum [{gnome-panel + all applets before the patch}

-- Ben

Revision history for this message
Ben Maurer (bmaurer) wrote : Re: [Bug 39155] Re: [Bug 39155] Re: In-proc some applets to reduce memory

On Sun, 2006-05-07 at 22:08 +0000, Ben Maurer wrote:
>
>
> Use the script here:
>
> http://www.contrib.andrew.cmu.edu/~bmaurer/memory/smem.pl
>
> You want to check that the non-shared rss for {gnome-panel after
> patch}
> < Sum [{gnome-panel + all applets before the patch}

Btw, an example of the measurements I did:

http://bmaurer.blogspot.com/2006/04/fighting-daemons.html

See the table in that post. That should give you a good number on the
reduction as a result of this patch.

-- Ben

Revision history for this message
Sebastien Bacher (seb128) wrote :

This upload fixes the issue:

 gnome-panel (2.14.1-0ubuntu10) dapper; urgency=low
 .
   * debian/gnome-panel.install:
     - list new .so built instead of the binaries to the new patch
   * debian/patches/10_some_applets_in_proc.patch:
      - patch by Ben Maurer <email address hidden>
      - change some applets to be in-proc for best memory usage on the desktop
        (Ubuntu: #39155)
   * debian/patches/12_autoconf.patch:
     - updated
   * debian/patches/99_about-ubuntu-translation.patch:
     - updated

Changed in gnome-panel:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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