[Ocfs2-devel] [PATCH 3/4] ocfs2/dlm: Use ast_lock to protect ast_list
Joel Becker
Joel.Becker at oracle.com
Tue Feb 10 23:31:54 PST 2009
whoops. sob.
On Tue, Feb 03, 2009 at 12:37:15PM -0800, Sunil Mushran wrote:
> The code was using dlm->spinlock instead of dlm->ast_lock to protect the
> ast_list. This patch fixes the issue.
>
> Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com>
> ---
> fs/ocfs2/dlm/dlmunlock.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/ocfs2/dlm/dlmunlock.c b/fs/ocfs2/dlm/dlmunlock.c
> index 86ca085..fcf879e 100644
> --- a/fs/ocfs2/dlm/dlmunlock.c
> +++ b/fs/ocfs2/dlm/dlmunlock.c
> @@ -117,11 +117,11 @@ static enum dlm_status dlmunlock_common(struct dlm_ctxt *dlm,
> else
> BUG_ON(res->owner == dlm->node_num);
>
> - spin_lock(&dlm->spinlock);
> + spin_lock(&dlm->ast_lock);
> /* We want to be sure that we're not freeing a lock
> * that still has AST's pending... */
> in_use = !list_empty(&lock->ast_list);
> - spin_unlock(&dlm->spinlock);
> + spin_unlock(&dlm->ast_lock);
> if (in_use) {
> mlog(ML_ERROR, "lockres %.*s: Someone is calling dlmunlock "
> "while waiting for an ast!", res->lockname.len,
> --
> 1.5.6.3
>
>
> _______________________________________________
> Ocfs2-devel mailing list
> Ocfs2-devel at oss.oracle.com
> http://oss.oracle.com/mailman/listinfo/ocfs2-devel
--
"Well-timed silence hath more eloquence than speech."
- Martin Fraquhar Tupper
Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127
More information about the Ocfs2-devel
mailing list