]> git.ipfire.org Git - thirdparty/bird.git/log
thirdparty/bird.git
3 years agoConf: Symbols are properly scoped
Maria Matejka [Mon, 2 May 2022 18:29:03 +0000 (20:29 +0200)] 
Conf: Symbols are properly scoped

Now there is a persistent root symbol scope and all scopes have their
symbol hashes to store local symbols and not leak any symbol out.

3 years agoReplaced boilerplate eattr allocation by ea_set_attr()
Maria Matejka [Thu, 14 Apr 2022 16:32:19 +0000 (18:32 +0200)] 
Replaced boilerplate eattr allocation by ea_set_attr()

3 years agoEnforcing certain data structure explicit paddings.
Maria Matejka [Thu, 14 Apr 2022 14:51:18 +0000 (16:51 +0200)] 
Enforcing certain data structure explicit paddings.

Implicit paddings have undefined values in C. We want the eattr blocks
to be comparable by memcmp and eattrs settable directly by structrure
literals. This check ensures that all paddings in eattr and bval are
explicit and therefore zeroed in all literals.

3 years agoProtocols use EA_LITERAL_* to set attributes
Maria Matejka [Sun, 10 Apr 2022 17:31:50 +0000 (19:31 +0200)] 
Protocols use EA_LITERAL_* to set attributes

3 years agoLocal route attributes are always allocated from tmp_linpool
Maria Matejka [Sun, 10 Apr 2022 17:15:18 +0000 (19:15 +0200)] 
Local route attributes are always allocated from tmp_linpool

3 years agoFilters always allocate from tmp_linpool
Maria Matejka [Sun, 10 Apr 2022 16:55:15 +0000 (18:55 +0200)] 
Filters always allocate from tmp_linpool

3 years agoAttribute list normalization cleanup
Maria Matejka [Sun, 10 Apr 2022 12:11:46 +0000 (14:11 +0200)] 
Attribute list normalization cleanup

3 years agoMoved filter value union to lib
Maria Matejka [Thu, 31 Mar 2022 17:29:17 +0000 (19:29 +0200)] 
Moved filter value union to lib

3 years agoComplex route attributes are data structures, shall be in lib also
Maria Matejka [Thu, 31 Mar 2022 17:22:07 +0000 (19:22 +0200)] 
Complex route attributes are data structures, shall be in lib also

3 years agoSplitting route data structures out to lib
Maria Matejka [Thu, 31 Mar 2022 17:09:38 +0000 (19:09 +0200)] 
Splitting route data structures out to lib

3 years agoFIB is a data structure generic enough to be in lib
Maria Matejka [Thu, 31 Mar 2022 17:00:00 +0000 (19:00 +0200)] 
FIB is a data structure generic enough to be in lib

3 years agoFilters: removing adata_empty() duplicating lp_alloc_adata()
Maria Matejka [Sat, 26 Mar 2022 14:05:03 +0000 (15:05 +0100)] 
Filters: removing adata_empty() duplicating lp_alloc_adata()

3 years agoUnified attribute and filter types
Maria Matejka [Sat, 26 Mar 2022 10:56:02 +0000 (11:56 +0100)] 
Unified attribute and filter types

This commit removes the EAF_TYPE_* namespace completely and also for
route attributes, filter-based types T_* are used. This simplifies
fetching and setting route attributes from filters.

Also, there is now union bval which serves as an universal value holder
instead of private unions held separately by eattr and filter code.

3 years agoOpaque types are named opaque also in filters
Maria Matejka [Sat, 26 Mar 2022 11:40:46 +0000 (12:40 +0100)] 
Opaque types are named opaque also in filters

3 years agoSpecial attribute types for enums
Maria Matejka [Sat, 26 Mar 2022 11:37:41 +0000 (12:37 +0100)] 
Special attribute types for enums

3 years agoDropping EAF_VAR_LENGTH as the attribute data is either embedded or adata
Maria Matejka [Thu, 14 Apr 2022 10:06:17 +0000 (12:06 +0200)] 
Dropping EAF_VAR_LENGTH as the attribute data is either embedded or adata

