Continue loop other port if not succeed

Signed-off-by: bingwang <wang.bing@microsoft.com>
sonic-net · Apr 24, 2022 · e3aa60f · e3aa60f
1 parent 05a55b6
commit e3aa60f
Showing 1 changed file with 17 additions and 21 deletions.
diff --git a/cfgmgr/buffermgr.cpp b/cfgmgr/buffermgr.cpp
@@ -497,32 +497,28 @@ void BufferMgr::doTask(Consumer &consumer)
 
                 if (m_speedLookup.count(port) != 0)
                 {
+                    // create/update profile for port
                     task_status = doSpeedUpdateTask(port, admin_up);
                 }
+            }
 
-                if (task_status != task_process_status::task_success)
-                {
+            switch (task_status)
+            {
+                case task_process_status::task_failed:
+                    SWSS_LOG_ERROR("Failed to process table update");
+                    return;
+                case task_process_status::task_need_retry:
+                    SWSS_LOG_INFO("Unable to process table update. Will retry...");
+                    ++it;
+                    break;
+                case task_process_status::task_invalid_entry:
+                    SWSS_LOG_ERROR("Failed to process invalid entry, drop it");
+                    it = consumer.m_toSync.erase(it);
+                    break;
+                default:
+                    it = consumer.m_toSync.erase(it);
                     break;
-                }
             }
         }
-
-        switch (task_status)
-        {
-            case task_process_status::task_failed:
-                SWSS_LOG_ERROR("Failed to process table update");
-                return;
-            case task_process_status::task_need_retry:
-                SWSS_LOG_INFO("Unable to process table update. Will retry...");
-                ++it;
-                break;
-            case task_process_status::task_invalid_entry:
-                SWSS_LOG_ERROR("Failed to process invalid entry, drop it");
-                it = consumer.m_toSync.erase(it);
-                break;
-            default:
-                it = consumer.m_toSync.erase(it);
-                break;
-        }
     }
 }