From 509cc066e0288a40f32d6facb8f0cacc866b3b46 Mon Sep 17 00:00:00 2001 From: Mateusz Jablonski Date: Fri, 4 Jul 2025 12:36:26 +0000 Subject: [PATCH] refactor: use NEO::SysCalls::close wrapper for close function call Signed-off-by: Mateusz Jablonski --- .../sysman/source/api/engine/linux/sysman_os_engine_imp.cpp | 4 ++-- .../source/api/vf_management/linux/sysman_os_vf_imp.cpp | 6 +++--- .../kmd_interface/sysman_kmd_interface_i915_prelim.cpp | 4 ++-- .../shared/linux/kmd_interface/sysman_kmd_interface_xe.cpp | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/level_zero/sysman/source/api/engine/linux/sysman_os_engine_imp.cpp b/level_zero/sysman/source/api/engine/linux/sysman_os_engine_imp.cpp index f7f739c6d8..cc456727f3 100644 --- a/level_zero/sysman/source/api/engine/linux/sysman_os_engine_imp.cpp +++ b/level_zero/sysman/source/api/engine/linux/sysman_os_engine_imp.cpp @@ -24,7 +24,7 @@ namespace Sysman { void LinuxEngineImp::cleanup() { for (auto &fdPair : fdList) { DEBUG_BREAK_IF(fdPair.first < 0); - close(static_cast(fdPair.first)); + NEO::SysCalls::close(static_cast(fdPair.first)); } fdList.clear(); } @@ -167,7 +167,7 @@ static void closeFds(std::vector &fdList) { if (!fdList.empty()) { for (auto &fd : fdList) { DEBUG_BREAK_IF(fd < 0); - close(static_cast(fd)); + NEO::SysCalls::close(static_cast(fd)); } fdList.clear(); } diff --git a/level_zero/sysman/source/api/vf_management/linux/sysman_os_vf_imp.cpp b/level_zero/sysman/source/api/vf_management/linux/sysman_os_vf_imp.cpp index 12bf39f150..13bdb567ac 100644 --- a/level_zero/sysman/source/api/vf_management/linux/sysman_os_vf_imp.cpp +++ b/level_zero/sysman/source/api/vf_management/linux/sysman_os_vf_imp.cpp @@ -157,7 +157,7 @@ ze_result_t LinuxVfImp::vfEngineDataInit() { int64_t totalTicksFd = pPmuInterface->pmuInterfaceOpen(totalTicksConfig, static_cast(busyTicksFd), PERF_FORMAT_TOTAL_TIME_ENABLED | PERF_FORMAT_GROUP); if (totalTicksFd < 0) { NEO::printDebugString(NEO::debugManager.flags.PrintDebugMessages.get(), stderr, "Error@ %s(): Could not open Total Ticks Handle and returning error:0x%x \n", __FUNCTION__, ZE_RESULT_ERROR_UNSUPPORTED_FEATURE); - close(static_cast(busyTicksFd)); + NEO::SysCalls::close(static_cast(busyTicksFd)); cleanup(); return ZE_RESULT_ERROR_UNSUPPORTED_FEATURE; } @@ -239,9 +239,9 @@ LinuxVfImp::LinuxVfImp( void LinuxVfImp::cleanup() { for (const auto &pEngineUtilsData : pEngineUtils) { DEBUG_BREAK_IF(pEngineUtilsData.busyTicksFd < 0); - close(static_cast(pEngineUtilsData.busyTicksFd)); + NEO::SysCalls::close(static_cast(pEngineUtilsData.busyTicksFd)); DEBUG_BREAK_IF(pEngineUtilsData.totalTicksFd < 0); - close(static_cast(pEngineUtilsData.totalTicksFd)); + NEO::SysCalls::close(static_cast(pEngineUtilsData.totalTicksFd)); } pEngineUtils.clear(); } diff --git a/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_i915_prelim.cpp b/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_i915_prelim.cpp index c03083824b..28d306a607 100644 --- a/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_i915_prelim.cpp +++ b/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_i915_prelim.cpp @@ -149,7 +149,7 @@ ze_result_t SysmanKmdInterfaceI915Prelim::getEngineActivityFdListAndConfigPair(z fd[1] = pPmuInterface->pmuInterfaceOpen(configPair.second, static_cast(fd[0]), PERF_FORMAT_TOTAL_TIME_ENABLED | PERF_FORMAT_GROUP); if (fd[1] < 0) { NEO::printDebugString(NEO::debugManager.flags.PrintDebugMessages.get(), stderr, "Error@ %s(): Could not open Total Active Ticks Handle \n", __FUNCTION__); - close(static_cast(fd[0])); + NEO::SysCalls::close(static_cast(fd[0])); return checkErrorNumberAndReturnStatus(); } @@ -247,4 +247,4 @@ void SysmanKmdInterfaceI915Prelim::setSysmanDeviceDirName(const bool isIntegrate } } // namespace Sysman -} // namespace L0 \ No newline at end of file +} // namespace L0 diff --git a/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_xe.cpp b/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_xe.cpp index 1060a95500..9a5cf29f0a 100644 --- a/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_xe.cpp +++ b/level_zero/sysman/source/shared/linux/kmd_interface/sysman_kmd_interface_xe.cpp @@ -180,7 +180,7 @@ ze_result_t SysmanKmdInterfaceXe::getEngineActivityFdListAndConfigPair(zes_engin fd[1] = pPmuInterface->pmuInterfaceOpen(configPair.second, static_cast(fd[0]), PERF_FORMAT_TOTAL_TIME_ENABLED | PERF_FORMAT_GROUP); if (fd[1] < 0) { NEO::printDebugString(NEO::debugManager.flags.PrintDebugMessages.get(), stderr, "Error@ %s(): Could not open Total Active Ticks Handle \n", __FUNCTION__); - close(static_cast(fd[0])); + NEO::SysCalls::close(static_cast(fd[0])); return checkErrorNumberAndReturnStatus(); }