Re: [Ocfs2-devel] [PATCH] ocfs2/dlm: clean unrelated comment

2018-02-23 Thread piaojun
Hi Changwei, On 2018/2/23 17:55, Changwei Ge wrote: > Hi Jun, > > On 2018/2/23 17:13, piaojun wrote: >> Hi changwei, >> >> On 2018/2/23 15:30, ge.chang...@h3c.com wrote: >>> From: Changwei Ge >> This line seems unnecessary, others looks good to me. > I used git send-email,

Re: [Ocfs2-devel] [PATCH] ocfs2/dlm: clean unrelated comment

2018-02-23 Thread Changwei Ge
Hi Jun, On 2018/2/23 17:13, piaojun wrote: > Hi changwei, > > On 2018/2/23 15:30, ge.chang...@h3c.com wrote: >> From: Changwei Ge > This line seems unnecessary, others looks good to me. I used git send-email, thus this line was added by parameter --from. I suppose you could

Re: [Ocfs2-devel] [PATCH] ocfs2/dlm: clean unrelated comment

2018-02-23 Thread piaojun
Hi changwei, On 2018/2/23 15:30, ge.chang...@h3c.com wrote: > From: Changwei Ge This line seems unnecessary, others looks good to me. thanks, Jun > > Obviously, the comment before dlm_do_local_recovery_cleanup() has > nothing to do with it. So remove it. > >

Re: [Ocfs2-devel] [PATCH] ocfs2/dlm: clean unrelated comment

2018-02-23 Thread Joseph Qi
On 18/2/23 15:30, ge.chang...@h3c.com wrote: > From: Changwei Ge > > Obviously, the comment before dlm_do_local_recovery_cleanup() has > nothing to do with it. So remove it. > > Signed-off-by: Changwei Ge It seems to say we can no longer trust the