Request for review: 8015903: Format issue with -XX:+PrintAdaptiveSizePolicy on JDK8
jesper.wilhelmsson at oracle.com
Fri Jun 7 14:37:26 UTC 2013
When I fixed the same thing yesterday I noticed that the method above,
PSAdaptiveSizePolicy::compute_survivor_space_size_and_threshold, writes its cr()
to tty instead of gclog_or_tty. Do you mind fixing that in the same change since
it is slightly related?
And I don't know how to stress this enough, you are not the first one who did
it, never ever work on a bug without assigning it to yourself, or talk to the
assigned engineer first. Having two engineers working on the same bug is not
efficient use of our time.
Vladimir Kempik skrev 7/6/13 3:46 PM:
> Hi all,
> Could I have a couple of reviews for this change?
> With next option -XX:+PrintAdapriveSizePolicy, java log looks like this:
> AdaptiveSizePolicy::update_averages: survived: 479272 promoted: 251648 overflow:
> falseAdaptiveSizeStart: 4,136 collection: 16
> avg_survived_padded_avg: 2223544,750000 avg_promoted_padded_avg:
> 1300947,750000 avg_pretenured_padded_avg: 286597,000000 tenuring_thresh: 1
> target_size: 1835008
> and there is no space between overflow: false and AdaptiveSizeStart:
> The patch adds line break after overflow: false.
More information about the hotspot-gc-dev