Skip to content

Commit

Permalink
fix a fdb-clear bug that introduced by SAI 1.2 modification (sonic-n…
Browse files Browse the repository at this point in the history
…et#301)

consolidated fdb flush event will carry a fdb_entry include
 a null bvid, in fdb flush case it shall not be considered as
 a invalid fdb_entry

 Changes to be committed:
	modified:   syncd/syncd_notifications.cpp
  • Loading branch information
keboliu authored and lguohan committed Feb 26, 2018
1 parent 07b9e00 commit 8990e25
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions syncd/syncd_notifications.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,9 @@ void redisPutFdbEntryToAsicView(

std::string key = ASIC_STATE_TABLE + (":" + strObjectType + ":" + strFdbEntry);

if (fdb->fdb_entry.switch_id == SAI_NULL_OBJECT_ID ||
fdb->fdb_entry.bv_id == SAI_NULL_OBJECT_ID)
if ((fdb->fdb_entry.switch_id == SAI_NULL_OBJECT_ID ||
fdb->fdb_entry.bv_id == SAI_NULL_OBJECT_ID) &&
(fdb->event_type != SAI_FDB_EVENT_FLUSHED))
{
SWSS_LOG_WARN("skipped to put int db: %s", strFdbEntry.c_str());
return;
Expand Down

0 comments on commit 8990e25

Please sign in to comment.