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