RFR: 8269295: Verification time before/after young collection only covers parts of the verification

Thomas Schatzl tschatzl at openjdk.java.net
Fri Jul 9 15:27:02 UTC 2021


  another mess-up of mine related to dependent merge-requests: after trying to integrate I assumed everything was fine but the bot did not agree - after the push of the base change there were merge errors. I did not notice that and just deleted the branch.
Then I tried to resurrect it, but it did not work, after the manual merge there were tons of changes. Since I'm too afraid to mess something up, I just made a new PR. Apologies for all the reviewers to be required to look at this again. :(
It looks like I'm now "healed" from dependent PRs...

Nothing changed, really - here's the original PR request text:

can I have reviews for this change that makes "Verify Before/After GC" contain all young collection time instead only a (large) part.

This change also removed some unnecessary and unused return values. They can certainly be re-added if necessary.

Testing: tier1



Commit messages:
 - Merge branch 'master' of https://git.openjdk.java.net/jdk into 8269295-verify-time-only-covers-parts
 - Initial implementation
 - Fix young gc verify stages
 - Initial implementation

Changes: https://git.openjdk.java.net/jdk/pull/4745/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4745&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8269295
  Stats: 28 lines in 3 files changed: 12 ins; 12 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4745.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4745/head:pull/4745

PR: https://git.openjdk.java.net/jdk/pull/4745

More information about the hotspot-gc-dev mailing list