Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inspur feature fix dynamic attr #1053

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Inspur feature fix dynamic attr #1053

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3b39c9a
Fix the attributes of fdbs from sai
inspurSDN May 30, 2022
f122207
Update Meta.cpp
inspurSDN May 30, 2022
a4c3f15
Update Meta.cpp
inspurSDN Jun 1, 2022
f89d2af
Update Meta.cpp
inspurSDN Jun 1, 2022
36f2fdd
Update Meta.cpp
inspurSDN Jun 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 21 additions & 10 deletions meta/Meta.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7446,18 +7446,29 @@ void Meta::meta_sai_on_fdb_event_single(
sai_attribute_t *list = data.attr;
uint32_t count = data.attr_count;

sai_attribute_t local[2]; // 2 for port id and type
/* Add an attr of type if there is no SAI_FDB_ENTRY_ATTR_TYPE in data.
* There are two attributes but no SAI_FDB_ENTRY_ATTR_TYPE in VXLAN environment.
*/
std::vector<sai_attribute_t> local_attributes;
sai_attribute_t local_type;

if (count == 1)
bool hasType = false;
for (uint32_t i = 0; i < count; i++)
{
// workaround for missing "TYPE" attribute on notification

local[0] = data.attr[0]; // copy 1st attr
local[1].id = SAI_FDB_ENTRY_ATTR_TYPE;
local[1].value.s32 = SAI_FDB_ENTRY_TYPE_DYNAMIC; // assume learned entries are always dynamic

list = local;
count = 2; // now we added type
if(data.attr[i].id == SAI_FDB_ENTRY_ATTR_TYPE)
{
hasType = true;
break;
}
local_attributes.push_back(data.attr[i]);//copy every attr
}
if(!hasType)
{
local_type.id = SAI_FDB_ENTRY_ATTR_TYPE;
local_type.value.s32 = SAI_FDB_ENTRY_TYPE_DYNAMIC;// assume learned entries are always dynamic
local_attributes.push_back(local_type);
list = local_attributes.data();
count++;
}

sai_status_t status = meta_generic_validation_create(meta_key_fdb, data.fdb_entry.switch_id, count, list);
Expand Down