Merge pull request #11232 from opensourcerouting/bgp-sendhold-derp

bgpd: fix oopsie with SendHoldTime==0

Merge pull request #11008 from patrasar/sec_addr_list_pimv6

Merge pull request #10903 from SaiGomathiN/pimv6debug

bgpd: fix oopsie with SendHoldTime==0

... *duh*

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #11225 from opensourcerouting/bgp-sendhold

bgpd: implement SendHoldTimer

Merge pull request #11216 from chiragshah6/fdev2

zebra: netlink registry of rtm tunnel notification

Merge pull request #11222 from donaldsharp/bgp_zebra_stuff

Bgp zebra stuff

Merge pull request #11204 from anlancs/fix/bgpd-check-vnihash

bgpd: remove unnecessary check for evpn

Merge pull request #11210 from anlancs/fix/zebra-leak-vtp

zebra: fix missing delete vtep during vni transition

bgpd: implement SendHoldTimer

As described by
https://www.ietf.org/archive/id/draft-spaghetti-idr-bgp-sendholdtimer-04.html

Since this replicates the HoldTime check on the receiver that is already
part of the protocol, I do not believe it necessary to wait for IETF
progress on this draft.  It's just replicating an existing element of
the protocol at the other side of the session.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #11183 from opensourcerouting/feature/handle_bgp_gr_notification

bgpd: Activate Graceful-Restart when receiving CEASE/HOLDTIME notifications

Merge pull request #11214 from kuldeepkash/multicast_pim_dr_nondr

tests: Fix for test_pim_dr_nondr_with_ospf_topo2 test failure

Merge pull request #11218 from opensourcerouting/fix/adv_routes_best_path_wrong_local_as

bgpd: Show a correct local AS for the advertised/bestpath routes

Merge pull request #11219 from pguibert6WIND/doc_show_isis

doc: use real wording for the show isis segment-routing node command

zebra: Fix newline in log message

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

bgpd: Allow nht to work when connection succeeds

Originally commit: 8761cd6ddb5437767625f58c8e9cc3ccda7887ab
implemented the ability for LL nexthop tracking to be
interface based.  During some more testing the code
to make the interface event happen was noticed to
not properly run in some cases.  This was because
it was originally assumed that the connect could
not succeed.  Testing has shown that the connect
can succeed and FRR needs to handle the nht tracking
in this case too.

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

Merge pull request #11205 from chiragshah6/fdev1

zebra: new netlink parse utility for rta used to send nhg msg

zebra: add netlink tunnel msg to dump routine

This patch parses vxlan vnifilter rtm tunnel
message which contains vni mapping to vxlan device.
The new notifications are RTM_NEWTUNNEL,
RTM_DELTUNNEL, and RTM_GETTUNNEL.

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/
linux.git/commit/?h=v5.18-rc7&id=7b8135f4df98b155b23754b6065c157861e268f1

Testing Done:

