RFR: 8151436: Leaner ArrayAllocator and BitMaps

[Stefan Karlsson]

Thanks, Per.

StefanK

On 2016-03-08 13:35, Per Liden wrote:
> Looks good!
>
> /Per
>
> On 2016-03-08 11:33, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this patch to remove the state variables from
>> ArrayAllocator and make it an AllStatic class instead. The main
>> motivation for this patch is to cleanup the BitMaps data structure and
>> make the instances smaller.
>>
>> The patch builds on the fact that the current code invokes AllocateHeap
>> with the default value AllocFailEnum:EXIT_OOM, and therefore never
>> returns NULL. This means that use_malloc will never be reset and the
>> allocation strategy used (malloc or mmap) can be determined by simply
>> looking at the size input parameter. Instead of changing the code to use
>> AllocateHeap with AllocFailEnum::RETURN_NULL, I chose to keep the
>> current behavior so that we could cleanup and minimize the size of our
>> BitMaps.
>>
>> http://cr.openjdk.java.net/~stefank/8151436/webrev/
>> https://bugs.openjdk.java.net/browse/JDK-8151436
>>
>> Thanks,
>> StefanK