RFR 8073423: Remove LazyClassPathEntry support if no longer needed

Karen Kinnear karen.kinnear at oracle.com
Fri Jun 26 16:33:28 UTC 2015


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