diff --git a/src/vnsw/agent/vrouter/ksync/linux/vnswif_listener.cc b/src/vnsw/agent/vrouter/ksync/linux/vnswif_listener.cc index 9de5e8dbb35..002c99f7e49 100644 --- a/src/vnsw/agent/vrouter/ksync/linux/vnswif_listener.cc +++ b/src/vnsw/agent/vrouter/ksync/linux/vnswif_listener.cc @@ -297,7 +297,7 @@ VnswInterfaceListenerLinux::HandleNetlinkIntfMsg(struct nlmsghdr *nlh) { assert(port_name != NULL); LOG(DEBUG, "Handle netlink interface message " << NetlinkTypeToString(nlh->nlmsg_type) - << " for interface " << port_name); + << " for interface " << port_name << " flags " << ifi->ifi_flags); Event::Type type; if (nlh->nlmsg_type == RTM_DELLINK) { diff --git a/src/vnsw/agent/vrouter/ksync/vnswif_listener_base.cc b/src/vnsw/agent/vrouter/ksync/vnswif_listener_base.cc index a49e4163f55..002aae9f632 100644 --- a/src/vnsw/agent/vrouter/ksync/vnswif_listener_base.cc +++ b/src/vnsw/agent/vrouter/ksync/vnswif_listener_base.cc @@ -191,18 +191,13 @@ void VnswInterfaceListenerBase::SetSeen(const std::string &name, bool oper) { host_interface_table_.insert(make_pair(name, entry)); } - bool old_active = IsInterfaceActive(entry); if (oper) { entry->oper_seen_ = true; } else { entry->host_seen_ = true; } - if (old_active == IsInterfaceActive(entry)) - return; - - if (old_active == false) - Activate(name, entry->oper_id_); + Activate(name, entry->oper_id_); } void VnswInterfaceListenerBase::ResetSeen(const std::string &name, bool oper) { @@ -210,7 +205,6 @@ void VnswInterfaceListenerBase::ResetSeen(const std::string &name, bool oper) { if (entry == NULL) return; - bool old_active = IsInterfaceActive(entry); if (oper) { entry->oper_seen_ = false; } else { @@ -224,11 +218,7 @@ void VnswInterfaceListenerBase::ResetSeen(const std::string &name, bool oper) { return; } - if (old_active == IsInterfaceActive(entry)) - return; - - if (old_active) - DeActivate(name, entry->oper_id_); + DeActivate(name, entry->oper_id_); } void VnswInterfaceListenerBase::SetLinkState(const std::string &name, bool link_up){ @@ -239,9 +229,6 @@ void VnswInterfaceListenerBase::SetLinkState(const std::string &name, bool link_ bool old_active = IsInterfaceActive(entry); entry->link_up_ = link_up; - if (old_active == IsInterfaceActive(entry)) - return; - if (old_active) DeActivate(name, entry->oper_id_); else