Bug 767862

Summary:	Cleanup graphs which are not active
Product:	[Community] GlusterFS	Reporter:	Raghavendra G <rgowdapp>
Component:	core	Assignee:	Raghavendra G <rgowdapp>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Anush Shetty <ashetty>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	amarts, gluster-bugs, rfortier
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.4.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-07-24 17:59:25 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	817967

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