src/hotspot/share/classfile/packageEntry.cpp
author erikj
Tue, 12 Sep 2017 19:03:39 +0200
changeset 47216 71c04702a3d5
parent 46729 hotspot/src/share/vm/classfile/packageEntry.cpp@c62d2e8b2728
child 50113 caf115bb98ad
permissions -rw-r--r--
8187443: Forest Consolidation: Move files to unified layout Reviewed-by: darcy, ihse
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
     1
/*
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
     2
 * Copyright (c) 2016, 2017, 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 "trace/traceMacros.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    33
#include "utilities/events.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    34
#include "utilities/growableArray.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    35
#include "utilities/hashtable.inline.hpp"
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
    36
#include "utilities/ostream.hpp"
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    37
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    38
// 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
    39
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
    40
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    41
  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
    42
  if (is_exported_allUnnamed() && !m->is_named()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    43
    return true;
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    44
  } else if (!has_qual_exports_list()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    45
    return false;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    46
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    47
    return _qualified_exports->contains(m);
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
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    51
// Add a module to the package's qualified export list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    52
void PackageEntry::add_qexport(ModuleEntry* m) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
    53
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    54
  if (!has_qual_exports_list()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    55
    // Lazily create a package's qualified exports list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    56
    // 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
    57
    _qualified_exports = new (ResourceObj::C_HEAP, mtModule) GrowableArray<ModuleEntry*>(QUAL_EXP_SIZE, true);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    58
  }
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
    59
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
  // 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
    61
  // 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
    62
  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
    63
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
    64
  // Establish exportability to module m
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    65
  _qualified_exports->append_if_missing(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    66
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    67
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
    68
// 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
    69
// 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
    70
// 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
    71
// 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
    72
// 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
    73
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
    74
  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
    75
  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
    76
  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
    77
      (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
    78
      !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
    79
    _must_walk_exports = true;
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    80
    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
    81
      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
    82
      assert(name() != NULL, "PackageEntry without a valid name");
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    83
      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
    84
                        name()->as_C_string(),
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    85
                        (this_pkg_mod == NULL || this_pkg_mod->name() == NULL) ?
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
    86
                          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
    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
}
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
    90
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
    91
// Set the package's exported states based on the value of the ModuleEntry.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    92
void PackageEntry::set_exported(ModuleEntry* m) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    93
  MutexLocker m1(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    94
  if (is_unqual_exported()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    95
    // An exception could be thrown, but choose to simply ignore.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    96
    // Illegal to convert an unqualified exported package to be qualifiedly exported
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
    97
    return;
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
  if (m == NULL) {
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   101
    // 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
   102
    // the qualified list at the next safepoint.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   103
    set_unqual_exported();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   104
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   105
    // Add the exported module
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   106
    add_qexport(m);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   107
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   108
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   109
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   110
// 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
   111
// already unqualifiedly exported.
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   112
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
   113
  if (module()->is_open()) {
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   114
    // 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
   115
    return;
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   116
  }
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 43471
diff changeset
   117
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   118
  MutexLocker m1(Module_lock);
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   119
  if (!is_unqual_exported()) {
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   120
   _export_flags = PKG_EXP_ALLUNNAMED;
38940
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
}
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   123
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   124
// 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
   125
// 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
   126
// 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
   127
// exported to non-exported.
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   128
void PackageEntry::purge_qualified_exports() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   129
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
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
   130
  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
   131
      _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
   132
      !_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
   133
    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
   134
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
    // 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
   136
    // 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
   137
    _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
   138
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   139
    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
   140
      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
   141
      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
   142
      ModuleEntry* pkg_mod = module();
44993
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   143
      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
   144
                        name()->as_C_string(),
f61bcd80ec1f 8178380: Module system implementation refresh (5/2017)
alanb
parents: 43471
diff changeset
   145
                        (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
   146
    }
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
   147
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   148
    // Go backwards because this removes entries that are dead.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   149
    int len = _qualified_exports->length();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   150
    for (int idx = len - 1; idx >= 0; idx--) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   151
      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
   152
      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
   153
      if (cld_idx->is_unloading()) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   154
        _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
   155
      } 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
   156
        // 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
   157
        set_export_walk_required(cld_idx);
36508
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
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   163
void PackageEntry::delete_qualified_exports() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   164
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   165
  if (_qualified_exports != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   166
    delete _qualified_exports;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   167
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   168
  _qualified_exports = NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   169
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   170
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   171
PackageEntryTable::PackageEntryTable(int table_size)
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 37503
diff changeset
   172
  : Hashtable<Symbol*, mtModule>(table_size, sizeof(PackageEntry))
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   173
{
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   174
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   175
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   176
PackageEntryTable::~PackageEntryTable() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   177
  // Walk through all buckets and all entries in each bucket,
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   178
  // freeing each entry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   179
  for (int i = 0; i < table_size(); ++i) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   180
    for (PackageEntry* p = bucket(i); p != NULL;) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   181
      PackageEntry* to_remove = p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   182
      // read next before freeing.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   183
      p = p->next();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   184
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   185
      // Clean out the C heap allocated qualified exports list first before freeing the entry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   186
      to_remove->delete_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   187
      to_remove->name()->decrement_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   188
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   189
      // Unlink from the Hashtable prior to freeing
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   190
      unlink_entry(to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   191
      FREE_C_HEAP_ARRAY(char, to_remove);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   192
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   193
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   194
  assert(number_of_entries() == 0, "should have removed all entries");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   195
  assert(new_entry_free_list() == NULL, "entry present on PackageEntryTable's free list");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   196
  free_buckets();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   197
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   198
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   199
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
   200
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46475
diff changeset
   201
  PackageEntry* entry = (PackageEntry*)Hashtable<Symbol*, mtModule>::allocate_new_entry(hash, name);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   202
42639
762117d57d05 8170672: Event-based tracing to support classloader instances
mgronlun
parents: 42636
diff changeset
   203
  TRACE_INIT_ID(entry);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   204
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   205
  // Initialize fields specific to a PackageEntry
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   206
  entry->init();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   207
  entry->name()->increment_refcount();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   208
  entry->set_module(module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   209
  return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   210
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   211
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   212
void PackageEntryTable::add_entry(int index, PackageEntry* new_entry) {
39290
0cc9f5028562 8156871: Possible concurrency issue with JVM_AddModuleExports
lfoltan
parents: 38940
diff changeset
   213
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 37503
diff changeset
   214
  Hashtable<Symbol*, mtModule>::add_entry(index, (HashtableEntry<Symbol*, mtModule>*)new_entry);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   215
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   216
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   217
// Create package in loader's package entry table and return the entry.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   218
// If entry already exists, return null.  Assume Module lock was taken by caller.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   219
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
   220
  assert(Module_lock->owned_by_self(), "should have the Module_lock");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   221
  // Check if package already exists.  Return NULL if it does.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   222
  if (lookup_only(name) != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   223
    return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   224
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   225
    PackageEntry* entry = new_entry(compute_hash(name), name, module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   226
    add_entry(index_for(name), entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   227
    return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   228
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   229
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   230
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   231
PackageEntry* PackageEntryTable::lookup(Symbol* name, ModuleEntry* module) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   232
  PackageEntry* p = lookup_only(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   233
  if (p != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   234
    return p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   235
  } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   236
    // If not found, add to table. Grab the PackageEntryTable lock first.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   237
    MutexLocker ml(Module_lock);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   238
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   239
    // Since look-up was done lock-free, we need to check if another thread beat
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   240
    // us in the race to insert the package.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   241
    PackageEntry* test = lookup_only(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   242
    if (test != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   243
      // A race occurred and another thread introduced the package.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   244
      return test;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   245
    } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   246
      assert(module != NULL, "module should never be null");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   247
      PackageEntry* entry = new_entry(compute_hash(name), name, module);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   248
      add_entry(index_for(name), entry);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   249
      return entry;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   250
    }
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
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   254
PackageEntry* PackageEntryTable::lookup_only(Symbol* name) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   255
  int index = index_for(name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   256
  for (PackageEntry* p = bucket(index); p != NULL; p = p->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   257
    if (p->name()->fast_compare(name) == 0) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   258
      return p;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   259
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   260
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   261
  return NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   262
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   263
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   264
// Called when a define module for java.base is being processed.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   265
// Verify the packages loaded thus far are in java.base's package list.
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   266
void PackageEntryTable::verify_javabase_packages(GrowableArray<Symbol*> *pkg_list) {
46420
227f72691ac1 8179235: PackageEntryTables should be created eagerly
hseigel
parents: 46404
diff changeset
   267
  assert_lock_strong(Module_lock);
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   268
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   269
    for (PackageEntry* entry = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   270
                       entry != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   271
                       entry = entry->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   272
      ModuleEntry* m = entry->module();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   273
      Symbol* module_name = (m == NULL ? NULL : m->name());
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   274
      if (module_name != NULL &&
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   275
          (module_name->fast_compare(vmSymbols::java_base()) == 0) &&
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   276
          !pkg_list->contains(entry->name())) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   277
        ResourceMark rm;
42636
aafc434ba580 8169734: Update uses of string "java.base" to macro
rprotacio
parents: 42073
diff changeset
   278
        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
   279
      }
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
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   284
38940
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   285
// iteration of qualified exports
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   286
void PackageEntry::package_exports_do(ModuleClosure* const f) {
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   287
  assert_locked_or_safepoint(Module_lock);
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   288
  assert(f != NULL, "invariant");
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   289
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   290
  if (has_qual_exports_list()) {
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   291
    int qe_len = _qualified_exports->length();
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   292
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   293
    for (int i = 0; i < qe_len; ++i) {
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   294
      f->do_module(_qualified_exports->at(i));
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   295
    }
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   296
  }
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   297
}
7de797f32b5f 8152404: Stabilize PackageEntry::package_exports_do
rprotacio
parents: 38733
diff changeset
   298
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   299
bool PackageEntry::exported_pending_delete() const {
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   300
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   301
  return (is_unqual_exported() && _qualified_exports != NULL);
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   302
}
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   303
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   304
// Remove dead entries from all packages' exported list
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   305
void PackageEntryTable::purge_all_package_exports() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   306
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   307
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   308
    for (PackageEntry* entry = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   309
                       entry != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   310
                       entry = entry->next()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   311
      if (entry->exported_pending_delete()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   312
        // exported list is pending deletion due to a transition
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   313
        // from qualified to unqualified
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   314
        entry->delete_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   315
      } else if (entry->is_qual_exported()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   316
        entry->purge_qualified_exports();
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   317
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   318
    }
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
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   322
void PackageEntryTable::print(outputStream* st) {
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   323
  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
   324
               table_size(), number_of_entries());
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   325
  for (int i = 0; i < table_size(); i++) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   326
    for (PackageEntry* probe = bucket(i);
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   327
                       probe != NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   328
                       probe = probe->next()) {
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   329
      probe->print(st);
36508
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
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   333
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   334
// 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
   335
// 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
   336
// getter methods.
37503
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   337
void PackageEntry::print(outputStream* st) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   338
  ResourceMark rm;
39625
8702e5b9bfed 8160356: invalid suffix on literal warning is occurred with GCC 6
kbarrett
parents: 39616
diff changeset
   339
  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
   340
               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
   341
               p2i(this), name()->as_C_string(),
77531df4dad3 8152845: Improve PackageEntry and ModuleEntry print methods for future logging
rprotacio
parents: 36508
diff changeset
   342
               (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
   343
               _classpath_index, _export_flags == PKG_EXP_UNQUALIFIED,
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 42639
diff changeset
   344
               _export_flags == PKG_EXP_ALLUNNAMED, p2i(next()));
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   345
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   346
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   347
void PackageEntryTable::verify() {
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46458
diff changeset
   348
  verify_table<PackageEntry>("Package Entry Table");
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   349
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   350
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   351
void PackageEntry::verify() {
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   352
  guarantee(name() != NULL, "A package entry must have a corresponding symbol name.");
5f9eee6b383b 8142968: Module System implementation
alanb
parents:
diff changeset
   353
}