RFR: 8320239: add dynamic switch for JvmtiVTMSTransitionDisabler sync protocol [v3]

Patricio Chilano Mateo pchilanomate at openjdk.org
Thu Nov 30 16:56:04 UTC 2023


On Thu, 30 Nov 2023 07:34:23 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> This is an update for a performance/scalability enhancement.
>> 
>> The `JvmtiVTMSTransitionDisabler`sync protocol is on a performance critical path of the virtual threads mount state transitions (VTMS transitions). It has a noticeable performance overhead (about 10%) which contributes to the combined JVMTI performance overhead when Java apps are executed with loaded JVMTI agents.
>> 
>> Please, also see another/related performance issue which contributes around 70% of total performance overhead:
>> [8308614](https://bugs.openjdk.org/browse/JDK-8308614): Enabling JVMTI ClassLoad event slows down vthread creation by factor 10
>> 
>> Testing:
>>  - Ran mach5 tiers 1-6 with no regressions noticed.
>
> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
> 
>   review: addressed a race condition

Thanks Serguei, looks good to me.

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

Marked as reviewed by pchilanomate (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16688#pullrequestreview-1757941041


More information about the serviceability-dev mailing list