Re: [Ocfs2-devel] [PATCH] fix a compiling warning

2018-05-08 Thread Changwei Ge
Hi Larry, It would be better for you to paste the compilation warning and gcc version. So we can justify if it is deserved to be fixed. Thanks, Changwei On 2018/5/7 11:06 AM, Larry Chen wrote: > Hi Jun > > Yeah,I know your logic is right. > It's just a compile warning that made me feel

Re: [Ocfs2-devel] [PATCH] fix a compiling warning

2018-05-06 Thread Larry Chen
Hi Jun Yeah,I know your logic is right. It's just a compile warning that made me feel uncomfortable. :) On 05/07/2018 08:52 AM, piaojun wrote: > Hi Larry, > > 'had_lock' will be initialized by ocfs2_inode_lock_tracker(), and the > 'bail' branch above won't use it either as 'inode_locked' is

Re: [Ocfs2-devel] [PATCH] fix a compiling warning

2018-05-06 Thread piaojun
Hi Larry, 'had_lock' will be initialized by ocfs2_inode_lock_tracker(), and the 'bail' branch above won't use it either as 'inode_locked' is still zero. thanks, Jun On 2018/5/6 17:49, Larry Chen wrote: > The variable had_lock might be used uninitialized. > > Signed-off-by: Larry Chen