RFR: 8344032: InterpreterRuntime::verify_mdp() missing lock while printing MethodData on failure [v2]

David Holmes dholmes at openjdk.org
Thu Nov 14 08:02:30 UTC 2024


On Wed, 13 Nov 2024 19:53:04 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Added locking.  I injected some MethodData printing with logging to test this to verify that there wasn't a lock ranking assert.
>> 
>> Tested with tier1-4.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use ConditionalMutexLocker instead.

LGTM2

Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22082#pullrequestreview-2435294789


More information about the hotspot-dev mailing list