RFR(S) : 8166925 : several native TESTs should be changed to TEST_VM
Igor Ignatyev
igor.ignatyev at oracle.com
Fri Sep 30 15:34:55 UTC 2016
http://cr.openjdk.java.net/~iignatyev/8166925/webrev.00
> 109 lines changed: 0 ins; 0 del; 109 mod;
Hi all,
could you please review this patch which fixes type of tests? the fix is basically 's/TEST/TEST_VM/‘.
there are several tests which depend on initialized JVM, but do not declare it (by using TEST_VM macros). here is the list of those tests along w/ observations where/when they need JVM:
1. all arrayOopDesc tests use arrayOopDesc::header_size_in_bytes which depends on a jvm state, it reads HeapWordSize and UseCompressedClassPointers and furthermore has an assert which checks that their values haven't changed.
2. all utilities/json tests use tty variable (as JSON::_st field), tty initialization is a port of jvm initialization.
3. SymbolTable.temp_new_symbol calls JavaThread::current()
4. LogFileOutput tests use ResourceMark which calls JavaThread::current()
5. LogDecorations.level, uptime, tags and timestamps tests directly or indirectly call os::javaTimeNanos() which assumes that some structures, e.g. Bsd::_timebase_info, are inited
6. LogConfigurationTest tests require ResourceMark or os::javaTimeNanos
JBS: https://bugs.openjdk.java.net/browse/JDK-8166925
webrev: http://cr.openjdk.java.net/~iignatyev/8166925/webrev.00
Thanks,
— Igor
More information about the hotspot-dev
mailing list