ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

vom: Add bridge domain unknown unicast flooding flag

Browse Source 
UU flood can be disabled or enabled using this flag in a bd.

Change-Id: I799be2742b599783eec019b5fd295c3b940eb3e8
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
This commit is contained in:
Mohsin Kazmi
2019-05-29 11:16:34 -04:00
committed by Neale Ranns
parent 0d635dacb7
commit 1bcb31102f
4 changed files with 54 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
extras/vom/vom/bridge_domain.cpp
View File

@ -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<bridge_domain::uu_flood_mode_t>(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();

23
extras/vom/vom/bridge_domain.hpp
View File

@ -97,7 +97,7 @@ public:
};
/**
* Bridge Domain Learning mode
* Bridge Domain flood mode
*/
struct flood_mode_t : enum_base<flood_mode_t>
{
@ -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<uu_flood_mode_t>
{
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
*/

4
extras/vom/vom/bridge_domain_cmds.cpp
View File

@ -24,12 +24,14 @@ create_cmd::create_cmd(HW::item<uint32_t>& 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();

5
extras/vom/vom/bridge_domain_cmds.hpp
View File

@ -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
*/