hotspot/src/share/vm/classfile/moduleEntry.cpp
author lfoltan
Thu, 21 Apr 2016 09:23:04 -0400
changeset 38014 8731fa11f766
parent 37503 77531df4dad3
child 38733 2b65f4db449e
permissions -rw-r--r--
8152949: Jigsaw crash when Klass in _fixup_module_field_list is unloaded Summary: During start up anonymous classes must be kept alive until after their mirror's module field is patched with java.base Reviewed-by: coleenp, hseigel Contributed-by: per.liden@oracle.com, stefan.karlsson@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     1
/*
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     2
 * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     4
 *
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     7
 * published by the Free Software Foundation.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     8
 *
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    13
 * accompanied this code).
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    14
 *
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    18
 *
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    21
 * questions.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    22
 *
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    23
 */
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    24
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    25
#include "precompiled.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    26
#include "classfile/classLoaderData.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    27
#include "classfile/javaClasses.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    28
#include "classfile/moduleEntry.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    29
#include "logging/log.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    30
#include "memory/resourceArea.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    31
#include "oops/symbol.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    32
#include "prims/jni.h"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    33
#include "runtime/handles.inline.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    34
#include "runtime/safepoint.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    35
#include "trace/traceMacros.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    36
#include "utilities/events.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    37
#include "utilities/growableArray.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    38
#include "utilities/hashtable.inline.hpp"
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
    39
