On 6 Dec 2024, at 21:26, Frode Nordahl wrote:

> Hello,
>
> This is a continuation of the work started in [0], which has gone
> through a couple of iterations by Felix Huettner [1][2].
>
> As agreed in the OVN A/V Community meeting December 2nd 2024, I took
> over herding these patches again.
>
> The main rationale for this is that I have done a fair bit of work on
> Netlink related code before, so it would be a good way to spread the
> workload of this epic, with the goal of getting the most out of this
> community effort prior to looming freeze deadlines.
>
> Main change from previous iterations is:
> * Series has been properly separated into logical units of change.
> * Nexthops stored as a linked list as opposed to fixed size array.
> * Support for parsing IPv4 routes over IPv6 next hops through the use
>   of the RTA_VIA attribute.
> * As agreed the namespaces code proposed by Felix Huettner has been
>   dropped.
> * More complete tests have been provided.

Hi Frode,

I have not yet started looking at this revision, but I noticed failures on the 
unit tests from patch five onwards. Did you figure out why?

//Eelco


> 0: https://mail.openvswitch.org/pipermail/ovs-dev/2024-July/416042.html.
> 1: https://mail.openvswitch.org/pipermail/ovs-dev/2024-October/417872.html
> 2: https://mail.openvswitch.org/pipermail/ovs-dev/2024-November/418547.html
>
> Frode Nordahl (10):
>   route-table: Store route table ID.
>   route-table: Store route priority.
>   route-table: Store route protocol.
>   route-table: Split header and attribute parsing.
>   route-table: Store nexthops in linked list.
>   route-table: Support parsing RTA_VIA attribute.
>   route-table: Support parsing multipath routes.
>   route-table: Use RTA_TABLE for route table filtering.
>   route-table: Use callback for handling route messages.
>   route-table: Export route table dump function.
>
>  lib/route-table.c     | 224 ++++++++++++++++++++++++++++++------------
>  lib/route-table.h     |  31 ++++++
>  tests/system-route.at |  92 +++++++++++++++++
>  3 files changed, 286 insertions(+), 61 deletions(-)
>
> -- 
> 2.45.2

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to