Skip to content

Commit e350a01

Browse files
authored
Reduce the test heap size for concurrent immix (#330)
The heap size for concurrent immix was set to 7x, as initially it was non-moving. Later we allowed moving for the full pause in concurrent Immix. This PR reduces the heap size for our concurrent immix tests.
1 parent e970e68 commit e350a01

File tree

3 files changed

+7
-14
lines changed

3 files changed

+7
-14
lines changed

.github/configs/large-heap.yml

Lines changed: 0 additions & 9 deletions
This file was deleted.

.github/configs/normal-heap.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ configs:
1010
- "jdk11-master|ms|s|fail_on_oom|tph|preserve|mmtk_gc-MarkSweep"
1111
- "jdk11-master|ms|s|fail_on_oom|tph|preserve|mmtk_gc-MarkCompact"
1212
- "jdk11-master|ms|s|fail_on_oom|tph|preserve|mmtk_gc-Compressor"
13+
- "jdk11-master|ms|s|fail_on_oom|tph|preserve|mmtk_gc-ConcurrentImmix"
1314

1415
# This will be expanded in CI when we run with the config. Keep a new line at the end.
1516
benchmarks:

.github/workflows/run-dacapo-chopin.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,9 @@ jobs:
1010
config-file: "normal-heap.yml"
1111
heap-factor: "2.5"
1212

13-
large-heap:
14-
uses: ./.github/workflows/run-dacapo-chopin-inner.yml
15-
with:
16-
config-file: "large-heap.yml"
17-
heap-factor: "7"
13+
# In case that we need to run some plans with a larger heap, we can use a different config file.
14+
# large-heap:
15+
# uses: ./.github/workflows/run-dacapo-chopin-inner.yml
16+
# with:
17+
# config-file: "large-heap.yml"
18+
# heap-factor: "7"

0 commit comments

Comments
 (0)