Re: RFR: 8247281: migrate ObjectMonitor::_object to OopStorage [v3]

2020-09-14 Thread Daniel D . Daugherty
On Mon, 14 Sep 2020 02:00:05 GMT, David Holmes wrote: >> Thanks for confirmation. > > From the spec I'm not clear on exactly what JVMTI_HEAP_REFERENCE_MONITOR is > intended to be. Serviceability folk should > be giving some input here though. I've taken a first pass at creating a CSR: JDK-82531

Re: RFR: 8247281: migrate ObjectMonitor::_object to OopStorage [v3]

2020-09-14 Thread Daniel D . Daugherty
On Mon, 14 Sep 2020 01:52:04 GMT, David Holmes wrote: >> Thanks for confirmation. > > I don't see anything in the HPROF format description that claims this is a > strong root. At a minimum this seems to be a > behavioural change that would warrant a CSR request. This also seems to be > somethin

Re: RFR: 8247281: migrate ObjectMonitor::_object to OopStorage [v3]

2020-09-14 Thread Daniel D . Daugherty
> This RFE is to migrate the following field to OopStorage: > > class ObjectMonitor { > > 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 tedi