Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(v0.29.1-release) JVMTI trigger VMStart event according to can_generate_early_vmstart #13747

Merged

Conversation

JasonFengJ9
Copy link
Member

Moved TRIGGER_J9HOOK_JAVA_BASE_LOADED to standardInit() after initializeAttachedThread() for mainThread, which is the earliest point to run an agent such as YourKit Java profiler agent;
Fixed processEvent() for J9HOOK_JAVA_BASE_LOADED;
When can_generate_early_vmstart capability is specified, VMStart event is triggered in the jvmtiHookModuleSystemStarted handler, otherwise in jvmtiHookVMStarted after JavaVM initialization;
Refactoring the functions affected, replaced JAVA_SPEC_VERSION 9 with 11.

Cherry-pick #13737

Proposing this to v0.29.1-release based on

Signed-off-by: Jason Feng fengj@ca.ibm.com

@pshipton
Copy link
Member

Pls fix the copyright check failure in stdinit.c

Moved TRIGGER_J9HOOK_JAVA_BASE_LOADED to standardInit() after
initializeAttachedThread() for mainThread, which is the earliest point
to run an agent such as YourKit Java profiler agent;
Fixed processEvent() for J9HOOK_JAVA_BASE_LOADED;
When can_generate_early_vmstart capability is specified, VMStart event
is triggered in the jvmtiHookModuleSystemStarted handler, otherwise in
jvmtiHookVMStarted after JavaVM initialization;
Refactoring the functions affected, replaced JAVA_SPEC_VERSION 9 with
11.

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
@JasonFengJ9
Copy link
Member Author

Pls fix the copyright check failure in stdinit.c

Fixed.

@pshipton pshipton merged commit ed73caf into eclipse-openj9:v0.29.1-release Oct 21, 2021
@JasonFengJ9 JasonFengJ9 deleted the earlyvmstartV0.29.1 branch October 21, 2021 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants