Code and Design Feedback request: 7114376: tune system dictionary size

Karen Kinnear karen.kinnear at oracle.com
Sun Jan 29 19:12:50 PST 2012


Final review please -

I reduced this to just add the command-line flag -XX:PredictedLoadedClassCount=#
and left the dynamic resizing for later.

webrev: http://cr.openjdk.java.net/~acorn/7114376.02/webrev/

Testing:
Manual testing of command-line options, CDS, scimark.
In parallel: JPRT, nsk, benchmarks

thanks,
Karen

p.s. the following files did not change between webrev.01 and webrev.02
agent/...
globals.hpp
vmStructs.cpp

On Jan 27, 2012, at 6:13 PM, Karen Kinnear wrote:

> Dan & David - 
> 
> Thank you both for the detailed reviews.
> 
> I will make the changes suggested - but I will put the code changes on hold. I think
> the best approach right now is to just add the command-line flag and then after
> we get the permanent generation changes, rethink the loaded class cache storage
> options - and do a whole lot more measurements than I am able to do in this timeframe -
> even thanks to the help I have had.
> 
> Details below,
> thanks,
> Karen



More information about the hotspot-dev mailing list