hotspot/src/share/vm/classfile/loaderConstraints.cpp
author never
Mon, 31 Jan 2011 17:48:21 -0800
changeset 8314 057b1c20fd7e
parent 8076 96d498ec7ae1
child 8921 14bfe81f2a9d
permissions -rw-r--r--
6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111" Reviewed-by: jrose, acorn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
     2
 * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5402
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5402
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5402
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    26
#include "classfile/loaderConstraints.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "runtime/safepoint.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "utilities/hashtable.inline.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
LoaderConstraintTable::LoaderConstraintTable(int nof_buckets)
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    34
  : Hashtable<klassOop>(nof_buckets, sizeof(LoaderConstraintEntry)) {};
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
LoaderConstraintEntry* LoaderConstraintTable::new_entry(
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    38
                                 unsigned int hash, Symbol* name,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
                                 klassOop klass, int num_loaders,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
                                 int max_loaders) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
  LoaderConstraintEntry* entry;
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    42
  entry = (LoaderConstraintEntry*)Hashtable<klassOop>::new_entry(hash, klass);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  entry->set_name(name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  entry->set_num_loaders(num_loaders);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  entry->set_max_loaders(max_loaders);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  return entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    49
void LoaderConstraintTable::free_entry(LoaderConstraintEntry *entry) {
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    50
  // decrement name refcount before freeing
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    51
  entry->name()->decrement_refcount();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    52
  Hashtable<klassOop>::free_entry(entry);
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    53
}
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    54
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
void LoaderConstraintTable::oops_do(OopClosure* f) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  for (int index = 0; index < table_size(); index++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
    for (LoaderConstraintEntry* probe = bucket(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
                                probe != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
                                probe = probe->next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
      if (probe->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
        f->do_oop((oop*)probe->klass_addr());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
      for (int n = 0; n < probe->num_loaders(); n++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
        if (probe->loader(n) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
          f->do_oop(probe->loader_addr(n));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
// The loaderConstraintTable must always be accessed with the
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
// SystemDictionary lock held. This is true even for readers as
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
// entries in the table could be being dynamically resized.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
LoaderConstraintEntry** LoaderConstraintTable::find_loader_constraint(
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    79
                                    Symbol* name, Handle loader) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  unsigned int hash = compute_hash(name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  int index = hash_to_index(hash);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  LoaderConstraintEntry** pp = bucket_addr(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  while (*pp) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    LoaderConstraintEntry* p = *pp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
    if (p->hash() == hash) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    87
      if (p->name() == name) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
        for (int i = p->num_loaders() - 1; i >= 0; i--) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
          if (p->loader(i) == loader()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
            return pp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    pp = p->next_addr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  return pp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
void LoaderConstraintTable::purge_loader_constraints(BoolObjectClosure* is_alive) {
5402
c51fd0c1d005 6888953: some calls to function-like macros are missing semicolons
jcoomes
parents: 5124
diff changeset
   102
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  // Remove unloaded entries from constraint table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  for (int index = 0; index < table_size(); index++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
    LoaderConstraintEntry** p = bucket_addr(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
    while(*p) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
      LoaderConstraintEntry* probe = *p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
      klassOop klass = probe->klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
      // Remove klass that is no longer alive
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
      if (klass != NULL && !is_alive->do_object_b(klass)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
        probe->set_klass(NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
        if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
          ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
          tty->print_cr("[Purging class object from constraint for name %s,"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
                     " loader list:",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
                     probe->name()->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
          for (int i = 0; i < probe->num_loaders(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
            tty->print_cr("[   [%d]: %s", i,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
                          SystemDictionary::loader_name(probe->loader(i)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
      // Remove entries no longer alive from loader array
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
      int n = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
      while (n < probe->num_loaders()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
        if (probe->loader(n) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
          if (!is_alive->do_object_b(probe->loader(n))) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
            if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
              ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
              tty->print_cr("[Purging loader %s from constraint for name %s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
                            SystemDictionary::loader_name(probe->loader(n)),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
                            probe->name()->as_C_string()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
                            );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
            // Compact array
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
            int num = probe->num_loaders() - 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
            probe->set_num_loaders(num);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
            probe->set_loader(n, probe->loader(num));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
            probe->set_loader(num, NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
            if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
              ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
              tty->print_cr("[New loader list:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
              for (int i = 0; i < probe->num_loaders(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
                tty->print_cr("[   [%d]: %s", i,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
                              SystemDictionary::loader_name(probe->loader(i)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
            continue;  // current element replaced, so restart without
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
                       // incrementing n
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
        n++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
      // Check whether entry should be purged
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
      if (probe->num_loaders() < 2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
            if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
              ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
              tty->print("[Purging complete constraint for name %s\n",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
                         probe->name()->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
        // Purge entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
        *p = probe->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
        FREE_C_HEAP_ARRAY(oop, probe->loaders());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
        free_entry(probe);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
        if (probe->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
          assert(is_alive->do_object_b(probe->klass()), "klass should be live");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
        for (n = 0; n < probe->num_loaders(); n++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
          if (probe->loader(n) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
            assert(is_alive->do_object_b(probe->loader(n)), "loader should be live");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
        // Go to next entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
        p = probe->next_addr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   187
bool LoaderConstraintTable::add_entry(Symbol* class_name,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
                                      klassOop klass1, Handle class_loader1,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
                                      klassOop klass2, Handle class_loader2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
  int failure_code = 0; // encode different reasons for failing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  if (klass1 != NULL && klass2 != NULL && klass1 != klass2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
    failure_code = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
    klassOop klass = klass1 != NULL ? klass1 : klass2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
    LoaderConstraintEntry** pp1 = find_loader_constraint(class_name,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
                                                         class_loader1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    if (*pp1 != NULL && (*pp1)->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
      if (klass != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
        if (klass != (*pp1)->klass()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
          failure_code = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
        klass = (*pp1)->klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
    LoaderConstraintEntry** pp2 = find_loader_constraint(class_name,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
                                                         class_loader2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
    if (*pp2 != NULL && (*pp2)->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
      if (klass != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
        if (klass != (*pp2)->klass()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
          failure_code = 3;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
        klass = (*pp2)->klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
    if (failure_code == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
      if (*pp1 == NULL && *pp2 == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
        unsigned int hash = compute_hash(class_name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
        int index = hash_to_index(hash);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
        LoaderConstraintEntry* p;
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   226
        p = new_entry(hash, class_name, klass, 2, 2);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
        p->set_loaders(NEW_C_HEAP_ARRAY(oop, 2));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
        p->set_loader(0, class_loader1());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
        p->set_loader(1, class_loader2());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
        p->set_klass(klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
        p->set_next(bucket(index));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
        set_entry(index, p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
        if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
          ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
          tty->print("[Adding new constraint for name: %s, loader[0]: %s,"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
                     " loader[1]: %s ]\n",
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   237
                     class_name->as_C_string(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
                     SystemDictionary::loader_name(class_loader1()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
                     SystemDictionary::loader_name(class_loader2())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
                     );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
      } else if (*pp1 == *pp2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
        /* constraint already imposed */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
        if ((*pp1)->klass() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
          (*pp1)->set_klass(klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
          if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
            ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
            tty->print("[Setting class object in existing constraint for"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
                       " name: %s and loader %s ]\n",
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   250
                       class_name->as_C_string(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
                       SystemDictionary::loader_name(class_loader1())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
                       );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
        } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
          assert((*pp1)->klass() == klass, "loader constraints corrupted");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
      } else if (*pp1 == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
        extend_loader_constraint(*pp2, class_loader1, klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
      } else if (*pp2 == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
        extend_loader_constraint(*pp1, class_loader2, klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
        merge_loader_constraints(pp1, pp2, klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  if (failure_code != 0 && TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
    ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
    const char* reason = "";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
    switch(failure_code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
    case 1: reason = "the class objects presented by loader[0] and loader[1]"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
              " are different"; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
    case 2: reason = "the class object presented by loader[0] does not match"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
              " the stored class object in the constraint"; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
    case 3: reason = "the class object presented by loader[1] does not match"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
              " the stored class object in the constraint"; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    default: reason = "unknown reason code";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
    tty->print("[Failed to add constraint for name: %s, loader[0]: %s,"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
               " loader[1]: %s, Reason: %s ]\n",
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   281
               class_name->as_C_string(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
               SystemDictionary::loader_name(class_loader1()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
               SystemDictionary::loader_name(class_loader2()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
               reason
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
               );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  return failure_code == 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
// return true if the constraint was updated, false if the constraint is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
// violated
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
bool LoaderConstraintTable::check_or_update(instanceKlassHandle k,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
                                                   Handle loader,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   296
                                                   Symbol* name) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  LoaderConstraintEntry* p = *(find_loader_constraint(name, loader));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
  if (p && p->klass() != NULL && p->klass() != k()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
    if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
      ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
      tty->print("[Constraint check failed for name %s, loader %s: "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
                 "the presented class object differs from that stored ]\n",
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   303
                 name->as_C_string(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
                 SystemDictionary::loader_name(loader()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
    if (p && p->klass() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
      p->set_klass(k());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
      if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
        ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
        tty->print("[Updating constraint for name %s, loader %s, "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
                   "by setting class object ]\n",
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   314
                   name->as_C_string(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
                   SystemDictionary::loader_name(loader()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   322
klassOop LoaderConstraintTable::find_constrained_klass(Symbol* name,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
                                                       Handle loader) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
  LoaderConstraintEntry *p = *(find_loader_constraint(name, loader));
8314
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   325
  if (p != NULL && p->klass() != NULL) {
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   326
    if (Klass::cast(p->klass())->oop_is_instance() && !instanceKlass::cast(p->klass())->is_loaded()) {
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   327
      // Only return fully loaded classes.  Classes found through the
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   328
      // constraints might still be in the process of loading.
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   329
      return NULL;
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   330
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
    return p->klass();
8314
057b1c20fd7e 6354181: nsk.logging.stress.threads.scmhml001 fails assertion in "src/share/vm/oops/instanceKlass.cpp, 111"
never
parents: 8076
diff changeset
   332
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
  // No constraints, or else no klass loaded yet.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
void LoaderConstraintTable::ensure_loader_constraint_capacity(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
                                                     LoaderConstraintEntry *p,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
                                                    int nfree) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
    if (p->max_loaders() - p->num_loaders() < nfree) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
        int n = nfree + p->num_loaders();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
        oop* new_loaders = NEW_C_HEAP_ARRAY(oop, n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
        memcpy(new_loaders, p->loaders(), sizeof(oop) * p->num_loaders());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
        p->set_max_loaders(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
        FREE_C_HEAP_ARRAY(oop, p->loaders());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
        p->set_loaders(new_loaders);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
void LoaderConstraintTable::extend_loader_constraint(LoaderConstraintEntry* p,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
                                                     Handle loader,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
                                                     klassOop klass) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
  ensure_loader_constraint_capacity(p, 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
  int num = p->num_loaders();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
  p->set_loader(num, loader());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
  p->set_num_loaders(num + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
  if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
    ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
    tty->print("[Extending constraint for name %s by adding loader[%d]: %s %s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
               p->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
               num,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
               SystemDictionary::loader_name(loader()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
               (p->klass() == NULL ? " and setting class object ]\n" : " ]\n")
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
               );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
  if (p->klass() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
    p->set_klass(klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
    assert(klass == NULL || p->klass() == klass, "constraints corrupted");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
void LoaderConstraintTable::merge_loader_constraints(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
                                                   LoaderConstraintEntry** pp1,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
                                                   LoaderConstraintEntry** pp2,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
                                                   klassOop klass) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
  // make sure *pp1 has higher capacity
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
  if ((*pp1)->max_loaders() < (*pp2)->max_loaders()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
    LoaderConstraintEntry** tmp = pp2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
    pp2 = pp1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
    pp1 = tmp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
  LoaderConstraintEntry* p1 = *pp1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
  LoaderConstraintEntry* p2 = *pp2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
  ensure_loader_constraint_capacity(p1, p2->num_loaders());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
  for (int i = 0; i < p2->num_loaders(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
    int num = p1->num_loaders();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
    p1->set_loader(num, p2->loader(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
    p1->set_num_loaders(num + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
  if (TraceLoaderConstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
    ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
    tty->print_cr("[Merged constraints for name %s, new loader list:",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
                  p1->name()->as_C_string()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
                  );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
    for (int i = 0; i < p1->num_loaders(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
      tty->print_cr("[   [%d]: %s", i,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
                    SystemDictionary::loader_name(p1->loader(i)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
    if (p1->klass() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
      tty->print_cr("[... and setting class object]");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
  // p1->klass() will hold NULL if klass, p2->klass(), and old
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
  // p1->klass() are all NULL.  In addition, all three must have
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  // matching non-NULL values, otherwise either the constraints would
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
  // have been violated, or the constraints had been corrupted (and an
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
  // assertion would fail).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
  if (p2->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
    assert(p2->klass() == klass, "constraints corrupted");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  if (p1->klass() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
    p1->set_klass(klass);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
    assert(p1->klass() == klass, "constraints corrupted");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
  *pp2 = p2->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
  FREE_C_HEAP_ARRAY(oop, p2->loaders());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
  free_entry(p2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
  return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
4899
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   434
void LoaderConstraintTable::verify(Dictionary* dictionary,
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   435
                                   PlaceholderTable* placeholders) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
  Thread *thread = Thread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
  for (int cindex = 0; cindex < _loader_constraint_size; cindex++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
    for (LoaderConstraintEntry* probe = bucket(cindex);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
                                probe != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
                                probe = probe->next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
      if (probe->klass() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
        instanceKlass* ik = instanceKlass::cast(probe->klass());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
        guarantee(ik->name() == probe->name(), "name should match");
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   444
        Symbol* name = ik->name();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
        Handle loader(thread, ik->class_loader());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
        unsigned int d_hash = dictionary->compute_hash(name, loader);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
        int d_index = dictionary->hash_to_index(d_hash);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
        klassOop k = dictionary->find_class(d_index, d_hash, name, loader);
4899
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   449
        if (k != NULL) {
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   450
          // We found the class in the system dictionary, so we should
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   451
          // make sure that the klassOop matches what we already have.
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   452
          guarantee(k == probe->klass(), "klass should be in dictionary");
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   453
        } else {
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   454
          // If we don't find the class in the system dictionary, it
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   455
          // has to be in the placeholders table.
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   456
          unsigned int p_hash = placeholders->compute_hash(name, loader);
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   457
          int p_index = placeholders->hash_to_index(p_hash);
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   458
          PlaceholderEntry* entry = placeholders->get_entry(p_index, p_hash,
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   459
                                                            name, loader);
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   460
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   461
          // The instanceKlass might not be on the entry, so the only
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   462
          // thing we can check here is whether we were successful in
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   463
          // finding the class in the placeholders table.
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   464
          guarantee(entry != NULL, "klass should be in the placeholders");
ab225bac579c 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails
tonyp
parents: 1
diff changeset
   465
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
      for (int n = 0; n< probe->num_loaders(); n++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
        guarantee(probe->loader(n)->is_oop_or_null(), "should be oop");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
// Called with the system dictionary lock held
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
void LoaderConstraintTable::print() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  assert_locked_or_safepoint(SystemDictionary_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
  tty->print_cr("Java loader constraints (entries=%d)", _loader_constraint_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
  for (int cindex = 0; cindex < _loader_constraint_size; cindex++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
    for (LoaderConstraintEntry* probe = bucket(cindex);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
                                probe != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
                                probe = probe->next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
      tty->print("%4d: ", cindex);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
      probe->name()->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
      tty->print(" , loaders:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
      for (int n = 0; n < probe->num_loaders(); n++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
        probe->loader(n)->print_value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
        tty->print(", ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
#endif