<div class="gmail_quote">On Wed, Jul 11, 2012 at 1:51 AM, Joel Becker <span dir="ltr">&lt;<a href="mailto:jlbec@evilplan.org" target="_blank">jlbec@evilplan.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Jul 11, 2012 at 02:49:56PM +0800, Junxiao Bi wrote:<br>
&gt; Signed-off-by: Junxiao Bi &lt;<a href="mailto:junxiao.bi@oracle.com">junxiao.bi@oracle.com</a>&gt;<br>
&gt; ---<br>
&gt;  fs/ocfs2/dlm/dlmmaster.c |    4 +++-<br>
&gt;  1 file changed, 3 insertions(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c<br>
&gt; index 005261c..1fadc39 100644<br>
&gt; --- a/fs/ocfs2/dlm/dlmmaster.c<br>
&gt; +++ b/fs/ocfs2/dlm/dlmmaster.c<br>
&gt; @@ -1888,8 +1888,10 @@ ok:<br>
&gt;                        * up nodes that this node contacted */<br>
&gt;                       while ((nn = find_next_bit (mle-&gt;response_map, O2NM_MAX_NODES,<br>
&gt;                                                   nn+1)) &lt; O2NM_MAX_NODES) {<br>
&gt; -                             if (nn != dlm-&gt;node_num &amp;&amp; nn != assert-&gt;node_idx)<br>
&gt; +                             if (nn != dlm-&gt;node_num &amp;&amp; nn != assert-&gt;node_idx) {<br>
&gt;                                       master_request = 1;<br>
&gt; +                                     break;<br>
&gt; +                             }<br>
<br>
</div>        Seems like this will save a tiny bit of CPU in the assert master<br>
handler.  Sunil, can you see any problem?<br>
<br></blockquote><div><br></div><div>Looks good to me. ACK. </div></div>