--- a/hotspot/src/share/vm/classfile/systemDictionary.cpp Tue Sep 06 13:01:27 2016 +0200
+++ b/hotspot/src/share/vm/classfile/systemDictionary.cpp Sun Sep 18 21:10:48 2016 -0400
@@ -1210,16 +1210,12 @@
instanceKlassHandle SystemDictionary::load_shared_class(
Symbol* class_name, Handle class_loader, TRAPS) {
- // Don't load shared class when JvmtiExport::should_post_class_file_load_hook()
- // is enabled since posting CFLH is not supported when loading shared class.
- if (!JvmtiExport::should_post_class_file_load_hook()) {
- instanceKlassHandle ik (THREAD, find_shared_class(class_name));
- // Make sure we only return the boot class for the NULL classloader.
- if (ik.not_null() &&
- ik->is_shared_boot_class() && class_loader.is_null()) {
- Handle protection_domain;
- return load_shared_class(ik, class_loader, protection_domain, THREAD);
- }
+ instanceKlassHandle ik (THREAD, find_shared_class(class_name));
+ // Make sure we only return the boot class for the NULL classloader.
+ if (ik.not_null() &&
+ ik->is_shared_boot_class() && class_loader.is_null()) {
+ Handle protection_domain;
+ return load_shared_class(ik, class_loader, protection_domain, THREAD);
}
return instanceKlassHandle();
}
@@ -1303,11 +1299,6 @@
Handle class_loader,
Handle protection_domain, TRAPS) {
instanceKlassHandle nh = instanceKlassHandle(); // null Handle
- if (JvmtiExport::should_post_class_file_load_hook()) {
- // Don't load shared class when JvmtiExport::should_post_class_file_load_hook()
- // is enabled since posting CFLH is not supported when loading shared class.
- return nh;
- }
if (ik.not_null()) {
Symbol* class_name = ik->name();
@@ -1358,6 +1349,14 @@
}
}
+ instanceKlassHandle new_ik = KlassFactory::check_shared_class_file_load_hook(
+ ik, class_name, class_loader, protection_domain, CHECK_(nh));
+ if (new_ik.not_null()) {
+ // The class is changed by CFLH. Return the new class. The shared class is
+ // not used.
+ return new_ik;
+ }
+
// Adjust methods to recover missing data. They need addresses for
// interpreter entry points and their default native method address
// must be reset.