-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[GR-61425] Update labsjdk to 25+6-jvmci-b01 #10513
Conversation
PullRequest: graal/19758
Added tag jdk-25+6 for changeset dff7737
8ee9266
to
2db390d
Compare
throw VMError.shouldNotReachHere("Paths from the image build must not be embedded into the Native Image."); | ||
} | ||
|
||
@Alias | ||
static native SafePath getPathInProperty(String prop, String subPath); | ||
static native Target_jdk_jfr_internal_SecuritySupport_SafePath getPathInProperty(String prop, String subPath); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that SubstrateJVM.getDumpPath()
needs to now accomodate the case where we're on JDKLatest
. Right now it still unconditionally uses Target_jdk_jfr_internal_SecuritySupport.getPathInProperty
which won't exist on JDKLatest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @roberttoyonaga!
@christianhaeubl do you know the answer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@roberttoyonaga : good point, I pushed a fix.
Update labsjdk to 25+6-jvmci-b01
Adopted:
JDK-8347287 - JFR: Remove use of Security Manager - PR
JDK-8347129 - cpuset cgroups controller is required for no good reason - PR
JDK-8346460 - NotifyFramePop should return JVMTI_ERROR_DUPLICATE - PR
JDK-8346143 - add ClearAllFramePops function to speedup debugger single stepping in some cases - PR