*: fix clang-6 SA warnings

I don't see these in CI, but my local clang-6 does emit warnings for
these.

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

lib: fix SA warning in skiplist code

Clang was thinking the random level could be negative.  (And, no, I
couldn't figure that out by reading its output... trial and error this
was.)

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

lib, ldpd: fix SA warnings from TAILQ oddness

Add a TAILQ_POP_FIRST so Clang understands it's the same item that is
getting removed from the list.

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

Merge pull request #2989 from donaldsharp/static_debug

Static debug

Merge pull request #2988 from donaldsharp/more_debugs

zebra: Modify nexthop checks to report inactive a bit more

Merge pull request #2981 from donaldsharp/v6_vxlan_bug

zebra: Fix RB-Tree storage comparison function for v6

Merge pull request #2982 from donaldsharp/smux_h

lib: Add missing smux.h to `make distrib` results

Merge pull request #2949 from donaldsharp/removal

ospf6d: Remove deprecated code after 1 year

Merge pull request #2987 from pacovn/Coverity_1473088_ovf_array_index_write

lib: array index check (Coverity 1473088)

sharpd: Fix missing 'show debug' command

The sharp daemon was not properly handling 'show debug'
commands.  Fix.

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

staticd: Fix 'show debug static" command

'show debugging' is returning a Command incomplete
error message as that it is being sent to staticd
and staticd has no knowledge of it, fix this.

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

lib: array index check (Coverity 1473088)

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

zebra: Modify nexthop checks to report inactive a bit more

Debugging inactive nexthops in zebra can be quite difficult
and non-obvious what has gone wrong.  Add detailed rib
debugs for the cases where we decide that a nexthop is
inactive so that we can more easily debug a reason
for the failure.

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

Merge pull request #2984 from donaldsharp/ak503_eigrp

eigrp: opnional metric in 'no redistribute'

Merge pull request #2974 from donaldsharp/v4_and_v6

V4 and v6

eigrp: opnional metric in 'no redistribute'

Signed-off-by: Dmitrii Turlupov <dturlupov@factor-rs.ru>

doc: Update sharp documentation

Update the sharp documentation to note v6 nexthops are able
to be installed now.

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

sharpd: Modify route install to allow v6 nexthops

Allow the user to create a v4 route with a v6 nexthop.

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

zebra: Fix _route_entry_dump to handle nexthop family as appropriate

The _route_entry_dump function was not handling the nexthop as passed
in from an upper level protocol appropriate and as such not displaying
the v4/v6 nexthop right in the case where we have both going.

Additionally dump the nexthop vrf as well.

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

lib: Add missing smux.h to `make distrib` results

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

Merge pull request #2959 from opensourcerouting/bfd-cleanup

bfdd: code clean up

Merge pull request #2873 from vivek-cumulus/evpn-extended-mobility

EVPN extended mobility support

zebra: Fix RB-Tree storage comparison function for v6

The RB-Tree used to store rmac information was not properly
handling the v6 address family.  Modify the code to allow
this handling.

Cleans up this error message:

zebra[2231]: host_rb_entry_compare: Unexpected family type: 10

That is being seen, This fixes some connectivity issues being seen.

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

bfdd: fix crash when receiving invalid echo packet

The log function would improperly format a string using an integer
causing a crash. This situation was found when switching echo mode with
a active connection.

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

bfdd: use THREAD_OFF() instead of thread_cancel()

Use a standardized function to cancel threads instead of hand rolling
our own.

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

Merge pull request #2970 from dslicenc/more-bool-use-json

bgpd, ospfd, pimd, zebra: Convert more use_json locations to bool

Merge pull request #2971 from qlyoung/misc-doc-updates

doc: fix building.rst index

doc: fix building.rst index

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

bgpd, ospfd, pimd, zebra: Convert more use_json locations to bool

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #2909 from netravnen/feature/git-pl-template

Add github pull-request-template.md

Merge pull request #2934 from LabNConsulting/working/master/rfapi_topotest_support

add some testing support for use in rfapi topotest

Merge pull request #2939 from netravnen/feature/git-issue-teplate

Add ISSUE_TEMPLATE

Merge pull request #2966 from donaldsharp/v6_cleanup

lib: Remove dead code

lib: Remove dead code

The ZEBRA_IPV4_ROUTE_IPV6_NEXTHOP_ADD zapi message has no creators and
no handlers.  Let's just remove.

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

bfdd: use UDP sockets instead of raw

Avoid all complexities of handling ethernet/IP/UDP headers and just use
UDP sockets: let the OS kernel handle this task.

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

bfdd: enumerate all diagnositic codes

Enumerate all codes, update the string representation and fix the
misdiagnosed echo failure.

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

bfdd: make bfd_recv_* more generic

