RFR: 8016276: CMS concurrentMarkSweepGeneration contains lots of unnecessary allocation failure handling

David Lindholm david.lindholm at oracle.com
Wed Apr 1 08:11:30 UTC 2015


Thanks for the review Stefan!

/David

On 2015-04-01 10:10, Stefan Karlsson wrote:
>
> On 2015-04-01 09:13, David Lindholm wrote:
>> Hi,
>>
>> Please review this fix that removes unnecessary error handling from 
>> concurrentMarkSweepGeneration.cpp. The error handling is performed in 
>> AllocateHeap(), which will exit the vm on malloc failure. The error 
>> handling in concurrentMarkSweepGeneration.cpp could never been reached.
>>
>> Webrev: http://cr.openjdk.java.net/~david/JDK-8016276/webrev.00/
>
> Looks good.
>
> StefanK
>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8016276
>>
>> Tesing: Passed JPRT
>>
>>
>> Thanks,
>> David
>



More information about the hotspot-gc-dev mailing list