RFR [XS] 8155239 [TESTBUG] Simple test setup for JVMTI ClassFileLoadHook
mikhailo
mikhailo.seledtsov at oracle.com
Wed Apr 27 18:04:10 UTC 2016
Hi Ioi,
Great. This is very helpful for the task at hand and more jvmti tests
to come.
Overall looks good, and I have couple of comments:
- I recommend moving the libSimpleClassFileLoadHook.c to a sub-directory
under hotpsot/test/testlibrary, such as
hotpsot/test/testlibrary/jvmti
- libSimpleClassFileLoadHook.c
init_options():
- would it be safer to make a copy of "CLASS_NAME", "FROM" and
"TO"; I am not sure of the convention
of ownership of the arguments passed to Agent_Initialize(),
and who is required to free them
Just checking as a precaution
ClassFileLoadHook():
- please add a brief comment, something like "this hook will
match the class name to CLASS_NAME, and attempt
to replace any occurrence of 'FROM' string to 'TO' string"
Thank you,
Misha
On 04/27/2016 06:44 AM, Ioi Lam wrote:
> Hi,
>
> Please review this simple test bug:
>
> https://bugs.openjdk.java.net/browse/JDK-8155239
> http://cr.openjdk.java.net/~iklam/jdk9/8155239_simple_classfileloadhook.v01/
>
>
> Testing ClassFileLoadHook is a pain. In many cases, we are not testing
> CFLH itself, but rather "how does my feature interact with CFLH. One
> example is the interaction between CDS and CFLH. This RFE makes it
> easy to write such tests.
>
> Please see SimpleClassFileLoadHookTest.java for an example.
>
> I've tested under Linux and I am now trying to test under all other
> platforms.
>
> Thanks
> - Ioi
More information about the hotspot-dev
mailing list