Please review 6687581: Make CMS work with compressed oops

Coleen Phillimore - Sun Microsystems Coleen.Phillimore at Sun.COM
Thu May 22 10:30:15 PDT 2008


Please email your comments to me regarding this bug fix, described below:

Fixed 6687581: Make CMS work with compressed oops

Make FreeChunk read markword instead of LSB in _klass pointer to 
indicate that it's a FreeChunk for compressed oops.  Moved the size 
field first so that it is used consistently for 64 bits w/out compressed 
oops, and 32 bits to reduce ifdefs.

Also needed to change set_klass to not zero the gap since CMS uses this 
to copy objects.  Decoupled set_klass and set_klass_gap in the case of 
compressed oops.

Changed FreeChunk and associated code in SA.

Performance results looked very good(not provided). Thanks to jmasa and 
ramki for their analysis, discussion of the change and prereview.

Webrev:  http://webrev.invokedynamic.info/coleenp/6687581/
Fix verified: y
Testing:  nsk stress, sajdi and my sanity testlists with CMS+compressedoops.
          runthese
          refworkload/GCOld
Reviewed-by: <your name here>



More information about the hotspot-runtime-dev mailing list