diff options
author | Alexander Aring <aahringo@redhat.com> | 2020-08-27 15:02:48 -0400 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2020-08-27 15:59:09 -0500 |
commit | aa7ab1e20882b04fc3e45da77a9dad5cbbefba99 (patch) | |
tree | 1630a2251e200d123131e0eacf6ca8e53a212cbf /fs/dlm | |
parent | d012a7190fc1fd72ed48911e77ca97ba4521bccd (diff) | |
download | linux-aa7ab1e20882b04fc3e45da77a9dad5cbbefba99.tar.gz linux-aa7ab1e20882b04fc3e45da77a9dad5cbbefba99.tar.bz2 linux-aa7ab1e20882b04fc3e45da77a9dad5cbbefba99.zip |
fs: dlm: synchronize dlm before shutdown
This patch moves the dlm workqueue dlm synchronization before shutdown
handling. The patch just flushes all pending work before starting to
shutdown the connection. At least for the send_workqeue we should flush
the workqueue to make sure there is no new connection handling going on
as dlm_allow_conn switch is turned to false before.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm')
-rw-r--r-- | fs/dlm/lowcomms.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 5050fe05769b..ed098870ba0d 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c @@ -1624,10 +1624,6 @@ static void work_flush(void) struct hlist_node *n; struct connection *con; - if (recv_workqueue) - flush_workqueue(recv_workqueue); - if (send_workqueue) - flush_workqueue(send_workqueue); do { ok = 1; foreach_conn(stop_conn); @@ -1659,6 +1655,12 @@ void dlm_lowcomms_stop(void) mutex_lock(&connections_lock); dlm_allow_conn = 0; mutex_unlock(&connections_lock); + + if (recv_workqueue) + flush_workqueue(recv_workqueue); + if (send_workqueue) + flush_workqueue(send_workqueue); + foreach_conn(shutdown_conn); work_flush(); clean_writequeues(); |