PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

[Hohensee, Paul]

Yes. I’ve added Fix Request comments to 8237499 and 8239886 and tagged both.

Thanks,
Paul

From: Yi Yang <qingfeng.yy at alibaba-inc.com>
Reply-To: Yi Yang <qingfeng.yy at alibaba-inc.com>
Date: Tuesday, September 7, 2021 at 7:59 PM
To: "Hohensee, Paul" <hohensee at amazon.com>, Andrew Hughes <gnu.andrew at redhat.com>
Cc: "jdk8u-dev at openjdk.java.net" <jdk8u-dev at openjdk.java.net>
Subject: RE: PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

Hi Paul,

Sorry for the late response, I missed this email inadvertently.

> Lgtm. Do you need a sponsor?

Sure! Can you help sponsor this patch?

Thanks.
Yang

------------------------------------------------------------------
From:Hohensee, Paul <hohensee at amazon.com>
Send Time:2021 Apr. 28 (Wed.) 02:32
To:"YANG, Yi" <qingfeng.yy at alibaba-inc.com>; Andrew Hughes <gnu.andrew at redhat.com>
Cc:jdk8u-dev at openjdk.java.net <jdk8u-dev at openjdk.java.net>
Subject:RE: PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

Lgtm. Do you need a sponsor?

Thanks,
Paul

 -----Original Message-----
From: jdk8u-dev <jdk8u-dev-retn at openjdk.java.net> on behalf of Yang Yi <qingfeng.yy at alibaba-inc.com>
Date: Wednesday, February 17, 2021 at 9:27 PM
To: Yang Yi <qingfeng.yy at alibaba-inc.com>, Andrew Hughes <gnu.andrew at redhat.com>
Cc: "jdk8u-dev at openjdk.java.net" <jdk8u-dev at openjdk.java.net>
Subject: RE: Re: PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

Gentle Ping v2 :-)

 ------------------Original Mail ------------------
Sender:Yang Yi <qingfeng.yy at alibaba-inc.com>
Send Date:Thu Feb 4 11:08:39 2021
Recipients:Andrew Hughes <gnu.andrew at redhat.com>
CC:jdk8u-dev at openjdk.java.net <jdk8u-dev at openjdk.java.net>
Subject:Re: PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

Hi Andrew,

Thanks to your explanation, it makes sense, I have updated my patch according to your change.

Webrev:
HotSpot Part: http://cr.openjdk.java.net/~ddong/yiyang/8237499/hotspot-webrev.01/
JDK Part: http://cr.openjdk.java.net/~ddong/yiyang/8237499/jdk-webrev/


Cheers,
Yang Yi


------------------------------------------------------------------
From:Andrew Hughes <gnu.andrew at redhat.com>
Send Time:2021 Feb. 4 (Thu.) 01:45
To:"YANG, Yi" <qingfeng.yy at alibaba-inc.com>
Cc:jdk8u-dev at openjdk.java.net <jdk8u-dev at openjdk.java.net>
Subject:Re: PING: RFR: 8u backport of JDK-8237499 JFR: Include stack trace in the ThreadStart event

On 19:52 Tue 02 Feb     , Yang Yi wrote:
> Hi Andrew,
>
> Do you mean I should update my patch to add these three functions together so that they work as a whole rather than adding them one by one in the future?
>
> Please let me know if I misunderstand your reply.
>
> Thanks,
> Yang Yi
>

Yes, that's correct. In other words, please add my patch to what you
have and provide an updated webrev :-)

Thanks,
--
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222