Ocfs2 devel - Apr 2016 - [PATCH V2] ocfs2/dlm: return zero if deref_done message is successfully handled

Function dlm_deref_lockres_done_handler() should return zero
if the message is successfully handled.

Signed-off-by: xuejiufei <xuejiufei at huawei.com>
---
 fs/ocfs2/dlm/dlmmaster.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c
index 9aed6e2..13719d3 100644
--- a/fs/ocfs2/dlm/dlmmaster.c
+++ b/fs/ocfs2/dlm/dlmmaster.c
@@ -2455,6 +2455,8 @@ int dlm_deref_lockres_done_handler(struct o2net_msg *msg,
u32 len, void *data,

 	spin_unlock(&dlm->spinlock);

+	ret = 0;
+
 done:
 	dlm_put(dlm);
 	return ret;
-- 
1.8.4.3

Thanks, Jiufei.
We'd better add:
Fixes: 60d663cb5273("ocfs2/dlm: add DEREF_DONE message")
Others looks good to me.

Reviewed-by: Joseph Qi <joseph.qi at huawei.com>

On 2016/4/7 16:39, xuejiufei wrote:
> Function dlm_deref_lockres_done_handler() should return zero
> if the message is successfully handled.
> 
> Signed-off-by: xuejiufei <xuejiufei at huawei.com>
> ---
>  fs/ocfs2/dlm/dlmmaster.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c
> index 9aed6e2..13719d3 100644
> --- a/fs/ocfs2/dlm/dlmmaster.c
> +++ b/fs/ocfs2/dlm/dlmmaster.c
> @@ -2455,6 +2455,8 @@ int dlm_deref_lockres_done_handler(struct o2net_msg
*msg, u32 len, void *data,
> 
>  	spin_unlock(&dlm->spinlock);
> 
> +	ret = 0;
> +
>  done:
>  	dlm_put(dlm);
>  	return ret;
>