3 years agoFilter: Bitfield eattrs reading / writing moved to filter code
Maria Matejka [Fri, 25 Mar 2022 18:51:35 +0000 (19:51 +0100)] 
Filter: Bitfield eattrs reading / writing moved to filter code

Before this change, fetch-update-write and bitmasking was hardcoded in
attribute access code cased by the attribute type. Several filter
instructions are used to do it instead.

As this is certainly going to be a little bit slower than before, the
switch block in attribute access code should be completely removed in
near future, helping with both performance and code cleanliness.

The user interface should have stayed intact.

3 years agoFilter operations: bitwise AND and OR
Maria Matejka [Fri, 25 Mar 2022 18:15:11 +0000 (19:15 +0100)] 
Filter operations: bitwise AND and OR

3 years agoImplicit ROA check converted to explicit filter instruction sequence
Maria Matejka [Sat, 19 Mar 2022 15:38:32 +0000 (16:38 +0100)] 
Implicit ROA check converted to explicit filter instruction sequence

3 years agoFixed spurious undef of route attributes
Alexander Zubkov [Wed, 4 May 2022 13:37:23 +0000 (15:37 +0200)] 
Fixed spurious undef of route attributes

3 years agoRIP: fixed the EA_RIP_FROM attribute
Maria Matejka [Wed, 13 Apr 2022 15:05:12 +0000 (17:05 +0200)] 
RIP: fixed the EA_RIP_FROM attribute

The interface pointer was improperly converted to u32 and back. Fixing
this by explicitly allocating an adata structure for it. It's not so
memory efficient, we'll optimize this later.

3 years agoBGP uses lp_save / lp_restore instead of linpool flushing
Maria Matejka [Tue, 5 Apr 2022 13:09:56 +0000 (15:09 +0200)] 
BGP uses lp_save / lp_restore instead of linpool flushing

It is too cryptic to flush tmp_linpool in these cases and we don't want
anybody in the future to break this code by adding an allocation
somewhere which should persist over that flush.

Saving and restoring linpool state is safer.

3 years agoAll linpools use pages to allocate regular blocks
Maria Matejka [Mon, 4 Apr 2022 20:34:14 +0000 (22:34 +0200)] 
All linpools use pages to allocate regular blocks

3 years agoBGP: Fixed LLGR depreferencing in bgp_rte_mergable
Maria Matejka [Tue, 22 Mar 2022 13:39:32 +0000 (14:39 +0100)] 
BGP: Fixed LLGR depreferencing in bgp_rte_mergable

3 years agoProtocols have their own explicit init routines
Maria Matejka [Fri, 18 Mar 2022 21:05:50 +0000 (22:05 +0100)] 
Protocols have their own explicit init routines

3 years agoUnsetting route attributes without messing with type system
Maria Matejka [Mon, 14 Mar 2022 10:13:48 +0000 (11:13 +0100)] 
Unsetting route attributes without messing with type system

3 years agoEattr flags (originated and fresh) get their own struct fields
Maria Matejka [Mon, 14 Mar 2022 09:06:44 +0000 (10:06 +0100)] 
Eattr flags (originated and fresh) get their own struct fields

3 years agoMinor fix: f_val literals should always have named struct fields
Maria Matejka [Wed, 16 Mar 2022 09:22:49 +0000 (10:22 +0100)] 
Minor fix: f_val literals should always have named struct fields

3 years agoConverted Slab allocator to typed lists
Maria Matejka [Mon, 4 Apr 2022 18:31:25 +0000 (20:31 +0200)] 
Converted Slab allocator to typed lists

3 years agoSlab allocator can free the blocks without knowing the parent structure
Maria Matejka [Mon, 4 Apr 2022 18:31:14 +0000 (20:31 +0200)] 
Slab allocator can free the blocks without knowing the parent structure

3 years agoTyped lists for easier walking and stronger type checking
Maria Matejka [Mon, 4 Apr 2022 18:28:23 +0000 (20:28 +0200)] 
Typed lists for easier walking and stronger type checking

3 years agoPrintf variant with a result allocated inside a pool / linpool
Maria Matejka [Tue, 15 Mar 2022 10:21:46 +0000 (11:21 +0100)] 
Printf variant with a result allocated inside a pool / linpool

