Integrated: 8367982: Unify ObjectSynchronizer and LightweightSynchronizer
Fredrik Bredberg
fbredberg at openjdk.org
Thu Nov 6 12:19:21 UTC 2025
On Tue, 21 Oct 2025 13:11:45 GMT, Fredrik Bredberg <fbredberg at openjdk.org> wrote:
> This is the last PR in a series of PRs (see: [JDK-8344261](https://bugs.openjdk.org/browse/JDK-8344261)) to obsolete the LockingMode flag and related code.
>
> The main focus is to to unify `ObjectSynchronizer` and `LightweightSynchronizer`.
> There used to be a number of "dispatch functions" to redirect calls depending on the setting of the `LockingMode` flag.
> Since we now only have lightweight locking, there is no longer any need for those dispatch functions, so I removed them.
> To remove the dispatch functions I renamed the corresponding lightweight functions and call them directly.
> This ultimately led me to remove "lightweight" from the function names and go back to "fast" instead, just to avoid having some with, and some without the "lightweight" part of the name.
>
> This PR also include a small simplification of `ObjectSynchronizer::FastHashCode`.
>
> Tested tier1-7 (on supported platforms) without seeing any problems that can be traced to this code change.
> All other platforms (`arm`, `ppc`, `riscv`, `s390`) has been sanity checked using QEMU.
This pull request has now been integrated.
Changeset: 3930b1d4
Author: Fredrik Bredberg <fbredberg at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/3930b1d4ddda9d56d0fb3626421283c72f4ad7f9
Stats: 2981 lines in 80 files changed: 1263 ins; 1429 del; 289 mod
8367982: Unify ObjectSynchronizer and LightweightSynchronizer
Reviewed-by: pchilanomate, coleenp
-------------
PR: https://git.openjdk.org/jdk/pull/27915
More information about the hotspot-dev
mailing list