> 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: Minor changes/deletions to address kimbarrett and sspitsyn CR comments. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/135/files - new: https://git.openjdk.java.net/jdk/pull/135/files/eeb9d761..215084ac Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=135&range=05 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=135&range=04-05 Stats: 9 lines in 2 files changed: 2 ins; 7 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