RFR(S): 8189425: Minor updates in support of closed changes
    Erik Gahlin 
    erik.gahlin at oracle.com
       
    Wed Oct 18 20:04:54 UTC 2017
    
    
  
Hi David,
> Hi Erik,
>
> On 18/10/2017 12:23 PM, Erik Gahlin wrote:
>> Hi,
>>
>> Could I have a review of this change that will adjust an assertion and 
>
> Can you explain the adjustment please.
We have closed code that modifies the mark word and then changes it back 
during a safepoint. When the mark word is modified, we reuse GC 
infrastructure that run into the assert. If we change the assert to 
ignore checking that the mark word is NULL, we don't run into the problem.
>
>> remove a lock associated with JFR.
>
I forgot to modify the header file, see updated webrev.
http://cr.openjdk.java.net/~egahlin/8189425_1/
I  also made a change to GrowableArray, the insert_sorted method now 
takes a const.
Thanks
Erik
> That bit is fine :)
>
> Thanks,
> David
>
>> Webrev:
>> http://cr.openjdk.java.net/~egahlin/8189425_0
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8189425
>>
>> Thanks
>> Erik
>>
>>
    
    
More information about the serviceability-dev
mailing list