26 Aug
2024
26 Aug
'24
9:47 p.m.
Hi Guys, I am looking at the given below function zebra_evpn_mac_is_static in zebra/zebra_evpn_mac.h file. /* if the mac has been added of a mac-route from the peer * or if it is being referenced by a neigh added by the * peer we cannot let it age out i.e. we set the static bit * in the dataplane */ static inline bool zebra_evpn_mac_is_static(struct zebra_mac *mac) { return ((mac->flags & ZEBRA_MAC_ALL_PEER_FLAGS) || mac->sync_neigh_cnt); } Is the use of ZEBRA_MAC_ALL_PEER_FLAGS correct here ? I feel that it should be ZEBRA_MAC_ES_PEER_ACTIVE instead. The current code is leading to creation of static mac on mac originating peer also since it will have the ZEBRA_MAC_ES_PEER_PROXY flag set. Rgds Atul