* [Ocfs2-devel] [PATCH V2] ocfs2/dlm: return zero if deref_done message is successfully handled
@ 2016-04-07 8:39 xuejiufei
2016-04-08 1:03 ` Joseph Qi
0 siblings, 1 reply; 2+ messages in thread
From: xuejiufei @ 2016-04-07 8:39 UTC (permalink / raw)
To: ocfs2-devel
Function dlm_deref_lockres_done_handler() should return zero
if the message is successfully handled.
Signed-off-by: xuejiufei <xuejiufei@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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Ocfs2-devel] [PATCH V2] ocfs2/dlm: return zero if deref_done message is successfully handled
2016-04-07 8:39 [Ocfs2-devel] [PATCH V2] ocfs2/dlm: return zero if deref_done message is successfully handled xuejiufei
@ 2016-04-08 1:03 ` Joseph Qi
0 siblings, 0 replies; 2+ messages in thread
From: Joseph Qi @ 2016-04-08 1:03 UTC (permalink / raw)
To: ocfs2-devel
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@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@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;
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-04-08 1:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-07 8:39 [Ocfs2-devel] [PATCH V2] ocfs2/dlm: return zero if deref_done message is successfully handled xuejiufei
2016-04-08 1:03 ` Joseph Qi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).