powerstats: Add INT state residency stats
Bug: 168833648 Test: dumpsys android.hardware.power.stats.IPowerStats/default Change-Id: Iecfb2ec05566a0d432a39904adddd442751c44e6
This commit is contained in:
parent
e5a49a9ae3
commit
6899dcc219
3 changed files with 172 additions and 0 deletions
111
powerstats/DevfreqStateResidencyDataProvider.cpp
Normal file
111
powerstats/DevfreqStateResidencyDataProvider.cpp
Normal file
|
@ -0,0 +1,111 @@
|
||||||
|
/*
|
||||||
|
* Copyright (C) 2021 The Android Open Source Project
|
||||||
|
*
|
||||||
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
* you may not use this file except in compliance with the License.
|
||||||
|
* You may obtain a copy of the License at
|
||||||
|
*
|
||||||
|
* http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
*
|
||||||
|
* Unless required by applicable law or agreed to in writing, software
|
||||||
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
* See the License for the specific language governing permissions and
|
||||||
|
* limitations under the License.
|
||||||
|
*/
|
||||||
|
#include "DevfreqStateResidencyDataProvider.h"
|
||||||
|
|
||||||
|
#include <android-base/logging.h>
|
||||||
|
|
||||||
|
static const std::string nameSuffix = "-DVFS";
|
||||||
|
static const std::string pathSuffix = "/time_in_state";
|
||||||
|
|
||||||
|
namespace aidl {
|
||||||
|
namespace android {
|
||||||
|
namespace hardware {
|
||||||
|
namespace power {
|
||||||
|
namespace stats {
|
||||||
|
|
||||||
|
DevfreqStateResidencyDataProvider::DevfreqStateResidencyDataProvider(const std::string& name,
|
||||||
|
const std::string& path) : mName(name + nameSuffix), mPath(path + pathSuffix) {}
|
||||||
|
|
||||||
|
bool DevfreqStateResidencyDataProvider::extractNum(const char *str, char **str_end, int base,
|
||||||
|
int64_t* num) {
|
||||||
|
// errno can be set to any non-zero value by a library function call
|
||||||
|
// regardless of whether there was an error, so it needs to be cleared
|
||||||
|
// in order to check the error set by strtoll
|
||||||
|
errno = 0;
|
||||||
|
*num = std::strtoll(str, str_end, base);
|
||||||
|
return (errno != ERANGE);
|
||||||
|
}
|
||||||
|
|
||||||
|
std::vector<std::pair<int64_t, int64_t>> DevfreqStateResidencyDataProvider::parseTimeInState() {
|
||||||
|
// Using FILE* instead of std::ifstream for performance reasons
|
||||||
|
std::unique_ptr<FILE, decltype(&fclose)> fp(fopen(mPath.c_str(), "r"), fclose);
|
||||||
|
if (!fp) {
|
||||||
|
PLOG(ERROR) << "Failed to open file " << mPath;
|
||||||
|
return {};
|
||||||
|
}
|
||||||
|
|
||||||
|
std::vector<std::pair<int64_t, int64_t>> timeInState;
|
||||||
|
|
||||||
|
char *line = nullptr;
|
||||||
|
size_t len = 0;
|
||||||
|
while (getline(&line, &len, fp.get()) != -1) {
|
||||||
|
char* pEnd;
|
||||||
|
int64_t frequencyHz, totalTimeMs;
|
||||||
|
if (!extractNum(line, &pEnd, 10, &frequencyHz) ||
|
||||||
|
!extractNum(pEnd, &pEnd, 10, &totalTimeMs)) {
|
||||||
|
PLOG(ERROR) << "Failed to parse " << mPath;
|
||||||
|
free(line);
|
||||||
|
return {};
|
||||||
|
}
|
||||||
|
|
||||||
|
timeInState.push_back({frequencyHz, totalTimeMs});
|
||||||
|
}
|
||||||
|
|
||||||
|
free(line);
|
||||||
|
return timeInState;
|
||||||
|
}
|
||||||
|
|
||||||
|
bool DevfreqStateResidencyDataProvider::getStateResidencies(
|
||||||
|
std::unordered_map<std::string, std::vector<StateResidency>> *residencies) {
|
||||||
|
std::vector<std::pair<int64_t, int64_t>> timeInState = parseTimeInState();
|
||||||
|
|
||||||
|
if (timeInState.empty()) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
int32_t id = 0;
|
||||||
|
std::vector<StateResidency> stateResidencies;
|
||||||
|
for (const auto[frequencyHz, totalTimeMs] : timeInState) {
|
||||||
|
StateResidency s = {.id = id++, .totalTimeInStateMs = totalTimeMs};
|
||||||
|
stateResidencies.push_back(s);
|
||||||
|
}
|
||||||
|
|
||||||
|
residencies->emplace(mName, stateResidencies);
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
std::unordered_map<std::string, std::vector<State>> DevfreqStateResidencyDataProvider::getInfo() {
|
||||||
|
std::vector<std::pair<int64_t, int64_t>> timeInState = parseTimeInState();
|
||||||
|
|
||||||
|
if (timeInState.empty()) {
|
||||||
|
return {};
|
||||||
|
}
|
||||||
|
|
||||||
|
int32_t id = 0;
|
||||||
|
std::vector<State> states;
|
||||||
|
for (const auto[frequencyHz, totalTimeMs] : timeInState) {
|
||||||
|
State s = {.id = id++, .name = std::to_string(frequencyHz / 1000) + "KHz"};
|
||||||
|
states.push_back(s);
|
||||||
|
}
|
||||||
|
|
||||||
|
return {{mName, states}};
|
||||||
|
}
|
||||||
|
|
||||||
|
} // namespace stats
|
||||||
|
} // namespace power
|
||||||
|
} // namespace hardware
|
||||||
|
} // namespace android
|
||||||
|
} // namespace aidl
|
53
powerstats/DevfreqStateResidencyDataProvider.h
Normal file
53
powerstats/DevfreqStateResidencyDataProvider.h
Normal file
|
@ -0,0 +1,53 @@
|
||||||
|
/*
|
||||||
|
* Copyright (C) 2021 The Android Open Source Project
|
||||||
|
*
|
||||||
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
* you may not use this file except in compliance with the License.
|
||||||
|
* You may obtain a copy of the License at
|
||||||
|
*
|
||||||
|
* http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
*
|
||||||
|
* Unless required by applicable law or agreed to in writing, software
|
||||||
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
* See the License for the specific language governing permissions and
|
||||||
|
* limitations under the License.
|
||||||
|
*/
|
||||||
|
#pragma once
|
||||||
|
|
||||||
|
#include <PowerStatsAidl.h>
|
||||||
|
|
||||||
|
namespace aidl {
|
||||||
|
namespace android {
|
||||||
|
namespace hardware {
|
||||||
|
namespace power {
|
||||||
|
namespace stats {
|
||||||
|
|
||||||
|
class DevfreqStateResidencyDataProvider : public PowerStats::IStateResidencyDataProvider {
|
||||||
|
public:
|
||||||
|
DevfreqStateResidencyDataProvider(const std::string& name, const std::string& path);
|
||||||
|
~DevfreqStateResidencyDataProvider() = default;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* See IStateResidencyDataProvider::getStateResidencies
|
||||||
|
*/
|
||||||
|
bool getStateResidencies(
|
||||||
|
std::unordered_map<std::string, std::vector<StateResidency>> *residencies) override;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* See IStateResidencyDataProvider::getInfo
|
||||||
|
*/
|
||||||
|
std::unordered_map<std::string, std::vector<State>> getInfo() override;
|
||||||
|
|
||||||
|
private:
|
||||||
|
bool extractNum(const char *str, char **str_end, int base, int64_t* num);
|
||||||
|
std::vector<std::pair<int64_t, int64_t>> parseTimeInState();
|
||||||
|
const std::string mName;
|
||||||
|
const std::string mPath;
|
||||||
|
};
|
||||||
|
|
||||||
|
} // namespace stats
|
||||||
|
} // namespace power
|
||||||
|
} // namespace hardware
|
||||||
|
} // namespace android
|
||||||
|
} // namespace aidl
|
|
@ -17,6 +17,7 @@
|
||||||
#include <PowerStatsAidl.h>
|
#include <PowerStatsAidl.h>
|
||||||
#include <Gs101CommonDataProviders.h>
|
#include <Gs101CommonDataProviders.h>
|
||||||
#include "AocStateResidencyDataProvider.h"
|
#include "AocStateResidencyDataProvider.h"
|
||||||
|
#include "DevfreqStateResidencyDataProvider.h"
|
||||||
#include "DvfsStateResidencyDataProvider.h"
|
#include "DvfsStateResidencyDataProvider.h"
|
||||||
#include "UfsStateResidencyDataProvider.h"
|
#include "UfsStateResidencyDataProvider.h"
|
||||||
#include <dataproviders/GenericStateResidencyDataProvider.h>
|
#include <dataproviders/GenericStateResidencyDataProvider.h>
|
||||||
|
@ -32,6 +33,7 @@
|
||||||
#include <log/log.h>
|
#include <log/log.h>
|
||||||
|
|
||||||
using aidl::android::hardware::power::stats::AocStateResidencyDataProvider;
|
using aidl::android::hardware::power::stats::AocStateResidencyDataProvider;
|
||||||
|
using aidl::android::hardware::power::stats::DevfreqStateResidencyDataProvider;
|
||||||
using aidl::android::hardware::power::stats::DvfsStateResidencyDataProvider;
|
using aidl::android::hardware::power::stats::DvfsStateResidencyDataProvider;
|
||||||
using aidl::android::hardware::power::stats::UfsStateResidencyDataProvider;
|
using aidl::android::hardware::power::stats::UfsStateResidencyDataProvider;
|
||||||
using aidl::android::hardware::power::stats::EnergyConsumerType;
|
using aidl::android::hardware::power::stats::EnergyConsumerType;
|
||||||
|
@ -667,6 +669,11 @@ void addPowerDomains(std::shared_ptr<PowerStats> p) {
|
||||||
"/sys/devices/platform/acpm_stats/pd_stats", cfgs));
|
"/sys/devices/platform/acpm_stats/pd_stats", cfgs));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void addDevfreq(std::shared_ptr<PowerStats> p) {
|
||||||
|
p->addStateResidencyDataProvider(std::make_unique<DevfreqStateResidencyDataProvider>(
|
||||||
|
"INT", "/sys/devices/platform/17000020.devfreq_int/devfreq/17000020.devfreq_int"));
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Unlike other data providers, which source power entity state residency data from the kernel,
|
* Unlike other data providers, which source power entity state residency data from the kernel,
|
||||||
* this data provider acts as a general-purpose channel for state residency data providers
|
* this data provider acts as a general-purpose channel for state residency data providers
|
||||||
|
@ -700,6 +707,7 @@ void addGs101CommonDataProviders(std::shared_ptr<PowerStats> p) {
|
||||||
addWifi(p);
|
addWifi(p);
|
||||||
addUfs(p);
|
addUfs(p);
|
||||||
addPowerDomains(p);
|
addPowerDomains(p);
|
||||||
|
addDevfreq(p);
|
||||||
|
|
||||||
// TODO (b/181070764) (b/182941084):
|
// TODO (b/181070764) (b/182941084):
|
||||||
// Remove this when Wifi/BT energy consumption models are available or revert before ship
|
// Remove this when Wifi/BT energy consumption models are available or revert before ship
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue