RFR(xxs): 8199432: metaspace: fix wrong comment and condition in SpaceManager::verify()
thomas.stuefe at gmail.com
Mon Mar 12 07:23:39 UTC 2018
may I please have sponsor/reviewers for this tiny fix:
Basically, the condition and the comment make no sense anymore.
They used to make sense: in an earlier version of the Metaspace ("6964458:
Reimplement class meta-data storage to use native memory") the Metablocks
inside a metachunk always had headers and therefore were walkable. So,
Metachunk::verify walked all the blocks - which was not possible if were
returned with Metaspace::deallocate() and added to the block dictionary.
This is not true anymore, now MetaBlocks generally do not have a header, so
MetaChunk::verify() does not walk them and is generally oblivious to
anything happening in the MetaChunk payload area. So, one can now always
call MetaChunk::verify(), regardless if MetaBlocks are in the block
dictionary or not.
More information about the hotspot-runtime-dev