3 years agoSlab memory allocator unit test
Maria Matejka [Mon, 14 Mar 2022 16:29:39 +0000 (17:29 +0100)] 
Slab memory allocator unit test

3 years agoFixed a static assert in page allocator
Maria Matejka [Wed, 9 Mar 2022 12:27:49 +0000 (13:27 +0100)] 
Fixed a static assert in page allocator

3 years agoMerge remote-tracking branch 'origin/master' into haugesund
Maria Matejka [Wed, 9 Mar 2022 10:51:00 +0000 (11:51 +0100)] 
Merge remote-tracking branch 'origin/master' into haugesund

3 years agoMerge commit '60880b539b8886f76961125d89a265c6e1112b7a' into haugesund
Maria Matejka [Wed, 9 Mar 2022 10:28:34 +0000 (11:28 +0100)] 
Merge commit '60880b539b8886f76961125d89a265c6e1112b7a' into haugesund

3 years agoBGP Flowspec validation: Removed in-route optimization for multithreading compatibility
Maria Matejka [Fri, 11 Feb 2022 21:29:13 +0000 (22:29 +0100)] 
BGP Flowspec validation: Removed in-route optimization for multithreading compatibility

3 years agoMerge commit 'e42eedb9' into haugesund
Maria Matejka [Wed, 9 Mar 2022 10:01:44 +0000 (11:01 +0100)] 
Merge commit 'e42eedb9' into haugesund

3 years agoMerge commit '5cff1d5f' into haugesund
Maria Matejka [Wed, 9 Mar 2022 09:56:06 +0000 (10:56 +0100)] 
Merge commit '5cff1d5f' into haugesund

Conflicts:
      proto/bgp/attrs.c
      proto/pipe/pipe.c

3 years agoMerge commit 'd5a32563' into haugesund
Maria Matejka [Wed, 9 Mar 2022 09:50:38 +0000 (10:50 +0100)] 
Merge commit 'd5a32563' into haugesund

3 years agoFixed resource initialization in unit tests
Maria Matejka [Wed, 9 Mar 2022 09:30:03 +0000 (10:30 +0100)] 
Fixed resource initialization in unit tests

3 years agoLinpools with pages fixed to the final page allocator version
Maria Matejka [Wed, 9 Mar 2022 09:30:33 +0000 (10:30 +0100)] 
Linpools with pages fixed to the final page allocator version

3 years agoMerge commit '0c59f7ff' into haugesund
Maria Matejka [Wed, 9 Mar 2022 08:13:55 +0000 (09:13 +0100)] 
Merge commit '0c59f7ff' into haugesund

3 years agoRevert "Bound allocated pages to resource pools with page caches to avoid unnecessary...
Maria Matejka [Wed, 9 Mar 2022 08:13:31 +0000 (09:13 +0100)] 
Revert "Bound allocated pages to resource pools with page caches to avoid unnecessary syscalls"

This reverts commit 7f0e59820899c30a243c18556ce2e3fb72d6d221.

3 years agoRevert "fixup! Bound allocated pages to resource pools with page caches to avoid...
Maria Matejka [Wed, 9 Mar 2022 08:13:21 +0000 (09:13 +0100)] 
Revert "fixup! Bound allocated pages to resource pools with page caches to avoid unnecessary syscalls"

This reverts commit bea582cbb53e30dd32a5b6829c7443e0e5558d11.

3 years agoRevert "Multipage allocation"
Maria Matejka [Wed, 9 Mar 2022 08:13:20 +0000 (09:13 +0100)] 
Revert "Multipage allocation"

This reverts commit 6cd37713781a3092f8166b2178fae35cbfec1e28.

3 years agoRevert "fixup! Multipage allocation"
Maria Matejka [Wed, 9 Mar 2022 08:13:18 +0000 (09:13 +0100)] 
Revert "fixup! Multipage allocation"

This reverts commit a54f75f454b7ed8c8ff2c1787a506528f22cbae7.

3 years agoSingle-threaded version of sark-branch memory page management
Maria Matejka [Wed, 9 Mar 2022 08:10:44 +0000 (09:10 +0100)] 
Single-threaded version of sark-branch memory page management

