RFR: JDK-8080511 - Refresh of jimage support

Mandy Chung mandy.chung at oracle.com
Thu Jun 18 18:48:25 UTC 2015


http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-top 
    modules.xml looks fine.
    
    test/lib/sun/hotspot/WhiteBox.java
        This adds the native entry points.  Where are they implemented?  It’s not obvious to me.

http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-langtools 
    the jdeps change looks fine.

http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-jdk/
This is a huge patch that I have to rely on the extensive testing to catch potential issues.  I tried running the jrt-fs tests using JDK 8 with jrt-fs.jar that work well.   Some comments and mostly minor.

make/gensrc/Gensrc-jdk.dev.gmk
 - you could revert this one

make/src/classes/build/tools/module/ImageBuilder.java
 Nit: line 366, 382 - left over commented lines
 some indentation/formatting nits line 386-397

jdk/tools/jimage/TaskHelper.java
43         static final long serialVersionUID = 8765093759964640721L;  // ## re-generate

jdk/tools/jimage/JImageTask.java
The start year is replaced in the copyright header.   You should add the end year instead.

test/java/nio/Buffer/LimitDirectMemory.sh
test/java/nio/file/spi/SetDefaultProvider.java
   Why this test is ignored? 

Mandy

> On Jun 17, 2015, at 5:08 PM, Jim Laskey (Oracle) <james.laskey at oracle.com> wrote:
> 
> https://bugs.openjdk.java.net/browse/JDK-8080511
> 
> This is an long overdue refresh of the jimage support in the JDK9-dev repo.  This includes native support for reading jimage files, improved jrt-fs (java runtime file system) support for retrieving modules and packages from the runtime, and improved performance for langtools in the presence of jrt-fs.
> 
> http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-top <http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-top>
> http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-jdk <http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-jdk>
> http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-hotspot <http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-hotspot>
> http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-langtools <http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-langtools>
> 
> 
> Details:
> 
> - jrt-fs provides access, via the nio FileSystem API, to the classes in a .jimage file, organized by module or by package.
> - Shared code for jimage support converted to native.  Currently residing in hotspot, but will migrate to it’s own jdk library https://bugs.openjdk.java.net/browse/JDK-8087181 <https://bugs.openjdk.java.net/browse/JDK-8087181>
> - A new archive abstraction for class/resource sources.
> - java based implementation layer for jimage reading to allow backport to JDK8 (jrt-fs.jar - IDE support.)
> - JNI support for jimage into hotspot.
> - White box tests written to exercise native jimage support.
> 



More information about the jigsaw-dev mailing list