Commit ce713ee5 authored Apr 05, 2012 by RongQing.Li Committed by David S. Miller Apr 05, 2012

net: replace continue with break to reduce unnecessary loop in xxx_xmarksources



The conditional which decides to skip inactive filters does not
change with the change of loop index, so it is unnecessary to
check them many times.

Signed-off-by: RongQing.Li <roy.qing.li@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 51c56b00

net/ipv4/igmp.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -775,7 +775,7 @@ static int igmp_xmarksources(struct ip_mc_list pmc, int nsrcs, __be32 srcs)
		if (psf->sf_count[MCAST_INCLUDE] \|\|
		pmc->sfcount[MCAST_EXCLUDE] !=
		psf->sf_count[MCAST_EXCLUDE])
		continue;
		break;
		if (srcs[i] == psf->sf_inaddr) {
		scount++;
		break;

net/ipv6/mcast.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1061,7 +1061,7 @@ static int mld_xmarksources(struct ifmcaddr6 *pmc, int nsrcs,
		if (psf->sf_count[MCAST_INCLUDE] \|\|
		pmc->mca_sfcount[MCAST_EXCLUDE] !=
		psf->sf_count[MCAST_EXCLUDE])
		continue;
		break;
		if (ipv6_addr_equal(&srcs[i], &psf->sf_addr)) {
		scount++;
		break;