diff options
| author | Shmulik Ladkani <shmulik.ladkani@gmail.com> | 2012-11-30 10:25:59 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-12-01 11:36:37 -0500 |
| commit | aeaf6e9d2f49d793d3eb8c1af4095cf25e061b94 (patch) | |
| tree | f668b46359bfa50f83563fb3ee0247f55f451760 /net/ipv6/addrconf.c | |
| parent | fd90b29d757827ab12d6669292612308ec249532 (diff) | |
| download | kernel_goldelico_gta04-aeaf6e9d2f49d793d3eb8c1af4095cf25e061b94.zip kernel_goldelico_gta04-aeaf6e9d2f49d793d3eb8c1af4095cf25e061b94.tar.gz kernel_goldelico_gta04-aeaf6e9d2f49d793d3eb8c1af4095cf25e061b94.tar.bz2 | |
ipv6: unify logic evaluating inet6_dev's accept_ra property
As of 026359b [ipv6: Send ICMPv6 RSes only when RAs are accepted], the
logic determining whether to send Router Solicitations is identical
to the logic determining whether kernel accepts Router Advertisements.
However the condition itself is repeated in several code locations.
Unify it by introducing 'ipv6_accept_ra()' accessor.
Also, simplify the condition expression, making it more readable.
No semantic change.
Signed-off-by: Shmulik Ladkani <shmulik.ladkani@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/addrconf.c')
| -rw-r--r-- | net/ipv6/addrconf.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index fc0e13a..4b644f6 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c @@ -3150,8 +3150,7 @@ static void addrconf_dad_completed(struct inet6_ifaddr *ifp) router advertisements, start sending router solicitations. */ - if (((ifp->idev->cnf.accept_ra == 1 && !ifp->idev->cnf.forwarding) || - ifp->idev->cnf.accept_ra == 2) && + if (ipv6_accept_ra(ifp->idev) && ifp->idev->cnf.rtr_solicits > 0 && (dev->flags&IFF_LOOPBACK) == 0 && (ipv6_addr_type(&ifp->addr) & IPV6_ADDR_LINKLOCAL)) { |