RFR (S): 8027295: Free CSet takes ~50% of young pause time

Bengt Rutisson bengt.rutisson at oracle.com
Fri Mar 14 13:15:06 UTC 2014


Hi Thomas,

Thanks for fixing this. Looks good.

Bengt


On 2014-03-14 14:16, Thomas Schatzl wrote:
> Hi all,
>
>    Bengt offered to review the change: he had two (three) comments:
>
> - OtherTable::clear_locked() did not have a method body and was
> unreferenced.
>
> - in the declaration of the G1CollectedHeap::free_region() method (in
> both the hpp and cpp files), every parameter should have its own line.
>
> All fixed in:
>
> http://cr.openjdk.java.net/~tschatzl/8027295/webrev.5/
>
> Change to last version:
>
> http://cr.openjdk.java.net/~tschatzl/8027295/webrev.5.diff/
>
> Testing:
> JPRT
>
> Thomas
>




More information about the hotspot-gc-dev mailing list