Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(310)

Issue 1421463006: Tweak CPU support; list all available CPU frequencies. (Closed)

Created:
5 years, 1 month ago by M-A Ruel
Modified:
5 years, 1 month ago
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org
Base URL:
git@github.com:luci/luci-py.git@5_fixes
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Tweak CPU support; list all available CPU frequencies. This is only exposed on newer kernels, so extract the old values on older kernels. Use stricter checking in SetCPUSpeed(). R=stip@chromium.org BUG=chromium:525873 Committed: https://github.com/luci/luci-py/commit/9beaef81eb32a0e412ac9dfe1157f67b75fd570b

Patch Set 1 #

Patch Set 2 : Update smoke_test.py #

Patch Set 3 : Fix high_test.py #

Patch Set 4 : Remove freqs, it's noisy in swarming stats and already available #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -23 lines) Patch
M appengine/third_party/python-adb/adb/high.py View 1 2 3 5 chunks +29 lines, -20 lines 0 comments Download
M appengine/third_party/python-adb/high_test.py View 1 2 1 chunk +1 line, -1 line 0 comments Download
M appengine/third_party/python-adb/smoke_test.py View 1 2 3 1 chunk +16 lines, -2 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 13 (8 generated)
M-A Ruel
5 years, 1 month ago (2015-11-04 20:37:47 UTC) #1
ghost stip (do not use)
lgtm
5 years, 1 month ago (2015-11-04 21:05:10 UTC) #2
commit-bot: I haz the power
This CL has an open dependency (Issue 1424923006 Patch 60001). Please resolve the dependency and ...
5 years, 1 month ago (2015-11-04 21:09:03 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1421463006/140001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1421463006/140001
5 years, 1 month ago (2015-11-05 20:44:01 UTC) #12
commit-bot: I haz the power
5 years, 1 month ago (2015-11-05 20:45:07 UTC) #13
Message was sent while issue was closed.
Committed patchset #4 (id:140001) as
https://github.com/luci/luci-py/commit/9beaef81eb32a0e412ac9dfe1157f67b75fd570b

Powered by Google App Engine
This is Rietveld 408576698