[Fwd: request for review (M): 4965777 GC changes to support use of discovered field for pending references]

Y. S. Ramakrishna y.s.ramakrishna at oracle.com
Tue Sep 6 18:45:35 UTC 2011


Just so that there is no confusion. These changes by themselves
are not intended to fix 4243978 and  4268317. They just fix 4965777 --
that is make GC and the ReferenceHandler thread use the discovered
field for linking the elements of the pending list.

I am assuming that 4243978 and 4268317 may need some more changes,
but those are beyond the scope of this review request.

-- ramki

On 09/06/11 10:27, Y. S. Ramakrishna wrote:
> 
> [Resent, as the original is still awaiting moderator approval -- i was 
> not a member
> when the attached email was sent. Please include 
> hotspot-gc-dev at openjdk.java.net in
> your responses, and visit the archives of that email list for emails 
> related to
> this thread. Thanks -- ramki.]
> 
> ------------------------------------------------------------------------
> 
> Subject:
> request for review (M): 4965777 GC changes to support use of discovered 
> field for pending references
> From:
> Ramki Ramakrishna <y.s.ramakrishna at oracle.com>
> Date:
> Mon, 05 Sep 2011 02:16:18 -0700
> To:
> "hotspot-gc-dev at openjdk.java.net" <hotspot-gc-dev at openjdk.java.net>, 
> core-libs-dev at openjdk.java.net
> 
> To:
> "hotspot-gc-dev at openjdk.java.net" <hotspot-gc-dev at openjdk.java.net>, 
> core-libs-dev at openjdk.java.net
> 
> 
> 
> CR's 4243978 and  4268317 have proposed to use the discovered field of
> a java.lang.ref.reference object to link the objects in the pending list.
> This requires changes in both GC and in the reference handler code.
> The JVM adaptsso as to allow it to run either inside a newer JDK which 
> uses the
> discovered field to link the references or inside an older JDK which 
> uses the
> next field for that purpose. Although the JDK part of the changes are
> also being submitted for review here, that part will be integrated as a 
> separate
> CR and changeset into the JDK repo following the integration of the JVM 
> changes
> into an appropriate version of the HotSpot (express) repos.
> 
>   JVM webrev:  http://cr.openjdk.java.net/~ysr/4965777/hotspot/webrev/
>   JDK webrev:   http://cr.openjdk.java.net/~ysr/4965777/jdk/webrev/
> 
> Many thanks to Mandy Chung and John Coomes for advice and for
> earlier reviews; and to Mandy for additional testing help.
> 
> Thanks for any other reviews or comments.
> -- ramki



More information about the hotspot-gc-dev mailing list