1162 Commits

Author SHA1 Message Date
Jesper Wilhelmsson
0f62f198b2 Merge 2017-03-28 00:03:23 +02:00
Jesper Wilhelmsson
7e8d75378e Merge 2017-03-16 16:40:41 -07:00
Coleen Phillimore
4b9562288f 8155672: Remove instanceKlassHandles and KlassHandles
Use unhandled pointers for Klass and InstanceKlass, remove handles with no implementation.

Reviewed-by: dholmes, lfoltan, vlivanov, sspitsyn
2017-03-15 10:25:37 -04:00
Coleen Phillimore
35e8433c1c Merge 2017-03-14 14:07:24 -04:00
Ioi Lam
f5777dc9bd 8005165: Remove CPU-dependent code in self-patching vtables
Reviewed-by: coleenp, stuefe, jiangli
2017-03-03 23:08:35 -08:00
Harold Seigel
8a5f33bd16 8172514: Stop checking for duplicate packages in the JVM_DefineModule() package list
Use GrowableArray append() instead of append_if_missing() to stop searching for duplicates

Reviewed-by: redestad, coleenp, dholmes, gtriantafill
2017-03-10 08:04:17 -05:00
Shafi Ahmad
90c3e97cbd 8171194: Exception "Duplicate field name&signature in class file" should report the name and signature of the field
Added code to emit name and signature of duplicate field in java.lang.ClassFormatError exception message

Reviewed-by: dholmes, coleenp
2017-02-28 20:32:06 -08:00
Harold Seigel
5485a03844 8172307: Remove ununsed JVM API JVM_GetModuleByPackageName()
Remove get_module_by_package_name() etc., and unneeded test.

Reviewed-by: sspitsyn, gtriantafill
2017-03-01 07:59:20 -05:00
Mandy Chung
150d28bb0e 6479237: (cl) Add support for classloader names
Reviewed-by: alanb, bchristi, coleenp, dfuchs, lfoltan, psandoz, sspitsyn
2016-11-03 18:08:02 -07:00
Rachel Protacio
7246df4cca Merge 2016-11-17 21:59:52 +00:00
Rachel Protacio
3f2be39bb3 8165550: Add class loader names to ClassCastException message
Adds new format of class loader names to VM ClassCastException messages

Reviewed-by: dholmes, mchung, lfoltan
2016-11-17 16:28:45 -05:00
Harold Seigel
ccbb28f6a0 8166304: Skipping access check for classes generated by core reflection
Only allow boot and reflection class loader to load sub-types of jdk.internal.reflect

Reviewed-by: acorn, lfoltan, rehn
2016-11-17 14:23:29 -05:00
Jiangli Zhou
70abdc1e0e 8168790: CDS: assert(max_delta <= (0x7FFFFFFF)) failed: range check
Change the assert to check the current delta is less than 0x7FFFFFFF.

Reviewed-by: iklam, ddmitriev
2016-11-14 18:03:44 -05:00
Jesper Wilhelmsson
bc72fc2242 Merge 2016-11-11 16:44:36 +01:00
Harold Seigel
300390f275 8167104: Additional class construction refinements
Reviewed-by: acorn, mschoene, asmotrak
2016-10-26 15:12:53 -04:00
Bob Vandette
929fc23c45 Merge 2017-02-24 12:41:26 -05:00
Coleen Phillimore
fcc9699132 8175104: Unhandled oop in ProtectionDomainCacheTable::compute_hash
Pass Handle for protection_domain as it is mostly a Handle up the call stack

Reviewed-by: hseigel, jiangli, dcubed
2017-02-17 14:47:46 -05:00
Coleen Phillimore
0b3bc5c4f0 8169881: Remove implicit Handle conversions oop->Handle
Pass THREAD to Handle as argument instead of implicit Thread::current() call.

