diff options
| author | Anuradha Karuppiah <anuradhak@cumulusnetworks.com> | 2020-07-31 15:21:57 +0200 |
|---|---|---|
| committer | Anuradha Karuppiah <anuradhak@nvidia.com> | 2021-02-22 20:56:30 +0100 |
| commit | 736475cdf6be530b03b14c49cfc9be5a2a9d245c (patch) | |
| tree | 1da9de9e74e1081cca7b6575b260f4a3b7010d5f /zebra/rt_netlink.c | |
| parent | Merge pull request #8105 from donaldsharp/reduce_monotime (diff) | |
| download | frr-736475cdf6be530b03b14c49cfc9be5a2a9d245c.tar.xz frr-736475cdf6be530b03b14c49cfc9be5a2a9d245c.zip | |
zebra: disable setting weak override flag in neigh updates
This is causing problems with VM move i.e. transition from remote
neigh to local neigh. This transition involves changing the NUD_STATE
NUD_NOARP to NUD_STALE. And the weak override flag prevents changing
the state from connected (REACHABLE, NOARP, PERMANENT) to STALE.
PS: Weak-override was originally used to prevent race conditions where
FRR can end up making a REACHABLE neigh STALE. We may need to revisit
and address that case at a later point.
Ticket: CM-30273
Signed-off-by: Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Diffstat (limited to 'zebra/rt_netlink.c')
| -rw-r--r-- | zebra/rt_netlink.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/zebra/rt_netlink.c b/zebra/rt_netlink.c index 602805be3..b98030614 100644 --- a/zebra/rt_netlink.c +++ b/zebra/rt_netlink.c @@ -3695,14 +3695,6 @@ static ssize_t netlink_neigh_update_ctx(const struct zebra_dplane_ctx *ctx, /* local neigh */ if (update_flags & DPLANE_NEIGH_SET_STATIC) ext_flags |= NTF_E_MH_PEER_SYNC; - - /* the ndm_state set for local entries can be REACHABLE or - * STALE. if the dataplane has already establish reachability - * (in the meantime) FRR must not over-write it with STALE. - * this accidental race/over-write is avoided by using the - * WEAK_OVERRIDE_STATE - */ - ext_flags |= NTF_E_WEAK_OVERRIDE_STATE; } if (IS_ZEBRA_DEBUG_KERNEL) { char buf[INET6_ADDRSTRLEN]; |