Remove TdiVirtualPortManager since its unused

Signed-off-by: Sabeel Ansari <sabeel.ansari@intel.com>

Author: 5abeel

stratum/hal/lib/tdi/CMakeLists.txt

@@ -1,6 +1,6 @@
 # Build file for //stratum/hal/lib/tdi
 #
-# Copyright 2022-2025 Intel Corporation
+# Copyright 2022-2024 Intel Corporation
 # SPDX-License-Identifier: Apache 2.0
 #
 
@@ -48,8 +48,6 @@ add_library(stratum_tdi_common_o OBJECT
     tdi_sde_wrapper.h
     tdi_table_manager.cc
     tdi_table_manager.h
-    tdi_virtual_port_manager.cc
-    tdi_virtual_port_manager.h
     utils.cc
     utils.h
 )

stratum/hal/lib/tdi/es2k/es2k_chassis_manager.h

@@ -19,7 +19,6 @@
 #include "stratum/hal/lib/common/writer_interface.h"
 #include "stratum/hal/lib/tdi/tdi_global_vars.h"
 #include "stratum/hal/lib/tdi/tdi_port_manager.h"
-#include "stratum/hal/lib/tdi/tdi_virtual_port_manager.h"
 #include "stratum/lib/channel/channel.h"
 
 namespace stratum {

stratum/hal/lib/tdi/es2k/es2k_virtual_port_manager.cc

@@ -44,6 +44,8 @@ namespace tdi {
 
 Es2kVirtualPortManager* Es2kVirtualPortManager::singleton_ = nullptr;
 
+ABSL_CONST_INIT absl::Mutex Es2kVirtualPortManager::init_lock_(absl::kConstInit);
+
 Es2kVirtualPortManager* Es2kVirtualPortManager::CreateSingleton() {
   absl::WriterMutexLock l(&init_lock_);
   if (!singleton_) {

stratum/hal/lib/tdi/es2k/es2k_virtual_port_manager.h

@@ -13,7 +13,6 @@
 #include "absl/synchronization/mutex.h"
 #include "stratum/hal/lib/tdi/tdi_fixed_function_manager.h"
 #include "stratum/hal/lib/tdi/tdi_sde_interface.h"
-#include "stratum/hal/lib/tdi/tdi_virtual_port_manager.h"
 
 // Suppress clang errors
 #undef LOCKS_EXCLUDED
@@ -23,7 +22,7 @@ namespace stratum {
 namespace hal {
 namespace tdi {
 
-class Es2kVirtualPortManager : public TdiVirtualPortManager {
+class Es2kVirtualPortManager {
  public:
   Es2kVirtualPortManager() {}
   virtual ~Es2kVirtualPortManager() {}
@@ -47,6 +46,11 @@ class Es2kVirtualPortManager : public TdiVirtualPortManager {
   static Es2kVirtualPortManager* GetSingleton() LOCKS_EXCLUDED(init_lock_);
 
  protected:
+  // RW mutex lock for protecting the singleton instance initialization and
+  // reading it back from other threads. Unlike other singleton classes, we
+  // use RW lock as we need the pointer to class to be returned.
+  static absl::Mutex init_lock_;
+
   // The singleton instance.
   static Es2kVirtualPortManager* singleton_ GUARDED_BY(init_lock_);