3 years agoReplacing BGP temporary linpools by the common temporary linpool
Maria Matejka [Wed, 2 Mar 2022 10:22:32 +0000 (11:22 +0100)] 
Replacing BGP temporary linpools by the common temporary linpool

3 years agoReplaced custom linpools in tests for the common tmp_linpool
Maria Matejka [Wed, 2 Mar 2022 09:59:52 +0000 (10:59 +0100)] 
Replaced custom linpools in tests for the common tmp_linpool

3 years agoIntroducing an universal temporary linpool flushed after every task
Maria Matejka [Wed, 2 Mar 2022 09:35:21 +0000 (10:35 +0100)] 
Introducing an universal temporary linpool flushed after every task

3 years agoStatic list initializer
Maria Matejka [Wed, 2 Mar 2022 10:22:01 +0000 (11:22 +0100)] 
Static list initializer

3 years agoSmall improvement in directory test
Ondrej Filip [Wed, 2 Mar 2022 09:52:09 +0000 (10:52 +0100)] 
Small improvement in directory test

3 years agoMerge commit '2c13759136951ef0e70a3e3c2b2d3c9a387f7ed9' into haugesund
Maria Matejka [Wed, 2 Mar 2022 09:01:44 +0000 (10:01 +0100)] 
Merge commit '2c13759136951ef0e70a3e3c2b2d3c9a387f7ed9' into haugesund

3 years agoBabel: Fix bug in iface reconfiguration
Ondrej Zajicek (work) [Sun, 27 Feb 2022 02:10:38 +0000 (03:10 +0100)] 
Babel: Fix bug in iface reconfiguration

A recent change in Babel causes ifaces to disappear after
reconfiguration. The patch fixes that.

Thanks to Johannes Kimmel for an insightful bugreport.

3 years agoSmall change to fix doc building on Debian v2.0.9
Ondrej Filip [Sun, 20 Feb 2022 14:30:47 +0000 (15:30 +0100)] 
Small change to fix doc building on Debian

3 years agoSmall changes related to the new release
Ondrej Filip [Sun, 20 Feb 2022 13:28:06 +0000 (14:28 +0100)] 
Small changes related to the new release

3 years agoNEWS: Fix formatting and add some notes
Ondrej Zajicek (work) [Sun, 13 Feb 2022 15:45:49 +0000 (16:45 +0100)] 
NEWS: Fix formatting and add some notes

3 years agoNEWS and version update
Ondrej Zajicek (work) [Wed, 9 Feb 2022 02:47:49 +0000 (03:47 +0100)] 
NEWS and version update

3 years agoAlloc: Use posix_memalign() instead of aligned_alloc()
Ondrej Zajicek (work) [Tue, 8 Feb 2022 21:42:00 +0000 (22:42 +0100)] 
Alloc: Use posix_memalign() instead of aligned_alloc()

For compatibility with older systems use posix_memalign(). We can
switch to aligned_alloc() when we commit to C11 for multithreading.

3 years agoNetlink: Minor cleanup
Ondrej Zajicek (work) [Tue, 8 Feb 2022 21:21:08 +0000 (22:21 +0100)] 
Netlink: Minor cleanup

3 years agoLib: Update alignment of slabs
Ondrej Zajicek (work) [Mon, 7 Feb 2022 03:39:49 +0000 (04:39 +0100)] 
Lib: Update alignment of slabs

Alignment of slabs should be at least sizeof(ptr) to avoid unaligned
pointers in slab structures. Fixme: Use proper way to choose alignment
for internal allocators.

3 years agoMerge branch 'oz-trie-table'
Ondrej Zajicek (work) [Sun, 6 Feb 2022 22:32:15 +0000 (23:32 +0100)] 
Merge branch 'oz-trie-table'

3 years agoTrie: Fix trie format
Ondrej Zajicek (work) [Sun, 6 Feb 2022 21:53:55 +0000 (22:53 +0100)] 
Trie: Fix trie format

After switching to 16-way tries, trie format ignored unaligned / internal
prefixes and only reported the primary prefix of a trie node.

Fix trie format by showing internal prefixes based on the 'local' bitmask
of a node. Also do basic (intra-node) reconstruction of prefix patterns
by finding common subtrees in 'local' bitmask.

