RFR (S) 8214408: Migrate EventsOnOff to using the same allocateAndCheck method

[JC Beyler]

Hi all,

Could I get a review for this webrev, it is an attempt to fix JDK-8214192,
which I cannot reproduce locally:

Webrev: http://cr.openjdk.java.net/~jcbeyler/8214408/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214408

Once pushed, I'll ask someone to check if the original failure JDK-8214192
has disappeared (or perhaps we could do a mach5 testing with this fix and
see?)

Thanks for your help,
Jc

[s***@oracle.com]

Hi Jc,

LGTM

Thanks,
Serguei

Post by JC Beyler
Hi all,
Could I get a review for this webrev, it is an attempt to fix
Webrev: http://cr.openjdk.java.net/~jcbeyler/8214408/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8214408/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8214408
Once pushed, I'll ask someone to check if the original failure
JDK-8214192 has disappeared (or perhaps we could do a mach5 testing
with this fix and see?)
Thanks for your help,
Jc

[Alex Menkov]

+1

--alex

Post by s***@oracle.com
Hi Jc,
LGTM
Thanks,
Serguei

Post by JC Beyler
Hi all,
Could I get a review for this webrev, it is an attempt to fix
Webrev: http://cr.openjdk.java.net/~jcbeyler/8214408/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8214408/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8214408
Once pushed, I'll ask someone to check if the original failure
JDK-8214192 has disappeared (or perhaps we could do a mach5 testing
with this fix and see?)
Thanks for your help,
Jc

[JC Beyler]

Thanks both, tested via submit repo and pushed!
Jc

Post by Alex Menkov
+1
--alex

Post by s***@oracle.com
Hi Jc,
LGTM
Thanks,
Serguei

Post by JC Beyler
Hi all,
Could I get a review for this webrev, it is an attempt to fix
Webrev: http://cr.openjdk.java.net/~jcbeyler/8214408/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8214408/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8214408
Once pushed, I'll ask someone to check if the original failure
JDK-8214192 has disappeared (or perhaps we could do a mach5 testing
with this fix and see?)
Thanks for your help,
Jc