Integrated: 8271293: Monitor class should use ThreadBlockInVMPreprocess

[Patricio Chilano Mateo]

On Tue, 3 Aug 2021 22:35:43 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Hi,
> 
> Please review the following small patch which changes the Monitor class to use the more appropriate ThreadBlockInVMPreprocess transition wrapper instead of ThreadBlockInVM. This allows to remove the embedded InFlightMutexRelease object from ThreadBlockInVM and to move it, along with the definition of the InFlightMutexRelease class, to mutex.cpp where they belong.
> 
> I also changed the default value of allow_suspend to false, even though more users set it to true, to make it consistent with the fact that ThreadBlockInVM doesn't process suspend requests. This avoids having to think twice when looking at a ThreadBlockInVM* object as to whether it processes suspend requests or not. Suspend requests are never processed unless explicitly allowed.
> 
> I also changed ThreadBlockInVM to be a typedef to avoid declaring a wrapper class of ThreadBlockInVMPreprocess. 
> 
> Testing in mach5 tiers 1-3.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: 62e72adf
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/62e72adfa7bcc8e2a70c1346f3051deb2488affc
Stats:     56 lines in 4 files changed: 15 ins; 23 del; 18 mod

8271293: Monitor class should use ThreadBlockInVMPreprocess

Reviewed-by: dholmes, dcubed

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

PR: https://git.openjdk.java.net/jdk/pull/4978