RFR (XS): 8146850, 8149850: Remove TraceHandleAllocation rather than converting to UL

Gerard Ziemski gerard.ziemski at oracle.com
Tue Feb 23 16:05:00 UTC 2016


Thank you for the review. Will update the copyright.

> On Feb 22, 2016, at 6:34 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> On 23/02/2016 4:35 AM, Gerard Ziemski wrote:
>> hi all,
>> 
>> Please review this XS fix, which:
>> 
>> - removes TraceJNIHandleAllocation (JDK-8146850)
> 
> That should be TraceHandleAllocation. :)
> 
>> - removes another 2 related flags, i.e. HandleAllocationLimit and TotalHandleAllocationLimit (JDK-8149850), for which I have filed compatibility request, and won’t push the fix is approved
>> 
>> bug 1 link at https://bugs.openjdk.java.net/browse/JDK-8146850
>> bug 2 link at https://bugs.openjdk.java.net/browse/JDK-8149850
> 
> Make sure you correctly reference both bugs when you commit the changeset.
> 
>> open webrev at http://cr.openjdk.java.net/~gziemski/8146850_rev0
> 
> Changes look fine - just need copyright update in handles.cpp.
> 
> Thanks,
> David
> 
>> compatibility request  approved for JDK-8146850, pending for JDK-8149850, passes JPRT hotspot and RBT hotspot_all
>> 
>> 
>> cheers
>> 



More information about the hotspot-runtime-dev mailing list