RFR: 8247281: migrate ObjectMonitor::_object to OopStorage [v5]

Daniel D.Daugherty dcubed at openjdk.java.net
Mon Sep 14 21:15:00 UTC 2020


> This RFE is to migrate the following field to OopStorage:
> 
> class ObjectMonitor {
> <snip>
>   void* volatile _object; // backward object pointer - strong root
> 
> Unlike the previous patches in this series, there are a lot of collateral
> changes so this is not a trivial review. Sorry for the tedious parts of
> the review. Since Erik and I are both contributors to this patch, we
> would like at least 1 GC team reviewer and 1 Runtime team reviewer.
> 
> This changeset was tested with Mach5 Tier[1-3],4,5,6,7,8 testing
> along with JDK-8252980 and JDK-8252981. I also ran it through my
> inflation stress kit for 48 hours on my Linux-X64 machine.

Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:

  rkennke, coleenp, fisk CR - delete random assert() that knows too much about markWords.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/135/files
  - new: https://git.openjdk.java.net/jdk/pull/135/files/9fa2bed1..eeb9d761

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=135&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=135&range=03-04

  Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/135.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/135/head:pull/135

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


More information about the hotspot-runtime-dev mailing list