Whamcloud - gitweb
Branch HEAD
[fs/lustre-release.git] / lustre / ldlm / ldlm_inodebits.c
index e3511dd..34c6295 100644 (file)
 # include <liblustre.h>
 #endif
 
-#include <linux/lustre_dlm.h>
-#include <linux/obd_support.h>
-#include <linux/lustre_lib.h>
+#include <lustre_dlm.h>
+#include <obd_support.h>
+#include <lustre_lib.h>
 
 #include "ldlm_internal.h"
 
 /* Determine if the lock is compatible with all locks on the queue. */
 static int
 ldlm_inodebits_compat_queue(struct list_head *queue, struct ldlm_lock *req,
-                            int send_cbs)
+                            struct list_head *work_list)
 {
-        struct list_head *tmp;
+        struct list_head *tmp, *tmp_tail;
         struct ldlm_lock *lock;
         ldlm_mode_t req_mode = req->l_req_mode;
         __u64 req_bits = req->l_policy_data.l_inodebits.bits;
         int compat = 1;
         ENTRY;
 
-        LASSERT(req_bits); /* There is no sence in lock with no bits set,
+        LASSERT(req_bits); /* There is no sense in lock with no bits set,
                               I think. Also such a lock would be compatible
                                with any other bit lock */
         list_for_each(tmp, queue) {
@@ -54,63 +54,98 @@ ldlm_inodebits_compat_queue(struct list_head *queue, struct ldlm_lock *req,
                         RETURN(compat);
 
                 /* locks are compatible, bits don't matter */
-                if (lockmode_compat(lock->l_req_mode, req_mode))
+                if (lockmode_compat(lock->l_req_mode, req_mode)) {
+                        /* jump to next mode group */
+                        if (LDLM_SL_HEAD(&lock->l_sl_mode))
+                                tmp = &list_entry(lock->l_sl_mode.next, 
+                                                  struct ldlm_lock,
+                                                  l_sl_mode)->l_res_link;
                         continue;
-
-                /* if bits don't overlap skip it */
-                if (!(lock->l_policy_data.l_inodebits.bits & req_bits))
-                        continue;
-
-                if (!send_cbs)
-                        RETURN(0);
-
-                compat = 0;
-                if (lock->l_blocking_ast)
-                        ldlm_add_ast_work_item(lock, req, NULL, 0);
-        }
+                }
+                
+                tmp_tail = tmp;
+                if (LDLM_SL_HEAD(&lock->l_sl_mode))
+                        tmp_tail = &list_entry(lock->l_sl_mode.next,
+                                               struct ldlm_lock,
+                                               l_sl_mode)->l_res_link;
+                for (;;) {
+                        /* locks with bits overlapped are conflicting locks */
+                        if (lock->l_policy_data.l_inodebits.bits & req_bits) {
+                                /* conflicting policy */
+                                if (!work_list)
+                                        RETURN(0);
+
+                                compat = 0;
+                                if (lock->l_blocking_ast)
+                                        ldlm_add_ast_work_item(lock, req, 
+                                                               work_list);
+                                /* add all members of the policy group */
+                                if (LDLM_SL_HEAD(&lock->l_sl_policy)) {
+                                        do {
+                                                tmp = lock->l_res_link.next;
+                                                lock = list_entry(tmp,
+                                                            struct ldlm_lock,
+                                                            l_res_link);
+                                                if (lock->l_blocking_ast)
+                                                        ldlm_add_ast_work_item(
+                                                                     lock,
+                                                                     req,
+                                                                     work_list);
+                                        } while (!LDLM_SL_TAIL(&lock->l_sl_policy));
+                                }
+                        } else if (LDLM_SL_HEAD(&lock->l_sl_policy)) {
+                                /* jump to next policy group */
+                                tmp = &list_entry(lock->l_sl_policy.next,
+                                                  struct ldlm_lock,
+                                                  l_sl_policy)->l_res_link;
+                        }
+                        if (tmp == tmp_tail)
+                                break;
+                        else
+                                tmp = tmp->next;
+                        lock = list_entry(tmp, struct ldlm_lock, l_res_link);
+                }       /* for locks in a mode group */
+        }       /* for each lock in the queue */
 
         RETURN(compat);
 }
 
 /* If first_enq is 0 (ie, called from ldlm_reprocess_queue):
   *   - blocking ASTs have already been sent
-  *   - the caller has already initialized req->lr_tmp
   *   - must call this function with the ns lock held
   *
   * If first_enq is 1 (ie, called from ldlm_lock_enqueue):
   *   - blocking ASTs have not been sent
-  *   - the caller has NOT initialized req->lr_tmp, so we must
   *   - must call this function with the ns lock held once */
 int ldlm_process_inodebits_lock(struct ldlm_lock *lock, int *flags,
-                                int first_enq, ldlm_error_t *err)
+                                int first_enq, ldlm_error_t *err,
+                                struct list_head *work_list)
 {
         struct ldlm_resource *res = lock->l_resource;
-        struct list_head rpc_list = LIST_HEAD_INIT(rpc_list);
+        struct list_head rpc_list = CFS_LIST_HEAD_INIT(rpc_list);
         int rc;
         ENTRY;
 
         LASSERT(list_empty(&res->lr_converting));
+        check_res_locked(res);
 
         if (!first_enq) {
-                LASSERT(res->lr_tmp != NULL);
-                rc = ldlm_inodebits_compat_queue(&res->lr_granted, lock, 0);
+                LASSERT(work_list != NULL);
+                rc = ldlm_inodebits_compat_queue(&res->lr_granted, lock, NULL);
                 if (!rc)
                         RETURN(LDLM_ITER_STOP);
-                rc = ldlm_inodebits_compat_queue(&res->lr_waiting, lock, 0);
+                rc = ldlm_inodebits_compat_queue(&res->lr_waiting, lock, NULL);
                 if (!rc)
                         RETURN(LDLM_ITER_STOP);
 
                 ldlm_resource_unlink_lock(lock);
-                ldlm_grant_lock(lock, NULL, 0, 1);
+                ldlm_grant_lock(lock, work_list);
                 RETURN(LDLM_ITER_CONTINUE);
         }
 
  restart:
-        LASSERT(res->lr_tmp == NULL);
-        res->lr_tmp = &rpc_list;
-        rc = ldlm_inodebits_compat_queue(&res->lr_granted, lock, 1);
-        rc += ldlm_inodebits_compat_queue(&res->lr_waiting, lock, 1);
-        res->lr_tmp = NULL;
+        rc = ldlm_inodebits_compat_queue(&res->lr_granted, lock, &rpc_list);
+        rc += ldlm_inodebits_compat_queue(&res->lr_waiting, lock, &rpc_list);
 
         if (rc != 2) {
                 /* If either of the compat_queue()s returned 0, then we
@@ -121,15 +156,15 @@ int ldlm_process_inodebits_lock(struct ldlm_lock *lock, int *flags,
                  * re-ordered!  Causes deadlock, because ASTs aren't sent! */
                 if (list_empty(&lock->l_res_link))
                         ldlm_resource_add_lock(res, &res->lr_waiting, lock);
-                l_unlock(&res->lr_namespace->ns_lock);
-                rc = ldlm_run_ast_work(res->lr_namespace, &rpc_list);
-                l_lock(&res->lr_namespace->ns_lock);
+                unlock_res(res);
+                rc = ldlm_run_ast_work(&rpc_list, LDLM_WORK_BL_AST);
+                lock_res(res);
                 if (rc == -ERESTART)
                         GOTO(restart, -ERESTART);
                 *flags |= LDLM_FL_BLOCK_GRANTED;
         } else {
                 ldlm_resource_unlink_lock(lock);
-                ldlm_grant_lock(lock, NULL, 0, 0);
+                ldlm_grant_lock(lock, NULL);
         }
         RETURN(0);
 }