[jdk25u] RFR: 8359472: JVM crashes when attaching a dynamic agent before JVMTI_PHASE_LIVE [v2]

[Goetz Lindenmaier]

On Thu, 6 Nov 2025 11:32:53 GMT, Francesco Andreuzzi <fandreuzzi at openjdk.org> wrote:

>> This PR contains a clean backport of JDK-8359472. JDK-8370730 is a follow-up which resolves a small problem in the test.
>
> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Backport b166b0d0826435c7965f3f11e8f3cec1392e1e01

Yes, the idea of saying it is "clean" is that it needed no reviewer.  Nevertheless, the information that you stacked two clean changes is helpful for the reviewer. So you should mention this in the PR text, which is directed at the reviewer, but not in the approval request comment.  I read in the JBS issue fix request comment that it is clean, then I look at the PR and find the "clean" label missing.  Now I need to research what is going on.  Many people claim a change is clean even after they edited something...

-------------

PR Comment: https://git.openjdk.org/jdk25u/pull/340#issuecomment-3585292159