RFR [XS] 8155239 [TESTBUG] Simple test setup for JVMTI ClassFileLoadHook
Mikhailo Seledtsov
mikhailo.seledtsov at oracle.com
Mon May 2 00:30:26 UTC 2016
Changes look good to me.
Misha
On 5/1/2016 1:11 AM, Ioi Lam wrote:
> Hi,
>
> I've posted a new version:
>
> http://cr.openjdk.java.net/~iklam/jdk9/8155239_simple_classfileloadhook.v02/
>
>
> [1] Moved libSimpleClassFileLoadHook.c to testlibrary/jvmti
> [2] Added comments in ClassFileLoadHook about what it does, with
> suggestions
> for good replacement strings ('cellphone').
> [3] strdup of CLASS_NAME, FROM and TO
>
> I ran with RBT and it passed on all 8 test platforms.
>
> Thanks
> - Ioi
>
> On 4/27/16 5:00 PM, Ioi Lam wrote:
>> Hi Misha,
>>
>> Thanks for the review. I'll fix the code as you suggested and post a
>> new version.
>>
>> - Ioo
>>
>> On 4/27/16 11:04 AM, mikhailo wrote:
>>> 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