Skip to content

JFR PhysicalMemory event fix-up#24315

Open
thallium wants to merge 1 commit into
eclipse-openj9:masterfrom
thallium:jfr-v2
Open

JFR PhysicalMemory event fix-up#24315
thallium wants to merge 1 commit into
eclipse-openj9:masterfrom
thallium:jfr-v2

Conversation

@thallium

@thallium thallium commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

In V2, respond to emit event request immediately instead of deferring to chunk generation. This allows multiple such events to be generated in one chunk.

Related: #24194

@thallium thallium force-pushed the jfr-v2 branch 4 times, most recently from 7dd0009 to bacb0ce Compare July 8, 2026 20:45
@thallium

thallium commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

@tajila please review

@tajila tajila self-requested a review July 9, 2026 14:58
@tajila tajila self-assigned this Jul 9, 2026
@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux jdk21,jdk17

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins compile win jdk17

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity.functional xlinux jdk17

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Please rebase these changes

In V2, respond to emit event request immediately instead of deferring to
chunk generation. This allows multiple such events to be generated in
one chunk.

Related: eclipse-openj9#24194

Signed-off-by: Gengchen Tuo <gengchen.tuo@ibm.com>
@thallium

Copy link
Copy Markdown
Contributor Author

Also added missing _threadAllocationStatisticsCount += 1; in ThreadAllocationStatistics event. and addressed comment #24216 (comment).

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux jdk21,jdk17

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins compile win jdk17

@thallium

Copy link
Copy Markdown
Contributor Author

I think the build failure on Windows was caused by overloaded writePhysicalMemoryEvent() so I changed name of one version.

@tajila

tajila commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

jenkins compile win jdk17

@tajila

tajila commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux jdk21,jdk17

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