[JFR] new patch bundle published

[Mario Torre]

Thanks for testing!

I'll backport JDK-8202142 once the patch has been integrated directly
into the jdk8u-dev.

We need to look more closely at the TestNative.java to see what's up,
do you have a patch for that already? The JTreg should have all the
necessary detail for the native library to be compiled.

Cheers,
Mario

On Fri, Feb 28, 2020 at 4:25 AM Denghui Dong
<denghui.ddh at alibaba-inc.com> wrote:
>
> Hi,
> jdk/jfr/event/io/TestInstrumentation was unstable now,  we can backport JDK-8202142 to fix this problem.
>
> Thanks,
> Denghui Dong
>
> ------------------------------------------------------------------
> From:Luo, Ziyi <luoziyi at amazon.com>
> Send Time:2020年2月28日(星期五) 09:33
> To:Mario Torre <neugens at redhat.com>
> Cc:jdk8u-dev <jdk8u-dev at openjdk.java.net>
> Subject:Re: [JFR] new patch bundle published
>
> Hi Mario,
>
> Amazon JDK team tested your JFR patches and found 1 failure & 1 error in Jtreg JDK test suite. The build platform is RHEL5 x64, test platform is CentOS 6.
>
> FAILED: jdk/jfr/event/io/TestInstrumentation.java (jtr snippet attached)
> Error:  jdk/jfr/event/sampling/TestNative.java
>
> The second test requires a native test dependency. Test passed after manually compiling the library and setting up the test native path.
>
> Cheers,
> Ziyi
>
>  On 2/17/20, 3:34 AM, "jdk8u-dev on behalf of Mario Torre" <jdk8u-dev-bounces at openjdk.java.net on behalf of neugens at redhat.com> wrote:
>
>     Hi all,
>
>     I just published a new set of patches to add on top of the latest jdk8u-dev:
>
>     http://cr.openjdk.java.net/~neugens/JDK8u-JFR.04/
>
>     This set includes all the patches currently in the incubator plus a
>     patch to enable the JFR by default (which isn't in the incubator yet,
>     pending discussion on the mailing list).
>
>     I really appreciate if I could get some feedback (even as additional
>     review eyes) and broader testing.
>
>     One notable change, this revision contains JDK-8230707, which makes
>     -XX:EnableTracing a no-op; the option is still recognised but it won't
>     print anymore information on the command line (since this is obviously
>     replaced by JFR).
>
>     Testing, especially for performance regression, is needed and very,
>     very welcomed :)
>
>     Cheers,
>     Mario
>     --
>     Mario Torre
>     Associate Manager, Software Engineering
>     Red Hat GmbH <https://www.redhat.com>
>     9704 A60C B4BE A8B8 0F30  9205 5D7E 4952 3F65 7898
>
>
>


-- 
Mario Torre
Associate Manager, Software Engineering
Red Hat GmbH <https://www.redhat.com>
9704 A60C B4BE A8B8 0F30  9205 5D7E 4952 3F65 7898