diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2021-10-04 13:05:18 +0200 | 
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2022-03-27 14:57:22 +0200 | 
| commit | 6bcc46657efbab070d36487b690258f8fa2bafbb (patch) | |
| tree | 3a26b75f7b4cc64320819b9402a69b5a87478282 /staticd/static_nht.c | |
| parent | e203efe87bd3cf989f0279861574f64dca695ad8 (diff) | |
staticd: track nexthops per-SAFI
This stops intermixing SAFI_UNICAST NHT into SAFI_MULTICAST static
routes.
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'staticd/static_nht.c')
| -rw-r--r-- | staticd/static_nht.c | 29 | 
1 files changed, 11 insertions, 18 deletions
diff --git a/staticd/static_nht.c b/staticd/static_nht.c index e1d6ba15d1..1d87a83c91 100644 --- a/staticd/static_nht.c +++ b/staticd/static_nht.c @@ -101,18 +101,15 @@ static void static_nht_update_safi(struct prefix *sp, struct prefix *nhp,  	}  } -void static_nht_update(struct prefix *sp, struct prefix *nhp, -		       uint32_t nh_num, afi_t afi, vrf_id_t nh_vrf_id) +void static_nht_update(struct prefix *sp, struct prefix *nhp, uint32_t nh_num, +		       afi_t afi, safi_t safi, vrf_id_t nh_vrf_id)  {  	struct vrf *vrf; -	RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) { -		static_nht_update_safi(sp, nhp, nh_num, afi, SAFI_UNICAST, -				       vrf, nh_vrf_id); -		static_nht_update_safi(sp, nhp, nh_num, afi, SAFI_MULTICAST, -				       vrf, nh_vrf_id); -	} +	RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) +		static_nht_update_safi(sp, nhp, nh_num, afi, safi, vrf, +				       nh_vrf_id);  }  static void static_nht_reset_start_safi(struct prefix *nhp, afi_t afi, @@ -166,16 +163,13 @@ static void static_nht_reset_start_safi(struct prefix *nhp, afi_t afi,  	}  } -void static_nht_reset_start(struct prefix *nhp, afi_t afi, vrf_id_t nh_vrf_id) +void static_nht_reset_start(struct prefix *nhp, afi_t afi, safi_t safi, +			    vrf_id_t nh_vrf_id)  {  	struct vrf *vrf; -	RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) { -		static_nht_reset_start_safi(nhp, afi, SAFI_UNICAST, -					    vrf, nh_vrf_id); -		static_nht_reset_start_safi(nhp, afi, SAFI_MULTICAST, -					    vrf, nh_vrf_id); -	} +	RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) +		static_nht_reset_start_safi(nhp, afi, safi, vrf, nh_vrf_id);  }  static void static_nht_mark_state_safi(struct prefix *sp, afi_t afi, @@ -212,7 +206,7 @@ static void static_nht_mark_state_safi(struct prefix *sp, afi_t afi,  	route_unlock_node(rn);  } -void static_nht_mark_state(struct prefix *sp, vrf_id_t vrf_id, +void static_nht_mark_state(struct prefix *sp, safi_t safi, vrf_id_t vrf_id,  			   enum static_install_states state)  {  	struct vrf *vrf; @@ -226,6 +220,5 @@ void static_nht_mark_state(struct prefix *sp, vrf_id_t vrf_id,  	if (!vrf || !vrf->info)  		return; -	static_nht_mark_state_safi(sp, afi, SAFI_UNICAST, vrf, state); -	static_nht_mark_state_safi(sp, afi, SAFI_MULTICAST, vrf, state); +	static_nht_mark_state_safi(sp, afi, safi, vrf, state);  }  | 
