hg: jdk7/hotspot-rt/hotspot: 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent

2011-01-10 Thread keith . mcguigan
Changeset: 7246a374a9f2 Author:kamg Date: 2011-01-10 17:14 -0500 URL: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/7246a374a9f2 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS Revi

hg: jdk7/tl/jdk: 7005608: diamond conversion of JCA and crypto providers

2011-01-10 Thread stuart . marks
Changeset: 2a0ff59928de Author:smarks Date: 2011-01-10 17:06 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/2a0ff59928de 7005608: diamond conversion of JCA and crypto providers Reviewed-by: wetmore ! src/share/classes/java/security/Security.java ! src/share/classes/sun/secur

Re: request for review (S): 6814943: getcpool001 catches more than one JvmtiThreadState problem

2011-01-10 Thread Keith McGuigan
On Jan 10, 2011, at 7:07 PM, David Holmes wrote: Hi Keith, Keith McGuigan said the following on 01/11/11 03:36: This closes a race condition hole between JvmtiThreadState::state_for_while_locked() and ~JavaThread(). Without this, the state_for_while_locked() could see a value of false f

Re: request for review (S): 6814943: getcpool001 catches more than one JvmtiThreadState problem

2011-01-10 Thread David Holmes
Hi Keith, Keith McGuigan said the following on 01/11/11 03:36: This closes a race condition hole between JvmtiThreadState::state_for_while_locked() and ~JavaThread(). Without this, the state_for_while_locked() could see a value of false for thread->is_exiting(), then the entirety of ~JavaThre

hg: jdk7/tl/langtools: 6396503: javac should not require assertions enabled

2011-01-10 Thread jonathan . gibbons
Changeset: 7c537f4298fb Author:jjg Date: 2011-01-10 15:08 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/7c537f4298fb 6396503: javac should not require assertions enabled Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/javac/Main.java ! src/share/classes/com

hg: jdk7/tl/langtools: 6992999: fully remove JSR 308 from langtools

2011-01-10 Thread jonathan . gibbons
Changeset: d17f37522154 Author:jjg Date: 2011-01-10 14:57 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/d17f37522154 6992999: fully remove JSR 308 from langtools Reviewed-by: mcimadamore ! src/share/classes/com/sun/source/tree/MethodTree.java ! src/share/classes/com/s

Re: request for review (S): 6814943: getcpool001 catches more than one JvmtiThreadState problem

2011-01-10 Thread Daniel D. Daugherty
On 1/10/2011 10:36 AM, Keith McGuigan wrote: This closes a race condition hole between JvmtiThreadState::state_for_while_locked() and ~JavaThread(). Without this, the state_for_while_locked() could see a value of false for thread->is_exiting(), then the entirety of ~JavaThread() could run, th

hg: jdk7/hotspot-rt/hotspot: 26 new changesets

2011-01-10 Thread daniel . daugherty
Changeset: 7737fa7ec2b5 Author:twisti Date: 2010-12-14 12:44 -0800 URL: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/7737fa7ec2b5 7006044: materialize cheap non-oop pointers on 64-bit SPARC Summary: After 6961690 we load non-oop pointers for the constant table which could

hg: jdk7/tl/jdk: 6544224: Remove the need of sun.reflect.misc

2011-01-10 Thread lance . andersen
Changeset: f4d755bbdabe Author:lancea Date: 2011-01-10 14:43 -0500 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/f4d755bbdabe 6544224: Remove the need of sun.reflect.misc Reviewed-by: alanb ! src/share/classes/javax/sql/rowset/serial/SerialJavaObject.java

request for review (S): 6814943: getcpool001 catches more than one JvmtiThreadState problem

2011-01-10 Thread Keith McGuigan
This closes a race condition hole between JvmtiThreadState::state_for_while_locked() and ~JavaThread(). Without this, the state_for_while_locked() could see a value of false for thread->is_exiting(), then the entirety of ~JavaThread() could run, the state_for_while_locked() could then fi

Re: Pls (re)review 6173675/7003271: per-thread memory allocation measurement

2011-01-10 Thread Paul Hohensee
Thanks! Paul On 1/10/11 10:59 AM, Mandy Chung wrote: On 1/9/11 5:10 PM, Paul Hohensee wrote: Thanks to all who reviewed the previous webrev. Revised versions are here http://cr.openjdk.java.net/~phh/6173675/webrev.02/ Looks good. Thanks for updating it and adding the tests. As for the

hg: jdk7/tl/jdk: 7002957: (fc) FileChannel.transferTo fails to load libsendfile on Solaris 64-bit

2011-01-10 Thread alan . bateman
Changeset: ee5bf287d0c4 Author:alanb Date: 2011-01-10 09:32 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/ee5bf287d0c4 7002957: (fc) FileChannel.transferTo fails to load libsendfile on Solaris 64-bit Reviewed-by: chegar ! make/java/nio/Makefile ! src/solaris/native/sun/nio/