diff --git a/syncd/VendorSai.cpp b/syncd/VendorSai.cpp index aaaa870ee..6a146f8be 100644 --- a/syncd/VendorSai.cpp +++ b/syncd/VendorSai.cpp @@ -1949,6 +1949,7 @@ sai_status_t VendorSai::logSet( _In_ sai_api_t api, _In_ sai_log_level_t log_level) { + MUTEX(); SWSS_LOG_ENTER(); m_logLevelMap[api] = log_level; @@ -1967,6 +1968,7 @@ sai_status_t VendorSai::queryApiVersion( sai_log_level_t VendorSai::logGet( _In_ sai_api_t api) { + MUTEX(); SWSS_LOG_ENTER(); auto it = m_logLevelMap.find(api); diff --git a/unittest/syncd/TestVendorSai.cpp b/unittest/syncd/TestVendorSai.cpp index f14d280a4..4d295c467 100644 --- a/unittest/syncd/TestVendorSai.cpp +++ b/unittest/syncd/TestVendorSai.cpp @@ -1544,6 +1544,17 @@ TEST(VendorSai, bulk_meter_rules) EXPECT_EQ(SAI_STATUS_SUCCESS, sai.remove((sai_object_type_t)SAI_OBJECT_TYPE_METER_POLICY, meter_policy1)); } +TEST(VendorSai, logSet_logGet) +{ + VendorSai sai; + sai.apiInitialize(0, &test_services); + + EXPECT_EQ(SAI_STATUS_SUCCESS, sai.logSet(SAI_API_PORT, SAI_LOG_LEVEL_DEBUG)); + + EXPECT_EQ(SAI_LOG_LEVEL_DEBUG, sai.logGet(SAI_API_PORT)); + EXPECT_EQ(SAI_LOG_LEVEL_NOTICE, sai.logGet(SAI_API_SWITCH)); +} + TEST_F(VendorSaiTest, bulk_prefix_compression_entry) { sai_prefix_compression_entry_t *e = nullptr;