sascha/devlib
1
0
Fork 0
mirror of https://github.com/ARM-software/devlib.git synced 2025-11-04 07:51:21 +00:00
Code Issues Releases Wiki Activity

module/cpufreq: Add userspace special case for use_governor()

Browse Source 
Frequencies are not considered as governor tunables by
list_governor_tunables(), so add a special case to restore userspace
frequency when using use_governor().

While at it, reword the internal "governor" variable to not clash
with the parameter of the same name.
This commit is contained in:
Valentin Schneider
2018-11-12 17:48:59 +00:00
committed by Marc Bonnici
parent 85e0fb08fe
commit 94c1339efd
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
devlib/module/cpufreq.py
View File

@@ -120,6 +120,12 @@ class CpufreqModule(Module):
prev_governors = {cpu : (self.get_governor(cpu), self.get_governor_tunables(cpu)) prev_governors = {cpu : (self.get_governor(cpu), self.get_governor_tunables(cpu))
for cpu in domains} for cpu in domains}
# Special case for userspace, frequency is not seen as a tunable
userspace_freqs = {}
for cpu, (prev_gov, _) in prev_governors.items():
if prev_gov == "userspace":
userspace_freqs[cpu] = self.get_frequency(cpu)
for cpu in domains: for cpu in domains:
self.set_governor(cpu, governor, **kwargs) self.set_governor(cpu, governor, **kwargs)
@@ -127,8 +133,10 @@ class CpufreqModule(Module):
yield yield
finally: finally:
for cpu, (governor, tunables) in prev_governors.items(): for cpu, (prev_gov, tunables) in prev_governors.items():
self.set_governor(cpu, governor, **tunables) self.set_governor(cpu, prev_gov, **tunables)
if prev_gov == "userspace":
self.set_frequency(cpu, userspace_freqs[cpu])
def list_governor_tunables(self, cpu): def list_governor_tunables(self, cpu):
"""Returns a list of tunables available for the governor on the specified CPU.""" """Returns a list of tunables available for the governor on the specified CPU."""