Merge pull request #3194 from opensourcerouting/bugfix/isis-misc

Bugfix: IS-IS fragmentation issues

isisd: adjust lsp size to area->lsp_size before packing

IS-IS would ignore any area lsp-mtu setting configured after initial
creation of the LSP since move to the new tlv serialized/deserializer.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Unlink LSP fragments from LSP0 on lsp_destroy

isisd would crash when lsp fragments aged out, since they got freed
correctly, but were not removed from LSP0's linked list of fragments.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3188 from opensourcerouting/bgp-snmp-fix-rename

bgpd: fix missed info->path rename in snmp

bgpd: fix missed info->path rename in snmp

Signed-off-by: David Lamparter <equinox@diac24.net>

Merge pull request #3157 from pguibert6WIND/doc_expanded

doc: inform user that expanded communities can not be used in set mode

Merge pull request #3179 from donaldsharp/vni_cmp

bgpd: The l2vni list compare function does not sort

Merge pull request #3181 from qlyoung/frr-sss-indent

tools: reindent frr

bgpd: Ensure that evpn_vtep_ip_cmp actually returns useful data

The evpn_vtep_ip_cmp function must return positive and negative
numbers for when we are doing sorted linked list inserts.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

tools: reindent frr

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

bgpd: The l2vni list compare function does not sort

The purpose of adding a l2vni as an sorted list is
shot in the foot when the l2vni compare function only
returns 0 or 1.  This will cause subtle crashes when
we add sorted and we end up with multiple list node pointing
to the same thing.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #3178 from vincentbernat/fix/json-route-ipv6

bgpd: fix IPv6 next-hop field name for routes in JSON

bgpd: fix IPv6 next-hop field name for routes in JSON

This may break existing clients but the original name contains two
errors.

Signed-off-by: Vincent Bernat <vincent@bernat.ch>

doc: inform user that expanded communities can not be used in set mode

Expanded communities can not be used in route-map set actions.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Merge pull request #3172 from opensourcerouting/bfd-mhop-lookup

bfdd: fix multi hop hash lookup

bfdd: fix multi hop hash lookup

Use the proper multi hop hash for matching multi hop peers.

Spotted by Dmitrii Turlupov.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #3024 from ton31337/fix/validate_route-map

bgpd: Check if route-map really exists before applying to the peer

Merge pull request #3142 from donaldsharp/bgp_peerhash

Bgp peerhash

Merge pull request #3134 from donaldsharp/bum_flooding

Bum flooding control for BGP and Zebra

Merge pull request #3168 from adharkar/frr-bgp_json

bgpd: BGP JSON new fields

Merge pull request #3167 from chiragshah6/ospfv3_dev

ospf6d: fix ospf6 crash in inp lsa processing

Merge pull request #3137 from opensourcerouting/feature/isis-bfd

Feature: BFD for IS-IS

ospf6d: fix ospf6 crash in inp lsa processing

Use ospf6_prefix_same for comparing two exact same prefix
to determine ECMP for a route.
ospf6_route_cmp expects two different prefix rather
being exactly same.

Ticket:CM-22630
Testing Done:
performed ECMP of intra network prefix route via
sending same route via different available
ospf6 paths.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

isisd: Fix license header of bfd code

isisd: Address code-style warnings

bgpd: Add '[no] flood <disable|head-end-replication>'

Add the '[no] flood <disable|head-end-replication>' command
to the l2vpn evpn afi/safi sub commands for bgp.  This command
when entered as 'flood disable' will turn off type 3 route
generation for the transmittal of the type 3 route necessary
for BUM replication on the remote VTEP.  Additionally it will
turn off the BUM handling via the new zebra command,
ZEBRA_VXLAN_FLOOD_CONTROL.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib, zebra: Allow the specification of BUM flooding

Allow the modification of whether or not we will allow
BUM flooding on the vxlan bridge.  To do this allow
the upper level protocol to specify via the ZEBRA_VXLAN_FLOOD_CONTROL
zapi message.

If flooding is disabled then BUM traffic will not be forwarded
to other VTEP's.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #3140 from vishaldhingra/b_2802

zebra : VRF aware route map in zebra

bgpd: BGP JSON new fields

1. "show bgp ipv4 json"
  - Added "network" field which displays a prefix in 'prefix/prefixlen' format.

2. "show bgp ipv6 json"
  - Added "network" field which displays a prefix in 'prefix/prefixlen' format.
  - JSON does not have "prefix", "prefixLen" fields which are present in IPv4
    command. Added these fields as they are useful.

3. "show bgp ipv4/ipv6 neighbor <neighbor_addr> advertised-routes json"
  - Added "network" field.

4. "show bgp ipv4/ipv6 summary json"
  - Added "pfxSnt" for peers. This count is obtained from corresponding
    update_subgroup.

5. "show bgp neighbor json"
  - Added "sentPrefixCounter"

Signed-off-by: Ameya Dharkar <adharkar@vmware.org>

