bgpd: Remove unused BGP_NEXTHOP_CONNECTED_CHANGED flag for nexthop

Also reduce the size of change_flags, which is way enough to be 1 byte.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
This commit is contained in:
Donatas Abraitis 2024-10-15 12:43:38 +03:00
parent 5c194b3639
commit a2e226b44e
2 changed files with 5 additions and 13 deletions

View File

@ -1377,16 +1377,9 @@ char *bgp_nexthop_dump_bnc_change_flags(struct bgp_nexthop_cache *bnc,
return buf; return buf;
} }
snprintfrr(buf, len, "%s%s%s", snprintfrr(buf, len, "%s%s",
CHECK_FLAG(bnc->change_flags, BGP_NEXTHOP_CHANGED) CHECK_FLAG(bnc->change_flags, BGP_NEXTHOP_CHANGED) ? "Changed " : "",
? "Changed " CHECK_FLAG(bnc->change_flags, BGP_NEXTHOP_METRIC_CHANGED) ? "Metric " : "");
: "",
CHECK_FLAG(bnc->change_flags, BGP_NEXTHOP_METRIC_CHANGED)
? "Metric "
: "",
CHECK_FLAG(bnc->change_flags, BGP_NEXTHOP_CONNECTED_CHANGED)
? "Connected "
: "");
return buf; return buf;
} }

View File

@ -45,11 +45,10 @@ struct bgp_nexthop_cache {
*/ */
bool is_evpn_gwip_nexthop; bool is_evpn_gwip_nexthop;
uint16_t change_flags; uint8_t change_flags;
#define BGP_NEXTHOP_CHANGED (1 << 0) #define BGP_NEXTHOP_CHANGED (1 << 0)
#define BGP_NEXTHOP_METRIC_CHANGED (1 << 1) #define BGP_NEXTHOP_METRIC_CHANGED (1 << 1)
#define BGP_NEXTHOP_CONNECTED_CHANGED (1 << 2) #define BGP_NEXTHOP_MACIP_CHANGED (1 << 2)
#define BGP_NEXTHOP_MACIP_CHANGED (1 << 3)
struct nexthop *nexthop; struct nexthop *nexthop;
time_t last_update; time_t last_update;