RFR 8068687: Remove meta-index support and cleanup hotspot code for rt.jar etc in non-modular jdk image
harold seigel
harold.seigel at oracle.com
Mon Feb 16 20:49:59 UTC 2015
Please review this updated webrev for JDK-8068687:
http://cr.openjdk.java.net/~hseigel/bug_8068687_metaindex.2/
The only changes from the previous webrev are to classLoader.cpp
starting around line 1314, and removal of rt.jar comments from
cfgnode.cpp, library_call.cpp, and type.cpp.
These latest changes were tested with jtreg hotspot, java/lang, java/io,
and java/util, and with JCK lang, VM, and API.
Thanks, Harold
On 2/12/2015 3:21 PM, Mandy Chung wrote:
> On 2/12/15 6:49 AM, harold seigel wrote:
>> Hi,
>>
>> Please review this fix for bug 8068687. The fix removes the code
>> that implements meta-index support and rt.jar related processing,
>> because this code is no longer used. Removing meta-index support may
>> allow removal of LazyClassPathEntry support. I will look into that
>> in a separate bug.
>>
>
> I was wondering that. It's fine to separate it from this bug.
>
>> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8068687_metaindex/
>
> classLoader.cpp - there are references to rt.jar that I think should
> be updated.
> 321 // Check whether zip name ends in "rt.jar"
> 1322 // This will match other archives named rt.jar as well, but this is
> 1323 // only used for debugging.
> 1324 return string_ends_with(zip->name, "rt.jar");
> 1352 // We stop at rt.jar, unless it is the first bootstrap path entry
> 1353 if (e->is_jrt() && e != _first_entry) break;
>
> Found rt.jar in comments - probably to take out those.
>
> src/share/vm/opto/library_call.cpp:
> 2601: // of safe & unsafe memory. Otherwise fails in a CTW of rt.jar
>
> src/share/vm/opto/type.cpp:
> 3125: // Happens in a CTW of rt.jar, 320-341, no extra flags
>
> Mandy
More information about the hotspot-runtime-dev
mailing list