zebra: vrf aware routmap is missing in Zebra #2802(Part 4 of 4)

The new cli for show output

Signed-off-by: vishaldhingra vdhingra@vmware.com

zebra: vrf aware routmap is missing in Zebra #2802(Part 3 of 4)

The cli changes to allow this to work.

Signed-off-by: vishaldhingra vdhingra@vmware.com

zebra: vrf aware routmap is missing in Zebra #2802(Part 2 of 4)

Function parameter replacement of using zvrf instead of vrf_id

Signed-off-by: vishaldhingra vdhingra@vmware.com

 zebra: vrf aware routmap is missing in Zebra #2802(Part 1 of 4)

 Work to handle the route-maps, namely the header changes in zebra_vrf.h
 and the mapping of using that everywhere

Signed-off-by: vishaldhingra vdhingra@vmware.com

Merge pull request #3032 from donaldsharp/bgp_info

`struct bgp_info` to `struct bgp_path_info`

bgpd: Check if route-map really exists before applying to the peer

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #3153 from adharkar/frr-scount

bgpd: Incorrect sent prefix count for a split subgroup

bgpd: Incorrect sent prefix count for a split subgroup

When a subgroup splits to form a new subgroup because of policy changes
for a peer, new subgroup copies adj out(state about advertised routes)
from the parent subgroup. At the same time, it should also copy
scount(advertised prefix count) to the new subgroup for the count to be
in sync with the adj_out for the subgroup.

Signed-off-by: Ameya Dharkar <adharkar@vmware.org>

Merge pull request #3155 from opensourcerouting/building_instructions

doc: update building instructions

Merge pull request #3159 from pguibert6WIND/doc_draft_bgp

doc: fix warning of doc compilation with bgp draft

Merge pull request #3154 from donaldsharp/bgp_mem_lem

Bgp mem issue with martian address interface strings

doc: simplify build instructions

Remove all --enable-xxx configuration flags for features that are enabled
by default.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

doc: update build instructions for Debian 9

The instruction to checkout to the stable/3.0 branch doesn't belong
here.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

doc: remove LibXML from the list of dependencies

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge pull request #3145 from pguibert6WIND/ipv6nexthop

bgpd: in case nexthop is ipv6, set nh attribute flag in mpls vpn case

bgpd: in case nexthop is ipv6, set nh attribute flag in mpls vpn case

In mpls vpn import/export, the nexthop attribute should also be
recognised for ipv6 too.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

doc: fix warning of doc compilation with bgp draft

Some draft ietf are not handle by :rfc: keyword. A workaround is
proposed, like it has been done for flowspec.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bgpd: Cleanup memory leak with the martian address intf string

When cleaning up a interface string, from the linked list we were
dropping the name pointer which held the allocated martian address
intf string.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Use a more appropriate memory type for a string

Track the memory type associated with the bartian address
interface a bit better, instead of using MTYPE_TMP.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Cleanup warnings issued by CI system from these changes

Modify code to cleanup warnings issued by the CI system from
this series of changes.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Rename various variable names to something more appropriate

ri -> pi
bi -> bpi
info -> path
info -> rmap_path ( for routemap applications )

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Convert binfo to path

Convert the binfo variable to path.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Convert all bgp_info_XXX functions to bgp_path_XXX functions

Rename all bgp_info_XXX functions to bgp_path_XXX functions

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Convert `struct bgp_info` to `struct bgp_path_info`

Do a straight conversion of `struct bgp_info` to `struct bgp_path_info`.
This commit will setup the rename of variables as well.

This is being done because `struct bgp_info` is not descriptive
of what this data actually is.  It is path information for routes
that we keep to build the actual routes nexthops plus some extra
information.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Convert BGP_INFO_XXX to BGP_PATH_XXX

Search and replace all BGP_INFO_XXX to BGP_PATH_XXX

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Rename some BGP_PATH_XXX to BGP_PATH_SHOW_XXX

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #3077 from rgirada/ospf_2980_fix

ospfd: Resolved the inconsistency behavior when “default-info origina…

isisd: don't add deprecated bfd command

Merge pull request #3037 from patrasar/2851

bgpd: remove ip prefix from as-path, <large,ext>community-list

Merge pull request #3143 from rgirada/fix_3124

ospfd: default route got flushed after lsa refresh timer.

ospfd: issue with the "default-information originate always" command #2980

Made changes such that message wont be sent to zebra to validate default
route existence  if user configured with “always”.

Signed-off-by: rgirada <rgirada@vmware.com>

Merge pull request #3132 from toreanderson/fix_routemap_prefixlist_match_doc

doc: correct route map match for prefix lists

ospfd: default route got flushed after lsa refresh timer.

Default route type is not considered while  processing  lsa
refresh timer  expiry   which intern makes it flushed from lsdb.

Signed-off-by: rgirada <rgirada@vmware.com>

Merge pull request #3125 from dslicenc/linklocal-nht

bgpd: allow nht registration on ipv6 link-local addresses

