It looks good to me.
On 2017/11/24 13:30, Changwei Ge wrote:> This code snippet is no longer used. So trim it, thus to make code neat.
>
> Signed-off-by: Changwei Ge <ge.changwei at h3c.com>
Reviewed-by: Alex Chen <alex.chen at huawei.com>> ---
> fs/ocfs2/dlm/dlmmaster.c | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c
> index 3e04279446e8..93b61c6c409e 100644
> --- a/fs/ocfs2/dlm/dlmmaster.c
> +++ b/fs/ocfs2/dlm/dlmmaster.c
> @@ -1122,13 +1122,6 @@ static int dlm_wait_for_lock_mastery(struct
> dlm_ctxt *dlm,
> /* sleep if we haven't finished voting yet */
> if (sleep) {
> unsigned long timeo = msecs_to_jiffies(DLM_MASTERY_TIMEOUT_MS);
> -
> - /*
> - if (kref_read(&mle->mle_refs) < 2)
> - mlog(ML_ERROR, "mle (%p) refs=%d, name=%.*s\n", mle,
> - kref_read(&mle->mle_refs),
> - res->lockname.len, res->lockname.name);
> - */
> atomic_set(&mle->woken, 0);
> (void)wait_event_timeout(mle->wq,
> (atomic_read(&mle->woken) == 1),
>