Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Heap size startup hints #3415

Merged
merged 1 commit into from
Jan 7, 2019
Merged

Heap size startup hints #3415

merged 1 commit into from
Jan 7, 2019

Conversation

amicic
Copy link
Contributor

@amicic amicic commented Jan 2, 2019

Some pre-req work to support Heap size startup hints feature in OpenJ9

  • swapped initial order of allocate and survivor in Nursery Physical
    Arena to be able to expand early
  • introduce new reasons for expansion, which would be reported by
    verbose GC during early startup
  • introduce two tuning parameters, that will control how aggressively
    the feature is used. heapSizeStatupHintWeightNewValue is for now set to
    0 (which effectively means the feature is disabled), but plan is to use
    more value of 0.8 or so (or remove historic averaging altogether), after
    some more testing is performed

Signed-off-by: Aleksandar Micic amicic@ca.ibm.com

@charliegracie
Copy link
Contributor

@genie-omr build all

Some pre-req work to support Heap size startup hints feature in OpenJ9
- swapped initial order of allocate and survivor in Nursery Physical
Arena to be able to expand early
- introduce new reasons for expansion, which would be reported by
verbose GC during early startup
- introduce two tuning parameters, that will control how aggressively
the feature is used. heapSizeStatupHintWeightNewValue is for now set to
0 (which effectively means the feature is disabled), but plan is to use
more value of 0.8 or so (or remove historic averaging altogether), after
some more testing is performed

Signed-off-by: Aleksandar Micic <amicic@ca.ibm.com>
@charliegracie
Copy link
Contributor

@genie-omr build all

@charliegracie charliegracie self-assigned this Jan 7, 2019
@charliegracie charliegracie merged commit 510a521 into eclipse:master Jan 7, 2019
@amicic amicic deleted the gchints branch February 24, 2020 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants