Re: RFR: 8160950 Agent JAR added to app class loader rather than system class loader when running with -Djava.system.class.loader

[Alan Bateman]

On 08/09/2016 11:06, David Holmes wrote:

Okay so ... that means this:

JvmtiEnv::AddToSystemClassLoaderSearch(const char* segment) {
  jvmtiPhase phase = get_phase();

  if (phase == JVMTI_PHASE_ONLOAD) {
    for (SystemProperty* p = Arguments::system_properties(); p != 
NULL; p = p->next()) {
      if (strcmp("java.class.path", p->key()) == 0) {
        p->append_value(segment);
        break;
      }
    }
    return JVMTI_ERROR_NONE;

is now dead code as we never call this in the ONLOAD phase?

It's not dead as there may be JVM TI agents that use it to extend it 
during the onload phase. There needs to be another follow-up issue to 
clarify the JVM TI spec but that is a separate issue.

-Alan