X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fldlm%2Fl_lock.c;h=6f3f859d8f08d43d2abe40826e4e95bbdb3a2e84;hb=89a668f6c8770c5af8c99984003ffd9844dc2904;hp=a0013e1cd88627019f62975af9d7479356d65846;hpb=9fb46705ae86aa2c0ac29427f0ff24f923560eb7;p=fs%2Flustre-release.git diff --git a/lustre/ldlm/l_lock.c b/lustre/ldlm/l_lock.c index a0013e1..6f3f859 100644 --- a/lustre/ldlm/l_lock.c +++ b/lustre/ldlm/l_lock.c @@ -26,6 +26,8 @@ /* * Copyright (c) 2002, 2010, Oracle and/or its affiliates. All rights reserved. * Use is subject to license terms. + * + * Copyright (c) 2012, Intel Corporation. */ /* * This file is part of Lustre, http://www.lustre.org/ @@ -33,41 +35,42 @@ */ #define DEBUG_SUBSYSTEM S_LDLM -#ifdef __KERNEL__ #include -#else -#include -#endif #include #include -/* - * ldlm locking uses resource to serialize access to locks +/** + * Lock a lock and its resource. + * + * LDLM locking uses resource to serialize access to locks * but there is a case when we change resource of lock upon - * enqueue reply. we rely on that lock->l_resource = new_res - * is atomic + * enqueue reply. We rely on lock->l_resource = new_res + * being an atomic operation. */ -struct ldlm_resource * lock_res_and_lock(struct ldlm_lock *lock) +struct ldlm_resource *lock_res_and_lock(struct ldlm_lock *lock) { /* on server-side resource of lock doesn't change */ - if (!lock->l_ns_srv) + if (!ldlm_is_ns_srv(lock)) spin_lock(&lock->l_lock); lock_res(lock->l_resource); - lock->l_res_locked = 1; + ldlm_set_res_locked(lock); return lock->l_resource; } EXPORT_SYMBOL(lock_res_and_lock); +/** + * Unlock a lock and its resource previously locked with lock_res_and_lock + */ void unlock_res_and_lock(struct ldlm_lock *lock) { /* on server-side resource of lock doesn't change */ - lock->l_res_locked = 0; + ldlm_clear_res_locked(lock); unlock_res(lock->l_resource); - if (!lock->l_ns_srv) + if (!ldlm_is_ns_srv(lock)) spin_unlock(&lock->l_lock); } EXPORT_SYMBOL(unlock_res_and_lock);