In future, we could improve that by doing inter-node reconstruction, so
prefixes entered as one pattern for a subtree (e.g. 192.168.0.0/18+)
would be reported as such, like with aligned prefixes.

3 years agoNest: Implement locking of prefix tries during walks
Ondrej Zajicek (work) [Fri, 4 Feb 2022 04:34:02 +0000 (05:34 +0100)] 
Nest: Implement locking of prefix tries during walks

The prune loop may may rebuild the prefix trie and therefore invalidate
walk state for asynchronous walks (used in 'show route in' cmd). Fix it
by adding locking that keeps the old trie in memory until current walks
are done.

In future this could be improved by rebuilding trie walk states (by
lookup for last found prefix) after the prefix trie rebuild.

3 years agoNest: Implement prefix trie pruning
Ondrej Zajicek (work) [Thu, 3 Feb 2022 05:08:51 +0000 (06:08 +0100)] 
Nest: Implement prefix trie pruning

When rtable is pruned and network fib nodes are removed, we also need to
prune prefix trie. Unfortunately, rebuilding prefix trie takes long time
(got about 400 ms for 1M networks), so must not be atomic, we have to
rebuild a new trie while current one is still active. That may require
some considerable amount of temporary memory, so we do that only if
we expect significant trie size reduction.

3 years agoTrie: Add prefix counter
Ondrej Zajicek (work) [Wed, 2 Feb 2022 04:06:49 +0000 (05:06 +0100)] 
Trie: Add prefix counter

Add counter of prefixes stored in trie. Works only for 'restricted' tries
composed of explicit prefixes (pxlen == l == h), like ones used in rtables.

3 years agoDoc: Describe routing table options
Ondrej Zajicek (work) [Fri, 31 Dec 2021 17:42:50 +0000 (18:42 +0100)] 
Doc: Describe routing table options

3 years agoBGP: Implement flowspec validation procedure
Ondrej Zajicek (work) [Mon, 20 Dec 2021 19:25:35 +0000 (20:25 +0100)] 
BGP: Implement flowspec validation procedure

Implement flowspec validation procedure as described in RFC 8955 sec. 6
and RFC 9117. The Validation procedure enforces that only routers in the
forwarding path for a network can originate flowspec rules for that
network.

The patch adds new mechanism for tracking inter-table dependencies, which
is necessary as the flowspec validation depends on IP routes, and flowspec
rules must be revalidated when best IP routes change.

The validation procedure is disabled by default and requires that
relevant IP table uses trie, as it uses interval queries for subnets.

3 years agoNest: Add routing table configuration blocks
Ondrej Zajicek (work) [Wed, 22 Dec 2021 03:32:26 +0000 (04:32 +0100)] 
Nest: Add routing table configuration blocks

Allow to specify sorted flag, trie fla, and min/max settle time.

Also do not enable trie by default, it must be explicitly enabled.

3 years agoNest: Add convenience functions to check rtable net type
Ondrej Zajicek (work) [Mon, 20 Dec 2021 19:44:36 +0000 (20:44 +0100)] 
Nest: Add convenience functions to check rtable net type

3 years agoNest: Avoid unnecessary net_format() in 'show route' command
Ondrej Zajicek (work) [Thu, 2 Dec 2021 03:05:17 +0000 (04:05 +0100)] 
Nest: Avoid unnecessary net_format() in 'show route' command

When output of 'show route' command was generated, the net_format() was
called for each network prematurely, even if the result was not needed.

Fix the code to call net_format() only when needed. This makes queries
that process many networks but show only few (e.g. 'show route where ..',
or 'show route count') much faster (like 5x - 10x faster).

3 years agoNest: Add trie iteration code to 'show route'
Ondrej Zajicek (work) [Thu, 2 Dec 2021 02:30:39 +0000 (03:30 +0100)] 
Nest: Add trie iteration code to 'show route'

Add trie iteration code to rt_show_cont() CLI hook and use it to
accelerate 'show route in <addr>' commands using interval queries.

3 years agoNest: Implement 'show route in <addr>' command
Ondrej Zajicek (work) [Thu, 2 Dec 2021 01:22:30 +0000 (02:22 +0100)] 
Nest: Implement 'show route in <addr>' command

