RFR: 8034025 Remove JPLIS agent dependency on canonicalize_md.c

[Alan Bateman]

On 06/03/2014 08:11, Staffan Larsen wrote:
> The JPLIS agent (libinstrument.so) compiles in code from libjava, 
> specifically the file functions in canonicalize_md.c. This dependence 
> has been there for a long time and will need to be changed to 
> re-organize the JDK sources so that the code is organized by module.
>
> With this change the instrument library links the same method 
> dynamically at runtime instead of statically at compile time.
>
> webrev: http://cr.openjdk.java.net/~sla/8034025/webrev.00/ 
> <http://cr.openjdk.java.net/%7Esla/8034025/webrev.00/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8034025
> testing: the jdk_instrument target in jprt
>
It's good to fix this dependency as it was always odd to compile into 
this libinstrument.

I'm now sure about adding -ljava to LIBINSTRUMENT_LDFLAGS_SUFFIX as I 
would this would need to be $(WIN_JAVA_LIB) on Windows. That is, I 
suspect  LDFLAGS_SUFFIX_<platform> needs to be used here.

The return from the JNI GetEnv isn't currently checked but for now I'll 
just believe the comment that the thread calling Agent_OnLoad early in 
the startup is already attached. As the JNIEnv parameter isn't used then 
it doesn't matter of course but the temptation may be to use it with 
other JNI functions that may not be usable early in the startup.

As Canonicalize is not in FileSystemSupport.c then maybe it would be 
better to remove the function prototype from the header file and just 
declare it as an extern in appendBootClassPath where it is used.

Otherwise thank for doing this.

-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20140306/d3003b02/attachment-0001.html