RFR(xs) 8196121 runtime/appcds/ClassLoaderTest.java fails silently

Ioi Lam ioi.lam at oracle.com
Wed Feb 28 19:10:19 UTC 2018


Thanks Misha, I will make the change as you suggested.

- Ioi


On 2/28/18 8:16 AM, Mikhailo Seledtsov wrote:
> Hi Ioi,
>
>   Overall looks good. Only one comment
>
> *ClassLoaderTest.java*
>      "-cp", appJar, bootClassPath, "-XX:+TraceClassPaths", "ForNameTest")
>      Since you are touching this file and line, perhaps change      
> -XX:+TraceClassPaths --> -Xlog:class+path=trace
>      Alternatively, this could be done in one pass with its own RFE - 
> up to you.
>      And if you decide to update this, no need to post a new webrev.
>
>
>   The rest looks good.
>
> Thank you,
> Misha
>
> On 2/27/18, 2:37 PM, Ioi Lam wrote:
>> Please review a very simple fix:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8196121
>> http://cr.openjdk.java.net/~iklam/jdk11/8196121-appcds-class-loader-test-fails-silently.v01/ 
>>
>>
>> Cause: the test was archiving the Hello class, but tried to assert that
>>        this class must not be archived.
>>
>> Fix:   flip the assertion
>>
>> Also some clean up:
>>
>> [1] Use the new TestCommon.run(...).assertNormalExit() API
>>     to make code more maintainable.
>> [2] Explicitly check the bootloader of the HelloWB class instead of
>>     relying on -Xlog:class+load
>>
>>
>> Thanks
>> - Ioi



More information about the hotspot-runtime-dev mailing list