2022/05/18 00:34:25 ZEBRA: netlink_recv_msg: << netlink message dump
[recv]
2022/05/18 00:34:25 ZEBRA: nlmsghdr [len=36 type=(120) NEWTUNNEL
flags=(0x0000) {} seq=0 pid=0]
2022/05/18 00:34:25 ZEBRA:   tnlm [family=(7) AF_BRIDGE ifindex=46
2022/05/18 00:34:25 ZEBRA:   vni_start 4001, vni_end 0

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

zebra: netlink registry rtm tunnel notif

The kernel supports l3vxlan device to have (l3vni)
vni filter similar to vlan filtering on bridge device.

To receive netlink notification, FRR to register
for new netlink RTNLGRP_TUNNEL message.
This message required to register via additional
socket option as it's beyond bitmap size.

kernel patches:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/
linux.git/commit/?h=v5.18-rc7&id=7b8135f4df98b155b23754b6065c157861e268f1

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/
linux.git/commit/?h=v5.18-rc7&id=f9c4bb0b245cee35ef66f75bf409c9573d934cf9

Ticket:#3073812
Testing Done:

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

doc: use real wording for the show isis segment-routing node command

The node keyword does not take 's' at the end.

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

bgpd: Show a correct local AS for the advertised/bestpath routes

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

doc: Updated the document for pimv6 debug commands

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding 'debug node' initialization

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6 zebra'

Implementing debug pimv6 zebra command for debugging about
the events from zebra that come up through the ZAPI.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6 trace' and 'debug pimv6 trace detail'

Implementing debugs pimv6 trace and trace detail for tracing pim code
and how it is running.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug_pimv6_packetdump_send_cmd' and 'debug_pimv6_packetdump_recv_cmd'

Implementing debug pimv6 packetdump send and recv coomands for debugging
of pimv6 packet sent and received which are dumped.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6 packets'

Implementing pimv6 packets debug to information about packet generation for sending
and about packet handling from a received packet.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6 events'

Implementing debug pimv6 events command for debugging PIMv6 system events.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6 nht' and 'debug pimv6 nht detial'

Implementing debug pimv6 nht command for PIMv6 nexthop tracking.
It will display information about RPF lookups and information about when a nexthop changes.

debug pimv6 nht detial for for PIMv6 nexthop in detail.This is not enabled by default.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'debug pimv6'

Implementing debug pimv6 command for PIMv6 protocol activity.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

Merge pull request #11203 from donaldsharp/coverity_bgp_memcpy

bgpd: Fix coverity SA issue with copying over prefix data

bgpd: Fix coverity SA issue with copying over prefix data

in bgp_nlri_parse_ip there is a `sanity` check to ensure
that the prefix length as specified by the packet
will fit inside of a `struct prefix` correctly.  The problem
here of course is that this is only v4 / v6 unicast/multicast
parsing and the bytes will never be more than 16, but we are copying
into a part of the struct prefix that is only 16 bytes, but with
this check the length may be up to 47 bytes( but not really possible ).

Limit the size check to at most 16 bytes (since we are only handling
v4 or v6 addresses here )

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

Merge pull request #11037 from SaiGomathiN/clearcli

Merge pull request #11192 from cyberstorm-mauritius/zebra_netlink

zebra: Add startup message and display netlink buffer size.

Merge pull request #11211 from donaldsharp/ospf6_time_left_in_hello

Merge pull request #10775 from opensourcerouting/pim6-mld-pr

pim6d: MLD code

doc: Updated the document for pimv6 clear commands

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'clear ipv6 mroute [vrf NAME] count' CLI

Adding clear ipv6 mroute count command for resetting
multicast routes and count.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'clear ipv6 pim [vrf NAME] oil' command

Adding clear ipv6 pim [vrf NAME] oil CLI for resetting
PIMv6 output interface list.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Add 'clear ipv6 mroute [vrf NAME]' command

Adding clear ipv6 mroute cli for resetting the PIMv6 mroutes.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding 'clear ipv6 pim statistics [vrf NAME]' cmd

Adding clear ipv6 pim statistics cli for resetting pimv6 statistics

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding pim_cmd_lookup api in pim_cmd_common file

Adding pim_cmd_lookup function for clear CLIs

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pimd: Moving the common functions from pim_cmd.c file

Moving the functions that are used by both IPV4 and IPV6 to a
common file pim_cmd_common.c file.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

ospf6d: Ensure the ospf6 interface hello timer pops in all cases

If a end users does something like this:

int enp39s0
  ipv6 ospf6 hello-interval 65535

And then the timer pops and we send the hello and immediately
if the end user does this:

  ipv6 ospf6 hello-interval 5

The timer is not being reset and FRR waits the full 65k seconds
before sending the hello again, which then immediately sets
the next hello to go out in 5 seconds.

When FRR receives the new timer value, look at how much time
is left on the timer in seconds.  If this value is greater
than the new hello timer, stop the timer and set it too that
value.

This should fix a CI system test failure found, where the
system is testing setting timer from things like 12 seconds
to 65k seconds then back down to 12 and that the ospf6 neighbor
relationship stays up.

The code was also changed from thread_add_event to thread_add_timer
in all cases.  I am not sure what would happen if a show command
comes in for a thread timer remaining with an event instead of a timer
just make it consistent.

This was chased down because the support bundle showed this:
r0# show ipv6 ospf6 vrf all interface
r0-r1-eth0 is up, type BROADCAST
  Interface ID: 6
  Internet Address:
    inet6: fe80::a4ea:d3ff:fe35:cef1/64
    inet6: fd00::1/64
  Instance ID 0, Interface MTU 1500 (autodetect: 1500)
  MTU mismatch detection: enabled
  Area ID 0.0.0.0, Cost 10
  State DR, Transmit Delay 1 sec, Priority 1
  Timer intervals configured:
   Hello 12(65480.960), Dead 48, Retransmit 5

And looking at the test code is doing stuff like this:
2022/05/16 17:08:15 OSPF6: [M7Q4P-46WDR] vty[5]@(config)# interface r1-r0-eth0

2022/05/16 17:08:15 OSPF6: [M7Q4P-46WDR] vty[5]@(config-if)# ipv6 ospf6 hello-interval 65535

2022/05/16 17:08:15 OSPF6: [M7Q4P-46WDR] vty[5]@(config-if)# no ipv6 ospf6 hello-interval
2022/05/16 17:08:16 OSPF6: [M7Q4P-46WDR] vty[5]@(config-if)# ipv6 ospf6 hello-interval 1
2022/05/16 17:08:16 OSPF6: [M7Q4P-46WDR] vty[5]@(config-if)# ipv6 ospf6 hello-interval 12

If the old timer value pops, the hello interval is set to 65k and never reset again.

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

Merge pull request #11152 from donaldsharp/dscp

bgpd: Allow bgp to control the DSCP session TOS value

zebra: new netlink parse utility for rta

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

zebra: add protocol name to nexthop dump

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

ospf6d: Give time left in hello timer for `show ipv6 ospf6 int`

When running `show ipv6 ospf6 interface` the hello timer period
is shown, but there is no indication on how much time is left
on the timer.  Add a clue:

sharpd@eva ~/frr5 (master)> vtysh -c "show ipv6 ospf6 int"
enp39s0 is up, type BROADCAST
  Interface ID: 2
  Internet Address:
    inet : 192.168.119.224/24
    inet6: 2603:6080:602:509e:9a14:998:b154:9e9/64
  Instance ID 0, Interface MTU 1500 (autodetect: 1500)
  MTU mismatch detection: enabled
  Area ID 0.0.0.0, Cost 1000
  State DR, Transmit Delay 1 sec, Priority 1
  Timer intervals configured:
   Hello 10(2.652), Dead 40, Retransmit 5
  DR: 192.168.122.1 BDR: 0.0.0.0
  Number of I/F scoped LSAs is 1
    0 Pending LSAs for LSUpdate in Time 00:00:00 [thread off]
    0 Pending LSAs for LSAck in Time 00:00:00 [thread off]
  Authentication Trailer is disabled

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

zebra: fix missing delete vtep during vni transition

All `vtep`s in dplane should be deleted/uninstalled during vni transition.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #11207 from anlancs/fix/zebra-remove-check-l3vni

zebra: remove unncecessary check for l3vni

Merge pull request #11199 from donaldsharp/nexthop_dump

zebra: Add encap and group type decoding to nexthop dump

tests: Fix for test_pim_dr_nondr_with_ospf_topo2 test failure

This test is failing intermittently because sometimes igmp
local join is not getting deleted. I did split the joins means
trying to delete igmp local joins one by one. I tried running
tests multiple times and it seems to be working fine with
current changes.

There was an issue found during debugging this test failure,
which was raised already:
Issue# https://github.com/FRRouting/frr/issues/11105

Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>

zebra: remove unncecessary check for l3vni

Since `l3vni` created by `zl3vni_add()` is always valid, remove the check
for it.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #11201 from donaldsharp/unused_in_netlink_compiles

Remove some unused functions in zebra

bgpd: remove unnecessary check for evpn

Since `bgp->vnihash` is naturally non-NULL, remove check for this field.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

doc: Add RFC8538 to our supported RFC list

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Activate Graceful-Restart when receiving CEASE/HOLDTIME notifications

If we receive CEASE Notification or HOLDTIME notification, retain STALE
routes if it's not a CEASE/Hard Reset.

When doing `clear ip bgp PEER`, we can control if this would be CEASE/Hard Reset
or not by using `bgp hard-administrative-reset` knob.

When `bgp graceful-restart notification` is disabled, STALE routes won't be
retained when receiving Notification message.

Co-authored-by: Biswajit Sadhu <biswajit.sadhu@gmail.com>
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #11180 from fdumontet6WIND/flag_pre_post

BMP fix invalid bmp POST POLICY flag

Merge pull request #11135 from donaldsharp/bgp_rpki_no_forms

bgpd: Fix no form of rpki commands to take values

Merge pull request #11149 from donaldsharp/update_group_debugging

Update group debugging

Merge pull request #11202 from anlancs/fix/check_zebra_vrf_get_evpn

zebra: remove unnecessary check for "zevpn_vrf"

zebra: remove unnecessary check for "zevpn_vrf"

The global vrf in zebra is always non-NULL. In general, it is bound to
default vrf by `zebra_vrf_init()`, at other times bound to some specific
vrf. Anyway, non-NULL.

So remove all redundant checkings for the returned value of
`zebra_vrf_get_evpn()`.

Additionally, remove the unnecessary check for `zvrf` in
`zebra_vxlan_cleanup_tables()`.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

zebra: Remove unused function `route_entry_copy_nexthops`

This function is no longer used.

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

zebra: Remove unused functions in netlink compiles

When compiling with netlink,  Remove the usage of these
functions.

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

bgpd: Allow bgp to control the DSCP session TOS value

Allow BGP to control the TOS DSCP value in the tcp header
via a new command at the bgp global level `bgp session-dscp <0-63>`

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Pavel Shirhov <pavelsh@microsoft.com>

Merge pull request #11198 from opensourcerouting/fix/show_ip_bgp_l2vpn_evpn_rd_neighbor_routes_unint

bgpd: Initialize prd for show_ip_bgp_l2vpn_evpn_rd_neighbor_routes()

Merge pull request #11195 from anlancs/fix/lib-qsort

lib: add one check in "list_sort()"

zebra: Add encap and group type decoding to nexthop dump

Add the ability to give data about the nexthop group type
and encap type so that it is human readable.

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

Merge pull request #10958 from patrasar/pim_passive

pimd: introduce ip pim passive command

bgpd: Add a bit of debug to give us data about how an update group is formed

The creation of the update group is a tiny bit of a black box and is hard
to figure out by hand if it is correct.  Add a bit of code so that FRR
operators/developers can determine if the update group categorization
makes us happy.

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

bgpd: Initialize prd for show_ip_bgp_l2vpn_evpn_rd_neighbor_routes()

*** CID 1517751:  Uninitialized variables  (UNINIT)
/bgpd/bgp_evpn_vty.c: 1648 in show_ip_bgp_l2vpn_evpn_rd_neighbor_routes()
1642
1643
1644            if (rd_all)
1645                    return bgp_show_ethernet_vpn(vty, NULL, bgp_show_type_neighbor,
1646                                                 peer, SHOW_DISPLAY_STANDARD, uj);
1647            else
>>>     CID 1517751:  Uninitialized variables  (UNINIT)
>>>     Using uninitialized element of array "prd.val" when calling "bgp_show_ethernet_vpn".
1648                    return bgp_show_ethernet_vpn(vty, &prd, bgp_show_type_neighbor,
1649                                                 peer, SHOW_DISPLAY_STANDARD, uj);
1650     }
1651
1652     DEFUN(show_ip_bgp_l2vpn_evpn_neighbor_advertised_routes,
1653           show_ip_bgp_l2vpn_evpn_neighbor_advertised_routes_cmd,

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Make bgp_debug.[ch] take `const struct peer *` and return bool

Several functions in bgp_debug.[ch] take a const struct peer *
and also return a bool instead of an int.

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

Merge pull request #11197 from opensourcerouting/feature/gr_notification_docs

doc: Add some more documentation about Notification support for GR

bgpd: Fix no form of rpki commands to take values

Fix the no forms of some rpki commands to take variable
values on the no form.  So that people cut-n-pasting while
adding a no works.

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

Merge pull request #11194 from donaldsharp/untrusted

Some more Coverity issues solved

Merge pull request #11177 from opensourcerouting/fix/memset_memcpy

*: memcpy/memset zeroing

Merge pull request #11188 from opensourcerouting/fix/argv_find

bgpd: Cleanup

Merge pull request #9454 from anlancs/fix-ospf6-gr-length

ospf6d: suppress coverity warnings of insecure data handling

zebra: Add startup message and display netlink buffer size.

Add startup message and display netlink buffer size.

Signed-off-by: Loganaden Velvindron <logan@cyberstorm.mu>

doc: Add some more documentation about Notification support for GR

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Inroduced pim passive test cases

Co-authored-by: Vijay Kumar Gupta <vijayg@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Northbound support for pim passive enable

Added nothbound callbacks for pim-passive-enable

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving BSM on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving assert msg on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving join/prune on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving register-stop on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving register msg on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable receiving hello on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handling hello send statistics for passive interface

Increment pim_ifstat_hello_sent only when interface is
passive disabled.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handling reg and reg_stop send statistics for passive interface

Increment pim_ifstat_reg_stop_send & pim_ifstat_reg_send when
interface is passive disabled.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handling Join/Prune statistics for passive interface

Increment pim_ifstat_prune_send only when interface is
passive disabled.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handling BSM send statistics for passive interface

Increment pim_ifstat_bsm_tx only when interface is
passive disabled.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handling assert send statistics for passive interface

Increment pim_ifstat_assert_send only when interface is
passive disabled.

Signed-off-by: sarita patra <saritap@vmware.com>

doc: add ip[v6] pim passive support

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: display passive in show ip pim interface cmd

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: show ip[v6] pim passive in show running config

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Disable sending of control packets on passive interface

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: introduced flags for pim_passive_enable

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: introduce ipv6 pim passive command

Added a new cli command "ip pim passive" in the interface context,
to disable sending of pim control packets on the interface.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: introduce ip pim passive command

Added a new cli command "ip pim passive" in the interface context,
to disable sending of pim control packets on the interface.

Signed-off-by: sarita patra <saritap@vmware.com>