Created attachment 593623 [details] Valgrind from about an hour of agonizingly slow desktop usage. :D Description of problem: Since upgrading to 3.3, my personal desktop machine has been oom killing the client about every 48 hours. Mine is only one of two machines that uses the home directory with an xwindows desktop and the other one logs out every night. Version-Release number of selected component (if applicable): How reproducible: About every 48 hours Steps to Reproduce: 1. Mount the home volume 2. Leave my desktop logged in Actual results: oom killer kills glusterfs after it consumes around 6 gig Expected results: It shouldn't use anywhere near that much Attached is some valgrind output. I've never really used it before, so I hope I picked the right options that would be helpful.
Created attachment 593942 [details] dump file from instance with 2.8g usage
Johny, can you look at the valgrind report and come up with the fix? Looks like simple one to crack.
Joe, sorry about *long* delay about this bug. Want to check if you are seeing similar issues with 3.3.1 (or 3.4.0qa2) releases. Anyways, will get this particular bug addressed soon.
reducing the priority (from urgent to high), as this issue is not as severe now with 3.4.0qa6 release.
not seeing much of the leak in fuse bridge now (mainly with proper fd and lock migration code in place). Also the valgrind didn't help us much on top of the recent master branch code. Reducing the priority to 'medium' and would be closed after running this with the 3.4.0 beta releases.
CHANGE: http://review.gluster.org/4523 (Better mechanism to handle memory accounting) merged in master by Anand Avati (avati)
CHANGE: http://review.gluster.org/4524 (mount.glusterfs: Introduce mem-accounting as an option) merged in master by Anand Avati (avati)
CHANGE: http://review.gluster.org/4525 (libglusterfs: Fix memory leaks in fd_lk_insert_and_merge) merged in master by Anand Avati (avati)
CHANGE: http://review.gluster.org/4530 (libglusterfs: Fix memory leaks in fd_lk_insert_and_merge) merged in release-3.3 by Anand Avati (avati)
CHANGE: http://review.gluster.org/4527 (Better mechanism to handle memory accounting) merged in release-3.4 by Vijay Bellur (vbellur)
CHANGE: http://review.gluster.org/4528 (mount.glusterfs: Introduce mem-accounting as an option) merged in release-3.4 by Vijay Bellur (vbellur)
CHANGE: http://review.gluster.org/4529 (libglusterfs: Fix memory leaks in fd_lk_insert_and_merge) merged in release-3.4 by Vijay Bellur (vbellur)
REVIEW: http://review.gluster.org/5343 (glusterfsd: Cleanup memory accounting to honor argp_parse properly) posted (#1) for review on master by Harshavardhana (harsha)
REVIEW: http://review.gluster.org/5343 (glusterfsd: Cleanup memory accounting to honor argp_parse properly) posted (#2) for review on master by Harshavardhana (harsha)