Page MenuHomeFreeBSD
Files F102682268

D35523.diff
No OneTemporary
Actions

D35523.diff
View Options

diff --git a/sys/netinet6/nd6_rtr.c b/sys/netinet6/nd6_rtr.c
--- a/sys/netinet6/nd6_rtr.c
+++ b/sys/netinet6/nd6_rtr.c
@@ -915,6 +915,18 @@
/* NOTREACHED */
}
+static bool
+is_dr_reachable(const struct nd_defrouter *dr) {
+ struct llentry *ln = NULL;
+
+ ln = nd6_lookup(&dr->rtaddr, LLE_SF(AF_INET6, 0), dr->ifp);
+ if (ln == NULL)
+ return (false);
+ bool reachable = ND6_IS_LLINFO_PROBREACH(ln);
+ LLE_RUNLOCK(ln);
+ return reachable;
+}
+
/*
* Default Router Selection according to Section 6.3.6 of RFC 2461 and
* draft-ietf-ipngwg-router-selection:
@@ -945,12 +957,12 @@
{
struct epoch_tracker et;
struct nd_defrouter *dr, *selected_dr, *installed_dr;
- struct llentry *ln = NULL;
if (fibnum == RT_ALL_FIBS) {
for (fibnum = 0; fibnum < rt_numfibs; fibnum++) {
defrouter_select_fib(fibnum);
}
+ return;
}
ND6_RLOCK();
@@ -969,21 +981,17 @@
* the ordering rule of the list described in defrtrlist_update().
*/
selected_dr = installed_dr = NULL;
+ NET_EPOCH_ENTER(et);
TAILQ_FOREACH(dr, &V_nd6_defrouter, dr_entry) {
- NET_EPOCH_ENTER(et);
- if (selected_dr == NULL && dr->ifp->if_fib == fibnum &&
- (ln = nd6_lookup(&dr->rtaddr, LLE_SF(AF_INET6, 0), dr->ifp)) &&
- ND6_IS_LLINFO_PROBREACH(ln)) {
+ if (dr->ifp->if_fib != fibnum)
+ continue;
+
+ if (selected_dr == NULL && is_dr_reachable(dr)) {
selected_dr = dr;
defrouter_ref(selected_dr);
}
- NET_EPOCH_EXIT(et);
- if (ln != NULL) {
- LLE_RUNLOCK(ln);
- ln = NULL;
- }
- if (dr->installed && dr->ifp->if_fib == fibnum) {
+ if (dr->installed) {
if (installed_dr == NULL) {
installed_dr = dr;
defrouter_ref(installed_dr);
@@ -997,6 +1005,7 @@
}
}
}
+
/*
* If none of the default routers was found to be reachable,
* round-robin the list regardless of preference.
@@ -1021,22 +1030,14 @@
}
}
} else if (installed_dr != NULL) {
- NET_EPOCH_ENTER(et);
- if ((ln = nd6_lookup(&installed_dr->rtaddr, 0,
- installed_dr->ifp)) &&
- ND6_IS_LLINFO_PROBREACH(ln) &&
- installed_dr->ifp->if_fib == fibnum &&
+ if (is_dr_reachable(installed_dr) &&
rtpref(selected_dr) <= rtpref(installed_dr)) {
defrouter_rele(selected_dr);
selected_dr = installed_dr;
}
- NET_EPOCH_EXIT(et);
- if (ln != NULL)
- LLE_RUNLOCK(ln);
}
ND6_RUNLOCK();
- NET_EPOCH_ENTER(et);
/*
* If we selected a router for this FIB and it's different
* than the installed one, remove the installed router and
@@ -1807,20 +1808,12 @@
{
struct epoch_tracker et;
struct nd_pfxrouter *pfxrtr;
- struct llentry *ln;
- int canreach;
ND6_LOCK_ASSERT();
NET_EPOCH_ENTER(et);
LIST_FOREACH(pfxrtr, &pr->ndpr_advrtrs, pfr_entry) {
- ln = nd6_lookup(&pfxrtr->router->rtaddr, LLE_SF(AF_INET6, 0),
- pfxrtr->router->ifp);
- if (ln == NULL)
- continue;
- canreach = ND6_IS_LLINFO_PROBREACH(ln);
- LLE_RUNLOCK(ln);
- if (canreach)
+ if (is_dr_reachable(pfxrtr->router))
break;
}
NET_EPOCH_EXIT(et);

File Metadata

Mime Type
text/plain
Expires
Sat, Nov 16, 7:33 PM (21 h, 59 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
14663525
Default Alt Text
D35523.diff (2 KB)

Event Timeline