Integrated: 8225631: Consider replacing muxAcquire/Release with PlatformMonitor

[David Holmes]

On Fri, 13 Nov 2020 00:51:02 GMT, David Holmes <dholmes at openjdk.org> wrote:

> This RFE was filed a while ago to see if we could get rid of `muxAcquire/release` to reduce overall code complexity. Initially some usages seemed to suffer some slight performance loss when I tried this. However, since then all but one usage have been removed and the final case, the `gInflationLock`'s in `ObjectSynchronizer read_stable_mark` does not suffer any performance degradation. It can also use `PlatformMutex` rather than `PlatformMonitor`.
> 
> On the plus side we remove a large chunk of complex synchronization code, we delete a `ParkEvent` per thread, and an int field per `ParkEvent` - which is good for footprint.
> 
> Testing:
> - GH actions
> - mach5 tiers 1-3
> - selected performance testing (guided by that used for JDK-8253064)
> 
> Thanks,
> David

This pull request has now been integrated.

Changeset: 99eac535
Author:    David Holmes <dholmes at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/99eac535
Stats:     195 lines in 6 files changed: 12 ins; 169 del; 14 mod

8225631: Consider replacing muxAcquire/Release with PlatformMonitor

Reviewed-by: coleenp, dcubed, kbarrett

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

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