Bug 763849 (GLUSTER-2117) - [glusterfs-3.1.1qa7]: memleak in glusterfsd
Summary: [glusterfs-3.1.1qa7]: memleak in glusterfsd
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-2117
Product: GlusterFS
Classification: Community
Component: glusterd
Version: 3.1.0
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: shishir gowda
QA Contact:
URL:
Whiteboard:
: 763846 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-11-16 15:57 UTC by Raghavendra Bhat
Modified: 2015-12-01 16:45 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
valgrind log file (75.11 KB, application/octet-stream)
2010-11-17 01:32 UTC, Raghavendra Bhat
no flags Details
glusterfs client valgrind log file (163.58 KB, application/octet-stream)
2010-11-17 04:29 UTC, Raghavendra Bhat
no flags Details

Description Raghavendra Bhat 2010-11-16 15:57:35 UTC
there is a memleak in glusterd. This is the valgrind report indicating the leak.




==18038== LEAK SUMMARY:
==18038==    definitely lost: 100,623 bytes in 525 blocks
==18038==    indirectly lost: 0 bytes in 0 blocks
==18038==      possibly lost: 17,456,525 bytes in 1,691 blocks
==18038==    still reachable: 3,372,663 bytes in 127 blocks
==18038==         suppressed: 0 bytes in 0 blocks
==18038== Reachable blocks (those to which a pointer was found) are not shown.
==18038== To see them, rerun with: --leak-check=full --show-reachable=yes
==18038== 


the volgrind log file is attached

Comment 1 Raghavendra Bhat 2010-11-17 01:32:09 UTC
Created attachment 386

Comment 2 Raghavendra Bhat 2010-11-17 04:29:07 UTC
I repeated the tests and these are the new valgrind logs indicating the leak. I cleaned up the mount point after the tests (posix compliance, dbench 100, areual /etc) and then unmounted it.

Comment 3 Raghavendra Bhat 2010-11-17 04:29:37 UTC
Created attachment 387

Comment 4 Anand Avati 2010-11-18 10:56:08 UTC
PATCH: http://patches.gluster.com/patch/5739 in master (Remove spurious inode_ref call on parent dir in fuse_create_cbk)

Comment 5 Amar Tumballi 2010-11-19 03:20:55 UTC
*** Bug 2114 has been marked as a duplicate of this bug. ***

Comment 6 shishir gowda 2010-11-19 03:26:56 UTC
some of the leaks are due to fini not being invoked.
Once that is fixed, we can revisit those leaks

Comment 7 Raghavendra Bhat 2011-02-21 04:44:32 UTC
Repeated the same tests (posix_compliance, arequal of /etc, dbench 100). This is the output of the valgrind.




CLIENT:
==6991== LEAK SUMMARY:
==6991==    definitely lost: 180 bytes in 2 blocks
==6991==    indirectly lost: 152 bytes in 1 blocks
==6991==      possibly lost: 20,111,663 bytes in 116 blocks
==6991==    still reachable: 3,366,649 bytes in 118 blocks
==6991==         suppressed: 0 bytes in 0 blocks



SERVER:

==6962== LEAK SUMMARY:
==6962==    definitely lost: 221 bytes in 3 blocks
==6962==    indirectly lost: 0 bytes in 0 blocks
==6962==      possibly lost: 2,857,856 bytes in 53 blocks
==6962==    still reachable: 8,995,178 bytes in 185 blocks
==6962==         suppressed: 0 bytes in 0 blocks


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