On Wed, Jul 11, 2012 at 1:51 AM, Joel Becker <jl...@evilplan.org> wrote:
> On Wed, Jul 11, 2012 at 02:49:56PM +0800, Junxiao Bi wrote: > > Signed-off-by: Junxiao Bi <junxiao...@oracle.com> > > --- > > fs/ocfs2/dlm/dlmmaster.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c > > index 005261c..1fadc39 100644 > > --- a/fs/ocfs2/dlm/dlmmaster.c > > +++ b/fs/ocfs2/dlm/dlmmaster.c > > @@ -1888,8 +1888,10 @@ ok: > > * up nodes that this node contacted */ > > while ((nn = find_next_bit (mle->response_map, > O2NM_MAX_NODES, > > nn+1)) < > O2NM_MAX_NODES) { > > - if (nn != dlm->node_num && nn != > assert->node_idx) > > + if (nn != dlm->node_num && nn != > assert->node_idx) { > > master_request = 1; > > + break; > > + } > > Seems like this will save a tiny bit of CPU in the assert master > handler. Sunil, can you see any problem? > > Looks good to me. ACK.
_______________________________________________ Ocfs2-devel mailing list Ocfs2-devel@oss.oracle.com https://oss.oracle.com/mailman/listinfo/ocfs2-devel