hg: jdk8/tl/jdk: 4243978: (ref) Race condition in Reference.enqueue(); ...

2011-10-31 Thread y . s . ramakrishna
Changeset: 5f2838744544 Author:ysr Date: 2011-10-31 17:38 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5f2838744544 4243978: (ref) Race condition in Reference.enqueue() 4268317: (ref) Reference.isEnqueued() can return true when instance not enqueued Summary: The reference

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

2011-09-06 Thread Y. S. Ramakrishna
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

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

2011-09-06 Thread Y. S. Ramakrishna
[i left the embedded inlines below for the cores-lib folk who may have missed part of the exchange because of non-membership of the posters.] Hi Stefan -- On 09/05/11 10:49, Stefan Karlsson wrote: Hi Ramki, On 09/05/2011 07:29 PM, Ramki Ramakrishna wrote: Hi Stefan -- Thanks for your

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

2011-09-06 Thread Y. S. Ramakrishna
Hi Mandy -- On 09/06/11 17:49, Mandy Chung wrote: Hi Ramki, I reviewed the JDK webrev: JDK webrev: http://cr.openjdk.java.net/~ysr/4965777/jdk/webrev/ The change looks good. I ran the JCK tests on solaris-i586 that I can reproduce the bug with JDK 7 and verify that they pass with your