refactor(sysman): Changes in the Sysman Power Module

Removed the call to getProperties during the power handle creation

Related-To: NEO-10484

Signed-off-by: Bari, Pratik <pratik.bari@intel.com>
This commit is contained in:
Bari, Pratik
2024-06-13 12:04:47 +00:00
committed by Compute-Runtime-Automation
parent a19639530a
commit 6ecf5a088f
4 changed files with 30 additions and 22 deletions

View File

@@ -1,5 +1,5 @@
/*
* Copyright (C) 2023 Intel Corporation
* Copyright (C) 2023-2024 Intel Corporation
*
* SPDX-License-Identifier: MIT
*
@@ -18,7 +18,10 @@ namespace Sysman {
ze_result_t PowerImp::powerGetProperties(zes_power_properties_t *pProperties) {
ze_result_t result = ZE_RESULT_SUCCESS;
void *pNext = pProperties->pNext;
*pProperties = powerProperties;
result = pOsPower->getProperties(pProperties);
if (result != ZE_RESULT_SUCCESS) {
return result;
}
pProperties->pNext = pNext;
while (pNext) {
zes_power_ext_properties_t *pExtProps = reinterpret_cast<zes_power_ext_properties_t *>(pNext);
@@ -65,15 +68,13 @@ PowerImp::PowerImp(OsSysman *pOsSysman, ze_bool_t isSubDevice, uint32_t subDevic
pOsPower = OsPower::create(pOsSysman, isSubDevice, subDeviceId);
UNRECOVERABLE_IF(nullptr == pOsPower);
this->isCardPower = isSubDevice ? false : true;
init();
}
void PowerImp::init() {
if (pOsPower->isPowerModuleSupported()) {
pOsPower->getProperties(&powerProperties);
this->initSuccess = true;
this->isCardPower = powerProperties.onSubdevice ? false : true;
}
}

View File

@@ -1,5 +1,5 @@
/*
* Copyright (C) 2023 Intel Corporation
* Copyright (C) 2023-2024 Intel Corporation
*
* SPDX-License-Identifier: MIT
*
@@ -31,9 +31,6 @@ class PowerImp : public Power, NEO::NonCopyableOrMovableClass {
OsPower *pOsPower = nullptr;
void init();
private:
zes_power_properties_t powerProperties = {};
};
} // namespace Sysman
} // namespace L0