Skip to content

Commit 8d6432f

Browse files
authored
Enforce logical to physical mapping in classifier fast-path (#235)
--- Signed-off-by: Kartik Nema <kartnema@qti.qualcomm.com>
1 parent d528798 commit 8d6432f

3 files changed

Lines changed: 3 additions & 5 deletions

File tree

contextual-classifier/ContextualClassifier.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,7 @@ void ContextualClassifier::ApplyActions(uint32_t sigId,
419419
this->mCurrRestuneHandles.push_back(request->getHandle());
420420

421421
// fast path to Request Queue
422-
submitResProvisionRequest(request, true);
422+
submitResProvisionRequest(request, false);
423423

424424
} else {
425425
Request::cleanUpRequest(request);
@@ -593,7 +593,7 @@ void ContextualClassifier::configureAppSignals(pid_t incomingPID,
593593
if(request->getResourcesCount() > 0) {
594594
// fast path to Request Queue
595595
this->mCurrRestuneHandles.push_back(request->getHandle());
596-
submitResProvisionRequest(request, true);
596+
submitResProvisionRequest(request, false);
597597

598598
} else {
599599
Request::cleanUpRequest(request);

resource-tuner/core/Include/RestuneInternal.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,4 @@ size_t submitPropGetRequest(const std::string& propName,
4747
std::string& result,
4848
const std::string& defVal);
4949

50-
ErrCode translateToPhysicalIDs(Resource* resource);
51-
5250
#endif

resource-tuner/core/RequestHandler.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ static int8_t performPhysicalMapping(int32_t& coreValue, int32_t& clusterValue)
5656
return true;
5757
}
5858

59-
ErrCode translateToPhysicalIDs(Resource* resource) {
59+
static ErrCode translateToPhysicalIDs(Resource* resource) {
6060
ResConfInfo* rConf = ResourceRegistry::getInstance()->getResConf(resource->getResCode());
6161
switch(rConf->mApplyType) {
6262
case ResourceApplyType::APPLY_CORE: {

0 commit comments

Comments
 (0)