Skip to content

Commit

Permalink
tests/eas/acceptance: s/EXPECTED_BIG_BUSY_TIME_PCT/OFFLOAD_EXPECTED_B…
Browse files Browse the repository at this point in the history
…USY_TIME_PCT/

EXPECTED_BIG_BUSY_TIME_PCT is a misleading name.  It actually is the
expected busy time for cpus in the offload migration test.  Rename it
accordingly.
  • Loading branch information
Javi Merino committed Sep 16, 2016
1 parent b78de30 commit a43cb2f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion tests/eas/acceptance.config
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"STEP_HIGH_DCYCLE" : 50,
"EXPECTED_RESIDENCY_PCT" : 85,
"OFFLOAD_MIGRATION_MIGRATOR_DELAY": 1,
"EXPECTED_BIG_BUSY_TIME_PCT": 99,
"OFFLOAD_EXPECTED_BUSY_TIME_PCT": 99,
"TEST_CONF" : {
"modules" : [ "bl", "cpufreq" ],
"tools" : [ "rt-app" ],
Expand Down
4 changes: 2 additions & 2 deletions tests/eas/acceptance.py
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,7 @@ def test_big_cpus_fully_loaded(self):
self.env.target.bl.bigs,
window=window, percent=True)

expected_busy_time = EXPECTED_BIG_BUSY_TIME_PCT * \
expected_busy_time = OFFLOAD_EXPECTED_BUSY_TIME_PCT * \
big_cpus_left / num_big_cpus
msg = "Big tasks were not running on big cpus from {} to {}".format(
window[0], window[1])
Expand All @@ -483,7 +483,7 @@ def test_little_cpus_run_tasks(self):

window_len = window[1] - window[0]
expected_busy_time = window_len * num_offloaded_tasks * \
EXPECTED_BIG_BUSY_TIME_PCT / 100.
OFFLOAD_EXPECTED_BUSY_TIME_PCT / 100.
msg = "Little cpus did not pick up big tasks while big cpus were fully loaded"

self.assertGreater(busy_time, expected_busy_time, msg=msg)
Expand Down

0 comments on commit a43cb2f

Please sign in to comment.