RFR: 8151436: Leaner ArrayAllocator and BitMaps
Kim Barrett
kim.barrett at oracle.com
Tue Mar 8 19:31:46 UTC 2016
> On Mar 8, 2016, at 5:33 AM, Stefan Karlsson <stefan.karlsson at oracle.com> 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
Looks good.
More information about the hotspot-dev
mailing list