summaryrefslogtreecommitdiffstats
path: root/fs/dlm/lowcomms.c
diff options
context:
space:
mode:
authorAlexander Aring <aahringo@redhat.com>2020-08-27 15:02:52 -0400
committerDavid Teigland <teigland@redhat.com>2020-08-27 15:59:09 -0500
commit0de984323ac56aa420e6f28d7ce205a293fdb649 (patch)
tree59fdaf9a20b05021e97f88c214c3b4777b7c1980 /fs/dlm/lowcomms.c
parent3d2825c8c6105b0f36f3ff72760799fa2e71420e (diff)
downloadlinux-stable-0de984323ac56aa420e6f28d7ce205a293fdb649.tar.gz
linux-stable-0de984323ac56aa420e6f28d7ce205a293fdb649.tar.bz2
linux-stable-0de984323ac56aa420e6f28d7ce205a293fdb649.zip
fs: dlm: move free writequeue into con free
This patch just move the free of struct connection member writequeue into the functionality when struct connection will be freed instead of doing two iterations. Signed-off-by: Alexander Aring <aahringo@redhat.com> Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/lowcomms.c')
-rw-r--r--fs/dlm/lowcomms.c9
1 files changed, 1 insertions, 8 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c
index d0ece252a0d9..04afc7178afb 100644
--- a/fs/dlm/lowcomms.c
+++ b/fs/dlm/lowcomms.c
@@ -1550,13 +1550,6 @@ static void process_send_sockets(struct work_struct *work)
send_to_sock(con);
}
-
-/* Discard all entries on the write queues */
-static void clean_writequeues(void)
-{
- foreach_conn(clean_one_writequeue);
-}
-
static void work_stop(void)
{
if (recv_workqueue)
@@ -1620,6 +1613,7 @@ static void free_conn(struct connection *con)
spin_lock(&connections_lock);
hlist_del_rcu(&con->list);
spin_unlock(&connections_lock);
+ clean_one_writequeue(con);
kfree_rcu(con, rcu);
}
@@ -1668,7 +1662,6 @@ void dlm_lowcomms_stop(void)
foreach_conn(shutdown_conn);
work_flush();
- clean_writequeues();
foreach_conn(free_conn);
work_stop();
deinit_local();