Bug 790062 - gvfs-udisks2-volume-monitor leaks memory
Summary: gvfs-udisks2-volume-monitor leaks memory
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gvfs
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: David Zeuthen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-13 15:17 UTC by Zdenek Kabelac
Modified: 2012-05-02 04:56 UTC (History)
5 users (show)

Fixed In Version: gvfs-1.12.2-1.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-02 04:56:56 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Zdenek Kabelac 2012-02-13 15:17:40 UTC
Description of problem:

There must be a serious memory leak since this daemon currently occupies 800MB
on my system.  I'm running on my system lvm2 test thus a lot of devices are appearing and disappearing during test suite run.

VMSIZE:    1058300 kb
RSS:        819528 kb total
                 4 kb shared
                 0 kb private clean
            819524 kb private dirty
PRIVATE MAPPINGS
     vmsize   rss clean   rss dirty   file
  816128 kb        0 kb   816080 kb   [heap]
    2964 kb        0 kb     2964 kb
      48 kb        0 kb       48 kb
      32 kb        0 kb       32 kb   /usr/lib64/libstdc++.so.6.0.17
      32 kb        0 kb       32 kb   /usr/lib64/libxml2.so.2.7.8
      24 kb        0 kb       24 kb   /usr/lib64/libudisks2.so.0.0.0
     136 kb        0 kb       24 kb   [stack]
      20 kb        0 kb       20 kb
      84 kb        0 kb       16 kb
      16 kb        0 kb       16 kb   /usr/lib64/libc-2.15.so
      16 kb        0 kb       16 kb   /usr/lib64/libgio-2.0.so.0.3116.0
    8192 kb        0 kb       12 kb
...

Version-Release number of selected component (if applicable):
gvfs-1.11.3-1.fc17.x86_64

How reproducible:


Steps to Reproduce:
1. probably running lvm2 test suite a few times
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 David Zeuthen 2012-02-16 10:28:30 UTC
Sure, could be there's a memory leak, not entirely impossible (I check for memory leaks from time to time while stressing the kernel block subsystem but it's been a while...). Can you also check if the udisksd process is leaking as well? Thanks

Comment 2 Zdenek Kabelac 2012-03-13 10:22:54 UTC
So far I'm only killing gvfs-udisks2-volume-monitor as it easily get to 1GB memory size after few runs of lvm test suite. Other daemons in my system do not seem to grow (at least not in such marginal way)

Comment 3 Richard W.M. Jones 2012-03-29 16:10:48 UTC
You guys are lightweights.  I've got this thing over 5GB ...

 1912 rjones    20   0 5042m 4.3g 3248 S  0.0 56.8   3:02.17 gvfs-udisks2-vo    

Very easy to reproduce too: just run some tests that continually
mount and unmount a filesystem.

Comment 4 Richard W.M. Jones 2012-03-29 16:12:05 UTC
udisksd is not using a large amount of memory.  It's definitely
a leak in gvfs-udisks2-volume-monitor alone.

Comment 5 David Zeuthen 2012-03-29 16:40:05 UTC
Yeah, sorry, I forgot about this... I'll start working on fixing it. Thanks!

Comment 6 David Zeuthen 2012-04-09 15:50:40 UTC
OK, I've fixed the memory leaks with this upstream commit

 http://git.gnome.org/browse/gvfs/commit/?id=d870015bff4eb8f34f465f4c45d91ca326f52a43

This should be included in F17.

Comment 7 Fedora Update System 2012-04-27 13:37:19 UTC
gvfs-1.12.2-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/gvfs-1.12.2-1.fc17

Comment 8 Fedora Update System 2012-04-27 23:35:07 UTC
Package gvfs-1.12.2-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing gvfs-1.12.2-1.fc17'
as soon as you are able to, then reboot.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-6873/gvfs-1.12.2-1.fc17
then log in and leave karma (feedback).

Comment 9 Zdenek Kabelac 2012-04-30 12:51:57 UTC
I can confirm, I'm no longer seeing this leak during my tests

Comment 10 Fedora Update System 2012-05-02 04:56:56 UTC
gvfs-1.12.2-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.