RFR: 8338007: [JVMCI] ResolvedJavaMethod.reprofile can crash ciMethodData [v2]

Vladimir Kozlov kvn at openjdk.org
Wed Oct 30 20:15:18 UTC 2024


On Wed, 30 Oct 2024 06:19:21 GMT, Tom Rodriguez <never at openjdk.org> wrote:

>> Graal unit testing uses ResolvedJavaMethod.reprofile to reset profiles between test but the current code rewrites the layout in a non-atomic way which can break other readers.  Instead perform the reinitialization at a safepoint which should protect all readers from seeing any transient initialization states.
>
> Tom Rodriguez has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments

Good.

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21746#pullrequestreview-2406185452


More information about the graal-dev mailing list