RFR 8073423: Remove LazyClassPathEntry support if no longer needed
harold seigel
harold.seigel at oracle.com
Fri Jun 26 17:35:17 UTC 2015
Hi Karen,
Thanks for the review. I'll try the compile-the-world test and see what
happens.
Thanks, Harold
On 6/26/2015 12:33 PM, Karen Kinnear wrote:
> Harold,
>
> Change looks good. Thanks for deprecating (and getting CCC approval).
>
> Just double-checking - I think this doesn't change the behavior of compile-the-world
> if you have bootclasspath/a as well as either the jimage or the exploded jimage. Is
> that accurate? Did you test compile-the-world?
>
> thanks,
> Karen
>
> On Jun 24, 2015, at 1:23 PM, harold seigel wrote:
>
>> Hi,
>>
>> Please review this updated webrev for bug JDK-8073423: http://cr.openjdk.java.net/~hseigel/bug_8073423.2/
>>
>> The updated webrev differs from the previous one in two ways:
>>
>> 1. The LazyBootClassLoader option is deprecated instead of just deleted.
>>
>> 2. To preserve compatibility, bad jar files detected while setting up
>> the boot class loader path are ignored. With the previous fix, they
>> caused the JVM to terminate. (This caused test JTReg test
>> hotspot/test/runtime/LoadClass/LoadClassNegative.java to fail.)
>>
>> This updated webrev was tested with hotspot JTreg tests, JCK lang and VM tests, and UTE Quick tests. Additionally, the deprecation of the LazyBootClassLoader option was tested by hand.
>>
>> Thanks, Harold
>>
>> -------- Forwarded Message --------
>> Subject: RFR 8073423: Remove LazyClassPathEntry support if no longer needed
>> Date: Wed, 01 Apr 2015 15:06:25 -0400
>> From: harold seigel <harold.seigel at oracle.com>
>> Organization: Oracle Corporation
>> To: hotspot dev runtime <hotspot-runtime-dev at openjdk.java.net>
>>
>>
>>
>> Hi,
>>
>> Please review this change to remove LazyClassPathEntry support from the
>> JVM. With the demise of rt.jar, this is no longer used.
>>
>> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8073423/
>>
>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8073423
>>
>> The change was tested with JCK lang, vm, and api tests, hotspot jtreg
>> tests, and testbase split_verifier and quick tests. Additionally, I
>> tested '$JAVA_HOME/bin/java -XX:+LazyBootClassLoader -version" to test
>> for the deprecation message for -XX:+LazyBootClassLoader.
>>
>> Thanks, Harold
>>
>>
>>
More information about the hotspot-runtime-dev
mailing list