On Fri, Jul 08, 2022 at 11:17:16AM +0200, Claudio Jeker wrote:
> This diff adds the required plumbing to support local path_ids in the
> output path. Mainly it extends prefix_adjout_update() to do the right
> thing. Since in normal mode of operation path_id_tx does not matter and
> only on prefix is in the Adj-RIB-Out the code uses prefix_adjout_lookup()
> to locate the prefix first and removes the lookup in
> prefix_adjout_update().
> 
> prefix_adjout_update() takes care if the passed in prefix switches
> path_id_tx and does the dance of unlinking the object from all RB trees
> before switching the path_id_tx.
> 
> In up_generate_default() just use 0 as the path_id_tx. There will only
> be the default route in this RIB so there is no conflict possible.

This all makes sense and reads perfectly fine.

ok tb

Reply via email to