Browse Source

Staging: lustre: remove unneeded variable

ldlm_lock_enqueue() always returns ELDLM_OK, no matter what happens, so
removed the unneeded variable that this value was being stored in and
just return the value itself.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org
Greg Kroah-Hartman 10 years ago
parent
commit
b5f0063778
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/staging/lustre/lustre/ldlm/ldlm_lock.c

+ 1 - 2
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c

@@ -1520,7 +1520,6 @@ ldlm_error_t ldlm_lock_enqueue(struct ldlm_namespace *ns,
 {
 	struct ldlm_lock *lock = *lockp;
 	struct ldlm_resource *res = lock->l_resource;
-	ldlm_error_t rc = ELDLM_OK;
 
 	lock->l_last_activity = ktime_get_real_seconds();
 
@@ -1558,7 +1557,7 @@ ldlm_error_t ldlm_lock_enqueue(struct ldlm_namespace *ns,
 
 out:
 	unlock_res_and_lock(lock);
-	return rc;
+	return ELDLM_OK;
 }
 
 /**