#include "utilities/ostream.hpp"
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    40
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    41
ModuleEntry* ModuleEntryTable::_javabase_module = NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    42
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    43
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    44
void ModuleEntry::set_location(Symbol* location) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    45
  if (_location != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    46
    // _location symbol's refcounts are managed by ModuleEntry,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    47
    // must decrement the old one before updating.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    48
    _location->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    49
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    50
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    51
  _location = location;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    52
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    53
  if (location != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    54
    location->increment_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    55
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    56
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    57
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    58
void ModuleEntry::set_version(Symbol* version) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    59
  if (_version != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    60
    // _version symbol's refcounts are managed by ModuleEntry,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    61
    // must decrement the old one before updating.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    62
    _version->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    63
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    64
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    65
  _version = version;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    66
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    67
  if (version != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    68
    version->increment_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    69
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    70
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    71
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    72
// Returns the shared ProtectionDomain
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    73
Handle ModuleEntry::shared_protection_domain() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    74
  return Handle(JNIHandles::resolve(_pd));
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    75
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    76
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    77
// Set the shared ProtectionDomain atomically
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    78
void ModuleEntry::set_shared_protection_domain(ClassLoaderData *loader_data,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    79
                                               Handle pd_h) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    80
  // Create a JNI handle for the shared ProtectionDomain and save it atomically.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    81
  // If someone beats us setting the _pd cache, the created JNI handle is destroyed.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    82
  jobject obj = loader_data->add_handle(pd_h);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    83
  if (Atomic::cmpxchg_ptr(obj, &_pd, NULL) != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    84
    loader_data->remove_handle(obj);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    85
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    86
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    87
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    88
// Returns true if this module can read module m
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    89
bool ModuleEntry::can_read(ModuleEntry* m) const {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    90
  assert(m != NULL, "No module to lookup in this module's reads list");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    91
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    92
  // Unnamed modules read everyone and all modules
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    93
  // read java.base.  If either of these conditions
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    94
  // hold, readability has been established.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    95
  if (!this->is_named() ||
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    96
      (m == ModuleEntryTable::javabase_module())) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    97
    return true;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    98
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    99
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   100
  MutexLocker m1(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   101
  if (!has_reads()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   102
    return false;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   103
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   104
    return _reads->contains(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   105
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   106
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   107
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   108
// Add a new module to this module's reads list
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   109
void ModuleEntry::add_read(ModuleEntry* m) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   110
  MutexLocker m1(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   111
  if (m == NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   112
    set_can_read_all_unnamed();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   113
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   114
    if (_reads == NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   115
      // Lazily create a module's reads list
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   116
      _reads = new (ResourceObj::C_HEAP, mtClass)GrowableArray<ModuleEntry*>(MODULE_READS_SIZE, true);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   117
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   118
    _reads->append_if_missing(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   119
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   120
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   121
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   122
bool ModuleEntry::has_reads() const {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   123
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   124
  return ((_reads != NULL) && !_reads->is_empty());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   125
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   126
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   127
// Purge dead module entries out of reads list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   128
void ModuleEntry::purge_reads() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   129
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   130
  if (has_reads()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   131
    // Go backwards because this removes entries that are dead.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   132
    int len = _reads->length();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   133
    for (int idx = len - 1; idx >= 0; idx--) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   134
      ModuleEntry* module_idx = _reads->at(idx);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   135
      ClassLoaderData* cld = module_idx->loader();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   136
      if (cld->is_unloading()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   137
        _reads->delete_at(idx);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   138
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   139
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   140
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   141
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   142
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   143
void ModuleEntry::module_reads_do(ModuleClosure* const f) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   144
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   145
  assert(f != NULL, "invariant");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   146
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   147
  if (has_reads()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   148
    int reads_len = _reads->length();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   149
    for (int i = 0; i < reads_len; ++i) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   150
      f->do_module(_reads->at(i));
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   151
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   152
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   153
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   154
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   155
void ModuleEntry::delete_reads() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   156
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   157
  delete _reads;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   158
  _reads = NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   159
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   160
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   161
ModuleEntryTable::ModuleEntryTable(int table_size)
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   162
  : Hashtable<Symbol*, mtClass>(table_size, sizeof(ModuleEntry)), _unnamed_module(NULL)
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   163
{
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   164
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   165
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   166
ModuleEntryTable::~ModuleEntryTable() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   167
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   168
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   169
  // Walk through all buckets and all entries in each bucket,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   170
  // freeing each entry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   171
  for (int i = 0; i < table_size(); ++i) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   172
    for (ModuleEntry* m = bucket(i); m != NULL;) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   173
      ModuleEntry* to_remove = m;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   174
      // read next before freeing.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   175
      m = m->next();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   176
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   177
      ResourceMark rm;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   178
      log_debug(modules)("ModuleEntryTable: deleting module: %s", to_remove->name() != NULL ?
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   179
                         to_remove->name()->as_C_string() : UNNAMED_MODULE);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   180
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   181
      // Clean out the C heap allocated reads list first before freeing the entry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   182
      to_remove->delete_reads();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   183
      if (to_remove->name() != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   184
        to_remove->name()->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   185
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   186
      if (to_remove->version() != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   187
        to_remove->version()->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   188
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   189
      if (to_remove->location() != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   190
        to_remove->location()->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   191
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   192
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   193
      // Unlink from the Hashtable prior to freeing
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   194
      unlink_entry(to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   195
      FREE_C_HEAP_ARRAY(char, to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   196
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   197
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   198
  assert(number_of_entries() == 0, "should have removed all entries");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   199
  assert(new_entry_free_list() == NULL, "entry present on ModuleEntryTable's free list");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   200
  free_buckets();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   201
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   202
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   203
void ModuleEntryTable::create_unnamed_module(ClassLoaderData* loader_data) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   204
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   205
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   206
  // Each ModuleEntryTable has exactly one unnamed module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   207
  if (loader_data->is_the_null_class_loader_data()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   208
    // For the boot loader, the java.lang.reflect.Module for the unnamed module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   209
    // is not known until a call to JVM_SetBootLoaderUnnamedModule is made. At
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   210
    // this point initially create the ModuleEntry for the unnamed module.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   211
    _unnamed_module = new_entry(0, Handle(NULL), NULL, NULL, NULL, loader_data);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   212
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   213
    // For all other class loaders the java.lang.reflect.Module for their
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   214
    // corresponding unnamed module can be found in the java.lang.ClassLoader object.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   215
    oop module = java_lang_ClassLoader::unnamedModule(loader_data->class_loader());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   216
    _unnamed_module = new_entry(0, Handle(module), NULL, NULL, NULL, loader_data);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   217
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   218
    // Store pointer to the ModuleEntry in the unnamed module's java.lang.reflect.Module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   219
    // object.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   220
    java_lang_reflect_Module::set_module_entry(module, _unnamed_module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   221
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   222
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   223
  // Add to bucket 0, no name to hash on
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   224
  add_entry(0, _unnamed_module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   225
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   226
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   227
ModuleEntry* ModuleEntryTable::new_entry(unsigned int hash, Handle module_handle, Symbol* name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   228
                                         Symbol* version, Symbol* location,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   229
                                         ClassLoaderData* loader_data) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   230
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   231
  ModuleEntry* entry = (ModuleEntry*) NEW_C_HEAP_ARRAY(char, entry_size(), mtClass);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   232
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   233
  // Initialize everything BasicHashtable would
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   234
  entry->set_next(NULL);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   235
  entry->set_hash(hash);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   236
  entry->set_literal(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   237
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   238
  // Initialize fields specific to a ModuleEntry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   239
  entry->init();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   240
  if (name != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   241
    name->increment_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   242
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   243
    // Unnamed modules can read all other unnamed modules.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   244
    entry->set_can_read_all_unnamed();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   245
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   246
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   247
  if (!module_handle.is_null()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   248
    entry->set_module(loader_data->add_handle(module_handle));
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   249
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   250
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   251
  entry->set_loader(loader_data);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   252
  entry->set_version(version);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   253
  entry->set_location(location);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   254
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   255
  TRACE_INIT_MODULE_ID(entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   256
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   257
  return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   258
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   259
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   260
void ModuleEntryTable::add_entry(int index, ModuleEntry* new_entry) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   261
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   262
  Hashtable<Symbol*, mtClass>::add_entry(index, (HashtableEntry<Symbol*, mtClass>*)new_entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   263
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   264
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   265
ModuleEntry* ModuleEntryTable::locked_create_entry_or_null(Handle module_handle,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   266
                                                           Symbol* module_name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   267
                                                           Symbol* module_version,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   268
                                                           Symbol* module_location,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   269
                                                           ClassLoaderData* loader_data) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   270
  assert(module_name != NULL, "ModuleEntryTable locked_create_entry_or_null should never be called for unnamed module.");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   271
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   272
  // Check if module already exists.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   273
  if (lookup_only(module_name) != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   274
    return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   275
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   276
    ModuleEntry* entry = new_entry(compute_hash(module_name), module_handle, module_name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   277
                                   module_version, module_location, loader_data);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   278
    add_entry(index_for(module_name), entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   279
    return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   280
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   281
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   282
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   283
// lookup_only by Symbol* to find a ModuleEntry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   284
ModuleEntry* ModuleEntryTable::lookup_only(Symbol* name) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   285
  if (name == NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   286
    // Return this table's unnamed module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   287
    return unnamed_module();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   288
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   289
  int index = index_for(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   290
  for (ModuleEntry* m = bucket(index); m != NULL; m = m->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   291
    if (m->name()->fast_compare(name) == 0) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   292
      return m;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   293
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   294
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   295
  return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   296
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   297
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   298
// Remove dead modules from all other alive modules' reads list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   299
// This should only occur at class unloading.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   300
void ModuleEntryTable::purge_all_module_reads() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   301
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   302
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   303
    for (ModuleEntry* entry = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   304
                      entry != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   305
                      entry = entry->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   306
      entry->purge_reads();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   307
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   308
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   309
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   310
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   311
void ModuleEntryTable::finalize_javabase(Handle module_handle, Symbol* version, Symbol* location) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   312
  assert_locked_or_safepoint(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   313
  ClassLoaderData* boot_loader_data = ClassLoaderData::the_null_class_loader_data();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   314
  ModuleEntryTable* module_table = boot_loader_data->modules();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   315
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   316
  assert(module_table != NULL, "boot loader's ModuleEntryTable not defined");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   317
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   318
  if (module_handle.is_null()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   319
    fatal("Unable to finalize module definition for java.base");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   320
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   321
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   322
  // Set java.lang.reflect.Module, version and location for java.base
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   323
  ModuleEntry* jb_module = javabase_module();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   324
  assert(jb_module != NULL, "java.base ModuleEntry not defined");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   325
  jb_module->set_module(boot_loader_data->add_handle(module_handle));
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   326
  jb_module->set_version(version);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   327
  jb_module->set_location(location);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   328
  // Store pointer to the ModuleEntry for java.base in the java.lang.reflect.Module object.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   329
  java_lang_reflect_Module::set_module_entry(module_handle(), jb_module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   330
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   331
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   332
void ModuleEntryTable::patch_javabase_entries(Handle module_handle) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   333
  if (module_handle.is_null()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   334
    fatal("Unable to patch the module field of classes loaded prior to java.base's definition, invalid java.lang.reflect.Module");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   335
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   336
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   337
  // Do the fixups for the basic primitive types
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   338
  java_lang_Class::set_module(Universe::int_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   339
  java_lang_Class::set_module(Universe::float_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   340
  java_lang_Class::set_module(Universe::double_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   341
  java_lang_Class::set_module(Universe::byte_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   342
  java_lang_Class::set_module(Universe::bool_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   343
  java_lang_Class::set_module(Universe::char_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   344
  java_lang_Class::set_module(Universe::long_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   345
  java_lang_Class::set_module(Universe::short_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   346
  java_lang_Class::set_module(Universe::void_mirror(), module_handle());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   347
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   348
  // Do the fixups for classes that have already been created.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   349
  GrowableArray <Klass*>* list = java_lang_Class::fixup_module_field_list();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   350
  int list_length = list->length();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   351
  for (int i = 0; i < list_length; i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   352
    Klass* k = list->at(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   353
    assert(k->is_klass(), "List should only hold classes");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   354
    Thread* THREAD = Thread::current();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   355
    KlassHandle kh(THREAD, k);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   356
    java_lang_Class::fixup_module_field(kh, module_handle);
38014
8731fa11f766 8152949: Jigsaw crash when Klass in _fixup_module_field_list is unloaded
lfoltan
parents: 37503
diff changeset
   357
    k->class_loader_data()->dec_keep_alive();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   358
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   359
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   360
  delete java_lang_Class::fixup_module_field_list();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   361
  java_lang_Class::set_fixup_module_field_list(NULL);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   362
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   363
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   364
void ModuleEntryTable::print(outputStream* st) {
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   365
  st->print_cr("Module Entry Table (table_size=%d, entries=%d)",
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   366
               table_size(), number_of_entries());
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   367
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   368
    for (ModuleEntry* probe = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   369
                              probe != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   370
                              probe = probe->next()) {
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   371
      probe->print(st);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   372
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   373
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   374
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   375
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   376
void ModuleEntry::print(outputStream* st) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   377
  ResourceMark rm;
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   378
  st->print_cr("entry "PTR_FORMAT" name %s module "PTR_FORMAT" loader %s version %s location %s strict %s next "PTR_FORMAT,
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   379
               p2i(this),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   380
               name() == NULL ? UNNAMED_MODULE : name()->as_C_string(),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   381
               p2i(module()),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   382
               loader()->loader_name(),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   383
               version() != NULL ? version()->as_C_string() : "NULL",
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   384
               location() != NULL ? location()->as_C_string() : "NULL",
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   385
               BOOL_TO_STR(!can_read_all_unnamed()), p2i(next()));
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   386
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   387
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   388
void ModuleEntryTable::verify() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   389
  int element_count = 0;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   390
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   391
    for (ModuleEntry* probe = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   392
                              probe != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   393
                              probe = probe->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   394
      probe->verify();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   395
      element_count++;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   396
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   397
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   398
  guarantee(number_of_entries() == element_count,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   399
            "Verify of Module Entry Table failed");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   400
  debug_only(verify_lookup_length((double)number_of_entries() / table_size()));
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   401
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   402
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   403
void ModuleEntry::verify() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   404
  guarantee(loader() != NULL, "A module entry must be associated with a loader.");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   405
}