Skip to content

Commit

Permalink
[syncd] Fix fdb flood queue size limit check (sonic-net#857)
Browse files Browse the repository at this point in the history
  • Loading branch information
kcudnik committed Jul 13, 2021
1 parent 84fa50a commit 6fe1e75
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion syncd/NotificationProcessor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ bool NotificationProcessor::check_fdb_event_notification_data(

if (!m_translator->checkRidExists(data.fdb_entry.switch_id) || data.fdb_entry.switch_id == SAI_NULL_OBJECT_ID)
{
SWSS_LOG_ERROR("switch_id RID 0x%" PRIx64 " is not present on local ASIC DB: %s", data.fdb_entry.bv_id,
SWSS_LOG_ERROR("switch_id RID 0x%" PRIx64 " is not present on local ASIC DB: %s", data.fdb_entry.switch_id,
sai_serialize_fdb_entry(data.fdb_entry).c_str());

result = false;
Expand Down
2 changes: 1 addition & 1 deletion syncd/NotificationQueue.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ bool NotificationQueue::enqueue(
*/
auto queueSize = m_queue.size();

if (queueSize < m_queueSizeLimit || kfvOp(item) != SAI_SWITCH_NOTIFICATION_NAME_FDB_EVENT) // TODO use enum instead of strings
if (queueSize < m_queueSizeLimit || kfvKey(item) != SAI_SWITCH_NOTIFICATION_NAME_FDB_EVENT) // TODO use enum instead of strings
{
m_queue.push(item);

Expand Down

0 comments on commit 6fe1e75

Please sign in to comment.