Skip to content

Commit

Permalink
configs: Fix stats name in arm/fs_power.py
Browse files Browse the repository at this point in the history
In the config, there are stats having name changed:
- overall_misses -> overallMisses
- sim_seconds -> simSeconds
- overall_accesses -> overallAccesses

JIRA: https://gem5.atlassian.net/browse/GEM5-957

Signed-off-by: Hoa Nguyen <[email protected]>
Change-Id: I35faa72b12320e6b41833f601eb23604358b3d42
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44626
Reviewed-by: Jason Lowe-Power <[email protected]>
Reviewed-by: Giacomo Travaglini <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
Tested-by: kokoro <[email protected]>
  • Loading branch information
hnpl committed Apr 21, 2021
1 parent 8d5a8f0 commit 503b60e
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions configs/example/arm/fs_power.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@ def __init__(self, cpu_path, **kwargs):
# 2A per IPC, 3pA per cache miss
# and then convert to Watt
self.dyn = "voltage * (2 * {}.ipc + 3 * 0.000000001 * " \
"{}.dcache.overall_misses / sim_seconds)".format(cpu_path,
cpu_path)
"{}.dcache.overallMisses / simSeconds)".format(cpu_path,
cpu_path)
self.st = "4 * temp"

class CpuPowerOff(MathExprPowerModel):
Expand All @@ -72,10 +72,10 @@ def __init__(self, cpu_path, **kwargs):
class L2PowerOn(MathExprPowerModel):
def __init__(self, l2_path, **kwargs):
super(L2PowerOn, self).__init__(**kwargs)
# Example to report l2 Cache overall_accesses
# Example to report l2 Cache overallAccesses
# The estimated power is converted to Watt and will vary based
# on the size of the cache
self.dyn = "{}.overall_accesses * 0.000018000".format(l2_path)
self.dyn = "{}.overallAccesses * 0.000018000".format(l2_path)
self.st = "(voltage * 3)/10"

class L2PowerOff(MathExprPowerModel):
Expand Down

0 comments on commit 503b60e

Please sign in to comment.