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

[Jaroslav Bachorík]

Hi Christoph,

I can confirm that your proposed patch is identical to the
original backport patch and the only change is in
test_singleWriterSynchronizer.cpp: #include "threadHelper.inline.hpp" ->
#include "utilitiesHelper.inline.hpp"

LGTM

-JB-

On Thu, Dec 19, 2019 at 1:08 PM Langer, Christoph <christoph.langer at sap.com>
wrote:

> 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-
> >
>
>