[11u] RFR: 8185005: Improve performance of ThreadMXBean.getThreadInfo(long ids[], int maxDepth)

[Doerr, Martin]

Hi Götz,

that makes sense. Looks good except the non-alphabetic include ordering (I don't need to see another webrev for that).

Best regards,
Martin


> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> Behalf Of Lindenmaier, Goetz
> Sent: Mittwoch, 15. Januar 2020 09:54
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; jdk-updates-
> dev at openjdk.java.net
> Subject: [CAUTION] RE: [11u] RFR: 8185005: Improve performance of
> ThreadMXBean.getThreadInfo(long ids[], int maxDepth)
> 
> Hi,
> 
> I had to do some more smaller fixes to this to get it working.
> 
> The third template argument of the hash table declaration was removed in
> "8214822: Move ConcurrentHashTable VALUE parameter to CONFIG". This is
> missing in 11,
> thus I need to specify a type for VALUE.
> 
> Also, "8213587: Speed up CDS dump time by using resizable hashtables"
> is missing which moves primitive_hash() from resourceHash.hpp
> to globalDefinitions.hpp, so I need to include resourceHash.hpp.
> 
> http://cr.openjdk.java.net/~goetz/wr20/8185005-
> getThreadInfo_performance-jdk11/02-incremental/
> http://cr.openjdk.java.net/~goetz/wr20/8185005-
> getThreadInfo_performance-jdk11/02/
> 
> Best regards,
>   Goetz.
> 
> 
> > -----Original Message-----
> > From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> > Behalf Of Lindenmaier, Goetz
> > Sent: Friday, January 3, 2020 2:41 PM
> > To: jdk-updates-dev at openjdk.java.net
> > Subject: [CAUTION] [11u] RFR: 8185005: Improve performance of
> > ThreadMXBean.getThreadInfo(long ids[], int maxDepth)
> >
> > Hi
> >
> > I had to do some trivial resolves due to context in mutexLocker.h|cpp  to
> > downport this:
> >
> > http://cr.openjdk.java.net/~goetz/wr20/8185005-
> > getThreadInfo_performance-jdk11/01/
> > https://bugs.openjdk.java.net/browse/JDK-8185005
> > https://hg.openjdk.java.net/jdk/jdk/rev/f4abe950c3b0
> >
> > Best regards,
> >   Goetz.