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

David Lindholm david.lindholm at oracle.com
Thu Apr 2 10:39:25 UTC 2015


Jon,

Thanks for the review!


/David

On 2015-04-01 20:44, Jon Masamitsu wrote:
> Changes look good.
>
> Reviewed.
>
> Jon
>
> On 4/1/2015 12:13 AM, 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/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8016276
>>
>> Tesing: Passed JPRT
>>
>>
>> Thanks,
>> David
>



More information about the hotspot-gc-dev mailing list