Make thread pool capacity configurable via YAML#202
Conversation
| UrmSettings::metaConfigs.mPluginCount = (uint32_t)std::stol(resultBuffer); | ||
|
|
||
| if(UrmSettings::metaConfigs.mDesiredThreadCount < 1) { | ||
| TYPELOGV(META_CONFIG_PARSE_FAILURE, "desiredThreadCount must be >= 1, setting to default value (5)"); |
There was a problem hiding this comment.
don't need these logs on 181, 186
| static ErrCode preAllocateWorkers() { | ||
| int32_t desiredThreadCapacity = UrmSettings::desiredThreadCount; | ||
| int32_t maxScalingCapacity = UrmSettings::maxScalingCapacity; | ||
| int32_t desiredThreadCapacity = UrmSettings::metaConfigs.mDesiredThreadCount; |
There was a problem hiding this comment.
Make these uint32_t and correct indentation
|
Tarun Balaji Nidiganti (@tnidigan) can you fix the merge conflict |
Signed-off-by: Tarun Balaji Nidiganti <tnidigan@qti.qualcomm.com>
|
Merge Check Failed: No CR Numbers Found Error: No Change Request numbers were found. Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests. |
3 similar comments
|
Merge Check Failed: No CR Numbers Found Error: No Change Request numbers were found. Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests. |
|
Merge Check Failed: No CR Numbers Found Error: No Change Request numbers were found. Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests. |
|
Merge Check Failed: No CR Numbers Found Error: No Change Request numbers were found. Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests. |
2 participants
Makes thread pool capacity configurable through YAML properties instead of hardcoded constants, enabling per-system tuning without recompilation.
Changes Made
Utils.h: Added property name constants
UrmSettings.h:
RestuneInit.cpp:
fetchMetaConfigs()