From: Andreas Gruenbacher <agrue...@redhat.com>

All that do_waiters() does is iterate the the waiters list of a
resource, so there is no need to check if the waiters list is empty
before calling do_waiters().
---
 dlm_controld/plock.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dlm_controld/plock.c b/dlm_controld/plock.c
index 85d6fdf2..ad9b0f27 100644
--- a/dlm_controld/plock.c
+++ b/dlm_controld/plock.c
@@ -2055,8 +2055,7 @@ void purge_plocks(struct lockspace *ls, int nodeid, int 
unmount)
                        send_pending_plocks(ls, r);
                }
                
-               if (!list_empty(&r->waiters))
-                       do_waiters(ls, r);
+               do_waiters(ls, r);
 
                if (!opt(plock_ownership_ind) &&
                    list_empty(&r->locks) && list_empty(&r->waiters)) {
-- 
2.31.1

Reply via email to