ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fib: fix covered_inherit_add

Browse Source 
When a FIB entry has multiple sources installed by covering inheritance,
the first walk installs the covering source, which can be worse than the
already used one (hence the source is not activated). If a covering path
is added and a new fib cover walk is done, the deactivated source is
compared to itself. This operation calls for a reactivation of the source which was not activated in the first place.

Type: fix
Change-Id: I2639401bed14eb5a2cb003664c04d6c1f9130de9
Signed-off-by: Maxime Peim <mpeim@cisco.com>
This commit is contained in:
Maxime Peim
2024-02-20 12:27:38 +01:00
committed by Neale Ranns
parent 0e3fcc0b7e
commit 6eac5bac60
2 changed files with 57 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/plugins/unittest/fib_test.c
+51
View File
@@ -10264,7 +10264,57 @@ fib_test_inherit (void)
&l99_o_10_10_10_3),
"%U via interposer label",
format_fib_prefix,&pfx_10_10_10_21_s_32);
fib_table_entry_special_remove(0,
&pfx_10_10_10_0_s_24,
FIB_SOURCE_SPECIAL);
const ip46_address_t nh_0_0_0_0 = {
.ip4.as_u32 = clib_host_to_net_u32(0x00000000),
};
const fib_prefix_t pfx_0_0_0_0_s_0 = {
.fp_len = 0,
.fp_proto = FIB_PROTOCOL_IP4,
.fp_addr = nh_0_0_0_0,
};
/* we have prio(API) < prio(hi_src) < prio(SPECIAL) */
/* Add/remove an interposer source from the top of the subtrie. The
* interposer source is inherited.
*/
fib_table_entry_special_dpo_add(0,
&pfx_0_0_0_0_s_0,
hi_src,
(FIB_ENTRY_FLAG_COVERED_INHERIT |
FIB_ENTRY_FLAG_INTERPOSE),
&interposer);
/*
* Add/remove an interposer source from the top of the subtrie. The
* interposer source is inherited, the previous inheritance is discarded.
*/
fib_table_entry_special_dpo_add(0,
&pfx_10_10_10_0_s_24,
FIB_SOURCE_SPECIAL,
(FIB_ENTRY_FLAG_COVERED_INHERIT |
FIB_ENTRY_FLAG_INTERPOSE),
&interposer);
/* force a tree walk */
fib_table_entry_update_one_path(0,
&pfx_0_0_0_0_s_0,
FIB_SOURCE_API,
FIB_ENTRY_FLAG_NONE,
DPO_PROTO_IP4,
&nh_10_10_10_3,
tm->hw[0]->sw_if_index,
~0,
1,
NULL,
FIB_ROUTE_PATH_FLAG_NONE);
fib_table_entry_special_remove(0,
&pfx_10_10_10_0_s_24,
FIB_SOURCE_SPECIAL);
fib_table_entry_special_remove(0,
&pfx_0_0_0_0_s_0,
hi_src);
/*
* cleanup
*/
@@ -10275,6 +10325,7 @@ fib_test_inherit (void)
fib_table_entry_delete(0, &pfx_10_10_10_0_s_24, FIB_SOURCE_API);
fib_table_entry_delete(0, &pfx_10_10_0_0_s_16, FIB_SOURCE_API);
fib_table_entry_delete(0, &pfx_10_10_10_0_s_24, FIB_SOURCE_SPECIAL);
fib_table_entry_delete(0, &pfx_0_0_0_0_s_0, FIB_SOURCE_API);
adj_unlock(ai_10_10_10_1);
adj_unlock(ai_10_10_10_2);
adj_unlock(ai_10_10_10_3);
src/vnet/fib/fib_entry_src.c
+6 -2
View File
@@ -875,8 +875,12 @@ fib_entry_src_covered_inherit_add_i (fib_entry_t *fib_entry,
* The covered's source data has been inherited, presumably
* from this cover, i.e. this is a modify.
*/
esrc = fib_entry_src_action_update_from_cover(fib_entry, cover_src);
fib_entry_source_change(fib_entry, esrc->fes_src, esrc->fes_src);
fib_source_t best_source;
best_source = fib_entry_get_best_source(
fib_entry_get_index(fib_entry));
fib_entry_src_action_update_from_cover(fib_entry, cover_src);
fib_entry_source_change(fib_entry, best_source, cover_src->fes_src);
}
else
{