We'll reuse these functions later to receive echo packets, so we need
to despecialize them first. Another improvement here is that we are
going to remove the global buffer variable and opt to use the stack.

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

bfdd: remove VxLAN code

Remove all VxLAN code. There are probably better ways to write VxLAN
support without having to use raw sockets and depend on extra user
configuration input.

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

bfdd: remove global variables and improve logs

Remove some more global variables and make the log messages look like
the rest of the code.

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

Merge pull request #2950 from qlyoung/misc-doc-updates

Misc doc updates

Merge pull request #2945 from dslicenc/bgp-ospf-json

bgpd/ospfd: make bgp and ospf json response a bit more consistent

.github: Update ISSUE_TEMPLATE.md

Addressed @rwestphal comments.

Signed-off-by: Christoffer <netravnen@gmail.com>

.github: Updated PULL_REQUEST_TEMPLATE.md

Addressed @qlyoung's suggestions.

Signed-off-by: Christoffer <netravnen@gmail.com>

.github: Updated ISSUE_TEMPALTE.md

Addressed @qlyoung's suggestions.

Signed-off-by: Christoffer <netravnen@gmail.com>

zebra: remove invalid ref "frr-configuration"

There is, in fact, no such additional information

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

doc: update subdir.am

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

doc: update misc build docs

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

doc: name correct sphinx package under FreeBSD 11

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

doc: rename building_frr_on* -> building_frr_for*

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

bgpd: convert from conditional operator to if-then for json changes

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #2913 from opensourcerouting/non-recursive-prep

pre-final non-recursive make

Merge pull request #2955 from ton31337/feature/nexthop_blackhole

plist: Add safety check to avoid going over 4294967295 for prefix-list

Merge pull request #2940 from ton31337/fix/as-path_validation

bgpd: Validate as-path access-list before continuing regarding invali…

plist: Add safety check to avoid going over 4294967295 for prefix-list

bgpd: change conditional operators for bgp/ospf json changes

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

bgpd: Validate as-path access-list before continuing regarding invalid chars

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

.github: Updated ISSUE_TEMPALTE.md

Addressed all of @qlyoung's "presentational nits
and a formatting suggestion". As well as
@rwestphal's idea of OS and FRR name+version
numbers.

Signed-off-by: Christoffer <netravnen@gmail.com>

Merge pull request #2785 from donaldsharp/AGGanomics

Abstract `void *aggregate` pointer in `struct route_node`

Merge pull request #2951 from qlyoung/sort-route-maps

lib: sort route-maps for display

static: Put vty_frame around vrf output in staticd.

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

staticd: Fix mixup in vrf translations

When we store the nexthop for ref-counting, keep
track of the nexthop vrf_id as well.  This will allow
us to track the nexthop per vrf!

Additionally when we get the callback from zebra about
a nexthop update, iterate over all static routes to
see if the nexthop we are getting a callback is
one we are concerned about.

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

vtysh: Let staticd know about vrf sub-mode

staticd was not transitioning into vrf sub-mode, this fixes it.

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

Merge pull request #2923 from pguibert6WIND/fix_veth_namespace

zebra: when veth link is used across vrf, the link may not be good

lib: Move aggregate pointer into aggregate route/table

Move the aggregate pointer from the route_node into agg_node
so that people using struct route_node will see a savings
in data size.

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

bgpd, ripngd: Convert to using new agg_table/route

Switch bgp and ripngd to use the new aggregate table and
route data structures.  This was mainly a search and replace
operation.

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

lib: Add Aggregate Table and Aggregate_node

Add a abstraction for `struct route_node` and `struct route_table`
such that we can have an aggregate route_node and table.  This
is because only bgp/rfapi and ripng use the aggregate data pointer
in `struct route_node`.  For full route tables other routing
protocols and tables are paying a 8 byte overhead per node.
A full bgp table ends up being ~1.2 million routes in bgp
and zebra.  This is not an insiginificant amount of data.

So create the data structures for this replacement, but
do not replace the aggregate pointer yet.  This is because
later commits will convert rfapi and ripng over to this
new data, and finally we'll move the aggregate pointer.

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

lib: sort route-maps for display

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

ospf6d: Remove deprecated code after 1 year

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

ospfd: fix more warnings with the bgp ospf json commit

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #2947 from mjstapp/fix_os_name

lib: fix use of frrpthread os_name

configure: Update version to 6.1-dev

We are now working on 6.1-dev for new work.

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

bgpd/ospfd: resolve warnings for bgp/ospf json commit

Signed-off-by: Don Slice <dslice@cumulusnetwork.com>

lib: fix use of frrpthread os_name

Fix test for presence of configured os pthread name.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #2859 from LabNConsulting/working/master/meminfo

lib: qmem show changes (header and max)

