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

(0.46) Disclaim Cold Code Cache #19581

Merged
merged 1 commit into from
May 31, 2024

Conversation

gita-omr
Copy link
Contributor

@gita-omr gita-omr commented May 30, 2024

  • use the same heuristcs for code cache disclaim as for data cache
  • disclaim starting from the cold code
  • move stack overflow outline instructions into the warm area to increase disclaim efficiency
  • code is enabled with -Xjit:enableCodeCacheDisclaiming

Port of #19397

- use the same heuristcs for code cache disclaim as for data cache
- disclaim starting from the cold code
- move stack overflow outline instructions into the warm area to
  increase disclaim efficiency
- code is enabled with -Xjit:enableCodeCacheDisclaiming
@gita-omr gita-omr requested a review from dsouzai as a code owner May 30, 2024 19:15
@gita-omr gita-omr changed the title Disclaim Cold Code Cache (0.46) Disclaim Cold Code Cache May 30, 2024
@pshipton pshipton requested a review from mpirvu May 30, 2024 20:35
@mpirvu
Copy link
Contributor

mpirvu commented May 31, 2024

@gita-omr Could you please confirm that all the prerequisites for this PR are already integrated in 0.46.0 branches?

@pshipton
Copy link
Member

I do confirm eclipse-omr/omr#7300 is included in 0.46

@mpirvu
Copy link
Contributor

mpirvu commented May 31, 2024

I have verified that the patch in this PR is identical to the original patch in #19397.
Merging into 0.46.0

@mpirvu mpirvu merged commit 714b75a into eclipse-openj9:v0.46.0-release May 31, 2024
2 checks passed
@gita-omr
Copy link
Contributor Author

@gita-omr Could you please confirm that all the prerequisites for this PR are already integrated in 0.46.0 branches?

Yes, all prerequisites are in. Essentially, it's just the new option in OMR.

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.

3 participants