Integrated: 8303908: Add missing check in VTMS_transition_disable_for_all() for suspend mode

Patricio Chilano Mateo pchilanomate at openjdk.org
Mon Mar 13 20:18:03 UTC 2023


On Thu, 9 Mar 2023 18:55:06 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Please review this small fix. A suspender is a JvmtiVTMSTransitionDisabler monopolist, meaning VTMS_transition_disable_for_all() should not return while there is any active jvmtiVTMSTransitionDisabler. The code though is checking for active "all-disablers" but it's missing the check for active "single disablers".
> I attached a simple reproducer to the bug which I used to test the patch. Not sure if it was worth adding a test so the patch contains just the fix.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: a8f662ec
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a8f662ecb2cf13ba7fa499b9a9150da4318306a8
Stats:     3 lines in 1 file changed: 1 ins; 0 del; 2 mod

8303908: Add missing check in VTMS_transition_disable_for_all() for suspend mode

Reviewed-by: sspitsyn, dholmes

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

PR: https://git.openjdk.org/jdk/pull/12956


More information about the serviceability-dev mailing list