Implement 'show route in <addr>' command, which shows all routes in
networks that are subnets of given network. Currently limited to IP
network types.

3 years agoNest: Attach prefix trie to rtable for faster LPM and interval queries
Ondrej Zajicek (work) [Mon, 29 Nov 2021 18:23:42 +0000 (19:23 +0100)] 
Nest: Attach prefix trie to rtable for faster LPM and interval queries

Attach a prefix trie to IP/VPN/ROA tables. Use it for net_route() and
net_roa_check(). This leads to 3-5x speedups for IPv4 and 5-10x
speedup for IPv6 of these calls.

TODO:
 - Rebuild the trie during rt_prune_table()
 - Better way to avoid trie_add_prefix() in net_get() for existing tables
 - Make it configurable (?)

3 years agoBGP: Make routing loops silent
Ondrej Zajicek (work) [Fri, 28 Jan 2022 17:13:18 +0000 (18:13 +0100)] 
BGP: Make routing loops silent

One of previous commits added error logging of invalid routes. This
also inadvertently caused error logging of route loops, which should
be ignored silently. Fix that.

3 years agoBGP: Use proper class in attribute error messages
Ondrej Zajicek (work) [Fri, 28 Jan 2022 04:35:22 +0000 (05:35 +0100)] 
BGP: Use proper class in attribute error messages

Most error messages in attribute processing are in rx/decode step and
these use L_REMOTE log class. But there are few that are in tx/export
step and these should use L_ERR log class.

Use tx-specific macro (REJECT()) in tx/export code and rename field
err_withdraw to err_reject in struct bgp_export_state to ensure that
appropriate error reporting macros are called in proper contexts.

3 years agoBGP: Improve 'invalid next hop' error reporting
Ondrej Zajicek (work) [Fri, 28 Jan 2022 04:03:03 +0000 (05:03 +0100)] 
BGP: Improve 'invalid next hop' error reporting

Distinguish multiple causes of 'invalid next hop' message and report
the relevant next hop address.

Thanks to Simon Ruderich for the original patch.

3 years agoBGP: Log route updates that were changed to withdraws
Ondrej Zajicek (work) [Mon, 24 Jan 2022 02:44:21 +0000 (03:44 +0100)] 
BGP: Log route updates that were changed to withdraws

Typical BGP error handling is treat-as-withdraw, where an invalid route
is replaced with a withdraw. Log route network when it happens.

3 years ago.gitlab-ci.yml: minor changes inside the .yml file.
Matous Holinka [Thu, 6 Jan 2022 08:53:23 +0000 (09:53 +0100)] 
.gitlab-ci.yml: minor changes inside the .yml file.

+ ubuntu:21.10 added into the pipeline,
- ubuntu:20.10 removed from the pipeline,

+ misc/docker/ubuntu-21.10-amd64/Dockerfile added,
- misc/docker/ubuntu-20.10-amd64/Dockerfile removed.

3 years agoNetlink: Add option to specify netlink socket receive buffer size
Ondrej Zajicek (work) [Mon, 17 Jan 2022 04:11:29 +0000 (05:11 +0100)] 
Netlink: Add option to specify netlink socket receive buffer size

Add option 'netlink rx buffer' to specify netlink socket receive buffer
size. Uses SO_RCVBUFFORCE, so it can override rmem_max limit.

Thanks to Trisha Biswas and Michal for the original patches.

3 years agoNetlink: Add another workaround for older kernel headers
Ondrej Zajicek (work) [Sat, 15 Jan 2022 21:39:40 +0000 (22:39 +0100)] 
Netlink: Add another workaround for older kernel headers

Unfortunately, SOL_NETLINK is both recently added and arch-dependent,
so we cannot just define it.

3 years agoNetlink: Add workaround for older kernel headers
Ondrej Zajicek (work) [Fri, 14 Jan 2022 22:15:05 +0000 (23:15 +0100)] 
Netlink: Add workaround for older kernel headers

3 years agoNetlink: Enable strict checking for KRT dumps
Ondrej Zajicek (work) [Fri, 14 Jan 2022 20:53:40 +0000 (21:53 +0100)] 
Netlink: Enable strict checking for KRT dumps

Add strict checking for netlink KRT dumps to avoid PMTU cache records
from FNHE table dump along with KRT.

