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

Thomas Schatzl thomas.schatzl at oracle.com
Mon Mar 3 11:33:37 UTC 2014


Hi all,

On Wed, 2014-02-26 at 10:38 +0100, Stefan Karlsson wrote: 
>[...]
>Could you add a comment that you do this padding to avoid false
>sharing?
> 
> It would have been nice to be able to use the code in padded.hpp. See 
> PaddedArray::create_unfreeable(). Do you think it would be worth 
> extracting the padding code to a new class for arrays of arrays with 
> primitives?
> 

  as suggested, I made the padding a separate changeset, RFR is out.

This is a request to review the renmants of 8027295. Basically only the
"locked" parameter of the free_region() method.

Webrev:
http://cr.openjdk.java.net/~tschatzl/8027295/webrev.4/

CR:
https://bugs.openjdk.java.net/browse/JDK-8027295

Testing:
jprt

Thanks,
  Thomas





More information about the hotspot-gc-dev mailing list