Request for Review (s) - 7012980: PSOldGen is increased if there is no space in Metaspace
tom.benson at oracle.com
Wed Jul 8 14:17:10 UTC 2015
Looks good to me as well. One thing I noticed is that this will also
prevent growth when GC cause is _gc_locker, which I think would have
happened before. I think that's OK, and possibly even good, since the
real reason for the GC is lost, but just noting that it is a difference.
On 7/8/2015 6:59 AM, Thomas Schatzl wrote:
> Hi Jon,
> On Thu, 2015-06-18 at 09:02 -0700, Jon Masamitsu wrote:
>> A recent integration collided with my patch so I had to merge
>> and redo the webrev. Previously in addition to the fix there was
>> a clean up that introduce a method is_system_gc(). The need
>> for is_system_gc() was overridden by the already added
>> New webrev
> looks good.
> One minor nit I can see is the naming of "should_update_promo_stats" - I
> am not sure why it is has "promo" in the name.
> The other, with the "eden" in the name can be immediately associated
> with young gen, the other not so much imo.
> Maybe there is a better name of it, but maybe it fits because other
> variables related to allocation in the old/tenured gen also have "promo"
> in their name. So it is probably best to keep it for uniformity.
More information about the hotspot-gc-dev