Bug 808400

Summary:	Lock migration
Product:	[Community] GlusterFS	Reporter:	Junaid <junaid>
Component:	protocol	Assignee:	Raghavendra G <rgowdapp>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	pre-release	CC:	amarts, gluster-bugs, jdarcy, rfortier, vagarwal
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.4.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	856985 (view as bug list)		Environment:
Last Closed:	2013-07-24 17:46:15 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	850501, 856985

Description Junaid 2012-03-30 10:38:00 UTC

Description of problem:
On a graph change, along with the fd migration the locks acquired must also be migrated.

How reproducible:
Always

Steps to Reproduce:
1. Acquire locks on a file
2. Perform volume set command that will change the graph
3. Then, try acquiring lock on the same range again from a different program.
  
Actual results:
The new lock is granted.

Expected results:
The new lock must not be granted.

Additional info:

Comment 1 Anand Avati 2012-05-15 23:52:45 UTC

CHANGE: http://review.gluster.com/3227 (mount/fuse: Use state->lk_lock to print lock information on EAGAIN.) merged in master by Anand Avati (avati)

Comment 2 Jeff Darcy 2012-10-31 14:13:30 UTC

http://review.gluster.org/3051 is still in review for this.

Comment 3 Vijay Bellur 2012-11-21 09:41:23 UTC

CHANGE: http://review.gluster.org/4135 (libglusterfs: implement gf_strndup) merged in master by Vijay Bellur (vbellur)

Comment 4 Vijay Bellur 2012-11-28 06:43:30 UTC

CHANGE: http://review.gluster.org/4121 (cluster/afr: handle GF_XATTR_LOCKINFO_KEY appropriately.) merged in master by Vijay Bellur (vbellur)

Comment 5 Vijay Bellur 2012-11-28 06:43:47 UTC

CHANGE: http://review.gluster.org/4122 (mount/fuse: migrate posix locks after a graph-switch) merged in master by Vijay Bellur (vbellur)

Comment 6 Vijay Bellur 2012-11-28 06:44:09 UTC

CHANGE: http://review.gluster.org/4123 (cluster/distribute: send getxattr on LOCKINFO to only cached subvolumes.) merged in master by Vijay Bellur (vbellur)

Comment 7 Vijay Bellur 2012-11-28 06:44:47 UTC

CHANGE: http://review.gluster.org/4124 (libglusterfs/mem-pool: implement gf_memdup) merged in master by Vijay Bellur (vbellur)

Comment 8 Vijay Bellur 2012-11-28 06:45:02 UTC

CHANGE: http://review.gluster.org/4125 (cluster/stripe: handle GF_XATTR_LOCKINFO_KEY in f(get)(set)xattr) merged in master by Vijay Bellur (vbellur)

Comment 9 Vijay Bellur 2012-11-28 06:45:13 UTC

CHANGE: http://review.gluster.org/4209 (mount/fuse: add test cases for lock migration) merged in master by Vijay Bellur (vbellur)

Comment 10 Vijay Bellur 2012-11-28 06:54:39 UTC

CHANGE: http://review.gluster.org/4120 (features/locks: implement fgetxattr and fsetxattr) merged in master by Vijay Bellur (vbellur)

Comment 11 Vijay Bellur 2013-01-31 07:41:29 UTC

CHANGE: http://review.gluster.org/4455 (fd-lk: implement new method to check for empty fd_lk_ctx_t) merged in master by Anand Avati (avati)

Comment 12 Vijay Bellur 2013-01-31 07:44:02 UTC

CHANGE: http://review.gluster.org/4453 (fuse-bridge: fix some breakages from lock migration patch) merged in master by Anand Avati (avati)

Comment 13 Anand Avati 2013-05-06 02:52:17 UTC

REVIEW: http://review.gluster.org/4951 (features/locks: call syncops only in syncenv threads.) posted (#1) for review on master by Raghavendra G (raghavendra)