Merge pull request #3133 from qlyoung/bgp-fix-broken-show-community-cmds

fix broken bgp community show cmds

bgpd: Do a bit better job of tracking the bgp->peerhash

When we add/remove peers we need to do a bit better job
of tracking them in the bgp->peerhash.

1) When we have the doppelganger take over, make sure the
winner is the one represented in the peerhash.

2) When creating the doppelganger, leave the current one
in place instead of blindly replacing it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Cleanup peer->su handling

Cleanup calls where we were passing in the su for
peer creation a tiny bit.

Creating a peer from the cli will always have a conf_if *or*
a su but not both. While a doppelganger will have both.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Add ability to dump the bgp peerhash

The bgp->peerhash is a secretive bit of data that we use
to quickly lookup data about peers.  Unfortunately
since we had not way to look at it, we had no way
of knowing if it had gotten in or out of sync.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #3135 from donaldsharp/zapi_handler_cabbage

zebra: Cleanup call parameters to some functions

Merge pull request #3119 from pacovn/Coverity_1465497_dead_code

zebra: dead code (Coverity 1465497)

zebra: Cleanup unused file

The zebra_vxlan_null.c file is never used.  Let's remove
from our build system.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

isisd: Track BFD state and take down adjacency on failure

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add debugging for BFD

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Implement BFD replay function

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Register/deregister BFD peers for adjacencies

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Write BFD config

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add a hook when writing interface config

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add BFD configuration commands

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Replace isis_event_adjacency_state_change with a hook

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Register as BFD client

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add forward declaration of struct isis_route_info

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

zebra_ptm: Add IS-IS and OpenFabric as protocols supporting BFD

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

zebra_ptm: Introduce IS_BFD_ENABLED_PROTOCOL macro

Unify the check if a client supports BFD into one location.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

doc: correct route map match for prefix lists

This corrects the route map documentation to add the missing "prefix-list"
keyword, which is necessary when matching against a prefix list (as opposed to
an access list).

Additionally, change hyphens for underscores in the variables the user is
supposed to substitute in those commands, to prevent any confusion with the
"prefix-list" keyword itself, and also to make it more consistent with the
other documented commands (which are already using underscores).

Signed-off-by: Tore Anderson <tore@redpill-linpro.com>

bgpd: remove ip prefix from as-path, <large,ext>community-list

The existing commands "ip as-path", "ip community list", "ip extcommunity
list" & "ip largecommunity list" is used to configure both for ipv4 and
ipv6. So the prefix "ip" is removed from these commands.
All the configuration, show related configuration, show running config
& boot up with write memory is also verified with the provided fix.

Signed-off-by: Sarita Patra <saritap@vmware.com>

Merge pull request #3098 from opensourcerouting/watchfrr-delay

tools/frr + watchfrr spring cleaning (southern hemisphere)

bgpd: complete description code for communities

When this description code was added, it was all dead code since none of
the bools that checked if the communities were present were ever changed
from 0.

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

bgpd: fix broken show community commands

SHA c5f1e1b broke about half of the community-related show commands; fix
them

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

Merge pull request #3126 from pacovn/Coverity_1424529_null_check

isisd: null check (Coverity 1424529)

Merge pull request #3127 from pacovn/Coverity_1472236_Improper_use_of_negative_value

zebra: netlink socket check (Coverity 1472236)

Merge pull request #3128 from donaldsharp/cpp_notice_build_failure

lib: Include compiler.h as early as is possible in the build

Merge pull request #3087 from opensourcerouting/bfd-memleak

bfdd: fix memory leak and echo-mode start

Merge pull request #3043 from qlyoung/doc-bgpd-multiple-ass

Doc bgpd multiple ASes

lib: refactor thread_execute

Don't allocate threads in the stack, but use the standardized
`thread_get` and `thread_add_unused` to avoid creating corner cases in
the thread API.

This fixes a thread mutex memory leak in FreeBSD.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

lib: fix a memory leak in FreeBSD

Two important changes:
* Centralize the thread teardown procedure;
* Save and restore thread mutex context to avoid losing the memory
  pointer;

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

zebra: netlink socket check (Coverity 1472236)

Signed-off-by: F. Aragon <paco@voltanet.io>

lib: Include compiler.h as early as is possible in the build

The compiler.h header provides us with some useful macro's
that we are using in the system.  We do not know exactly
where the CPP_NOTICE and CPP_WARN macros are used but
they can move around.  Place this header early in the
build then.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

isisd: null check (Coverity 1424529)

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd: allow nht registration on ipv6 link-local addresses

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>
Testing Done: bgp-smoke completed with no new failures

While testing 5549 support using global addresses, discovered that
ipv6 nexthop tracking thru a route-reflector didn't work.  Since
the next-hop used for remote nexthops resolves to the link-local
of the route-reflector, we need to track it in order to react to
interface down events.  Also tripped over a crash in certain cases
which is also resolved in this fix.

doc: embellish bgp view docs

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

doc: add extra information on bgp views and vrfs

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>