Skip to content

null initial ClassValueMap.cacheArray#9

Closed
plevart wants to merge 1 commit intoopenjdk:masterfrom
plevart:null_initial_ClassValue_cacheArray
Closed

null initial ClassValueMap.cacheArray#9
plevart wants to merge 1 commit intoopenjdk:masterfrom
plevart:null_initial_ClassValue_cacheArray

Conversation

@plevart
Copy link
Copy Markdown
Contributor

@plevart plevart commented Aug 15, 2020

Version 2 of null initial ClassValueMap.cacheArray

@bridgekeeper
Copy link
Copy Markdown

bridgekeeper Bot commented Aug 15, 2020

Welcome to the OpenJDK organization on GitHub!

This repository is currently a read-only git mirror of the official Mercurial repository (located at https://hg.openjdk.java.net/). As such, we are not currently accepting pull requests here. If you would like to contribute to the OpenJDK project, please see https://openjdk.java.net/contribute/ on how to proceed.

This pull request will be automatically closed.

@bridgekeeper bridgekeeper Bot closed this Aug 15, 2020
@plevart plevart deleted the null_initial_ClassValue_cacheArray branch September 25, 2020 16:18
JornVernee referenced this pull request in JornVernee/jdk Nov 16, 2020
VladimirKempik added a commit to VladimirKempik/jdk that referenced this pull request Feb 26, 2021
JimLaskey pushed a commit to JimLaskey/jdk that referenced this pull request Nov 17, 2022
caojoshua pushed a commit to caojoshua/jdk that referenced this pull request Mar 21, 2023
After materialization, we used to replace all appearances of the old cooked
oop with the new cooked oop along JVMState. This isn't correct if the
old cooked oop is alive in the map of an ancester.

When C2 encounters a real call, it needs to aggregate debug edges from
all ancestors. If we replace the old cook oop in materization before, we
may end up with a bad graph. The late control of the new cooked oop is
higher than its early control in dominator tree.

Co-authored-by: Xin Liu <xxinliu@amazon.com>
robehn pushed a commit to robehn/jdk that referenced this pull request Aug 15, 2023
openjdk-notifier Bot pushed a commit that referenced this pull request Aug 2, 2024
biboudis added a commit to biboudis/jdk that referenced this pull request May 8, 2025
* Introduce Deconstructor <: MemberPattern <: Executable

* Fix MemberPattern.setAccessible

* Retain signature of unreflectDeconstructor
snake66 pushed a commit to snake66/jdk that referenced this pull request Oct 14, 2025
Update bsd-port with changes to restore OpenBSD support and include changes from the jdku24-freebsd branch.
mo-beck added a commit to mo-beck/jdk that referenced this pull request Jan 2, 2026
Implement reset_free_region_timestamps() method that resets ALL free region
timestamps after GC operations. This prevents time-based uncommit from
interfering with GC-based heap sizing decisions.

The problem: Regions that were already free before GC retain their old
timestamps. When time-based evaluation runs, these regions appear idle
even though GC just evaluated them, causing too many regions to be uncommitted.

The fix: After GC completes (both young and full), reset ALL free region
timestamps to give them a fresh baseline. This ensures time-based uncommit
only considers regions that became idle AFTER the most recent GC.

Addresses critical issue openjdk#9 and related issue openjdk#29 from Thomas review
mo-beck added a commit to mo-beck/jdk that referenced this pull request Jan 10, 2026
Implement reset_free_region_timestamps() method that resets ALL free region
timestamps after GC operations. This prevents time-based uncommit from
interfering with GC-based heap sizing decisions.

The problem: Regions that were already free before GC retain their old
timestamps. When time-based evaluation runs, these regions appear idle
even though GC just evaluated them, causing too many regions to be uncommitted.

The fix: After GC completes (both young and full), reset ALL free region
timestamps to give them a fresh baseline. This ensures time-based uncommit
only considers regions that became idle AFTER the most recent GC.

Addresses critical issue openjdk#9 and related issue openjdk#29 from Thomas review
mo-beck added a commit to mo-beck/jdk that referenced this pull request Jan 24, 2026
…penjdk#10, openjdk#12: comment updates and default-initialize _last_access_timestamp
mo-beck added a commit to mo-beck/jdk that referenced this pull request Jan 24, 2026
…penjdk#10, openjdk#12: comment updates and default-initialize _last_access_timestamp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant