From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752464AbdJ0JYD convert rfc822-to-8bit (ORCPT ); Fri, 27 Oct 2017 05:24:03 -0400 Received: from mga06.intel.com ([134.134.136.31]:41886 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752016AbdJ0JYB (ORCPT ); Fri, 27 Oct 2017 05:24:01 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,304,1505804400"; d="scan'208";a="165681457" From: "Dilger, Andreas" To: NeilBrown CC: "Drokin, Oleg" , James Simmons , Greg Kroah-Hartman , "lustre-devel@lists.lustre.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 6/9] staging: lustre: ldlm: remove 'flags' arg from ldlm_flock_destroy() Thread-Topic: [PATCH 6/9] staging: lustre: ldlm: remove 'flags' arg from ldlm_flock_destroy() Thread-Index: AQHTS5mpOHTG5Z9yBEK+5uMkhDXbGKL36MmA Date: Fri, 27 Oct 2017 09:23:59 +0000 Message-ID: References: <150871988297.3340.4522589460981284121.stgit@noble> <150872002976.3340.17760141414233585370.stgit@noble> In-Reply-To: <150872002976.3340.17760141414233585370.stgit@noble> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.254.7.56] Content-Type: text/plain; charset="us-ascii" Content-ID: <13D8466DF5757C4C8CB3ACA8B46EC0F3@intel.com> Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Oct 22, 2017, at 18:53, NeilBrown wrote: > > The only value ever passed in LDLM_FL_WAIT_NOREPROC, so assume that > instead of passing it. > > Signed-off-by: NeilBrown Reviewed-by: Andreas Dilger > --- > drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 36 ++++++++++------------- > 1 file changed, 16 insertions(+), 20 deletions(-) > > diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c b/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c > index d5a5742a1171..1bf56892fcf5 100644 > --- a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c > +++ b/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c > @@ -88,24 +88,23 @@ ldlm_flocks_overlap(struct ldlm_lock *lock, struct ldlm_lock *new) > } > > static inline void > -ldlm_flock_destroy(struct ldlm_lock *lock, enum ldlm_mode mode, __u64 flags) > +ldlm_flock_destroy(struct ldlm_lock *lock, enum ldlm_mode mode) > { > - LDLM_DEBUG(lock, "%s(mode: %d, flags: 0x%llx)", > - __func__, mode, flags); > + LDLM_DEBUG(lock, "%s(mode: %d)", > + __func__, mode); > > /* Safe to not lock here, since it should be empty anyway */ > LASSERT(hlist_unhashed(&lock->l_exp_flock_hash)); > > list_del_init(&lock->l_res_link); > - if (flags == LDLM_FL_WAIT_NOREPROC) { > - /* client side - set a flag to prevent sending a CANCEL */ > - lock->l_flags |= LDLM_FL_LOCAL_ONLY | LDLM_FL_CBPENDING; > > - /* when reaching here, it is under lock_res_and_lock(). Thus, > - * need call the nolock version of ldlm_lock_decref_internal > - */ > - ldlm_lock_decref_internal_nolock(lock, mode); > - } > + /* client side - set a flag to prevent sending a CANCEL */ > + lock->l_flags |= LDLM_FL_LOCAL_ONLY | LDLM_FL_CBPENDING; > + > + /* when reaching here, it is under lock_res_and_lock(). Thus, > + * need call the nolock version of ldlm_lock_decref_internal > + */ > + ldlm_lock_decref_internal_nolock(lock, mode); > > ldlm_lock_destroy_nolock(lock); > } > @@ -208,8 +207,7 @@ static int ldlm_process_flock_lock(struct ldlm_lock *req) > } > > if (added) { > - ldlm_flock_destroy(lock, mode, > - LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(lock, mode); > } else { > new = lock; > added = 1; > @@ -233,8 +231,7 @@ static int ldlm_process_flock_lock(struct ldlm_lock *req) > new->l_policy_data.l_flock.end + 1; > break; > } > - ldlm_flock_destroy(lock, lock->l_req_mode, > - LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(lock, lock->l_req_mode); > continue; > } > if (new->l_policy_data.l_flock.end >= > @@ -265,8 +262,7 @@ static int ldlm_process_flock_lock(struct ldlm_lock *req) > NULL, 0, LVB_T_NONE); > lock_res_and_lock(req); > if (IS_ERR(new2)) { > - ldlm_flock_destroy(req, lock->l_granted_mode, > - LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(req, lock->l_granted_mode); > return LDLM_ITER_STOP; > } > goto reprocess; > @@ -323,7 +319,7 @@ static int ldlm_process_flock_lock(struct ldlm_lock *req) > * could be freed before the completion AST can be sent. > */ > if (added) > - ldlm_flock_destroy(req, mode, LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(req, mode); > > ldlm_resource_dump(D_INFO, res); > return LDLM_ITER_CONTINUE; > @@ -477,7 +473,7 @@ ldlm_flock_completion_ast(struct ldlm_lock *lock, __u64 flags, void *data) > LDLM_DEBUG(lock, "client-side enqueue deadlock received"); > rc = -EDEADLK; > } > - ldlm_flock_destroy(lock, mode, LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(lock, mode); > unlock_res_and_lock(lock); > > /* Need to wake up the waiter if we were evicted */ > @@ -498,7 +494,7 @@ ldlm_flock_completion_ast(struct ldlm_lock *lock, __u64 flags, void *data) > * in the lock changes we can decref the appropriate refcount. > */ > LASSERT(ldlm_is_test_lock(lock)); > - ldlm_flock_destroy(lock, getlk->fl_type, LDLM_FL_WAIT_NOREPROC); > + ldlm_flock_destroy(lock, getlk->fl_type); > switch (lock->l_granted_mode) { > case LCK_PR: > getlk->fl_type = F_RDLCK; > > Cheers, Andreas -- Andreas Dilger Lustre Principal Architect Intel Corporation