diff options
| author | David Teigland <teigland@redhat.com> | 2011-04-05 13:16:24 -0500 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2011-07-15 12:30:43 -0500 |
| commit | 23e8e1aaacb10d9f05e44a93e10ea4ee5b3838a5 (patch) | |
| tree | 7c94bc4eeb9dfd85a26869003c56dc45fc6fd697 /fs/dlm/recoverd.c | |
| parent | 883ba74f43092823d0ed4c6b21f0171e9b334607 (diff) | |
| download | op-kernel-dev-23e8e1aaacb10d9f05e44a93e10ea4ee5b3838a5.zip op-kernel-dev-23e8e1aaacb10d9f05e44a93e10ea4ee5b3838a5.tar.gz | |
dlm: use workqueue for callbacks
Instead of creating our own kthread (dlm_astd) to deliver
callbacks for all lockspaces, use a per-lockspace workqueue
to deliver the callbacks. This eliminates complications and
slowdowns from many lockspaces sharing the same thread.
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/recoverd.c')
| -rw-r--r-- | fs/dlm/recoverd.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/fs/dlm/recoverd.c b/fs/dlm/recoverd.c index fd677c8..774da3c 100644 --- a/fs/dlm/recoverd.c +++ b/fs/dlm/recoverd.c @@ -58,13 +58,7 @@ static int ls_recover(struct dlm_ls *ls, struct dlm_recover *rv) mutex_lock(&ls->ls_recoverd_active); - /* - * Suspending and resuming dlm_astd ensures that no lkb's from this ls - * will be processed by dlm_astd during recovery. - */ - - dlm_astd_suspend(); - dlm_astd_resume(); + dlm_callback_suspend(ls); /* * Free non-master tossed rsb's. Master rsb's are kept on toss @@ -202,6 +196,8 @@ static int ls_recover(struct dlm_ls *ls, struct dlm_recover *rv) dlm_adjust_timeouts(ls); + dlm_callback_resume(ls); + error = enable_locking(ls, rv->seq); if (error) { log_debug(ls, "enable_locking failed %d", error); @@ -222,8 +218,6 @@ static int ls_recover(struct dlm_ls *ls, struct dlm_recover *rv) dlm_grant_after_purge(ls); - dlm_astd_wake(); - log_debug(ls, "recover %llx done: %u ms", (unsigned long long)rv->seq, jiffies_to_msecs(jiffies - start)); |
