Re: RFR[9u-dev] 8130425: libjvm crash due to stack overflow in executables with 32k tbss/tdata

2016-02-24 Thread David Holmes
For the record this looks fine to me too. Thanks, David - On 18/02/2016 1:17 AM, cheleswer sahu wrote: Hi, I have made changes in the property name (jdk.lang.processReaperUseDefaultStackSize) and code as suggested in the earlier reviews. --- old/src/java.base/share/classes/java/lang/Proc

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread David Holmes
Thanks Serguei - but already pushed. :) David On 25/02/2016 12:16 PM, serguei.spit...@oracle.com wrote: On 2/23/16 23:03, David Holmes wrote: I stumbled across the fact that the following locks are no longer being used in the VM: Runtime: - Interrupt_lock - ProfileVM_lock - ObjAllocPost_lo

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread serguei.spit...@oracle.com
On 2/23/16 23:03, David Holmes wrote: I stumbled across the fact that the following locks are no longer being used in the VM: Runtime: - Interrupt_lock - ProfileVM_lock - ObjAllocPost_lock Serviceability: -JvmtiPendingEvent_lock This looks good too. Thanks, Serguei GC: - CMark_lock -

Re: RFR(XXS): 8149803: Adjust lock rankings for some Event-based tracing locks

2016-02-24 Thread Karen Kinnear
Markus, See below. > On Feb 23, 2016, at 11:58 PM, David Holmes wrote: > > Hi Markus, > > On 23/02/2016 8:00 PM, Markus Gronlund wrote: >> Thanks for taking a look David, >> >> I have verified the new lock rankings by running Kitchensink and by code >> inspection. >> >> The original intent w

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread David Holmes
Thanks Mikael! David On 24/02/2016 6:38 PM, Mikael Gerdin wrote: Hi David, On 2016-02-24 08:03, David Holmes wrote: I stumbled across the fact that the following locks are no longer being used in the VM: Runtime: - Interrupt_lock - ProfileVM_lock - ObjAllocPost_lock Serviceability:

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread David Holmes
Thanks Thomas! David On 24/02/2016 6:08 PM, Thomas Schatzl wrote: Hi David, On Wed, 2016-02-24 at 17:03 +1000, David Holmes wrote: I stumbled across the fact that the following locks are no longer being used in the VM: Runtime: - Interrupt_lock - ProfileVM_lock - ObjAllocPost_lock

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread Mikael Gerdin
Hi David, On 2016-02-24 08:03, David Holmes wrote: I stumbled across the fact that the following locks are no longer being used in the VM: Runtime: - Interrupt_lock - ProfileVM_lock - ObjAllocPost_lock Serviceability: -JvmtiPendingEvent_lock GC: - CMark_lock - CMRegionStack_lock GC

Re: (S) RFR: 8150506: Remove unused locks

2016-02-24 Thread Thomas Schatzl
Hi David, On Wed, 2016-02-24 at 17:03 +1000, David Holmes wrote: > I stumbled across the fact that the following locks are no longer > being > used in the VM: > > Runtime: > - Interrupt_lock > - ProfileVM_lock > - ObjAllocPost_lock > > Serviceability: > -JvmtiPendingEvent_lock > > GC: