Bug 767862 - Cleanup graphs which are not active
Summary: Cleanup graphs which are not active
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: All
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: Raghavendra G
QA Contact: Anush Shetty
URL:
Whiteboard:
Depends On:
Blocks: 817967
TreeView+ depends on / blocked
 
Reported: 2011-12-15 02:05 UTC by Raghavendra G
Modified: 2013-07-24 17:59 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:59:25 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:

Attachments (Terms of Use)

Description Raghavendra G 2011-12-15 02:05:02 UTC 
Description of problem:
After a graph switch, old graphs are not freed. The graphs cannot be freed till none of the entities like inodes, frames, rpc objects, threads etc that refer to them are present. Hence, the work falls into two broad categories:

1. the infrastructure code, that decides when the graphs can be freed.
2. Actual cleanup itself. This step majorly consists of implementing finis of all the translators to cleanup their respective states completely.

regards,
Raghavendra.
Comment 1 Anand Avati 2011-12-26 10:49:36 UTC 
CHANGE: http://review.gluster.com/811 (features/posix-acl: handle fini for features/posix-acl) merged in master by Vijay Bellur (vijay)
Comment 2 Anand Avati 2011-12-29 06:21:24 UTC 
CHANGE: http://review.gluster.com/2499 (debug/io-stats: Handle fini for io-stats xlator) merged in master by Vijay Bellur (vijay)
Comment 3 Anand Avati 2011-12-29 06:47:41 UTC 
CHANGE: http://review.gluster.com/800 (cluster/afr: Handle fini for afr,pump) merged in master by Vijay Bellur (vijay)
Comment 4 Anand Avati 2012-02-21 09:05:22 UTC 
CHANGE: http://review.gluster.com/2734 (fuse-bridge: Handle graph-switch.) merged in master by Vijay Bellur (vijay)
Comment 5 Anand Avati 2012-02-21 09:42:49 UTC 
CHANGE: http://review.gluster.com/2735 (rpc/clnt: handle PARENT_DOWN event appropriately) merged in master by Vijay Bellur (vijay)
Comment 6 Anand Avati 2012-03-14 10:49:23 UTC 
CHANGE: http://review.gluster.com/790 (performance/quick-read: implement fini) merged in master by Vijay Bellur (vijay)
Comment 7 Anand Avati 2012-03-14 10:50:23 UTC 
CHANGE: http://review.gluster.com/792 (performance/read-ahead: cleanup xlator private data completely in fini.) merged in master by Vijay Bellur (vijay)
Comment 8 Anand Avati 2012-03-14 10:57:46 UTC 
CHANGE: http://review.gluster.com/2870 (fuse-bridge/graph-cleanup: don't provide a cbk for cleanup syncop task) merged in master by Vijay Bellur (vijay)
Comment 9 Anand Avati 2012-03-14 10:59:41 UTC 
CHANGE: http://review.gluster.com/791 (performance/io-cache: cleanup the xlator private data completely in fini.) merged in master by Vijay Bellur (vijay)
Comment 10 Anand Avati 2012-03-26 22:09:27 UTC 
CHANGE: http://review.gluster.com/3006 (libglusterfs/fd: ref fds while copying the fdentries.) merged in master by Anand Avati (avati)
Comment 11 Anush Shetty 2012-05-30 13:26:37 UTC 
Graph cleanup is in release-3.3
Note You need to log in before you can comment on or make changes to this bug.