[11u] [JFR] RFR 8209850: Allow NamedThreads to use GlobalCounter critical sections

[Langer, Christoph]

Hi Jaroslav,

I just ran the patch through our test system and found that it wouldn't build.

I had to change one include in test_singleWriterSynchronizer.cpp: #include "threadHelper.inline.hpp" -> #include "utilitiesHelper.inline.hpp". I think you have that change in your webrev for JDK- 8209976 which comes next in your queue.

So, I think the correct backport of JDK-8209850 would look like this: http://cr.openjdk.java.net/~clanger/webrevs/8209850.11u/ 

Can I please get a review for this? I'd approve & push after I have a review and build results on Windows. The Linux/Unix world looks good so far.

Thanks
Christoph


> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> Behalf Of Hohensee, Paul
> Sent: Donnerstag, 12. Dezember 2019 17:10
> To: Jaroslav Bachorík <jaroslav.bachorik at datadoghq.com>; jdk-updates-dev
> <jdk-updates-dev at openjdk.java.net>
> Subject: Re: [11u] [JFR] RFR 8209850: Allow NamedThreads to use
> GlobalCounter critical sections
> 
> Lgtm.
> 
> Paul
> 
> On 12/12/19, 7:41 AM, "jdk-updates-dev on behalf of Jaroslav Bachorík"
> <jdk-updates-dev-bounces at openjdk.java.net on behalf of
> jaroslav.bachorik at datadoghq.com> wrote:
> 
>      Hi all,
> 
>     please, review this backport which is a prerequisite for the planned
>     backport of JDK-8225797
> 
>     JIRA .   : https://bugs.openjdk.java.net/browse/JDK-8209850
>     Webrev : http://cr.openjdk.java.net/~jbachorik/8209850/webrev.00/
> 
>     The patch applies with fuzz and the patch diff is listed at
>     http://cr.openjdk.java.net/~jbachorik/8209850/webrev.00/patch.diff
> 
>     Thanks,
> 
>     -JB-
>