src/hotspot/share/classfile/packageEntry.cpp
author coleenp
Fri, 16 Nov 2018 10:54:04 -0500
changeset 52587 6cd56deebb0d
parent 51375 b812a85b3aa4
child 52631 3009ca99de32
permissions -rw-r--r--
8213092: Add more runtime locks for concurrent class unloading Summary: Add locks for calling CLDG::purge concurrently as well and for calling SystemDictionary::do_unloading concurrently. Reviewed-by: eosterlund, hseigel
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     1
/*
51375
b812a85b3aa4 8207778: Add locking to ModuleEntry and PackageEntry tables
hseigel
parents: 50113
diff changeset
     2
 * Copyright (c) 2016, 2018, Oracle and/or its affiliates. All rights reserved.
36508
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/moduleEntry.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    27
#include "classfile/packageEntry.hpp"
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    28
#include "logging/log.hpp"
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    29
#include "memory/resourceArea.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    30
#include "oops/symbol.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    31
#include "runtime/handles.inline.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    32
#include "utilities/events.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    33
#include "utilities/growableArray.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    34
#include "utilities/hashtable.inline.hpp"
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
    35
#include "utilities/ostream.hpp"
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    36
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    37
// Returns true if this package specifies m as a qualified export, including through an unnamed export
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    38
bool PackageEntry::is_qexported_to(ModuleEntry* m) const {
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
    39
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    40
  assert(m != NULL, "No module to lookup in this package's qualified exports list");
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    41
  if (is_exported_allUnnamed() && !m->is_named()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    42
    return true;
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    43
  } else if (!has_qual_exports_list()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    44
    return false;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    45
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    46
    return _qualified_exports->contains(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    47
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    48
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    49
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    50
// Add a module to the package's qualified export list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    51
void PackageEntry::add_qexport(ModuleEntry* m) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
    52
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    53
  if (!has_qual_exports_list()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    54
    // Lazily create a package's qualified exports list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    55
    // Initial size is small, do not anticipate export lists to be large.
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    56
    _qualified_exports = new (ResourceObj::C_HEAP, mtModule) GrowableArray<ModuleEntry*>(QUAL_EXP_SIZE, true);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    57
  }
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    58
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    59
  // Determine, based on this newly established export to module m,
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    60
  // if this package's export list should be walked at a GC safepoint.
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    61
  set_export_walk_required(m->loader_data());
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    62
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    63
  // Establish exportability to module m
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    64
  _qualified_exports->append_if_missing(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    65
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    66
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    67
// If the module's loader, that an export is being established to, is
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    68
// not the same loader as this module's and is not one of the 3 builtin
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    69
// class loaders, then this package's export list must be walked at GC
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    70
// safepoint. Modules have the same life cycle as their defining class
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    71
// loaders and should be removed if dead.
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    72
void PackageEntry::set_export_walk_required(ClassLoaderData* m_loader_data) {
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    73
  assert_locked_or_safepoint(Module_lock);
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    74
  ModuleEntry* this_pkg_mod = module();
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    75
  if (!_must_walk_exports &&
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    76
      (this_pkg_mod == NULL || this_pkg_mod->loader_data() != m_loader_data) &&
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    77
      !m_loader_data->is_builtin_class_loader_data()) {
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    78
    _must_walk_exports = true;
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    79
    if (log_is_enabled(Trace, module)) {
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    80
      ResourceMark rm;
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    81
      assert(name() != NULL, "PackageEntry without a valid name");
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    82
      log_trace(module)("PackageEntry::set_export_walk_required(): package %s defined in module %s, exports list must be walked",
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    83
                        name()->as_C_string(),
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    84
                        (this_pkg_mod == NULL || this_pkg_mod->name() == NULL) ?
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    85
                          UNNAMED_MODULE : this_pkg_mod->name()->as_C_string());
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    86
    }
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    87
  }
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    88
}
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
    89
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    90
// Set the package's exported states based on the value of the ModuleEntry.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    91
void PackageEntry::set_exported(ModuleEntry* m) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    92
  MutexLocker m1(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    93
  if (is_unqual_exported()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    94
    // An exception could be thrown, but choose to simply ignore.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    95
    // Illegal to convert an unqualified exported package to be qualifiedly exported
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    96
    return;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    97
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    98
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    99
  if (m == NULL) {
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   100
    // NULL indicates the package is being unqualifiedly exported.  Clean up
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   101
    // the qualified list at the next safepoint.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   102
    set_unqual_exported();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   103
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   104
    // Add the exported module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   105
    add_qexport(m);
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
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   109
// Set the package as exported to all unnamed modules unless the package is
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   110
// already unqualifiedly exported.
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   111
void PackageEntry::set_is_exported_allUnnamed() {
46404
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   112
  if (module()->is_open()) {
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   113
    // No-op for open modules since all packages are unqualifiedly exported
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   114
    return;
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   115
  }
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   116
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   117
  MutexLocker m1(Module_lock);
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   118
  if (!is_unqual_exported()) {
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   119
   _export_flags = PKG_EXP_ALLUNNAMED;
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   120
  }
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   121
}
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   122
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   123
// Remove dead module entries within the package's exported list.  Note that
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   124
// if all of the modules on the _qualified_exports get purged the list does not
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   125
// get deleted.  This prevents the package from illegally transitioning from
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   126
// exported to non-exported.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   127
void PackageEntry::purge_qualified_exports() {
52587
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   128
  assert_locked_or_safepoint(Module_lock);
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   129
  if (_must_walk_exports &&
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   130
      _qualified_exports != NULL &&
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   131
      !_qualified_exports->is_empty()) {
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   132
    ModuleEntry* pkg_module = module();
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   133
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   134
    // This package's _must_walk_exports flag will be reset based
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   135
    // on the remaining live modules on the exports list.
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   136
    _must_walk_exports = false;
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   137
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   138
    if (log_is_enabled(Trace, module)) {
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   139
      ResourceMark rm;
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   140
      assert(name() != NULL, "PackageEntry without a valid name");
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   141
      ModuleEntry* pkg_mod = module();
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   142
      log_trace(module)("PackageEntry::purge_qualified_exports(): package %s defined in module %s, exports list being walked",
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   143
                        name()->as_C_string(),
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   144
                        (pkg_mod == NULL || pkg_mod->name() == NULL) ? UNNAMED_MODULE : pkg_mod->name()->as_C_string());
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   145
    }
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   146
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   147
    // Go backwards because this removes entries that are dead.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   148
    int len = _qualified_exports->length();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   149
    for (int idx = len - 1; idx >= 0; idx--) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   150
      ModuleEntry* module_idx = _qualified_exports->at(idx);
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   151
      ClassLoaderData* cld_idx = module_idx->loader_data();
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   152
      if (cld_idx->is_unloading()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   153
        _qualified_exports->delete_at(idx);
39616
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   154
      } else {
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   155
        // Update the need to walk this package's exports based on live modules
f82b1f888578 8159262: Walking PackageEntry Export and ModuleEntry Reads Must Occur Only When Neccessary And Wait Until ClassLoader's Aliveness Determined
lfoltan
parents: 39290
diff changeset
   156
        set_export_walk_required(cld_idx);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   157
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   158
    }
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
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   162
void PackageEntry::delete_qualified_exports() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   163
  if (_qualified_exports != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   164
    delete _qualified_exports;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   165
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   166
  _qualified_exports = NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   167
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   168
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   169
PackageEntryTable::PackageEntryTable(int table_size)
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 37503
diff changeset
   170
  : Hashtable<Symbol*, mtModule>(table_size, sizeof(PackageEntry))
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   171
{
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   172
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   173
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   174
PackageEntryTable::~PackageEntryTable() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   175
  // Walk through all buckets and all entries in each bucket,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   176
  // freeing each entry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   177
  for (int i = 0; i < table_size(); ++i) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   178
    for (PackageEntry* p = bucket(i); p != NULL;) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   179
      PackageEntry* to_remove = p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   180
      // read next before freeing.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   181
      p = p->next();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   182
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   183
      // Clean out the C heap allocated qualified exports list first before freeing the entry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   184
      to_remove->delete_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   185
      to_remove->name()->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   186
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   187
      // Unlink from the Hashtable prior to freeing
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   188
      unlink_entry(to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   189
      FREE_C_HEAP_ARRAY(char, to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   190
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   191
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   192
  assert(number_of_entries() == 0, "should have removed all entries");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   193
  assert(new_entry_free_list() == NULL, "entry present on PackageEntryTable's free list");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   194
  free_buckets();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   195
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   196
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   197
PackageEntry* PackageEntryTable::new_entry(unsigned int hash, Symbol* name, ModuleEntry* module) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
   198
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46475
diff changeset
   199
  PackageEntry* entry = (PackageEntry*)Hashtable<Symbol*, mtModule>::allocate_new_entry(hash, name);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   200
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents: 47216
diff changeset
   201
  JFR_ONLY(INIT_ID(entry);)
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   202
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   203
  // Initialize fields specific to a PackageEntry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   204
  entry->init();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   205
  entry->name()->increment_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   206
  entry->set_module(module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   207
  return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   208
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   209
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   210
void PackageEntryTable::add_entry(int index, PackageEntry* new_entry) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
   211
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 37503
diff changeset
   212
  Hashtable<Symbol*, mtModule>::add_entry(index, (HashtableEntry<Symbol*, mtModule>*)new_entry);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   213
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   214
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   215
// Create package in loader's package entry table and return the entry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   216
// If entry already exists, return null.  Assume Module lock was taken by caller.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   217
PackageEntry* PackageEntryTable::locked_create_entry_or_null(Symbol* name, ModuleEntry* module) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
   218
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   219
  // Check if package already exists.  Return NULL if it does.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   220
  if (lookup_only(name) != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   221
    return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   222
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   223
    PackageEntry* entry = new_entry(compute_hash(name), name, module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   224
    add_entry(index_for(name), entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   225
    return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   226
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   227
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   228
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   229
PackageEntry* PackageEntryTable::lookup(Symbol* name, ModuleEntry* module) {
52587
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   230
  MutexLocker ml(Module_lock);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   231
  PackageEntry* p = lookup_only(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   232
  if (p != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   233
    return p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   234
  } else {
52587
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   235
    assert(module != NULL, "module should never be null");
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   236
    PackageEntry* entry = new_entry(compute_hash(name), name, module);
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   237
    add_entry(index_for(name), entry);
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   238
    return entry;
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   239
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   240
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   241
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   242
PackageEntry* PackageEntryTable::lookup_only(Symbol* name) {
52587
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   243
  MutexLockerEx ml(Module_lock->owned_by_self() ? NULL : Module_lock);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   244
  int index = index_for(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   245
  for (PackageEntry* p = bucket(index); p != NULL; p = p->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   246
    if (p->name()->fast_compare(name) == 0) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   247
      return p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   248
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   249
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   250
  return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   251
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   252
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   253
// Called when a define module for java.base is being processed.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   254
// Verify the packages loaded thus far are in java.base's package list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   255
void PackageEntryTable::verify_javabase_packages(GrowableArray<Symbol*> *pkg_list) {
46420
227f72691ac1 8179235: PackageEntryTables should be created eagerly
hseigel
parents: 46404
diff changeset
   256
  assert_lock_strong(Module_lock);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   257
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   258
    for (PackageEntry* entry = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   259
                       entry != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   260
                       entry = entry->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   261
      ModuleEntry* m = entry->module();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   262
      Symbol* module_name = (m == NULL ? NULL : m->name());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   263
      if (module_name != NULL &&
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   264
          (module_name->fast_compare(vmSymbols::java_base()) == 0) &&
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   265
          !pkg_list->contains(entry->name())) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   266
        ResourceMark rm;
42636
aafc434ba580 8169734: Update uses of string "java.base" to macro
rprotacio
parents: 42073
diff changeset
   267
        vm_exit_during_initialization("A non-" JAVA_BASE_NAME " package was loaded prior to module system initialization", entry->name()->as_C_string());
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   268
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   269
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   270
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   271
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   272
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   273
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   274
// iteration of qualified exports
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents: 47216
diff changeset
   275
void PackageEntry::package_exports_do(ModuleClosure* f) {
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   276
  assert_locked_or_safepoint(Module_lock);
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   277
  assert(f != NULL, "invariant");
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   278
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   279
  if (has_qual_exports_list()) {
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   280
    int qe_len = _qualified_exports->length();
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   281
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   282
    for (int i = 0; i < qe_len; ++i) {
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   283
      f->do_module(_qualified_exports->at(i));
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   284
    }
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   285
  }
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   286
}
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   287
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   288
bool PackageEntry::exported_pending_delete() const {
52587
6cd56deebb0d 8213092: Add more runtime locks for concurrent class unloading
coleenp
parents: 51375
diff changeset
   289
  assert_locked_or_safepoint(Module_lock);
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   290
  return (is_unqual_exported() && _qualified_exports != NULL);
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   291
}
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   292
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   293
// Remove dead entries from all packages' exported list
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   294
void PackageEntryTable::purge_all_package_exports() {
51375
b812a85b3aa4 8207778: Add locking to ModuleEntry and PackageEntry tables
hseigel
parents: 50113
diff changeset
   295
  assert_locked_or_safepoint(Module_lock);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   296
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   297
    for (PackageEntry* entry = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   298
                       entry != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   299
                       entry = entry->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   300
      if (entry->exported_pending_delete()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   301
        // exported list is pending deletion due to a transition
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   302
        // from qualified to unqualified
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   303
        entry->delete_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   304
      } else if (entry->is_qual_exported()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   305
        entry->purge_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   306
      }
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
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   311
void PackageEntryTable::print(outputStream* st) {
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   312
  st->print_cr("Package Entry Table (table_size=%d, entries=%d)",
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   313
               table_size(), number_of_entries());
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   314
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   315
    for (PackageEntry* probe = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   316
                       probe != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   317
                       probe = probe->next()) {
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   318
      probe->print(st);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   319
    }
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
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   323
// This function may be called from debuggers so access private fields directly
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   324
// to prevent triggering locking-related asserts that could result from calling
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   325
// getter methods.
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   326
void PackageEntry::print(outputStream* st) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   327
  ResourceMark rm;
39625
8702e5b9bfed 8160356: invalid suffix on literal warning is occurred with GCC 6
kbarrett
parents: 39616
diff changeset
   328
  st->print_cr("package entry " PTR_FORMAT " name %s module %s classpath_index "
8702e5b9bfed 8160356: invalid suffix on literal warning is occurred with GCC 6
kbarrett
parents: 39616
diff changeset
   329
               INT32_FORMAT " is_exported_unqualified %d is_exported_allUnnamed %d " "next " PTR_FORMAT,
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   330
               p2i(this), name()->as_C_string(),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   331
               (module()->is_named() ? module()->name()->as_C_string() : UNNAMED_MODULE),
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   332
               _classpath_index, _export_flags == PKG_EXP_UNQUALIFIED,
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   333
               _export_flags == PKG_EXP_ALLUNNAMED, p2i(next()));
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   334
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   335
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   336
void PackageEntryTable::verify() {
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46458
diff changeset
   337
  verify_table<PackageEntry>("Package Entry Table");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   338
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   339
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   340
void PackageEntry::verify() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   341
  guarantee(name() != NULL, "A package entry must have a corresponding symbol name.");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   342
}