RFR: 8202095: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test

[Robin Westberg]

HI Markus,

Thanks for pushing!

Best regards,
Robin

> On 1 Jun 2018, at 10:33, Markus Gronlund <markus.gronlund at oracle.com> wrote:
> 
> I will take care of it.
>  
> Markus
>  
> From: Robin Westberg 
> Sent: den 1 juni 2018 10:24
> To: Markus Gronlund <markus.gronlund at oracle.com>
> Cc: hotspot-jfr-dev at openjdk.java.net
> Subject: Re: RFR: 8202095: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
>  
> Hi Markus,
>  
> Thanks for the review!
>  
> I’ll need a sponsor to push this, I’ve created http://cr.openjdk.java.net/~rwestberg/8203237/webrev.01/ <http://cr.openjdk.java.net/~rwestberg/8203237/webrev.01/> with a proper mercurial changeset if anyone would like to volunteer.
>  
> Best regards,
> Robin
> 
> 
> On 30 May 2018, at 22:46, Markus Gronlund <markus.gronlund at oracle.com <mailto:markus.gronlund at oracle.com>> wrote:
>  
> Hi Robin,
> 
> Looks good, thanks for fixing this.
> 
> Markus
> 
> -----Original Message-----
> From: Robin Westberg 
> Sent: den 29 maj 2018 15:05
> To: hotspot-jfr-dev at openjdk.java.net <mailto:hotspot-jfr-dev at openjdk.java.net>
> Subject: RFR: 8202095: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
> 
> Hi all,
> 
> Please review the following test case fix for the TestBiasedLockRevocationEvents test. It was possible that other biased lock revocation events generated in unrelated threads could interfere with the ones generated by the test. To avoid this, the test should only look at events related to its own locks.
> 
> Issue: https://bugs.openjdk.java.net/browse/JDK-8203237 <https://bugs.openjdk.java.net/browse/JDK-8203237><https://bugs.openjdk.java.net/browse/JDK-8203237 <https://bugs.openjdk.java.net/browse/JDK-8203237>>
> Webrev: http://cr.openjdk.java.net/~rwestberg/8202095/webrev.00/ <http://cr.openjdk.java.net/~rwestberg/8202095/webrev.00/><http://cr.openjdk.java.net/~rwestberg/8202095/webrev.00/ <http://cr.openjdk.java.net/~rwestberg/8202095/webrev.00/>>
> Testing: linux-x64, macosx-x64, solaris-sparcv9, windows-x64: open/test/jdk/jdk/jfr/event/*
> 
> Best regards.
> Robin Westberg