Bug 808400 - Lock migration
Summary: Lock migration
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: protocol
Version: pre-release
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: ---
Assignee: Raghavendra G
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 850501 856985
TreeView+ depends on / blocked
 
Reported: 2012-03-30 10:38 UTC by Junaid
Modified: 2013-08-06 22:38 UTC (History)
5 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 856985 (view as bug list)
Environment:
Last Closed: 2013-07-24 17:46:15 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:


Attachments (Terms of Use)

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)


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