summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Connor O'Brien <connoro@google.com> 2020-08-17 19:02:15 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2020-08-17 19:02:15 +0000
commit149b19d97bc1dea015ef92efc2096a115e52cd57 (patch)
tree1be35226fc9ec05f51a86b164687b0c55d53c35f
parent4dbc9223bbe76bd75dead5c509263e6e4002c579 (diff)
parentdee8b7398de739719f5022fab8017edff85aac1c (diff)
Merge "libtimeinstate: improve error handling in single-UID functions" am: 5be652dbf2 am: e30924539a am: 6b35867b33 am: dee8b7398d
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1401471 Change-Id: Ibc78393072da21f7ca3b8d01048be04130ab3954
-rw-r--r--libs/cputimeinstate/cputimeinstate.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/cputimeinstate/cputimeinstate.cpp b/libs/cputimeinstate/cputimeinstate.cpp
index 50f6289726..5e785b67c8 100644
--- a/libs/cputimeinstate/cputimeinstate.cpp
+++ b/libs/cputimeinstate/cputimeinstate.cpp
@@ -251,7 +251,7 @@ std::optional<std::vector<std::vector<uint64_t>>> getUidCpuFreqTimes(uint32_t ui
for (uint32_t i = 0; i <= (maxFreqCount - 1) / FREQS_PER_ENTRY; ++i) {
key.bucket = i;
if (findMapEntry(gTisMapFd, &key, vals.data())) {
- if (errno != ENOENT) return {};
+ if (errno != ENOENT || getFirstMapKey(gTisMapFd, &key)) return {};
continue;
}
@@ -362,7 +362,7 @@ std::optional<concurrent_time_t> getUidConcurrentTimes(uint32_t uid, bool retry)
time_key_t key = {.uid = uid};
for (key.bucket = 0; key.bucket <= (gNCpus - 1) / CPUS_PER_ENTRY; ++key.bucket) {
if (findMapEntry(gConcurrentMapFd, &key, vals.data())) {
- if (errno != ENOENT) return {};
+ if (errno != ENOENT || getFirstMapKey(gConcurrentMapFd, &key)) return {};
continue;
}
auto offset = key.bucket * CPUS_PER_ENTRY;