Skip to content

Commit 11f7bb3

Browse files
marian-pritsaklguohan
authored andcommitted
[syncd]: Rename PFC WD to Flex Counter (sonic-net#242)
* [syncd]: Rename PFC WD to Flex Counter Signed-off-by: marian-pritsak <[email protected]> * Update Makefile.am
1 parent d22bffb commit 11f7bb3

File tree

5 files changed

+119
-119
lines changed

5 files changed

+119
-119
lines changed

syncd/Makefile.am

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ syncd_SOURCES = \
2222
syncd_notifications.cpp \
2323
syncd_counters.cpp \
2424
syncd_applyview.cpp \
25-
syncd_pfc_watchdog.cpp
25+
syncd_flex_counter.cpp
2626

2727
syncd_CPPFLAGS = $(DBGFLAGS) $(AM_CPPFLAGS) $(CFLAGS_COMMON) $(SAIFLAGS)
2828
syncd_LDADD = -lhiredis -lswsscommon $(SAILIB) -lpthread -L$(top_srcdir)/meta/.libs -lsaimetadata -lsaimeta -ldl
@@ -44,7 +44,7 @@ tests_SOURCES = \
4444
syncd_notifications.cpp \
4545
syncd_counters.cpp \
4646
syncd_applyview.cpp \
47-
syncd_pfc_watchdog.cpp
47+
syncd_flex_counter.cpp
4848

4949
tests_CPPFLAGS = $(DBGFLAGS) $(AM_CPPFLAGS) $(CFLAGS_COMMON)
5050
tests_LDADD = -lhiredis -lswsscommon -lpthread -L$(top_srcdir)/lib/src/.libs -lsairedis -L$(top_srcdir)/meta/.libs -lsaimetadata -lsaimeta

syncd/syncd.cpp

+20-20
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#include "syncd.h"
22
#include "syncd_saiswitch.h"
33
#include "sairedis.h"
4-
#include "syncd_pfc_watchdog.h"
4+
#include "syncd_flex_counter.h"
55
#include "swss/tokenize.h"
66
#include <limits.h>
77

@@ -2352,7 +2352,7 @@ sai_status_t processEvent(
23522352
return status;
23532353
}
23542354

2355-
void processPfcWdEvent(
2355+
void processFlexCounterEvent(
23562356
_In_ swss::ConsumerStateTable &consumer)
23572357
{
23582358
std::lock_guard<std::mutex> lock(g_mutex);
@@ -2380,11 +2380,11 @@ void processPfcWdEvent(
23802380
{
23812381
if (objectType == SAI_OBJECT_TYPE_PORT)
23822382
{
2383-
PfcWatchdog::removePort(vid);
2383+
FlexCounter::removePort(vid);
23842384
}
23852385
else if (objectType == SAI_OBJECT_TYPE_QUEUE)
23862386
{
2387-
PfcWatchdog::removeQueue(vid);
2387+
FlexCounter::removeQueue(vid);
23882388
}
23892389
else
23902390
{
@@ -2404,7 +2404,7 @@ void processPfcWdEvent(
24042404
sai_deserialize_port_stat(str, stat);
24052405
portCounterIds.push_back(stat);
24062406
}
2407-
PfcWatchdog::setPortCounterList(vid, rid, portCounterIds);
2407+
FlexCounter::setPortCounterList(vid, rid, portCounterIds);
24082408
}
24092409
else if (objectType == SAI_OBJECT_TYPE_QUEUE && field == PFC_WD_QUEUE_COUNTER_ID_LIST)
24102410
{
@@ -2415,7 +2415,7 @@ void processPfcWdEvent(
24152415
sai_deserialize_queue_stat(str, stat);
24162416
queueCounterIds.push_back(stat);
24172417
}
2418-
PfcWatchdog::setQueueCounterList(vid, rid, queueCounterIds);
2418+
FlexCounter::setQueueCounterList(vid, rid, queueCounterIds);
24192419
}
24202420
else if (objectType == SAI_OBJECT_TYPE_QUEUE && field == PFC_WD_QUEUE_ATTR_ID_LIST)
24212421
{
@@ -2427,7 +2427,7 @@ void processPfcWdEvent(
24272427
queueAttrIds.push_back(attr);
24282428
}
24292429

2430-
PfcWatchdog::setQueueAttrList(vid, rid, queueAttrIds);
2430+
FlexCounter::setQueueAttrList(vid, rid, queueAttrIds);
24312431
}
24322432
else
24332433
{
@@ -2437,7 +2437,7 @@ void processPfcWdEvent(
24372437
}
24382438
}
24392439

2440-
void processPfcWdPluginEvent(
2440+
void processFlexCounterPluginEvent(
24412441
_In_ swss::ConsumerStateTable &consumer)
24422442
{
24432443
std::lock_guard<std::mutex> lock(g_mutex);
@@ -2452,7 +2452,7 @@ void processPfcWdPluginEvent(
24522452

24532453
if (op == DEL_COMMAND)
24542454
{
2455-
PfcWatchdog::removeCounterPlugin(key);
2455+
FlexCounter::removeCounterPlugin(key);
24562456
return;
24572457
}
24582458

@@ -2469,11 +2469,11 @@ void processPfcWdPluginEvent(
24692469

24702470
if (value == sai_serialize_object_type(SAI_OBJECT_TYPE_PORT))
24712471
{
2472-
PfcWatchdog::addPortCounterPlugin(key);
2472+
FlexCounter::addPortCounterPlugin(key);
24732473
}
24742474
else if (value == sai_serialize_object_type(SAI_OBJECT_TYPE_QUEUE))
24752475
{
2476-
PfcWatchdog::addQueueCounterPlugin(key);
2476+
FlexCounter::addQueueCounterPlugin(key);
24772477
}
24782478
else
24792479
{
@@ -3055,14 +3055,14 @@ int syncd_main(int argc, char **argv)
30553055

30563056
std::shared_ptr<swss::DBConnector> dbAsic = std::make_shared<swss::DBConnector>(ASIC_DB, swss::DBConnector::DEFAULT_UNIXSOCKET, 0);
30573057
std::shared_ptr<swss::DBConnector> dbNtf = std::make_shared<swss::DBConnector>(ASIC_DB, swss::DBConnector::DEFAULT_UNIXSOCKET, 0);
3058-
std::shared_ptr<swss::DBConnector> dbPfcWatchdog = std::make_shared<swss::DBConnector>(PFC_WD_DB, swss::DBConnector::DEFAULT_UNIXSOCKET, 0);
3058+
std::shared_ptr<swss::DBConnector> dbFlexCounter = std::make_shared<swss::DBConnector>(PFC_WD_DB, swss::DBConnector::DEFAULT_UNIXSOCKET, 0);
30593059

30603060
g_redisClient = std::make_shared<swss::RedisClient>(dbAsic.get());
30613061

30623062
std::shared_ptr<swss::ConsumerTable> asicState = std::make_shared<swss::ConsumerTable>(dbAsic.get(), ASIC_STATE_TABLE);
30633063
std::shared_ptr<swss::NotificationConsumer> restartQuery = std::make_shared<swss::NotificationConsumer>(dbAsic.get(), "RESTARTQUERY");
3064-
std::shared_ptr<swss::ConsumerStateTable> pfcWdState = std::make_shared<swss::ConsumerStateTable>(dbPfcWatchdog.get(), PFC_WD_STATE_TABLE);
3065-
std::shared_ptr<swss::ConsumerStateTable> pfcWdPlugin = std::make_shared<swss::ConsumerStateTable>(dbPfcWatchdog.get(), PLUGIN_TABLE);
3064+
std::shared_ptr<swss::ConsumerStateTable> flexCounterState = std::make_shared<swss::ConsumerStateTable>(dbFlexCounter.get(), PFC_WD_STATE_TABLE);
3065+
std::shared_ptr<swss::ConsumerStateTable> flexCounterPlugin = std::make_shared<swss::ConsumerStateTable>(dbFlexCounter.get(), PLUGIN_TABLE);
30663066

30673067
/*
30683068
* At the end we cant use producer consumer concept since if one proces
@@ -3158,8 +3158,8 @@ int syncd_main(int argc, char **argv)
31583158

31593159
s.addSelectable(asicState.get());
31603160
s.addSelectable(restartQuery.get());
3161-
s.addSelectable(pfcWdState.get());
3162-
s.addSelectable(pfcWdPlugin.get());
3161+
s.addSelectable(flexCounterState.get());
3162+
s.addSelectable(flexCounterPlugin.get());
31633163

31643164
SWSS_LOG_NOTICE("starting main loop");
31653165

@@ -3184,13 +3184,13 @@ int syncd_main(int argc, char **argv)
31843184
warmRestartHint = handleRestartQuery(*restartQuery);
31853185
break;
31863186
}
3187-
else if (sel == pfcWdState.get())
3187+
else if (sel == flexCounterState.get())
31883188
{
3189-
processPfcWdEvent(*(swss::ConsumerStateTable*)sel);
3189+
processFlexCounterEvent(*(swss::ConsumerStateTable*)sel);
31903190
}
3191-
else if (sel == pfcWdPlugin.get())
3191+
else if (sel == flexCounterPlugin.get())
31923192
{
3193-
processPfcWdPluginEvent(*(swss::ConsumerStateTable*)sel);
3193+
processFlexCounterPluginEvent(*(swss::ConsumerStateTable*)sel);
31943194
}
31953195
else if (result == swss::Select::OBJECT)
31963196
{

0 commit comments

Comments
 (0)