Linux Kernel added FNHE table dump to the netlink API in patch:

https://patchwork.ozlabs.org/project/netdev/patch/8d3b68cd37fb5fddc470904cdd6793fcf480c6c1.1561131177.git.sbrivio@redhat.com/

Therefore, since Linux 5.3 these route cache entries are dumped together
with regular routes during periodic KRT scans, which in some cases may be
huge amount of useless data. This can be avoided by using strict checking
for netlink dumps:

https://lore.kernel.org/netdev/20181008031644.15989-1-dsahern@kernel.org/

The patch mitigates the risk of receiving unknown and potentially large
number of FNHE records that would block BIRD I/O in each sync. There is a
known issue caused by the GRE tunnels on Linux that seems to be creating
one FNHE record for each destination IP address that is routed through
the tunnel, even when the PMTU equals to GRE interface MTU.

Thanks to Tomas Hlavacek for the original patch.

3 years agoNetlink: Explicitly skip received cloned routes
Ondrej Zajicek (work) [Fri, 14 Jan 2022 18:07:57 +0000 (19:07 +0100)] 
Netlink: Explicitly skip received cloned routes

Kernel uses cloned routes to keep route cache entries, but reports them
together with regular routes. They were skipped implicitly as they
do not have rtm_protocol filled. Add explicit check for cloned flag
and skip such routes explicitly.

Also, improve debug logs of skipped routes.

3 years agoBGP: Add option 'free bind'
Ondrej Zajicek (work) [Sun, 9 Jan 2022 01:40:58 +0000 (02:40 +0100)] 
BGP: Add option 'free bind'

The BGP 'free bind' option applies the IP_FREEBIND/IPV6_FREEBIND
socket option for the BGP listening socket.

Thanks to Alexander Zubkov for the idea.

3 years agoIO: Support nonlocal bind in socket interface
Alexander Zubkov [Sat, 8 Jan 2022 17:31:56 +0000 (18:31 +0100)] 
IO: Support nonlocal bind in socket interface

Add option to socket interface for nonlocal binding, i.e. binding to an
IP address that is not present on interfaces. This behaviour is enabled
when SKF_FREEBIND socket flag is set. For Linux systems, it is
implemented by IP_FREEBIND socket flag.

Minor changes done by commiter.

3 years agoTest: Activate some remaining build tests
Ondrej Zajicek (work) [Wed, 5 Jan 2022 19:07:27 +0000 (20:07 +0100)] 
Test: Activate some remaining build tests

3 years agoNetlink: Do not ignore dead routes from BIRD
Ondrej Zajicek (work) [Wed, 5 Jan 2022 18:25:42 +0000 (19:25 +0100)] 
Netlink: Do not ignore dead routes from BIRD

Currently, BIRD ignores dead routes to consider them absent. But it also
ignores its own routes and thus it can not correctly manage such routes
in some cases. This patch makes an exception for routes with proto bird
when ignoring dead routes, so they can be properly updated or removed.

Thanks to Alexander Zubkov for the original patch.

3 years agoNetlink: Improve multipath parsing errors
Ondrej Zajicek (work) [Wed, 5 Jan 2022 17:46:41 +0000 (18:46 +0100)] 
Netlink: Improve multipath parsing errors

Function nl_parse_multipath() should handle errors internally.

3 years agoConf: Fix parsing full-length IPv6 addresses
Ondrej Zajicek (work) [Wed, 5 Jan 2022 15:38:49 +0000 (16:38 +0100)] 
Conf: Fix parsing full-length IPv6 addresses

Lexer expression for bytestring was too loose, accepting also
full-length IPv6 addresses. It should be restricted such that
colon is used between every byte or never.

Fix the regex and also add some test cases for it.

Thanks to Alexander Zubkov for the bugreport

3 years agogitlab-ci.yml: failing gitlab runner fixed.
Matous [Thu, 14 Oct 2021 10:16:46 +0000 (12:16 +0200)] 
gitlab-ci.yml: failing gitlab runner fixed.

'registry.labs.nic.cz' -> 'registry.nic.cz' changed

