On 17/03/2016 12:14 AM, Dmitry Samersoff wrote:
Everybody,
Please, review small fix.
http://cr.openjdk.java.net/~dsamersoff/JDK-8151991/webrev.01/
New diagnostic command (JVMTI.agent_load) should be guarded by
#if INCLUDE_SERVICES and don't brake minimal VM build.
Initially I was confused as to why this was associated with
INCLUDE_SERVICES as it seems unrelated to the all the other things
guarded by INCLUDE_SERVICES. But now I see that the attachListener code
is completely excluded by INCLUDE_SERVICES (excludeSrc.gmk) so it makes
sense that the same guard is used in the diagnosticCommand sources (or
else an independent guard introduced?).
However you would then also need the same guard in:
src/share/vm/prims/jvmtiExport.cpp
to allow INCLUDE_JVMTI to be true and INCLUDE_SERVICES to be false.
lso can you update this comment:
64 #if INCLUDE_SERVICES // Heap dumping/inspection supported
to also refer to the JVM TI agent/attach support
Thanks,
David
-----
-Dmitry
