Review request: 8025096: Move the ChunkManager instances out of the VirtualSpaceLists

Stefan Karlsson stefan.karlsson at
Thu Sep 19 23:54:13 PDT 2013

New webrev based on Coleen's comments.

- Moved the _vs_list pointer out from SpaceManager
- Removed the unused _medium_chunk_bunch field.


On 9/19/13 9:48 PM, Stefan Karlsson wrote:
> In the fix for 'JDK-8024547 
> <>: MaxMetaspaceSize 
> should limit the committed memory used by the metaspaces' we need to 
> make a clearer distinction between when new memory is committed and 
> when memory is allocated inside already committed memory.
> Moving the ChunkManager, which handles the free chunk list, out of the 
> VirtualSpaceList, which handles the committed metaspace memory, is a 
> step in that direction.
> Changes:
> - Move the ChunkManager instances from VirtualSpaceList to SpaceManager.
> - Move the inc_container_count() into ChunkManager::free_chunks_get. 
> Now the Chunk manager calls dec_container_count when chunks are 
> inserted in the chunk free list and inc_container_count when chunks 
> are removed from the chunk free list.
> - Moved the ChunkManager::chunk_freelist_allocate() call out from the 
> VirtualSpaceList::get_new_chunk()
> thanks,
> StefanK

More information about the hotspot-dev mailing list