Reviewed-by: dholmes, sspitsyn
2017-02-15 22:59:57 -05:00
Harold Seigel
6e88661a89 8174092: Remove array-related access checks from Reflection::verify_class_access()
Change the parameter type to InstanceKlass* and fix the method's callers

Reviewed-by: acorn, coleenp, gtriantafill, lfoltan
2017-02-10 07:46:07 -05:00
Alejandro Murillo
af03100a51 Merge 2016-10-20 16:53:56 -07:00
Calvin Cheung
c5dc50c03f 8166931: Do not include classes which are unusable during run time in the classlist file
Added check to exclude classes found in the --patch-module list

Reviewed-by: jiangli, lfoltan, iklam
2016-10-18 16:15:03 -07:00
Jiangli Zhou
4d9fc9541d 8167333: Invalid source path info might be used when creating ClassFileStream after CFLH transforms a shared classes in some cases
Use NULL as 'source' when there is no valid source path info.

Reviewed-by: iklam, dholmes, dsamersoff
2016-10-10 20:50:33 -04:00
Coleen Phillimore
04bc07be95 8164921: Memory leaked when instrumentation.retransformClasses() is called repeatedly
Return Metablocks smaller than dictionary's dark matter.

Co-authored-by: Jon Masamitsu <jon.masamitsu@oracle.com>
Reviewed-by: mgerdin, sspitsyn, dsamersoff
2016-10-18 08:39:43 -04:00
Tobias Hartmann
4bb6761600 8153134: Infinite loop in handle_wrong_method in jmod
Use Patching_lock to synchronize access between set_code() and clear_code().

Reviewed-by: kvn, dlong
2016-10-18 09:43:12 +02:00
Harold Seigel
63e0cc39e9 8166364: fatal error: acquiring lock DirtyCardQ_CBL_mon/16 out of order with lock Module_lock/6 -- possible deadlock
Set the mirror's module field outside of the module lock.

Reviewed-by: dsamersoff, dholmes, rehn
2016-10-10 08:34:32 -04:00
Doug Simon
b7b3fb4266 8167353: [JVMCI] JVMCI re-initialization check is in the wrong location
Reviewed-by: kvn, twisti
2016-10-07 15:40:34 -07:00
Paul Sandoz
a1289cce5b 8166974: invokedynamic implementation should not wrap Errors
Reviewed-by: dholmes, jrose
2016-10-14 14:47:01 -07:00
Gerard Ziemski
e555a3e342 8166145: runtime/threads/ThreadInterruptTest3 fails with ExitCode 0
Added new hashtables logging tag and used it print out the table performance details.

Reviewed-by: coleenp, dholmes
2016-10-27 09:42:45 -05:00
Rachel Protacio
e715dd214c Merge 2016-10-26 20:13:29 +00:00
Rachel Protacio
420a682afa 8167995: -Xlog:defaultmethods=debug: lengthy method descriptor triggers "StringStream is re-allocated with a different ResourceMark"
Removed nested ResourceMark declarations

Reviewed-by: dholmes, hseigel
2016-10-26 15:46:13 -04:00
Tobias Hartmann
ed9adcbdfa 8164612: NoSuchMethodException when method name contains NULL or Latin-1 supplement character
String length needs to be updated when converting from unicode to utf8.

Reviewed-by: kvn, coleenp
2016-10-26 14:36:05 +02:00
Harold Seigel
6beed1f844 8161218: Better bytecode loading
Reviewed-by: acorn, mschoene, ctornqvi
2016-07-26 08:23:25 -04:00
Rachel Protacio
2ec4f5a76a 8159507: RuntimeVisibleAnnotation validation
Reviewed-by: coleenp, hseigel, mschoene, acorn
2016-07-21 15:49:17 -04:00
Rachel Protacio
53164dd532 8159503: Amend Annotation Actions
Reviewed-by: coleenp, hseigel, mschoene, acorn, ctornqvi
2016-07-01 15:11:38 -04:00
Harold Seigel
5f49daa2c4 8159511: Stack map validation
Reviewed-by: acorn, mschoene
2016-06-30 08:11:30 -04:00
Coleen Phillimore
5a2d8cb819 8140685: Fix backtrace building to not rely on constant pool merging
Store Symbol* for the name in the backtrace

