RE: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation event

2017-10-27 Thread Markus Gronlund
Hi Robin, Looks good, thanks for adding this information to the event. Cheers Markus -Original Message- From: Robin Westberg Sent: den 25 oktober 2017 15:52 To: serviceability-dev@openjdk.java.net Subject: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation

Re: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation event

2017-10-27 Thread Robin Westberg
Hi Erik, Thanks for reviewing! Best regards, Robin > On 26 Oct 2017, at 18:50, Erik Gahlin wrote: > > Looks good. > > Erik >> Hi all, >> >> Please review the following changes which add a new field to the >> BiasedLockRevocation event, previousOwner, containing information on which >> thre

Re: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation event

2017-10-27 Thread Robin Westberg
Hi David, Thanks for taking a look! > On 27 Oct 2017, at 12:13, David Holmes wrote: > > Hi Robin, > > On 25/10/2017 11:52 PM, Robin Westberg wrote: >> Hi all, >> Please review the following changes which add a new field to the >> BiasedLockRevocation event, previousOwner, containing informati

Re: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation event

2017-10-27 Thread David Holmes
Hi Robin, On 25/10/2017 11:52 PM, Robin Westberg wrote: Hi all, Please review the following changes which add a new field to the BiasedLockRevocation event, previousOwner, containing information on which thread (if any) owned the bias before it was revoked. So to be clear the previous bias

Re: RFR(S): 8189368: Add information on current bias holder for BiasedLockRevocation event

2017-10-26 Thread Erik Gahlin
Looks good. Erik Hi all, Please review the following changes which add a new field to the BiasedLockRevocation event, previousOwner, containing information on which thread (if any) owned the bias before it was revoked. Issue: https://bugs.openjdk.java.net/browse/JDK-8189368 Webrev: http://c