Hi Ioi,
It looks good to me except for:
filemap.hpp:
313 assert(i >= 0 && i <= NUM_CDS_REGIONS, "invalid region");
Shouldn't the assert be:
assert(i >= 0 && i < NUM_CDS_REGIONS, "invalid region");
Thanks!
Jini.
On 8/21/2018 1:53 AM, Ioi Lam wrote:
Hi,
I've updated the webrev to merge with Calvin's change in the latest repo.
http://cr.openjdk.java.net/~iklam/jdk12/8209657-shared-FileMapHeader-decl.v02/
Thanks
- Ioi
On 8/17/2018 2:22 PM, Ioi Lam wrote:
[Resending to include bug number in e-mail subject line]
https://bugs.openjdk.java.net/browse/JDK-8209657
http://cr.openjdk.java.net/~iklam/jdk12/8209657-shared-FileMapHeader-decl.v01/
Summary:
The CDS FileMapHeader type was big, and was duplicated 4 times in our
sources.
I moved the parts that's common to HotSpot and Serviceability Agent
into a new
common header file, cds.h.
I also did various clean up in filemap.cpp/hpp:
- avoid using unwieldy nested types such as
FileMapInfo::FileMapHeader::space_info
- added convenience function space_at(), so you have
struct FileMapInfo::FileMapHeader::space_info* si =
&_header->_space[i];
=>
CDSFileMapRegion* si = space_at(i);
Testing:
hs tiers 1,2,3 on all supported platforms.