3 years agoDoc: Document min/max operators for lists
Alexander Zubkov [Tue, 28 Dec 2021 03:09:36 +0000 (04:09 +0100)] 
Doc: Document min/max operators for lists

3 years agoFilter: Add operators to find minimum and maximum element of sets
Alexander Zubkov [Tue, 28 Dec 2021 03:05:05 +0000 (04:05 +0100)] 
Filter: Add operators to find minimum and maximum element of sets

Add operators .min and .max to find minumum or maximum element in sets
of types: clist, eclist, lclist. Example usage:

bgp_community.min
bgp_ext_community.max
filter(bgp_large_community, [(as1, as2, *)]).min

Signed-off-by: Alexander Zubkov <green@qrator.net>
3 years agoDoc: Document community components access operators
Alexander Zubkov [Tue, 28 Dec 2021 02:48:42 +0000 (03:48 +0100)] 
Doc: Document community components access operators

3 years agoFilter: Add operators to pick community components
Alexander Zubkov [Tue, 28 Dec 2021 02:46:13 +0000 (03:46 +0100)] 
Filter: Add operators to pick community components

Add operators that can be used to pick components from
pair (standard community) or lc (large community) types.
For example:

(10, 20).asn --> 10
(10, 20).data --> 20

(10, 20, 30).asn --> 10
(10, 20, 30).data1 --> 20
(10, 20, 30).data2 --> 30

Signed-off-by: Alexander Zubkov <green@qrator.net>
3 years agoBSD: Assume onlink flag on ifaces with only host addresses
Ondrej Zajicek (work) [Mon, 27 Dec 2021 18:10:35 +0000 (19:10 +0100)] 
BSD: Assume onlink flag on ifaces with only host addresses

The BSD kernel does not support the onlink flag and BIRD does not use
direct routes for next hop validation, instead depends on interface
address ranges. We would like to handle PtMP cases with only host
addresses configured, like:

  ifconfig wg0 192.168.0.10/32
  route add 192.168.0.4 -iface wg0
  route add 192.168.0.8 -iface wg0

To accept BIRD routes with onlink next-hop, like:

  route 192.168.42.0/24 via 192.168.0.4%wg0 onlink

BIRD would dismiss the route when receiving from the kernel, as the
next-hop 192.168.0.4 is not part of any interface subnet and onlink
flag is not kept by the BSD kernel.

The commit fixes this by assuming that for routes received from the
kernel, any next-hop is onlink on ifaces with only host addresses.

Thanks to Stefan Haller for the original patch.

3 years agoRPKI: Add contextual out-of-bound checks in RTR Prefix PDU handler
Job Snijders [Sat, 18 Dec 2021 15:35:28 +0000 (16:35 +0100)] 
RPKI: Add contextual out-of-bound checks in RTR Prefix PDU handler

RFC 6810 and RFC 8210 specify that the "Max Length" value MUST NOT be
less than the Prefix Length element (underflow). On the other side,
overflow of the Max Length element also is possible, it being an 8-bit
unsigned integer allows for values larger than 32 or 128. This also
implicitly ensures there is no overflow of "Length" value.

When a PDU is received where the Max Length field is corrputed, the RTR
client (BIRD) should immediately terminate the session, flush all data
learned from that cache, and log an error for the operator.

Minor changes done by commiter.

3 years agoDoc: bgp: remove "advertise ipv4"
Simon Ruderich [Sat, 18 Dec 2021 02:17:48 +0000 (03:17 +0100)] 
Doc: bgp: remove "advertise ipv4"

The option was removed in d15b0b0a ("BGP redesign", 2016-12-07)
but the documentation wasn't updated.

3 years agoNest: Do not ignore secondary flag changes in ifa updates
Ondrej Zajicek (work) [Fri, 17 Dec 2021 23:58:47 +0000 (00:58 +0100)] 
Nest: Do not ignore secondary flag changes in ifa updates

Compare all IA_* flags that are set by sysdep iface code.

The old code ignores IA_SECONDARY flag when comparing whether iface
address updates from kernel changed anything. This is usually not an
issue as kernel removes all secondary addresses due to removal of the
primary one, but it breaks when sysctl 'promote_secondaries' is enabled
and kernel promotes secondary addresses to primary ones.

Thanks to 'Alexander' for the bugreport.