bgpd/ospfd: make bgp and ospf json response a bit more consistent

Problem reported that some bgp and ospf json commands did not return
any json output at all if the bgp/ospf instance did not exist.
Additionally, some bgp and ospf json commands did not return any json
output if the instance existed but no neighbors were defined.  This
fix makes these commands more consistent in returning empty braces for
json output and issue a message if not using json output.  Additionally,
made the flag "use_json" a bool to make it consistent since previously,
it had been defined as an int, char, u_char, and bool at various places.

Ticket: CM-21040
Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

doc: drop colspan/rowspan elements

The sphinx 'text' writer apparently doesn't support these, and the
'dummy' writer is only available on recent versions, and all of this
makes the build a bit annoying...

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

doc: html default off

... RPM builds break if we enable this unconditionally; should really
change this in CI instead.

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

build: add Makefile variable easy access

This adds 2 helper targets for use in scripts to easily get at Makefile
variables without parsing the Makefile or config.status.

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

build: improve python search pattern

- try pythonN.N-config after pythonN-config
- use "python-config --ldflags" instead of --libs
- add Python 3.6 to explicitly searched versions
- if linking fails, try with "-lz" added

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

build: add warning when overwriting vars on "make"

doing things like `make CC="mmix-linux-musl-gcc"` breaks the hosttools/
cross-compilation setup pretty hard and just straight up should not be
done.  These vars belong on `configure`, not `make`.

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

build: cache a few more results in ./configure

Makes configure re-runs faster if the `-C` option is used.

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

build: -Werror does not belong in AM_CPPFLAGS

(probably neither do the ASAN/... FLAGS, but let's leave that alone.)

Also, it was intentional that it's $(WERROR) and not @WERROR@.  The
former can be overwritten at comandline as "make WERROR=" while the
latter can't.

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

build: non-recursive solaris

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

build: fix some oddities in solaris/

target_* is always wrong for FRR because we are not a compiler.  We only
use host_* and build_*.

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

build: trigger fewer unneccessary clippy rebuilds

Make sure we include the clippy binary file in our timestamp mangling so
`make` will correctly determine when clippy files don't need to be
rebuilt.

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

build: non-recursive debianpkg

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

build: non-recursive doc + parallel sphinx

Sphinx actually does work with a parallel build, if the doctree creation
is a separate step (which the other builds will then just read
unmodified.)  This can be done with the "dummy" target.

This also adds "-j6" to sphinx-build and adds a "--disable-doc-html"
switch on ./configure to turn on/off building HTML docs separately.

Also, HTML docs are now installed by "make install" to
/usr/share/doc/frr/html.

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

Merge pull request #2942 from mjstapp/fix_new_atomics

libs: fix missing atomics for some gcc versions

Merge pull request #2754 from chiragshah6/mdev1

*: pthread set name abstraction

*: pthread set name abstraction

Testing Done:

 TOR#cat /proc/2670/task/2672/comm
 bgpd_ka

TOR# ps H -C bgpd -o 'pid tid cmd comm'
  PID   TID CMD                         COMMAND
  2670  2670 /usr/lib/frr/bgpd -M snmp - bgpd
  2670  2671 /usr/lib/frr/bgpd -M snmp - bgpd
  2670  2672 /usr/lib/frr/bgpd -M snmp - bgpd_ka

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

libs: fix missing atomics for some gcc versions

stdatomic.h does not have aliases for all of the useful gcc
atomic primitives; add them in for that path through
frratomic.h.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #2938 from pguibert6WIND/spurious_message

lib: remove spurious message when netns is not available

Merge pull request #2917 from opensourcerouting/doc-devel-openwrt

doc/developer/openwrt: Update instructions for building for OpenWRT

rfp_example: add 'rfp full-table-download' config to support topotest

Signed-off-by: Lou Berger <lberger@labn.net>

rfapi: debug related changes to support topotest -- add cost

Signed-off-by: Lou Berger <lberger@labn.net>

zebra: do not update link if interface is veth interface

when interface is a virtual ethernet interface, then there is no need to
update link pointer of interface.

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

.github: Add ISSUE_TEMPLATE.md

Content of file ISSUE_TEMPLATE.md lossly based upon
https://github.com/sysrepo/sysrepo/blob/e38fcc3c00638f97e9d0c27cb6ca2d0bc2b645dc/.github/ISSUE_TEMPLATE.md

Signed-off-by: Christoffer <netravnen@gmail.com>

.github: Updated PULL_REQUEST_TEMPLATE.md per Quentin's request.

Signed-off-by: Christoffer <netravnen@gmail.com>

doc: add comment on closing PRs

Signed-off-by: Lou Berger <lberger@labn.net>

doc: add comment on PR title and description

Signed-off-by: Lou Berger <lberger@labn.net>