Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: openjdk/jdk
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: tschatzl/jdk
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: submit/8350621-code-cache-mgmt-hang
Choose a head ref
  • 3 commits
  • 7 files changed
  • 1 contributor

Commits on Jul 4, 2025

  1. 8350621

    Hi all,
    
      please review this change to avoid CodeCache triggered GCs temporarily being ignored.
    
    In particular, G1 does not make sure when its `collect()` method is called during a
    concurrent cycle, that a `Remark` pause that does code unloading etc. actually occurs
    after that request. This makes it so that some internal flag is not reset appropriately,
    stuck until the next code unloading (caused by e.g. a regular concurrent cycle being
    triggered).
    
    Testing: tier1-5
    
    Thanks,
      Thomas
    tschatzl committed Jul 4, 2025
    Configuration menu
    Copy the full SHA
    9240022 View commit details
    Browse the repository at this point in the history

Commits on Jul 8, 2025

  1. * some whitespace fixes

    tschatzl committed Jul 8, 2025
    Configuration menu
    Copy the full SHA
    348a789 View commit details
    Browse the repository at this point in the history

Commits on Jul 10, 2025

  1. * kbarret review:

      - restructure code in `try_collect_concurrently`
      - fix asserts - they only worked in the test because of whitebox being active
      - fix comments
    * threalmdoerr review:
      - fix test to be more lenient
    tschatzl committed Jul 10, 2025
    Configuration menu
    Copy the full SHA
    36b023f View commit details
    Browse the repository at this point in the history
Loading