Skip to content

Commit 6429d9f

Browse files
[GR-59039] Fix energy regex patterns
PullRequest: mx/1943
2 parents 8cac862 + 7de8758 commit 6429d9f

File tree

4 files changed

+17
-12
lines changed

4 files changed

+17
-12
lines changed

ci/common.jsonnet

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,11 @@ local common_json = import "../common.json";
142142

143143
common_catch_files: {
144144
catch_files+: [
145+
# There are additional catch_files-like patterns in buildbot/graal/catcher.py for:
146+
# * hs_err_pid*.log files
147+
# * Dumping IGV graphs to (?P<filename>.+(\.gv\.xml|\.bgv))
148+
# * CFGPrinter: Output to file (?P<filename>.*compilations-.+\.cfg)
149+
# There are defined there for efficiency reasons.
145150
# Keep in sync with jdk.graal.compiler.debug.StandardPathUtilitiesProvider#DIAGNOSTIC_OUTPUT_DIRECTORY_MESSAGE_REGEXP
146151
"Graal diagnostic output saved in '(?P<filename>[^']+)'",
147152
# Keep in sync with jdk.graal.compiler.debug.DebugContext#DUMP_FILE_MESSAGE_REGEXP

common.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
"COMMENT.jdks": "When adding or removing JDKs keep in sync with JDKs in ci/common.jsonnet",
1010
"jdks": {
11-
"galahad-jdk": {"name": "jpg-jdk", "version": "25", "build_id": "jdk-25+26-3319", "platformspecific": true, "extrabundles": ["static-libs"]},
11+
"galahad-jdk": {"name": "jpg-jdk", "version": "25", "build_id": "jdk-26+2-83", "platformspecific": true, "extrabundles": ["static-libs"]},
1212

1313
"oraclejdk17": {"name": "jpg-jdk", "version": "17.0.7", "build_id": "jdk-17.0.7+8", "platformspecific": true, "extrabundles": ["static-libs"]},
1414
"labsjdk-ce-17": {"name": "labsjdk", "version": "ce-17.0.7+4-jvmci-23.1-b02", "platformspecific": true },
@@ -45,13 +45,13 @@
4545

4646
"oraclejdk24": {"name": "jpg-jdk", "version": "24", "build_id": "jdk-24.0.1+9", "platformspecific": true, "extrabundles": ["static-libs"]},
4747

48-
"oraclejdk-latest": {"name": "jpg-jdk", "version": "25", "build_id": "jdk-26+1", "platformspecific": true, "extrabundles": ["static-libs"]},
49-
"labsjdk-ce-latest": {"name": "labsjdk", "version": "ce-26+1-jvmci-b01", "platformspecific": true },
50-
"labsjdk-ce-latestDebug": {"name": "labsjdk", "version": "ce-26+1-jvmci-b01-debug", "platformspecific": true },
51-
"labsjdk-ce-latest-llvm": {"name": "labsjdk", "version": "ce-26+1-jvmci-b01-sulong", "platformspecific": true },
52-
"labsjdk-ee-latest": {"name": "labsjdk", "version": "ee-26+1-jvmci-b01", "platformspecific": true },
53-
"labsjdk-ee-latestDebug": {"name": "labsjdk", "version": "ee-26+1-jvmci-b01-debug", "platformspecific": true },
54-
"labsjdk-ee-latest-llvm": {"name": "labsjdk", "version": "ee-26+1-jvmci-b01-sulong", "platformspecific": true }
48+
"oraclejdk-latest": {"name": "jpg-jdk", "version": "26", "build_id": "jdk-26+2", "platformspecific": true, "extrabundles": ["static-libs"]},
49+
"labsjdk-ce-latest": {"name": "labsjdk", "version": "ce-26+2-jvmci-b01", "platformspecific": true },
50+
"labsjdk-ce-latestDebug": {"name": "labsjdk", "version": "ce-26+2-jvmci-b01-debug", "platformspecific": true },
51+
"labsjdk-ce-latest-llvm": {"name": "labsjdk", "version": "ce-26+2-jvmci-b01-sulong", "platformspecific": true },
52+
"labsjdk-ee-latest": {"name": "labsjdk", "version": "ee-26+2-jvmci-b01", "platformspecific": true },
53+
"labsjdk-ee-latestDebug": {"name": "labsjdk", "version": "ee-26+2-jvmci-b01-debug", "platformspecific": true },
54+
"labsjdk-ee-latest-llvm": {"name": "labsjdk", "version": "ee-26+2-jvmci-b01-sulong", "platformspecific": true }
5555
},
5656

5757
"eclipse": {

src/mx/_impl/mx_benchmark.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3507,9 +3507,9 @@ def get_rules(self, bmSuiteArgs):
35073507
rules = []
35083508

35093509
energy_patterns = {
3510-
"total-machine-energy": r"Total system energy consumption (including idle): (?P<total_machine_energy>[0-9]*\.?[0-9]+) Joules",
3511-
"total-compute-energy": r"Total energy used by the benchmark (excluding idle): (?P<total_compute_energy>[0-9]*\.?[0-9]+) Joules",
3512-
"avg-machine-power": r"Average system power (including idle): (?P<avg_machine_power>[0-9]*\.?[0-9]+) Watts",
3510+
"total-machine-energy": r"Total system energy consumption \(including idle\): (?P<total_machine_energy>[0-9]*\.?[0-9]+) Joules",
3511+
"total-compute-energy": r"Total energy used by the benchmark \(excluding idle\): (?P<total_compute_energy>[0-9]*\.?[0-9]+) Joules",
3512+
"avg-machine-power": r"Average system power \(including idle\): (?P<avg_machine_power>[0-9]*\.?[0-9]+) Watts",
35133513
"avg-compute-power": r"Average power used by benchmark: (?P<avg_compute_power>[0-9]*\.?[0-9]+) Watts"
35143514
}
35153515
for metric_name, pattern in energy_patterns.items():

src/mx/mx_version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
# The version must be updated for every PR (checked in CI) and the comment should reflect the PR's issue
2-
version = "7.56.0" # GR-63250: Fetch-jdk: Add architecture to JDK path
2+
version = "7.57.0" # GR-59039 Fix energy regex pattern

0 commit comments

Comments
 (0)