From 1bcb31102fbbf4a960c70f517b18f697d4fd1132 Mon Sep 17 00:00:00 2001 From: Mohsin Kazmi Date: Wed, 29 May 2019 11:16:34 -0400 Subject: vom: Add bridge domain unknown unicast flooding flag UU flood can be disabled or enabled using this flag in a bd. Change-Id: I799be2742b599783eec019b5fd295c3b940eb3e8 Signed-off-by: Mohsin Kazmi --- extras/vom/vom/bridge_domain.cpp | 27 ++++++++++++++++++++++++--- extras/vom/vom/bridge_domain.hpp | 23 ++++++++++++++++++++++- extras/vom/vom/bridge_domain_cmds.cpp | 4 +++- extras/vom/vom/bridge_domain_cmds.hpp | 5 +++++ 4 files changed, 54 insertions(+), 5 deletions(-) (limited to 'extras') diff --git a/extras/vom/vom/bridge_domain.cpp b/extras/vom/vom/bridge_domain.cpp index 7535af35b34..e86fc26b268 100644 --- a/extras/vom/vom/bridge_domain.cpp +++ b/extras/vom/vom/bridge_domain.cpp @@ -39,6 +39,16 @@ bridge_domain::flood_mode_t::flood_mode_t(int v, const std::string& s) { } +const bridge_domain::uu_flood_mode_t bridge_domain::uu_flood_mode_t::ON(1, + "on"); +const bridge_domain::uu_flood_mode_t bridge_domain::uu_flood_mode_t::OFF(0, + "off"); + +bridge_domain::uu_flood_mode_t::uu_flood_mode_t(int v, const std::string& s) + : enum_base(v, s) +{ +} + const bridge_domain::mac_age_mode_t bridge_domain::mac_age_mode_t::ON(1, "on"); const bridge_domain::mac_age_mode_t bridge_domain::mac_age_mode_t::OFF(0, "off"); @@ -83,12 +93,14 @@ bridge_domain::bridge_domain(uint32_t id, const arp_term_mode_t& amode, const arp_ufwd_mode_t& aumode, const flood_mode_t& fmode, + const uu_flood_mode_t& uufmode, const mac_age_mode_t& mmode) : m_id(id) , m_learning_mode(lmode) , m_arp_term_mode(amode) , m_arp_ufwd_mode(aumode) , m_flood_mode(fmode) + , m_uu_flood_mode(uufmode) , m_mac_age_mode(mmode) { } @@ -99,6 +111,7 @@ bridge_domain::bridge_domain(const bridge_domain& o) , m_arp_term_mode(o.m_arp_term_mode) , m_arp_ufwd_mode(o.m_arp_ufwd_mode) , m_flood_mode(o.m_flood_mode) + , m_uu_flood_mode(o.m_uu_flood_mode) , m_mac_age_mode(o.m_mac_age_mode) { } @@ -120,6 +133,7 @@ bridge_domain::operator==(const bridge_domain& b) const { return ((m_learning_mode == b.m_learning_mode) && (m_flood_mode == b.m_flood_mode) && + (m_uu_flood_mode == b.m_uu_flood_mode) && (m_mac_age_mode == b.m_mac_age_mode) && (m_arp_term_mode == b.m_arp_term_mode) && (m_arp_ufwd_mode == b.m_arp_ufwd_mode) && id() == b.id()); @@ -140,7 +154,7 @@ bridge_domain::replay() if (rc_t::OK == m_id.rc()) { HW::enqueue(new bridge_domain_cmds::create_cmd( m_id, m_learning_mode, m_arp_term_mode, m_arp_ufwd_mode, m_flood_mode, - m_mac_age_mode)); + m_uu_flood_mode, m_mac_age_mode)); } } @@ -177,7 +191,7 @@ bridge_domain::update(const bridge_domain& desired) if (rc_t::OK != m_id.rc()) { HW::enqueue(new bridge_domain_cmds::create_cmd( m_id, m_learning_mode, m_arp_term_mode, m_arp_ufwd_mode, m_flood_mode, - m_mac_age_mode)); + m_uu_flood_mode, m_mac_age_mode)); } } @@ -214,7 +228,14 @@ bridge_domain::event_handler::handle_populate(const client_db::key_t& key) for (auto& record : *cmd) { auto& payload = record.get_payload(); - bridge_domain bd(payload.bd_id); + bridge_domain bd( + payload.bd_id, + (payload.learn ? learning_mode_t::ON : learning_mode_t::OFF), + (payload.arp_term ? arp_term_mode_t::ON : arp_term_mode_t::OFF), + (payload.arp_ufwd ? arp_ufwd_mode_t::ON : arp_ufwd_mode_t::OFF), + (payload.flood ? flood_mode_t::ON : flood_mode_t::OFF), + (payload.uu_flood ? uu_flood_mode_t::ON : uu_flood_mode_t::OFF), + (payload.mac_age ? mac_age_mode_t::ON : mac_age_mode_t::OFF)); VOM_LOG(log_level_t::DEBUG) << "dump: " << bd.to_string(); diff --git a/extras/vom/vom/bridge_domain.hpp b/extras/vom/vom/bridge_domain.hpp index ecaae04496d..8ece1cc89d1 100644 --- a/extras/vom/vom/bridge_domain.hpp +++ b/extras/vom/vom/bridge_domain.hpp @@ -97,7 +97,7 @@ public: }; /** - * Bridge Domain Learning mode + * Bridge Domain flood mode */ struct flood_mode_t : enum_base { @@ -111,6 +111,21 @@ public: flood_mode_t(int v, const std::string& s); }; + /** + * Bridge Domain Unknown Unicast Flood mode + */ + struct uu_flood_mode_t : enum_base + { + const static uu_flood_mode_t ON; + const static uu_flood_mode_t OFF; + + private: + /** + * Private constructor taking the value and the string name + */ + uu_flood_mode_t(int v, const std::string& s); + }; + /** * The value of the defaultbridge domain */ @@ -124,6 +139,7 @@ public: const arp_term_mode_t& amode = arp_term_mode_t::ON, const arp_ufwd_mode_t& aumode = arp_ufwd_mode_t::OFF, const flood_mode_t& fmode = flood_mode_t::ON, + const uu_flood_mode_t& uufmode = uu_flood_mode_t::ON, const mac_age_mode_t& mmode = mac_age_mode_t::OFF); /** @@ -262,6 +278,11 @@ private: */ flood_mode_t m_flood_mode; + /** + * The unknown unicast flood mode of the bridge + */ + uu_flood_mode_t m_uu_flood_mode; + /** * The MAC aging mode of the bridge */ diff --git a/extras/vom/vom/bridge_domain_cmds.cpp b/extras/vom/vom/bridge_domain_cmds.cpp index a7dccf28a07..1aeafcee44f 100644 --- a/extras/vom/vom/bridge_domain_cmds.cpp +++ b/extras/vom/vom/bridge_domain_cmds.cpp @@ -24,12 +24,14 @@ create_cmd::create_cmd(HW::item& item, const bridge_domain::arp_term_mode_t& amode, const bridge_domain::arp_ufwd_mode_t& aumode, const bridge_domain::flood_mode_t& fmode, + const bridge_domain::uu_flood_mode_t& uufmode, const bridge_domain::mac_age_mode_t& mmode) : rpc_cmd(item) , m_learning_mode(lmode) , m_arp_term_mode(amode) , m_arp_ufwd_mode(aumode) , m_flood_mode(fmode) + , m_uu_flood_mode(uufmode) , m_mac_age_mode(mmode) { } @@ -48,7 +50,7 @@ create_cmd::issue(connection& con) auto& payload = req.get_request().get_payload(); payload.bd_id = m_hw_item.data(); payload.flood = m_flood_mode.value(); - payload.uu_flood = m_flood_mode.value(); + payload.uu_flood = m_uu_flood_mode.value(); payload.forward = 1; payload.learn = m_learning_mode.value(); payload.arp_term = m_arp_term_mode.value(); diff --git a/extras/vom/vom/bridge_domain_cmds.hpp b/extras/vom/vom/bridge_domain_cmds.hpp index 818de674823..395b3560d7a 100644 --- a/extras/vom/vom/bridge_domain_cmds.hpp +++ b/extras/vom/vom/bridge_domain_cmds.hpp @@ -39,6 +39,7 @@ public: const bridge_domain::arp_term_mode_t& amode, const bridge_domain::arp_ufwd_mode_t& aumode, const bridge_domain::flood_mode_t& fmode, + const bridge_domain::uu_flood_mode_t& uufmode, const bridge_domain::mac_age_mode_t& mmode); /** @@ -72,6 +73,10 @@ private: * the flood mode for the bridge */ bridge_domain::flood_mode_t m_flood_mode; + /** + * the unknown unicast flood mode for the bridge + */ + bridge_domain::uu_flood_mode_t m_uu_flood_mode; /** * the flood mode for the bridge */ -- cgit 1.2.3-korg