Reviewed-by: gtriantafill, dholmes, kbarrett, lfoltan
2017-02-01 17:56:22 -05:00
David Holmes
d1856645bc 8081800: AbstractMethodError when evaluating a private method in an interface via debugger
Reviewed-by: acorn, dcubed, coleenp
2016-10-03 21:48:21 -04:00
Calvin Cheung
ac6b2ca3dc 8166930: minor cleanups 1) remove reference to ZIP_ReadMappedEntry 2) checking of st_mode
Reviewed-by: jiangli, lfoltan
2016-09-30 12:11:02 -07:00
Dean Long
42ec83b459 Merge 2016-09-26 14:21:21 -04:00
Kirill Zhaldybin
616083ca76 8165439: Convert Test_TempNewSymbol to GTest
Reviewed-by: dholmes, coleenp
2016-09-05 20:31:03 +03:00
Calvin Cheung
f9707ab4cd 8164011: --patch-module support for CDS
Allows the use of the --patch-module vm option with CDS. However, classes found in --patch-module during dump time will not be archived.

Reviewed-by: iklam, dcubed, lfoltan
2016-09-20 10:37:19 -07:00
Lois Foltan
368585d0db 8163406: The fixup_module_list must be protected by Module_lock when inserting new entries
In java_lang_Class::create_mirror, restructure the check for adding a class to the fixup_module_list, guarded by Module_lock.

Reviewed-by: acorn, coleenp, dholmes, zgu
2016-09-19 12:04:28 -04:00
Jiangli Zhou
eb52950704 8078644: CDS needs to support JVMTI CFLH
Support posting CLFH for shared classes. Tests are contributed by Misha Seledtsov.

Reviewed-by: iklam, coleenp, acorn, dcubed, sspitsyn
2016-09-18 21:10:48 -04:00
Harold Seigel
228097de7c 8149607: [Verifier] Do not verify pop, pop2, swap, dup* against top
Throw VerifyError exception if type top is illegally popped from the stack.

Reviewed-by: coleenp, acorn, ddmitriev
2016-09-14 10:02:49 -04:00
Coleen Phillimore
3c2621dbdd 8165246: [REDO] InstanceKlass::_previous_version_count goes negative
Make _has_previous_version a boolean that is set to true when previous version of a class is added or during class unloading call to purge_previous_versions

Reviewed-by: gtriantafill, dcubed, sspitsyn
2016-09-07 15:25:21 -04:00
Harold Seigel
e8e6415b7a 8058575: IllegalAccessError trying to access package-private class from VM anonymous class
Put anonymous classes in unnamed package into host class's package.  Throw exception if host class's package differs from anonymous class.

Reviewed-by: coleenp, acorn
2016-09-07 07:19:48 -04:00
Rachel Protacio
180c089047 Merge 2016-09-06 22:52:35 +02:00
Rachel Protacio
a8147854b7 8161224: CONSTANT_NameAndType_info permits references to illegal names and descriptors
Enforces proper format checking for NameAndType string content, and that the checking occurs even when not referenced in classfile

Reviewed-by: coleenp, hseigel, ddmitriev
2016-09-06 16:29:32 -04:00
Coleen Phillimore
f03ab592cc 8165268: [BACKOUT] InstanceKlass::_previous_version_count goes negative
Reviewed-by: dcubed
2016-09-01 18:02:13 -04:00
Rachel Protacio
fca8d5b7b1 8163973: VM Anonymous classes should not call Class File Load Hooks
Ensures CFLH's are not called for VM anonymous classes

Reviewed-by: lfoltan, dholmes, coleenp, vlivanov, acorn
2016-08-22 11:06:18 -04:00