diff options
author | qingli <qingli@FreeBSD.org> | 2008-04-13 05:45:14 +0000 |
---|---|---|
committer | qingli <qingli@FreeBSD.org> | 2008-04-13 05:45:14 +0000 |
commit | 4e8901ea7a04d2d803067647c0641e41494b8868 (patch) | |
tree | 03815f4a4313c90b705a6c025d169df0eddd29c1 /sys/net | |
parent | 5a49f99cf6d02cec123b5d9a859677c5ab42a0b3 (diff) | |
download | FreeBSD-src-4e8901ea7a04d2d803067647c0641e41494b8868.zip FreeBSD-src-4e8901ea7a04d2d803067647c0641e41494b8868.tar.gz |
This patch provides the back end support for equal-cost multi-path
(ECMP) for both IPv4 and IPv6. Previously, multipath route insertion
is disallowed. For example,
route add -net 192.103.54.0/24 10.9.44.1
route add -net 192.103.54.0/24 10.9.44.2
The second route insertion will trigger an error message of
"add net 192.103.54.0/24: gateway 10.2.5.2: route already in table"
Multiple default routes can also be inserted. Here is the netstat
output:
default 10.2.5.1 UGS 0 3074 bge0 =>
default 10.2.5.2 UGS 0 0 bge0
When multipath routes exist, the "route delete" command requires
a specific gateway to be specified or else an error message would
be displayed. For example,
route delete default
would fail and trigger the following error message:
"route: writing to routing socket: No such process"
"delete net default: not in table"
On the other hand,
route delete default 10.2.5.2
would be successful: "delete net default: gateway 10.2.5.2"
One does not have to specify a gateway if there is only a single
route for a particular destination.
I need to perform more testings on address aliases and multiple
interfaces that have the same IP prefixes. This patch as it
stands today is not yet ready for prime time. Therefore, the ECMP
code fragments are fully guarded by the RADIX_MPATH macro.
Include the "options RADIX_MPATH" in the kernel configuration
to enable this feature.
Reviewed by: robert, sam, gnn, julian, kmacy
Diffstat (limited to 'sys/net')
-rw-r--r-- | sys/net/radix.c | 22 | ||||
-rw-r--r-- | sys/net/radix.h | 1 | ||||
-rw-r--r-- | sys/net/route.c | 122 | ||||
-rw-r--r-- | sys/net/route.h | 3 | ||||
-rw-r--r-- | sys/net/rtsock.c | 20 |
5 files changed, 167 insertions, 1 deletions
diff --git a/sys/net/radix.c b/sys/net/radix.c index 0f718b7..030aa58 100644 --- a/sys/net/radix.c +++ b/sys/net/radix.c @@ -48,6 +48,13 @@ #include <net/radix.h> #endif +#include "opt_mpath.h" + +#ifdef RADIX_MPATH +#include <net/radix_mpath.h> +#endif + + static int rn_walktree_from(struct radix_node_head *h, void *a, void *m, walktree_f_t *f, void *w); static int rn_walktree(struct radix_node_head *, walktree_f_t *, void *); @@ -630,6 +637,21 @@ rn_addroute(v_arg, n_arg, head, treenodes) saved_tt = tt = rn_insert(v, head, &keyduplicated, treenodes); if (keyduplicated) { for (t = tt; tt; t = tt, tt = tt->rn_dupedkey) { +#ifdef RADIX_MPATH + /* permit multipath, if enabled for the family */ + if (rn_mpath_capable(head) && netmask == tt->rn_mask) { + /* + * go down to the end of multipaths, so that + * new entry goes into the end of rn_dupedkey + * chain. + */ + do { + t = tt; + tt = tt->rn_dupedkey; + } while (tt && t->rn_mask == tt->rn_mask); + break; + } +#endif if (tt->rn_mask == netmask) return (0); if (netmask == 0 || diff --git a/sys/net/radix.h b/sys/net/radix.h index ca53095..376fdda 100644 --- a/sys/net/radix.h +++ b/sys/net/radix.h @@ -130,6 +130,7 @@ struct radix_node_head { void (*rnh_close) /* do something when the last ref drops */ (struct radix_node *rn, struct radix_node_head *head); struct radix_node rnh_nodes[3]; /* empty tree for common case */ + int rnh_multipath; /* multipath capable ? */ #ifdef _KERNEL struct mtx rnh_mtx; /* locks entire radix tree */ #endif diff --git a/sys/net/route.c b/sys/net/route.c index 757ed6d..c41af97 100644 --- a/sys/net/route.c +++ b/sys/net/route.c @@ -32,6 +32,7 @@ #include "opt_inet.h" #include "opt_mrouting.h" +#include "opt_mpath.h" #include <sys/param.h> #include <sys/systm.h> @@ -44,6 +45,10 @@ #include <net/if.h> #include <net/route.h> +#ifdef RADIX_MPATH +#include <net/radix_mpath.h> +#endif + #include <netinet/in.h> #include <netinet/ip_mroute.h> @@ -700,6 +705,67 @@ rtrequest1(int req, struct rt_addrinfo *info, struct rtentry **ret_nrt) } switch (req) { case RTM_DELETE: +#ifdef RADIX_MPATH + /* + * if we got multipath routes, we require users to specify + * a matching RTAX_GATEWAY. + */ + if (rn_mpath_capable(rnh)) { + struct rtentry *rto = NULL; + + rn = rnh->rnh_matchaddr(dst, rnh); + if (rn == NULL) + senderr(ESRCH); + rto = rt = RNTORT(rn); + rt = rt_mpath_matchgate(rt, gateway); + if (!rt) + senderr(ESRCH); + /* + * this is the first entry in the chain + */ + if (rto == rt) { + rn = rn_mpath_next((struct radix_node *)rt); + /* + * there is another entry, now it's active + */ + if (rn) { + rto = RNTORT(rn); + RT_LOCK(rto); + rto->rt_flags |= RTF_UP; + RT_UNLOCK(rto); + } else if (rt->rt_flags & RTF_GATEWAY) { + /* + * For gateway routes, we need to + * make sure that we we are deleting + * the correct gateway. + * rt_mpath_matchgate() does not + * check the case when there is only + * one route in the chain. + */ + if (gateway && + (rt->rt_gateway->sa_len != gateway->sa_len || + memcmp(rt->rt_gateway, gateway, gateway->sa_len))) + senderr(ESRCH); + } + /* + * use the normal delete code to remove + * the first entry + */ + goto normal_rtdel; + } + /* + * if the entry is 2nd and on up + */ + if (!rt_mpath_deldup(rto, rt)) + panic ("rtrequest1: rt_mpath_deldup"); + RT_LOCK(rt); + RT_ADDREF(rt); + rt->rt_flags &= ~RTF_UP; + goto deldone; /* done with the RTM_DELETE command */ + } +#endif + +normal_rtdel: /* * Remove the item from the tree and return it. * Complain if it is not there and do no more processing. @@ -740,6 +806,7 @@ rtrequest1(int req, struct rt_addrinfo *info, struct rtentry **ret_nrt) if ((ifa = rt->rt_ifa) && ifa->ifa_rtrequest) ifa->ifa_rtrequest(RTM_DELETE, rt, info); +deldone: /* * One more rtentry floating around that is not * linked to the routing table. rttrash will be decremented @@ -822,6 +889,22 @@ rtrequest1(int req, struct rt_addrinfo *info, struct rtentry **ret_nrt) rt->rt_ifa = ifa; rt->rt_ifp = ifa->ifa_ifp; +#ifdef RADIX_MPATH + /* do not permit exactly the same dst/mask/gw pair */ + if (rn_mpath_capable(rnh) && + rt_mpath_conflict(rnh, rt, netmask)) { + if (rt->rt_gwroute) + RTFREE(rt->rt_gwroute); + if (rt->rt_ifa) { + IFAFREE(rt->rt_ifa); + } + Free(rt_key(rt)); + RT_LOCK_DESTROY(rt); + uma_zfree(rtzone, rt); + senderr(EEXIST); + } +#endif + /* XXX mtu manipulation will be done in rnh_addaddr -- itojun */ rn = rnh->rnh_addaddr(ndst, netmask, rnh, rt->rt_nodes); if (rn == NULL) { @@ -1166,7 +1249,7 @@ rtinit(struct ifaddr *ifa, int cmd, int flags) struct mbuf *m = NULL; struct rtentry *rt = NULL; struct rt_addrinfo info; - int error; + int error=0; if (flags & RTF_HOST) { dst = ifa->ifa_dstaddr; @@ -1208,10 +1291,32 @@ rtinit(struct ifaddr *ifa, int cmd, int flags) if ((rnh = rt_tables[dst->sa_family]) == NULL) goto bad; RADIX_NODE_HEAD_LOCK(rnh); +#ifdef RADIX_MPATH + if (rn_mpath_capable(rnh)) { + + rn = rnh->rnh_matchaddr(dst, rnh); + if (rn == NULL) + error = ESRCH; + else { + rt = RNTORT(rn); + /* + * for interface route the rt->rt_gateway is + * sockaddr_intf for cloning ARP entries, so + * rt_mpath_matchgate must use the interface + * address + */ + rt = rt_mpath_matchgate(rt, ifa->ifa_addr); + if (!rt) + error = ESRCH; + } + } + else +#endif error = ((rn = rnh->rnh_lookup(dst, netmask, rnh)) == NULL || (rn->rn_flags & RNF_ROOT) || RNTORT(rn)->rt_ifa != ifa || !sa_equal((struct sockaddr *)rn->rn_key, dst)); + RADIX_NODE_HEAD_UNLOCK(rnh); if (error) { bad: @@ -1235,6 +1340,21 @@ bad: * notify any listening routing agents of the change */ RT_LOCK(rt); +#ifdef RADIX_MPATH + /* + * in case address alias finds the first address + * e.g. ifconfig bge0 192.103.54.246/24 + * e.g. ifconfig bge0 192.103.54.247/24 + * the address set in the route is 192.103.54.246 + * so we need to replace it with 192.103.54.247 + */ + if (memcmp(rt->rt_ifa->ifa_addr, ifa->ifa_addr, ifa->ifa_addr->sa_len)) { + IFAFREE(rt->rt_ifa); + IFAREF(ifa); + rt->rt_ifp = ifa->ifa_ifp; + rt->rt_ifa = ifa; + } +#endif rt_newaddrmsg(cmd, ifa, error, rt); if (cmd == RTM_DELETE) { /* diff --git a/sys/net/route.h b/sys/net/route.h index 01b2957..e9f4980 100644 --- a/sys/net/route.h +++ b/sys/net/route.h @@ -97,6 +97,9 @@ struct mbuf; */ #ifndef RNF_NORMAL #include <net/radix.h> +#ifdef RADIX_MPATH +#include <net/radix_mpath.h> +#endif #endif struct rtentry { struct radix_node rt_nodes[2]; /* tree glue, and other values */ diff --git a/sys/net/rtsock.c b/sys/net/rtsock.c index 2893f4b..5ea93d3 100644 --- a/sys/net/rtsock.c +++ b/sys/net/rtsock.c @@ -30,6 +30,8 @@ * $FreeBSD$ */ #include "opt_sctp.h" +#include "opt_mpath.h" + #include <sys/param.h> #include <sys/domain.h> #include <sys/kernel.h> @@ -420,6 +422,24 @@ route_output(struct mbuf *m, struct socket *so) RADIX_NODE_HEAD_UNLOCK(rnh); senderr(ESRCH); } +#ifdef RADIX_MPATH + /* + * for RTM_CHANGE/LOCK, if we got multipath routes, + * we require users to specify a matching RTAX_GATEWAY. + * + * for RTM_GET, gate is optional even with multipath. + * if gate == NULL the first match is returned. + * (no need to call rt_mpath_matchgate if gate == NULL) + */ + if (rn_mpath_capable(rnh) && + (rtm->rtm_type != RTM_GET || info.rti_info[RTAX_GATEWAY])) { + rt = rt_mpath_matchgate(rt, info.rti_info[RTAX_GATEWAY]); + if (!rt) { + RADIX_NODE_HEAD_UNLOCK(rnh); + senderr(ESRCH); + } + } +#endif RT_LOCK(rt); RT_ADDREF(rt); RADIX_NODE_HEAD_UNLOCK(rnh); |