| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of having an ?: expression embedded in every single caller of
vty_out, just expand \n to \r\n in the vty code if neccessary.
(Deprecation warnings will be enabled in the next commits which will do
the search-and-replace over the codebase.)
[This reverts commit 4d5f445 "lib: add vty_outln()"]
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
|
|
| |
Signed-off-by: Lou Berger <lberger@labn.net>
|
|
|
|
| |
Signed-off-by: Lou Berger <lberger@labn.net>
|
|
|
|
| |
Signed-off-by: Lou Berger <lberger@labn.net>
|
|
|
|
| |
Signed-off-by: Lou Berger <lberger@labn.net>
|
|\
| |
| |
| |
| |
| | |
First 12-and-a-half commits from PR #619
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
| |
| |
| |
| |
| |
| |
| |
| | |
The ipaddr_t type was conflicting with code on omnios.
Remove the typedef
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| |\
| | |
| | |
| | |
| | |
| | |
| | | |
Conflicts:
lib/Makefile.am
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Add checks related to AFI_L2VPN/SAFI_EVPN that were missing in some parts
of the code. Fix incorrect check skipping EVPN when sending End of RIB.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
For certain (sub) address families such as EVPN or L3VPN, the routing
table is organized as a 2-level tree. Ensure that code walking the
routing table does the correct handling in such cases.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When zebra issues read (GET) requests to the kernel using the netlink
interface, it is incorrect to format all of them in a generic manner
using 'struct ifinfomsg' or 'struct rtgenmsg'. Rather, messages for a
particular entity (e.g., routes) should use the corresponding structure
for encoding (e.g., 'struct rtmsg'). Of course, this has to correlate
with what the kernel expects.
In the absence of this, there is the possibility of sending extraneous
information in the request which the kernel wouldn't like.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by: David Ahern <dsa@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
While it is not essential to set this, it seems a good thing to do.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Define addattr16().
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Modify EVPN prefix to use the generic IP address structure. Add support
for EVPN type-2 and type-3 prefix dump. Fix references to modified fields
as needed.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Define an IP address structure which is a union of an IPv4 and IPv6
address. This is for subsequent use in EVPN.
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Added APIs to:
a) pre-assign 0th bit in the bitfield
b) free 0th bit in the bitfield
c) free the allocated bitfield data
Signed-off-by: Mitesh Kanjariya <mitesh@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
Reviewed-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | | |
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| | |
| | |
| | |
| | |
| | | |
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|\ \ \
| |/ /
|/| | |
bgpd: peer-group members 'activate' when they shouldn't
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
Today if you configure the following where the 'fabric' peer-group has
been deactivated for ipv4 unicast and then assign swp1/swp2 to that
peer-group we end up activating those two peers for ipv4 unicast.
conf t
no router bgp 100
router bgp 100
neighbor fabric peer-group
neighbor fabric capability extended-nexthop
neighbor fabric remote-as external
!
address-family ipv4 unicast
no neighbor fabric activate
!
neighbor swp1 interface peer-group fabric
neighbor swp2 interface peer-group fabric
neighbor 1.1.1.1 peer-group fabric
end
cel-redxp-10# show run bgp
!
router bgp 100
neighbor fabric peer-group
neighbor fabric remote-as external
neighbor fabric capability extended-nexthop
neighbor swp1 interface peer-group fabric
neighbor swp2 interface peer-group fabric
neighbor 1.1.1.1 peer-group fabric
!
address-family ipv4 unicast
no neighbor fabric activate
neighbor swp1 activate
neighbor swp2 activate
exit-address-family
!
cel-redxp-10#
With the patch we do not activate swp1/swp2
cel-redxp-10# show run bgp
!
router bgp 100
neighbor fabric peer-group
neighbor fabric remote-as external
neighbor fabric capability extended-nexthop
neighbor swp1 interface peer-group fabric
neighbor swp2 interface peer-group fabric
neighbor 1.1.1.1 peer-group fabric
!
address-family ipv4 unicast
no neighbor fabric activate
exit-address-family
!
cel-redxp-10#
|
|\ \ \
| | | |
| | | | |
bgpd: Install SAFI_LABELED_UNICAST routes in SAFI_UNICAST table
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
|
| |\ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
bgpd-ipv4-plus-label-misc3
Conflicts:
bgpd/bgp_route.c
bgpd/bgp_zebra.c
|
|\ \ \ \
| |_|/ /
|/| | | |
Bgp buffer
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The data for each nexthop is stored off of the info
pointer, instead of handling the first one and then looping
over the remaining, just loop over them all and handle the
first one as a special case.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In the bgp code to send routes to zebra we were spending
a non trivial amount of time managing buffer sizes.
We know a priori the multipath supported by our system
so let's just use that value to appropriately size
the buffers.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|\ \ \ \
| | | | |
| | | | | |
Fix excess docstring
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|\ \ \ \ \
| | | | | |
| | | | | | |
doc: update build docs for ubuntu
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
cumulus/etc --> tools/etc
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|\ \ \ \ \
| | | | | |
| | | | | | |
zebra: fix mpls_str2label()
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When making improvements to error handling in this code I accidentally
introduced an off-by-one. Fix it.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|\ \ \ \ \
| | | | | |
| | | | | | |
lib: fix vty_out with >1024 bytes of output
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Consuming va_args modifies its internal bits, hence the need to copy
it... but the copying wasn't quite right just yet.
Fixes: 4d5f445 ("lib: add vty_outln()")
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
bgpd: "address-family" not displayed in configuration
|
|/ / / /
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
|
| | |\ \
| |_|/ /
|/| | |
| | | | |
bgpd-ipv4-plus-label-misc3
|
|\ \ \ \
| | | | |
| | | | | |
Hashtable statistics
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|