Skip to content

[ZGC] Add support for heap capacity, nMethod and relocation summary#426

Merged
kcpeppe merged 3 commits intomicrosoft:mainfrom
j-bahr:gen-zgc-fixes3
Apr 10, 2025
Merged

[ZGC] Add support for heap capacity, nMethod and relocation summary#426
kcpeppe merged 3 commits intomicrosoft:mainfrom
j-bahr:gen-zgc-fixes3

Conversation

@j-bahr
Copy link
Copy Markdown
Collaborator

@j-bahr j-bahr commented Mar 30, 2025

This PR includes 3 commits to add support for more of the missing info level logging output for Generational ZGC cycles. It includes the following:

  1. Support for heap capacity measurements, Min, Max, Soft Max
  2. Support for nMethod Tracking, registered and unregistered
  3. Support for Relocation information including Page Summary for Small, Medium and Large pages as well as an Age Table with a summary broken down by generation, Eden, Survivor 1-14.

j-bahr added 3 commits March 29, 2025 17:02
Adds support for heap capacity summary. Adds fixes to ensure that temporary long[]'s and booleans are reset between phases to ensure no data is carried over between cycles.
Adds support for nMethod tracking in generational ZGC young and old generations.
This commit adds support for all the information for gc,reloc stats. This includes Small, Medium and Large page stats, bytes forward and the age table for all the pages, including Eden and Survivor generations.

The Small, Medium and Large table stats are directly accessible on the ZGC cycle, where the page age summaries for Eden and all Survivor generations are included as a variable sized list. As far as I can tell this list can be as long at 14.
@j-bahr j-bahr requested review from dsgrieve, karianna and kcpeppe March 31, 2025 15:04
@kcpeppe
Copy link
Copy Markdown
Contributor

kcpeppe commented Mar 31, 2025

Looks great and thank you for taking this on. Is it possible to add a test data to gctoolkit-testdata and add tests to ZGCParserTest, ZGCParserRulesTest? Unfortunately there isn't the equivalent of UnifiedG1GCParserTest for ZGC. This should be added but not as part of this PR.

@j-bahr
Copy link
Copy Markdown
Collaborator Author

j-bahr commented Apr 1, 2025

@kcpeppe, Adding test data is something on my to-do list. I plan to to add testing for Generational ZGC in a follow up PR.

@kcpeppe kcpeppe merged commit 0df38d5 into microsoft:main Apr 10, 2025
8 checks passed
@karianna karianna mentioned this pull request Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants