RFR: 8151436: Leaner ArrayAllocator and BitMaps

Stefan Karlsson stefan.karlsson at oracle.com
Tue Mar 8 10:33:30 UTC 2016


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


More information about the hotspot-dev mailing list