Thanks, Christoph!
I forgot to tell that my recommendation is for the serviceability
(j.l.instrument) coverage only.
Thanks,
Serguei
On 11/19/19 23:47, Langer, Christoph wrote:
Hi Serguei,
thanks for the review.
The patch successfully ran through our nightly test system which covers all
these tests on several platforms.
Best regards
Christoph
-----Original Message-----
From: [email protected] <[email protected]>
Sent: Mittwoch, 20. November 2019 03:21
To: Langer, Christoph <[email protected]>; core-libs-
[email protected]; [email protected]; gerard
ziemski <[email protected]>
Subject: Re: RFR: 8234185: Cleanup usage of canonicalize function between
libjava, hotspot and libinstrument
Hi Christoph,
The fix looks good to me.
I'd recommend to run the jdk_instrument and vmTestbase_nsk_jvmti tests.
Also, it can be safe to run:
open/test/hotspot/jtreg/serviceability/jvmti
open/test/hotspot/jtreg/runtime/cds/appcds
open/test/hotspot/jtreg/runtime/BootClassAppendProp
Thanks,
Serguei
On 11/14/19 07:37, Langer, Christoph wrote:
Hi,
please review this cleanup change regarding function "canonicalize" of
libjava.
Bug: https://bugs.openjdk.java.net/browse/JDK-8234185
Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8234185.0/
The goal is to cleanup how this function is defined and used. One thing is,
that there was an unnecessary wrapper function "Canonicalize" in jni_util.c.
It wrapped the call to "canonicalize". We can get rid of this wrapper.
Unfortunately, it is not possible to just export "canonicalize" since this will
conflict with a method signature from the math library, at least on modern
Linuxes. So I decided to call the method JDK_Canonicalize and will correctly
define it in jdk_util.h which can be included everywhere.
Hotspot's classloader.cpp will dynamically resolve this method, so I add a
local declaration of the function pointer in there.
This change shall be predecessor of JDK-8223261, where a review was
already started here: https://mail.openjdk.java.net/pipermail/core-libs-
dev/2019-November/063398.html
Thanks
Christoph
