[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