<p dir="ltr">Hi,</p>
<p dir="ltr">2015. 2. 18. 오전 8:30에 "Andrew Morton" <<a href="mailto:akpm@linux-foundation.org">akpm@linux-foundation.org</a>>님이 작성:<br>
><br>
> On Tue, 17 Feb 2015 16:13:50 +0900 Daeseok Youn <<a href="mailto:daeseok.youn@gmail.com">daeseok.youn@gmail.com</a>> wrote:<br>
><br>
> > The Local variable "i" in for loop is always less then<br>
> > O2CB_MAP_STABILIZE_COUNT.<br>
> ><br>
> > Signed-off-by: Daeseok Youn <<a href="mailto:daeseok.youn@gmail.com">daeseok.youn@gmail.com</a>><br>
> > ---<br>
> > fs/ocfs2/stack_o2cb.c | 4 ++--<br>
> > 1 files changed, 2 insertions(+), 2 deletions(-)<br>
> ><br>
> > diff --git a/fs/ocfs2/stack_o2cb.c b/fs/ocfs2/stack_o2cb.c<br>
> > index 1724d43..813d726 100644<br>
> > --- a/fs/ocfs2/stack_o2cb.c<br>
> > +++ b/fs/ocfs2/stack_o2cb.c<br>
> > @@ -295,8 +295,8 @@ static int o2cb_cluster_check(void)<br>
> > set_bit(node_num, netmap);<br>
> > if (!memcmp(hbmap, netmap, sizeof(hbmap)))<br>
> > return 0;<br>
> > - if (i < O2CB_MAP_STABILIZE_COUNT)<br>
> > - msleep(1000);<br>
> > +<br>
> > + msleep(1000);<br>
> > }<br>
><br>
> I assume the code was intended to do<br>
><br>
> if (i < O2CB_MAP_STABILIZE_COUNT - 1)<br>
> msleep(1000);<br>
><br>
> to avoid a pointless 1-second delay when the operation times out.</p>
<p dir="ltr">Ok. I will send this patch after makes it as your comment.</p>
<p dir="ltr">Thanks.<br>
Regards,<br>
Daeseok Youn<br>
</p>