From 86501cd36b9784a6c0247326092a50fb44524656 Mon Sep 17 00:00:00 2001 From: Anoop C S Date: Sun, 20 Dec 2015 21:47:37 +0530 Subject: features/locks: Remove unneeded function parameter We use list_del_init() inside __delete_lock() to remove a lock from inode's lock list where pl_inode_t is not required at all. This patch removes pl_inode_t from list of parameters required for __delete_lock(). Change-Id: Ic5701bcae231172d4fd7feda1b25752343ee81cf BUG: 1293227 Signed-off-by: Anoop C S Reviewed-on: http://review.gluster.org/13033 Smoke: Gluster Build System CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Jeff Darcy --- xlators/features/locks/src/common.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'xlators/features/locks/src/common.c') diff --git a/xlators/features/locks/src/common.c b/xlators/features/locks/src/common.c index 0abc9353c9e..1c43ef16179 100644 --- a/xlators/features/locks/src/common.c +++ b/xlators/features/locks/src/common.c @@ -483,7 +483,7 @@ out: /* Delete a lock from the inode's lock list */ void -__delete_lock (pl_inode_t *pl_inode, posix_lock_t *lock) +__delete_lock (posix_lock_t *lock) { list_del_init (&lock->list); } @@ -562,7 +562,7 @@ __delete_unlck_locks (pl_inode_t *pl_inode) list_for_each_entry_safe (l, tmp, &pl_inode->ext_list, list) { if (l->fl_type == F_UNLCK) { - __delete_lock (pl_inode, l); + __delete_lock (l); __destroy_lock (l); } } @@ -800,7 +800,7 @@ __insert_and_merge (pl_inode_t *pl_inode, posix_lock_t *lock) sum->client_pid = lock->client_pid; sum->owner = lock->owner; - __delete_lock (pl_inode, conf); + __delete_lock (conf); __destroy_lock (conf); __destroy_lock (lock); @@ -820,10 +820,10 @@ __insert_and_merge (pl_inode_t *pl_inode, posix_lock_t *lock) v = subtract_locks (sum, lock); - __delete_lock (pl_inode, conf); + __delete_lock (conf); __destroy_lock (conf); - __delete_lock (pl_inode, lock); + __delete_lock (lock); __destroy_lock (lock); __destroy_lock (sum); -- cgit