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

David Lindholm david.lindholm at oracle.com
Wed Apr 1 09:22:51 UTC 2015


Thanks for the review Per.


/David

On 2015-04-01 11:10, Per Liden wrote:
> Looks good!
>
> /Per
>
> 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/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8016276
>>
>> Tesing: Passed JPRT
>>
>>
>> Thanks,
>> David



More information about the hotspot-gc-dev mailing list