commit b8fe077ca456c991ad0f18ce3e5fb905ea4fed34
author Connor O'Brien <connoro@google.com> Wed Sep 11 18:09:28 2019 -0700
committer Steven Moreland <smoreland@google.com> Mon Jan 13 16:08:09 2020 -0800
tree 9371d05f4b43ddd4e2f173d3fb3b168d2377ba6e
parent d65f2a02225b93255dc458cd435b34e2f45f586b

libtimeinstate: correctly handle devices with no boost freqs

initGlobals() should only fail if a cpufreq policy doesn't report any
frequencies at all; if scaling_available_frequencies reports some
freqs then it's fine if scaling_boost_frequencies is absent

Bug: 138317993
Test: libtimeinstate_test passes on a device with no boost freqs
Change-Id: I3fe0f2fbdf61888406a687ba4016f9bf7fe4796a
Signed-off-by: Connor O'Brien <connoro@google.com>
(cherry picked from commit e0b4d825be2551b94c3f3ea8510cff70ca7fc982)
Merged-In: I3fe0f2fbdf61888406a687ba4016f9bf7fe4796a

libs/cputimeinstate/cputimeinstate.cpp

diff --git a/libs/cputimeinstate/cputimeinstate.cpp b/libs/cputimeinstate/cputimeinstate.cpp
index 2d2536c..45fea85 100644
--- a/libs/cputimeinstate/cputimeinstate.cpp
+++ b/libs/cputimeinstate/cputimeinstate.cpp
@@ -110,9 +110,10 @@
             std::string path =
                     StringPrintf("%s/%s/scaling_%s_frequencies", basepath, policy.c_str(), name);
             auto nums = readNumbersFromFile(path);
-            if (!nums) return false;
+            if (!nums) continue;
             freqs.insert(freqs.end(), nums->begin(), nums->end());
         }
+        if (freqs.empty()) return false;
         std::sort(freqs.begin(), freqs.end());
         gPolicyFreqs.emplace_back(freqs);