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
[email protected] 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 <[email protected]>
Date:
Mon, 05 Sep 2011 02:16:18 -0700
To:
"[email protected]" <[email protected]>,
[email protected]
To:
"[email protected]" <[email protected]>,
[email protected]
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
