On Thu, 9 Mar 2023 18:55:06 GMT, Patricio Chilano Mateo <pchilanom...@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 <pchilanom...@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
