Whamcloud - gitweb
b=14379
authorgreen <green>
Wed, 19 Mar 2008 06:40:28 +0000 (06:40 +0000)
committergreen <green>
Wed, 19 Mar 2008 06:40:28 +0000 (06:40 +0000)
r=vitaly,adilger

Always walk entire list of granted locks on match for now.

lustre/ChangeLog
lustre/ldlm/ldlm_lock.c

index 08463bc..492499a 100644 (file)
@@ -834,6 +834,12 @@ Bugzilla   : 15033
 Description: build for x2 fails
 Details    : fix compile issue on Cray systems.
 
+Severity   : normal
+Bugzilla   : 14379
+Description: Properly match for duplicate locks
+Details    : Due to different lock order from skiplists code, we need to
+             traverse entire list for now
+
 --------------------------------------------------------------------------------
 
 2007-08-10         Cluster File Systems, Inc. <info@clusterfs.com>
index 825dfb8..6f7f5cf 100644 (file)
@@ -911,7 +911,7 @@ static struct ldlm_lock *search_queue(struct list_head *queue,
                 lock = list_entry(tmp, struct ldlm_lock, l_res_link);
 
                 if (lock == old_lock)
-                        break;
+                        continue;
 
                 /* llite sometimes wants to match locks that will be
                  * canceled when their users drop, but we allow it to match