RFR(M): [Backport of] 8007036: G1: Too many old regions added to last mixed GC

Bengt Rutisson bengt.rutisson at oracle.com
Wed Apr 24 05:49:03 UTC 2013


Hi John,

Looks good!

Since I reviewed the original change I only looked at the diff of the 
patches this time. As you said the only difference is that 
finalize_cset() now takes the evacuation_info parameter. I guess once 
the event tracing is integrated into 8 this code will look the same 
again in hsx24 and hsx25.

Bengt

On 4/24/13 2:29 AM, John Cuthbertson wrote:
> Hi Everyone,
>
> Can I have review of the backport of the fix for 800007036 to hsx24? 
> One of the changes to g1CollectorPolicy.cpp did not apply cleanly 
> because of the additional parameter to finalize_cset. There was 
> actually no conflict in the change but importing the changeset 
> rejected that particular chunk.
>
> The webrev can be found at: 
> http://cr.openjdk.java.net/~johnc/8007036-hsx24-backport/webrev.0/
>
> Thanks,
>
> JohnC




More information about the hotspot-gc-dev mailing list