Defer Sysman Temperature Module Initialization

With this change, init for sysman Temperature API would not be done
during zeInit.
Rather init and thereby Temperature API handle creation would be done
only when user explicitly requests to enumerate handles
using zesDeviceEnumTemperatureSensors.

Modification to temperature ULTs

Related-To: LOCI-3127

Signed-off-by: Kulkarni, Ashwin Kumar <ashwin.kumar.kulkarni@intel.com>

Author: AshwinKumarKulkarni

level_zero/tools/source/sysman/sysman_imp.cpp

@@ -103,9 +103,6 @@ ze_result_t SysmanDeviceImp::init() {
     if (pFabricPortHandleContext) {
         pFabricPortHandleContext->init();
     }
-    if (pTempHandleContext) {
-        pTempHandleContext->init(deviceHandles);
-    }
     if (pPci) {
         pPci->init();
     }

level_zero/tools/source/sysman/temperature/temperature.cpp

@@ -1,12 +1,13 @@
 /*
- * Copyright (C) 2020-2021 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
  */
 
 #include "shared/source/helpers/basic_math.h"
 
+#include "level_zero/tools/source/sysman/os_sysman.h"
 #include "level_zero/tools/source/sysman/temperature/temperature_imp.h"
 
 namespace L0 {
@@ -35,6 +36,9 @@ void TemperatureHandleContext::init(std::vector<ze_device_handle_t> &deviceHandl
 }
 
 ze_result_t TemperatureHandleContext::temperatureGet(uint32_t *pCount, zes_temp_handle_t *phTemperature) {
+    std::call_once(initTemperatureOnce, [this]() {
+        this->init(pOsSysman->getDeviceHandles());
+    });
     uint32_t handleListSize = static_cast<uint32_t>(handleList.size());
     uint32_t numToCopy = std::min(*pCount, handleListSize);
     if (0 == *pCount || *pCount > handleListSize) {

level_zero/tools/source/sysman/temperature/temperature.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020-2021 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -8,6 +8,7 @@
 #pragma once
 #include <level_zero/zes_api.h>
 
+#include <mutex>
 #include <vector>
 
 struct _zes_temp_handle_t {
@@ -45,6 +46,7 @@ struct TemperatureHandleContext {
 
   private:
     void createHandle(const ze_device_handle_t &deviceHandle, zes_temp_sensors_t type);
+    std::once_flag initTemperatureOnce;
 };
 
-} // namespace L0
+} // namespace L0

level_zero/tools/test/unit_tests/sources/sysman/temperature/linux/CMakeLists.txt

@@ -6,12 +6,15 @@
 
 set(L0_TESTS_TOOLS_SYSMAN_TEMPERATURE_LINUX
     ${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt
-    ${CMAKE_CURRENT_SOURCE_DIR}${BRANCH_DIR_SUFFIX}test_zes_temperature.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/mock_sysfs_temperature.h
 )
 
-if((NEO_ENABLE_i915_PRELIM_DETECTION) AND ("${BRANCH_TYPE}" STREQUAL ""))
-  list(REMOVE_ITEM L0_TESTS_TOOLS_SYSMAN_TEMPERATURE_LINUX
+if(NEO_ENABLE_i915_PRELIM_DETECTION)
+  list(APPEND L0_TESTS_TOOLS_SYSMAN_TEMPERATURE_LINUX
+       ${CMAKE_CURRENT_SOURCE_DIR}/test_zes_temperature_prelim.cpp
+  )
+else()
+  list(APPEND L0_TESTS_TOOLS_SYSMAN_TEMPERATURE_LINUX
        ${CMAKE_CURRENT_SOURCE_DIR}/test_zes_temperature.cpp
   )
 endif()

level_zero/tools/test/unit_tests/sources/sysman/temperature/linux/test_zes_temperature.cpp

@@ -114,7 +114,7 @@ class SysmanMultiDeviceTemperatureFixture : public SysmanMultiDeviceFixture {
             pLinuxSysmanImp->mapOfSubDeviceIdToPmtObject.emplace(deviceProperties.subdeviceId, pPmt);
         }
 
-        pSysmanDeviceImp->pTempHandleContext->init(deviceHandles);
+        getTempHandles(0);
     }
     void TearDown() override {
         if (!sysmanUltsEnable) {
@@ -230,7 +230,7 @@ class SysmanDeviceTemperatureFixture : public SysmanDeviceFixture {
             pLinuxSysmanImp->mapOfSubDeviceIdToPmtObject.emplace(deviceProperties.subdeviceId, pPmt);
         }
 
-        pSysmanDeviceImp->pTempHandleContext->init(deviceHandles);
+        getTempHandles(0);
     }
     void TearDown() override {
         if (!sysmanUltsEnable) {