hotspot/src/share/vm/oops/klassVtable.cpp
author dcubed
Wed, 06 Feb 2013 14:31:37 -0800
changeset 15591 b8aa0577f137
parent 14488 ab48109f7d1b
child 16591 73047b0eb568
permissions -rw-r--r--
7182152: Instrumentation hot swap test incorrect monitor count Summary: Add/refine new tracing support using -XX:TraceRedefineClasses=16384. Reviewed-by: coleenp, acorn, sspitsyn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
     2
 * Copyright (c) 1997, 2013, 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: 5403
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5403
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: 5403
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/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "classfile/vmSymbols.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "gc_implementation/shared/markSweep.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "memory/gcLocker.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "memory/universe.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "oops/instanceKlass.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "oops/klassVtable.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    34
#include "oops/method.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "oops/objArrayOop.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "prims/jvmtiRedefineClassesTrace.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "runtime/arguments.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    40
#include "utilities/copy.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    42
inline InstanceKlass* klassVtable::ik() const {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    43
  Klass* k = _klass();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    44
  assert(k->oop_is_instance(), "not an InstanceKlass");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    45
  return (InstanceKlass*)k;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
// this function computes the vtable size (including the size needed for miranda
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
// methods) and the number of miranda methods in this class
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
// Note on Miranda methods: Let's say there is a class C that implements
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
// interface I.  Let's say there is a method m in I that neither C nor any
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
// of its super classes implement (i.e there is no method of any access, with
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
// the same name and signature as m), then m is a Miranda method which is
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
// entered as a public abstract method in C's vtable.  From then on it should
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
// treated as any other public method in C for method over-ride purposes.
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    57
void klassVtable::compute_vtable_size_and_num_mirandas(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    58
    int* vtable_length_ret, int* num_new_mirandas,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    59
    GrowableArray<Method*>* all_mirandas, Klass* super,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    60
    Array<Method*>* methods, AccessFlags class_flags,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    61
    Handle classloader, Symbol* classname, Array<Klass*>* local_interfaces,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    62
    TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  No_Safepoint_Verifier nsv;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  // set up default result values
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    66
  int vtable_length = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  // start off with super's vtable length
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    69
  InstanceKlass* sk = (InstanceKlass*)super;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  vtable_length = super == NULL ? 0 : sk->vtable_length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  // go thru each method in the methods table to see if it needs a new entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  int len = methods->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  for (int i = 0; i < len; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    75
    assert(methods->at(i)->is_method(), "must be a Method*");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
    76
    methodHandle mh(THREAD, methods->at(i));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
    78
    if (needs_new_vtable_entry(mh, super, classloader, classname, class_flags, THREAD)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
      vtable_length += vtableEntry::size(); // we need a new entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    83
  GrowableArray<Method*> new_mirandas(20);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  // compute the number of mirandas methods that must be added to the end
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    85
  get_mirandas(&new_mirandas, all_mirandas, super, methods, local_interfaces);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    86
  *num_new_mirandas = new_mirandas.length();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    87
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
    88
  vtable_length += *num_new_mirandas * vtableEntry::size();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
  if (Universe::is_bootstrapping() && vtable_length == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
    // array classes don't have their superclass set correctly during
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
    // bootstrapping
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
    vtable_length = Universe::base_vtable_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  if (super == NULL && !Universe::is_bootstrapping() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
      vtable_length != Universe::base_vtable_size()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    // Someone is attempting to redefine java.lang.Object incorrectly.  The
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
    // only way this should happen is from
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
    // SystemDictionary::resolve_from_stream(), which will detect this later
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
    // and throw a security exception.  So don't assert here to let
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
    // the exception occur.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
    vtable_length = Universe::base_vtable_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  assert(super != NULL || vtable_length == Universe::base_vtable_size(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
         "bad vtable size for class Object");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  assert(vtable_length % vtableEntry::size() == 0, "bad vtable length");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  assert(vtable_length >= Universe::base_vtable_size(), "vtable too small");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   109
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   110
  *vtable_length_ret = vtable_length;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   113
int klassVtable::index_of(Method* m, int len) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  assert(m->vtable_index() >= 0, "do not ask this of non-vtable methods");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  return m->vtable_index();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
int klassVtable::initialize_from_super(KlassHandle super) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  if (super.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
    return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
    // copy methods from superKlass
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   123
    // can't inherit from array class, so must be InstanceKlass
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
    assert(super->oop_is_instance(), "must be instance klass");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   125
    InstanceKlass* sk = (InstanceKlass*)super();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
    klassVtable* superVtable = sk->vtable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    assert(superVtable->length() <= _length, "vtable too short");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    superVtable->verify(tty, true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
    superVtable->copy_vtable_to(table());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
    if (PrintVtables && Verbose) {
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   134
      ResourceMark rm;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
      tty->print_cr("copy vtable from %s to %s size %d", sk->internal_name(), klass()->internal_name(), _length);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
    return superVtable->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
// Revised lookup semantics   introduced 1.3 (Kestral beta)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
void klassVtable::initialize_vtable(bool checkconstraints, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  // Note:  Arrays can have intermediate array supers.  Use java_super to skip them.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
  KlassHandle super (THREAD, klass()->java_super());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  int nofNewEntries = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
  if (PrintVtables && !klass()->oop_is_array()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
    tty->print_cr("Initializing: %s", _klass->name()->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  oop* end_of_obj = (oop*)_klass() + _klass()->size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  oop* end_of_vtable = (oop*)&table()[_length];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  assert(end_of_vtable <= end_of_obj, "vtable extends beyond end");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
  if (Universe::is_bootstrapping()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
    // just clear everything
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
    for (int i = 0; i < _length; i++) table()[i].clear();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  int super_vtable_len = initialize_from_super(super);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  if (klass()->oop_is_array()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
    assert(super_vtable_len == _length, "arrays shouldn't introduce new methods");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
  } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   171
    assert(_klass->oop_is_instance(), "must be InstanceKlass");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   173
    Array<Method*>* methods = ik()->methods();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   174
    int len = methods->length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
    int initialized = super_vtable_len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   177
    // update_inherited_vtable can stop for gc - ensure using handles
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
    for (int i = 0; i < len; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
      HandleMark hm(THREAD);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   180
      assert(methods->at(i)->is_method(), "must be a Method*");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   181
      methodHandle mh(THREAD, methods->at(i));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   183
      bool needs_new_entry = update_inherited_vtable(ik(), mh, super_vtable_len, checkconstraints, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
      if (needs_new_entry) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
        put_method_at(mh(), initialized);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
        mh()->set_vtable_index(initialized); // set primary vtable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
        initialized++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
    // add miranda methods; it will also update the value of initialized
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   193
    fill_in_mirandas(&initialized);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   195
    // In class hierarchies where the accessibility is not increasing (i.e., going from private ->
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
    // package_private -> publicprotected), the vtable might actually be smaller than our initial
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
    // calculation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
    assert(initialized <= _length, "vtable initialization failed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    for(;initialized < _length; initialized++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
      put_method_at(NULL, initialized);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
    NOT_PRODUCT(verify(tty, true));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   206
// Called for cases where a method does not override its superclass' vtable entry
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   207
// For bytecodes not produced by javac together it is possible that a method does not override
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   208
// the superclass's method, but might indirectly override a super-super class's vtable entry
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   209
// If none found, return a null superk, else return the superk of the method this does override
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   210
InstanceKlass* klassVtable::find_transitive_override(InstanceKlass* initialsuper, methodHandle target_method,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   211
                            int vtable_index, Handle target_loader, Symbol* target_classname, Thread * THREAD) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   212
  InstanceKlass* superk = initialsuper;
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   213
  while (superk != NULL && superk->super() != NULL) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   214
    InstanceKlass* supersuperklass = InstanceKlass::cast(superk->super());
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   215
    klassVtable* ssVtable = supersuperklass->vtable();
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   216
    if (vtable_index < ssVtable->length()) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   217
      Method* super_method = ssVtable->method_at(vtable_index);
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   218
#ifndef PRODUCT
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   219
      Symbol* name= target_method()->name();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   220
      Symbol* signature = target_method()->signature();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   221
      assert(super_method->name() == name && super_method->signature() == signature, "vtable entry name/sig mismatch");
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   222
#endif
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   223
      if (supersuperklass->is_override(super_method, target_loader, target_classname, THREAD)) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   224
#ifndef PRODUCT
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   225
        if (PrintVtables && Verbose) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   226
          ResourceMark rm(THREAD);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   227
          tty->print("transitive overriding superclass %s with %s::%s index %d, original flags: ",
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   228
           supersuperklass->internal_name(),
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   229
           _klass->internal_name(), (target_method() != NULL) ?
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   230
           target_method()->name()->as_C_string() : "<NULL>", vtable_index);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   231
           super_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   232
           tty->print("overriders flags: ");
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   233
           target_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   234
           tty->cr();
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   235
        }
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   236
#endif /*PRODUCT*/
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   237
        break; // return found superk
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   238
      }
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   239
    } else  {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   240
      // super class has no vtable entry here, stop transitive search
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   241
      superk = (InstanceKlass*)NULL;
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   242
      break;
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   243
    }
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   244
    // if no override found yet, continue to search up
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   245
    superk = InstanceKlass::cast(superk->super());
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   246
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   248
  return superk;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   251
// Methods that are "effectively" final don't need vtable entries.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   252
bool method_is_effectively_final(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   253
    AccessFlags klass_flags, methodHandle target) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   254
  return target->is_final() || klass_flags.is_final() && !target->is_overpass();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   255
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
// Update child's copy of super vtable for overrides
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
// OR return true if a new vtable entry is required
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   259
// Only called for InstanceKlass's, i.e. not for arrays
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
// If that changed, could not use _klass as handle for klass
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   261
bool klassVtable::update_inherited_vtable(InstanceKlass* klass, methodHandle target_method, int super_vtable_len,
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   262
                  bool checkconstraints, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  bool allocate_new = true;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   265
  assert(klass->oop_is_instance(), "must be InstanceKlass");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  // Initialize the method's vtable index to "nonvirtual".
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  // If we allocate a vtable entry, we will update it to a non-negative number.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   269
  target_method()->set_vtable_index(Method::nonvirtual_vtable_index);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  // Static and <init> methods are never in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  if (target_method()->is_static() || target_method()->name() ==  vmSymbols::object_initializer_name()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   276
  if (method_is_effectively_final(klass->access_flags(), target_method)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    // a final method never needs a new entry; final methods can be statically
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
    // resolved and they have to be present in the vtable only if they override
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
    // a super's method, in which case they re-use its entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    allocate_new = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
  // we need a new entry if there is no superclass
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  if (klass->super() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
    return allocate_new;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  // private methods always have a new entry in the vtable
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   289
  // specification interpretation since classic has
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   290
  // private methods not overriding
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  if (target_method()->is_private()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
    return allocate_new;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  // search through the vtable and update overridden entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  // Since check_signature_loaders acquires SystemDictionary_lock
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   297
  // which can block for gc, once we are in this loop, use handles
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   298
  // For classfiles built with >= jdk7, we now look for transitive overrides
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   300
  Symbol* name = target_method()->name();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   301
  Symbol* signature = target_method()->signature();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   302
  Handle target_loader(THREAD, _klass()->class_loader());
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   303
  Symbol*  target_classname = _klass->name();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
  for(int i = 0; i < super_vtable_len; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   305
    Method* super_method = method_at(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
    // Check if method name matches
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   307
    if (super_method->name() == name && super_method->signature() == signature) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   309
      // get super_klass for method_holder for the found method
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   310
      InstanceKlass* super_klass =  super_method->method_holder();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   312
      if ((super_klass->is_override(super_method, target_loader, target_classname, THREAD)) ||
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   313
      ((klass->major_version() >= VTABLE_TRANSITIVE_OVERRIDE_VERSION)
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   314
        && ((super_klass = find_transitive_override(super_klass, target_method, i, target_loader,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   315
             target_classname, THREAD)) != (InstanceKlass*)NULL))) {
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   316
        // overriding, so no new entry
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   317
        allocate_new = false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
        if (checkconstraints) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
        // Override vtable entry if passes loader constraint check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
        // if loader constraint checking requested
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
        // No need to visit his super, since he and his super
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
        // have already made any needed loader constraints.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
        // Since loader constraints are transitive, it is enough
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
        // to link to the first super, and we get all the others.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
          Handle super_loader(THREAD, super_klass->class_loader());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   328
          if (target_loader() != super_loader()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
            ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
            char* failed_type_name =
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   331
              SystemDictionary::check_signature_loaders(signature, target_loader,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
                                                        super_loader, true,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
                                                        CHECK_(false));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
            if (failed_type_name != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
              const char* msg = "loader constraint violation: when resolving "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
                "overridden method \"%s\" the class loader (instance"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
                " of %s) of the current class, %s, and its superclass loader "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
                "(instance of %s), have different Class objects for the type "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
                "%s used in the signature";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
              char* sig = target_method()->name_and_sig_as_C_string();
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   341
              const char* loader1 = SystemDictionary::loader_name(target_loader());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
              char* current = _klass->name()->as_C_string();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
              const char* loader2 = SystemDictionary::loader_name(super_loader());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
              size_t buflen = strlen(msg) + strlen(sig) + strlen(loader1) +
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
                strlen(current) + strlen(loader2) + strlen(failed_type_name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
              char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
              jio_snprintf(buf, buflen, msg, sig, loader1, current, loader2,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
                           failed_type_name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
              THROW_MSG_(vmSymbols::java_lang_LinkageError(), buf, false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
          }
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   352
       }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   354
        put_method_at(target_method(), i);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   355
        target_method()->set_vtable_index(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
#ifndef PRODUCT
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   357
        if (PrintVtables && Verbose) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   358
          tty->print("overriding with %s::%s index %d, original flags: ",
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   359
           _klass->internal_name(), (target_method() != NULL) ?
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   360
           target_method()->name()->as_C_string() : "<NULL>", i);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   361
           super_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   362
           tty->print("overriders flags: ");
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   363
           target_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   364
           tty->cr();
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   365
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
#endif /*PRODUCT*/
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   367
      } else {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   368
        // allocate_new = true; default. We might override one entry,
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   369
        // but not override another. Once we override one, not need new
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
#ifndef PRODUCT
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   371
        if (PrintVtables && Verbose) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   372
          tty->print("NOT overriding with %s::%s index %d, original flags: ",
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   373
           _klass->internal_name(), (target_method() != NULL) ?
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   374
           target_method()->name()->as_C_string() : "<NULL>", i);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   375
           super_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   376
           tty->print("overriders flags: ");
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   377
           target_method->access_flags().print_on(tty);
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   378
           tty->cr();
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   379
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
#endif /*PRODUCT*/
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
  return allocate_new;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   387
void klassVtable::put_method_at(Method* m, int index) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
  if (PrintVtables && Verbose) {
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   390
    ResourceMark rm;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
    tty->print_cr("adding %s::%s at index %d", _klass->internal_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
      (m != NULL) ? m->name()->as_C_string() : "<NULL>", index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
  table()[index].set(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
// Find out if a method "m" with superclass "super", loader "classloader" and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
// name "classname" needs a new vtable entry.  Let P be a class package defined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
// by "classloader" and "classname".
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
// NOTE: The logic used here is very similar to the one used for computing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
// the vtables indices for a method. We cannot directly use that function because,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   403
// we allocate the InstanceKlass at load time, and that requires that the
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   404
// superclass has been loaded.
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   405
// However, the vtable entries are filled in at link time, and therefore
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   406
// the superclass' vtable may not yet have been filled in.
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   407
bool klassVtable::needs_new_vtable_entry(methodHandle target_method,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   408
                                         Klass* super,
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   409
                                         Handle classloader,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   410
                                         Symbol* classname,
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   411
                                         AccessFlags class_flags,
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   412
                                         TRAPS) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   413
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   414
  if (method_is_effectively_final(class_flags, target_method) ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
      // a final method never needs a new entry; final methods can be statically
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
      // resolved and they have to be present in the vtable only if they override
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
      // a super's method, in which case they re-use its entry
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   418
      (target_method()->is_static()) ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
      // static methods don't need to be in vtable
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   420
      (target_method()->name() ==  vmSymbols::object_initializer_name())
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
      // <init> is never called dynamically-bound
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
      ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
  // we need a new entry if there is no superclass
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
  if (super == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
  // private methods always have a new entry in the vtable
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   432
  // specification interpretation since classic has
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   433
  // private methods not overriding
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   434
  if (target_method()->is_private()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
  // search through the super class hierarchy to see if we need
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
  // a new entry
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   440
  ResourceMark rm;
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   441
  Symbol* name = target_method()->name();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   442
  Symbol* signature = target_method()->signature();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   443
  Klass* k = super;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   444
  Method* super_method = NULL;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   445
  InstanceKlass *holder = NULL;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   446
  Method* recheck_method =  NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  while (k != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
    // lookup through the hierarchy for a method with matching name and sign.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   449
    super_method = InstanceKlass::cast(k)->lookup_method(name, signature);
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   450
    if (super_method == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
      break; // we still have to search for a matching miranda method
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
    // get the class holding the matching method
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   454
    // make sure you use that class for is_override
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   455
    InstanceKlass* superk = super_method->method_holder();
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   456
    // we want only instance method matches
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   457
    // pretend private methods are not in the super vtable
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   458
    // since we do override around them: e.g. a.m pub/b.m private/c.m pub,
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   459
    // ignore private, c.m pub does override a.m pub
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   460
    // For classes that were not javac'd together, we also do transitive overriding around
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   461
    // methods that have less accessibility
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   462
    if ((!super_method->is_static()) &&
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   463
       (!super_method->is_private())) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   464
      if (superk->is_override(super_method, classloader, classname, THREAD)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
        return false;
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   466
      // else keep looking for transitive overrides
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   470
    // Start with lookup result and continue to search up
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   471
    k = superk->super(); // haven't found an override match yet; continue to look
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
  // if the target method is public or protected it may have a matching
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
  // miranda method in the super, whose entry it should re-use.
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   476
  // Actually, to handle cases that javac would not generate, we need
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   477
  // this check for all access permissions.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   478
  InstanceKlass *sk = InstanceKlass::cast(super);
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   479
  if (sk->has_miranda_methods()) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   480
    if (sk->lookup_method_in_all_interfaces(name, signature) != NULL) {
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   481
      return false;  // found a matching miranda; we do not need a new entry
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
  return true; // found no match; we need a new entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
// Support for miranda methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
// get the vtable index of a miranda method with matching "name" and "signature"
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   490
int klassVtable::index_of_miranda(Symbol* name, Symbol* signature) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
  // search from the bottom, might be faster
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
  for (int i = (length() - 1); i >= 0; i--) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   493
    Method* m = table()[i].method();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
    if (is_miranda_entry_at(i) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
        m->name() == name && m->signature() == signature) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
      return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   499
  return Method::invalid_vtable_index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
// check if an entry is miranda
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
bool klassVtable::is_miranda_entry_at(int i) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   504
  Method* m = method_at(i);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   505
  Klass* method_holder = m->method_holder();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   506
  InstanceKlass *mhk = InstanceKlass::cast(method_holder);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
  // miranda methods are interface methods in a class's vtable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
  if (mhk->is_interface()) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   510
    assert(m->is_public(), "should be public");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
    assert(ik()->implements_interface(method_holder) , "this class should implement the interface");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
    assert(is_miranda(m, ik()->methods(), ik()->super()), "should be a miranda_method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
// check if a method is a miranda method, given a class's methods table and it's super
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
// the caller must make sure that the method belongs to an interface implemented by the class
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   520
bool klassVtable::is_miranda(Method* m, Array<Method*>* class_methods, Klass* super) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   521
  Symbol* name = m->name();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   522
  Symbol* signature = m->signature();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   523
  if (InstanceKlass::find_method(class_methods, name, signature) == NULL) {
7443
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   524
    // did not find it in the method table of the current class
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
    if (super == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
      // super doesn't exist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
      return true;
7443
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   528
    }
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   529
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   530
    Method* mo = InstanceKlass::cast(super)->lookup_method(name, signature);
7443
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   531
    if (mo == NULL || mo->access_flags().is_private() ) {
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   532
      // super class hierarchy does not implement it or protection is different
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   533
      return true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
  }
7443
39c85d63abf2 6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents: 7397
diff changeset
   536
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   540
void klassVtable::add_new_mirandas_to_lists(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   541
    GrowableArray<Method*>* new_mirandas, GrowableArray<Method*>* all_mirandas,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   542
    Array<Method*>* current_interface_methods, Array<Method*>* class_methods,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   543
    Klass* super) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
  // iterate thru the current interface's method to see if it a miranda
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
  int num_methods = current_interface_methods->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
  for (int i = 0; i < num_methods; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   547
    Method* im = current_interface_methods->at(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
    bool is_duplicate = false;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   549
    int num_of_current_mirandas = new_mirandas->length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
    // check for duplicate mirandas in different interfaces we implement
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
    for (int j = 0; j < num_of_current_mirandas; j++) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   552
      Method* miranda = new_mirandas->at(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
      if ((im->name() == miranda->name()) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
          (im->signature() == miranda->signature())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
        is_duplicate = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
    if (!is_duplicate) { // we don't want duplicate miranda entries in the vtable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
      if (is_miranda(im, class_methods, super)) { // is it a miranda at all?
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   562
        InstanceKlass *sk = InstanceKlass::cast(super);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
        // check if it is a duplicate of a super's miranda
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
        if (sk->lookup_method_in_all_interfaces(im->name(), im->signature()) == NULL) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   565
          new_mirandas->append(im);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   566
        }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   567
        if (all_mirandas != NULL) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   568
          all_mirandas->append(im);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   575
void klassVtable::get_mirandas(GrowableArray<Method*>* new_mirandas,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   576
                               GrowableArray<Method*>* all_mirandas,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   577
                               Klass* super, Array<Method*>* class_methods,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   578
                               Array<Klass*>* local_interfaces) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   579
  assert((new_mirandas->length() == 0) , "current mirandas must be 0");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
  // iterate thru the local interfaces looking for a miranda
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
  int num_local_ifs = local_interfaces->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
  for (int i = 0; i < num_local_ifs; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   584
    InstanceKlass *ik = InstanceKlass::cast(local_interfaces->at(i));
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   585
    add_new_mirandas_to_lists(new_mirandas, all_mirandas,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   586
                              ik->methods(), class_methods, super);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
    // iterate thru each local's super interfaces
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   588
    Array<Klass*>* super_ifs = ik->transitive_interfaces();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
    int num_super_ifs = super_ifs->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
    for (int j = 0; j < num_super_ifs; j++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   591
      InstanceKlass *sik = InstanceKlass::cast(super_ifs->at(j));
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   592
      add_new_mirandas_to_lists(new_mirandas, all_mirandas,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   593
                                sik->methods(), class_methods, super);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
// fill in mirandas
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   599
void klassVtable::fill_in_mirandas(int* initialized) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   600
  GrowableArray<Method*> mirandas(20);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   601
  get_mirandas(&mirandas, NULL, ik()->super(), ik()->methods(),
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   602
               ik()->local_interfaces());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   603
  for (int i = 0; i < mirandas.length(); i++) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   604
    put_method_at(mirandas.at(i), *initialized);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13952
diff changeset
   605
    ++(*initialized);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
void klassVtable::copy_vtable_to(vtableEntry* start) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
  Copy::disjoint_words((HeapWord*)table(), (HeapWord*)start, _length * vtableEntry::size());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   613
#if INCLUDE_JVMTI
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   614
void klassVtable::adjust_method_entries(Method** old_methods, Method** new_methods,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
                                        int methods_length, bool * trace_name_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
  // search the vtable for uses of either obsolete or EMCP methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
  for (int j = 0; j < methods_length; j++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   618
    Method* old_method = old_methods[j];
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   619
    Method* new_method = new_methods[j];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
    // In the vast majority of cases we could get the vtable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
    // by using:  old_method->vtable_index()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
    // However, there are rare cases, eg. sun.awt.X11.XDecoratedPeer.getX()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
    // in sun.awt.X11.XFramePeer where methods occur more than once in the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
    // vtable, so, alas, we must do an exhaustive search.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
    for (int index = 0; index < length(); index++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
      if (unchecked_method_at(index) == old_method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
        put_method_at(new_method, index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
        if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
          if (!(*trace_name_printed)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
            // RC_TRACE_MESG macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
            RC_TRACE_MESG(("adjust: name=%s",
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   634
                           old_method->method_holder()->external_name()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
            *trace_name_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
          // RC_TRACE macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
          RC_TRACE(0x00100000, ("vtable method update: %s(%s)",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
                                new_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
                                new_method->signature()->as_C_string()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
        }
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   642
        // cannot 'break' here; see for-loop comment above.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   648
// a vtable should never contain old or obsolete methods
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   649
bool klassVtable::check_no_old_or_obsolete_entries() {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   650
  for (int i = 0; i < length(); i++) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   651
    Method* m = unchecked_method_at(i);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   652
    if (m != NULL &&
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   653
        (NOT_PRODUCT(!m->is_valid() ||) m->is_old() || m->is_obsolete())) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   654
      return false;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   655
    }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   656
  }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   657
  return true;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   658
}
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   659
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   660
void klassVtable::dump_vtable() {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   661
  tty->print_cr("vtable dump --");
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   662
  for (int i = 0; i < length(); i++) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   663
    Method* m = unchecked_method_at(i);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   664
    if (m != NULL) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   665
      tty->print("      (%5d)  ", i);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   666
      m->access_flags().print_on(tty);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   667
      tty->print(" --  ");
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   668
      m->print_name(tty);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   669
      tty->cr();
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   670
    }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   671
  }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   672
}
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   673
#endif // INCLUDE_JVMTI
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   674
9172
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   675
// CDS/RedefineClasses support - clear vtables so they can be reinitialized
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   676
void klassVtable::clear_vtable() {
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   677
  for (int i = 0; i < _length; i++) table()[i].clear();
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   678
}
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   679
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   680
bool klassVtable::is_initialized() {
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   681
  return _length == 0 || table()[0].method() != NULL;
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   682
}
a4e13ccafc44 7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents: 8725
diff changeset
   683
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
//-----------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
// Itable code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
// Initialize a itableMethodEntry
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   688
void itableMethodEntry::initialize(Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
  if (m == NULL) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
  _method = m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
klassItable::klassItable(instanceKlassHandle klass) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
  _klass = klass;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
  if (klass->itable_length() > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
    itableOffsetEntry* offset_entry = (itableOffsetEntry*)klass->start_of_itable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
    if (offset_entry  != NULL && offset_entry->interface_klass() != NULL) { // Check that itable is initialized
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
      // First offset entry points to the first method_entry
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   701
      intptr_t* method_entry  = (intptr_t *)(((address)klass()) + offset_entry->offset());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
      intptr_t* end         = klass->end_of_itable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   704
      _table_offset      = (intptr_t*)offset_entry - (intptr_t*)klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
      _size_offset_table = (method_entry - ((intptr_t*)offset_entry)) / itableOffsetEntry::size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
      _size_method_table = (end - method_entry)                  / itableMethodEntry::size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
      assert(_table_offset >= 0 && _size_offset_table >= 0 && _size_method_table >= 0, "wrong computation");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   712
  // The length of the itable was either zero, or it has not yet been initialized.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
  _table_offset      = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
  _size_offset_table = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
  _size_method_table = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
static int initialize_count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
// Initialization
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
void klassItable::initialize_itable(bool checkconstraints, TRAPS) {
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   722
  // Cannot be setup doing bootstrapping, interfaces don't have
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   723
  // itables, and klass with only ones entry have empty itables
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   724
  if (Universe::is_bootstrapping() ||
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   725
      _klass->is_interface() ||
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   726
      _klass->itable_length() == itableOffsetEntry::size()) return;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   728
  // There's alway an extra itable entry so we can null-terminate it.
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   729
  guarantee(size_offset_table() >= 1, "too small");
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   730
  int num_interfaces = size_offset_table() - 1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
  if (num_interfaces > 0) {
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   732
    if (TraceItables) tty->print_cr("%3d: Initializing itables for %s", ++initialize_count,
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   733
                                    _klass->name()->as_C_string());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
2264
55d0115a54fe 4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents: 2136
diff changeset
   736
    // Iterate through all interfaces
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
    int i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
    for(i = 0; i < num_interfaces; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
      itableOffsetEntry* ioe = offset_entry(i);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   740
      HandleMark hm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
      KlassHandle interf_h (THREAD, ioe->interface_klass());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
      assert(interf_h() != NULL && ioe->offset() != 0, "bad offset entry in itable");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
      initialize_itable_for_interface(ioe->offset(), interf_h, checkconstraints, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   744
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   745
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
  }
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   747
  // Check that the last entry is empty
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   748
  itableOffsetEntry* ioe = offset_entry(size_offset_table() - 1);
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   749
  guarantee(ioe->interface_klass() == NULL && ioe->offset() == 0, "terminator entry missing");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   750
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
void klassItable::initialize_itable_for_interface(int method_table_offset, KlassHandle interf_h, bool checkconstraints, TRAPS) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   754
  Array<Method*>* methods = InstanceKlass::cast(interf_h())->methods();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   755
  int nof_methods = methods->length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  HandleMark hm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
  KlassHandle klass = _klass;
5402
c51fd0c1d005 6888953: some calls to function-like macros are missing semicolons
jcoomes
parents: 2343
diff changeset
   758
  assert(nof_methods > 0, "at least one method must exist for interface to be in vtable");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   759
  Handle interface_loader (THREAD, InstanceKlass::cast(interf_h())->class_loader());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
  int ime_num = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   762
  // Skip first Method* if it is a class initializer
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   763
  int i = methods->at(0)->is_static_initializer() ? 1 : 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
  // m, method_name, method_signature, klass reset each loop so they
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
  // don't need preserving across check_signature_loaders call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
  // methods needs a handle in case of gc from check_signature_loaders
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
  for(; i < nof_methods; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   769
    Method* m = methods->at(i);
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   770
    Symbol* method_name = m->name();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7443
diff changeset
   771
    Symbol* method_signature = m->signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
    // This is same code as in Linkresolver::lookup_instance_method_in_klasses
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   774
    Method* target = klass->uncached_lookup_method(method_name, method_signature);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
    while (target != NULL && target->is_static()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
      // continue with recursive lookup through the superclass
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   777
      Klass* super = target->method_holder()->super();
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   778
      target = (super == NULL) ? (Method*)NULL : super->uncached_lookup_method(method_name, method_signature);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
    if (target == NULL || !target->is_public() || target->is_abstract()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
      // Entry do not resolve. Leave it empty
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
      // Entry did resolve, check loader constraints before initializing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
      // if checkconstraints requested
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
      methodHandle  target_h (THREAD, target); // preserve across gc
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
      if (checkconstraints) {
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   787
        Handle method_holder_loader (THREAD, target->method_holder()->class_loader());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
        if (method_holder_loader() != interface_loader()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
          ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
          char* failed_type_name =
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
            SystemDictionary::check_signature_loaders(method_signature,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
                                                      method_holder_loader,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
                                                      interface_loader,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
                                                      true, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
          if (failed_type_name != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
            const char* msg = "loader constraint violation in interface "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
              "itable initialization: when resolving method \"%s\" the class"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
              " loader (instance of %s) of the current class, %s, "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
              "and the class loader (instance of %s) for interface "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   800
              "%s have different Class objects for the type %s "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
              "used in the signature";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
            char* sig = target_h()->name_and_sig_as_C_string();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   803
            const char* loader1 = SystemDictionary::loader_name(method_holder_loader());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
            char* current = klass->name()->as_C_string();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
            const char* loader2 = SystemDictionary::loader_name(interface_loader());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   806
            char* iface = InstanceKlass::cast(interf_h())->name()->as_C_string();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
            size_t buflen = strlen(msg) + strlen(sig) + strlen(loader1) +
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
              strlen(current) + strlen(loader2) + strlen(iface) +
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
              strlen(failed_type_name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
            char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
            jio_snprintf(buf, buflen, msg, sig, loader1, current, loader2,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
                         iface, failed_type_name);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
            THROW_MSG(vmSymbols::java_lang_LinkageError(), buf);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
      // ime may have moved during GC so recalculate address
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
      itableOffsetEntry::method_entry(_klass(), method_table_offset)[ime_num].initialize(target_h());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
    // Progress to next entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
    ime_num++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   826
// Update entry for specific Method*
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   827
void klassItable::initialize_with_method(Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
  itableMethodEntry* ime = method_entry(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
  for(int i = 0; i < _size_method_table; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
    if (ime->method() == m) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
      ime->initialize(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
    ime++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   837
#if INCLUDE_JVMTI
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   838
void klassItable::adjust_method_entries(Method** old_methods, Method** new_methods,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
                                        int methods_length, bool * trace_name_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
  // search the itable for uses of either obsolete or EMCP methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
  for (int j = 0; j < methods_length; j++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   842
    Method* old_method = old_methods[j];
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   843
    Method* new_method = new_methods[j];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
    itableMethodEntry* ime = method_entry(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
2136
c55428da3cec 6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents: 670
diff changeset
   846
    // The itable can describe more than one interface and the same
c55428da3cec 6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents: 670
diff changeset
   847
    // method signature can be specified by more than one interface.
c55428da3cec 6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents: 670
diff changeset
   848
    // This means we have to do an exhaustive search to find all the
c55428da3cec 6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents: 670
diff changeset
   849
    // old_method references.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
    for (int i = 0; i < _size_method_table; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
      if (ime->method() == old_method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
        ime->initialize(new_method);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
        if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
          if (!(*trace_name_printed)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
            // RC_TRACE_MESG macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
            RC_TRACE_MESG(("adjust: name=%s",
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   858
              old_method->method_holder()->external_name()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
            *trace_name_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
          // RC_TRACE macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
          RC_TRACE(0x00200000, ("itable method update: %s(%s)",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
            new_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
            new_method->signature()->as_C_string()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
        }
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   866
        // cannot 'break' here; see for-loop comment above.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
      ime++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   873
// an itable should never contain old or obsolete methods
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   874
bool klassItable::check_no_old_or_obsolete_entries() {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   875
  itableMethodEntry* ime = method_entry(0);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   876
  for (int i = 0; i < _size_method_table; i++) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   877
    Method* m = ime->method();
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   878
    if (m != NULL &&
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   879
        (NOT_PRODUCT(!m->is_valid() ||) m->is_old() || m->is_obsolete())) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   880
      return false;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   881
    }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   882
    ime++;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   883
  }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   884
  return true;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   885
}
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   886
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   887
void klassItable::dump_itable() {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   888
  itableMethodEntry* ime = method_entry(0);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   889
  tty->print_cr("itable dump --");
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   890
  for (int i = 0; i < _size_method_table; i++) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   891
    Method* m = ime->method();
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   892
    if (m != NULL) {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   893
      tty->print("      (%5d)  ", i);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   894
      m->access_flags().print_on(tty);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   895
      tty->print(" --  ");
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   896
      m->print_name(tty);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   897
      tty->cr();
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   898
    }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   899
    ime++;
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   900
  }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   901
}
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   902
#endif // INCLUDE_JVMTI
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14488
diff changeset
   903
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
// Setup
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
class InterfaceVisiterClosure : public StackObj {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   907
 public:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   908
  virtual void doit(Klass* intf, int method_count) = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   909
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
489c9b5090e2 Initial load
duke
parents:
diff changeset
   911
// Visit all interfaces with at-least one method (excluding <clinit>)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   912
void visit_all_interfaces(Array<Klass*>* transitive_intf, InterfaceVisiterClosure *blk) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
  // Handle array argument
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
  for(int i = 0; i < transitive_intf->length(); i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   915
    Klass* intf = transitive_intf->at(i);
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   916
    assert(intf->is_interface(), "sanity check");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   917
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
    // Find no. of methods excluding a <clinit>
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   919
    int method_count = InstanceKlass::cast(intf)->methods()->length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
    if (method_count > 0) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   921
      Method* m = InstanceKlass::cast(intf)->methods()->at(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   922
      assert(m != NULL && m->is_method(), "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
      if (m->name() == vmSymbols::object_initializer_name()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
        method_count--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
    // Only count interfaces with at least one method
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
    if (method_count > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   930
      blk->doit(intf, method_count);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
class CountInterfacesClosure : public InterfaceVisiterClosure {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  int _nof_methods;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
  int _nof_interfaces;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
   CountInterfacesClosure() { _nof_methods = 0; _nof_interfaces = 0; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
   int nof_methods() const    { return _nof_methods; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
   int nof_interfaces() const { return _nof_interfaces; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   945
   void doit(Klass* intf, int method_count) { _nof_methods += method_count; _nof_interfaces++; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   946
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
class SetupItableClosure : public InterfaceVisiterClosure  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  itableOffsetEntry* _offset_entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
  itableMethodEntry* _method_entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
  address            _klass_begin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
  SetupItableClosure(address klass_begin, itableOffsetEntry* offset_entry, itableMethodEntry* method_entry) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
    _klass_begin  = klass_begin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
    _offset_entry = offset_entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
    _method_entry = method_entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
  itableMethodEntry* method_entry() const { return _method_entry; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   962
  void doit(Klass* intf, int method_count) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
    int offset = ((address)_method_entry) - _klass_begin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
    _offset_entry->initialize(intf, offset);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
    _offset_entry++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
    _method_entry += method_count;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   968
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   969
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   970
int klassItable::compute_itable_size(Array<Klass*>* transitive_interfaces) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
  // Count no of interfaces and total number of interface methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
  CountInterfacesClosure cic;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   973
  visit_all_interfaces(transitive_interfaces, &cic);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   975
  // There's alway an extra itable entry so we can null-terminate it.
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   976
  int itable_size = calc_itable_size(cic.nof_interfaces() + 1, cic.nof_methods());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
  // Statistics
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
  update_stats(itable_size * HeapWordSize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
  return itable_size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
// Fill out offset table and interface klasses into the itable space
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
void klassItable::setup_itable_offset_table(instanceKlassHandle klass) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
  if (klass->itable_length() == 0) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
  assert(!klass->is_interface(), "Should have zero length itable");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   989
489c9b5090e2 Initial load
duke
parents:
diff changeset
   990
  // Count no of interfaces and total number of interface methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   991
  CountInterfacesClosure cic;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
  visit_all_interfaces(klass->transitive_interfaces(), &cic);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
  int nof_methods    = cic.nof_methods();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
  int nof_interfaces = cic.nof_interfaces();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
189
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   996
  // Add one extra entry so we can null-terminate the table
4248c8e21063 6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents: 1
diff changeset
   997
  nof_interfaces++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
   999
  assert(compute_itable_size(klass->transitive_interfaces()) ==
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
         calc_itable_size(nof_interfaces, nof_methods),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
         "mismatch calculation of itable size");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1003
  // Fill-out offset table
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
  itableOffsetEntry* ioe = (itableOffsetEntry*)klass->start_of_itable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
  itableMethodEntry* ime = (itableMethodEntry*)(ioe + nof_interfaces);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
  intptr_t* end               = klass->end_of_itable();
8725
8c1e3dd5fe1b 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 8653
diff changeset
  1007
  assert((oop*)(ime + nof_methods) <= (oop*)klass->start_of_nonstatic_oop_maps(), "wrong offset calculation (1)");
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 189
diff changeset
  1008
  assert((oop*)(end) == (oop*)(ime + nof_methods),                      "wrong offset calculation (2)");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
  // Visit all interfaces and initialize itable offset table
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1011
  SetupItableClosure sic((address)klass(), ioe, ime);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
  visit_all_interfaces(klass->transitive_interfaces(), &sic);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
  ime  = sic.method_entry();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
  oop* v = (oop*) klass->end_of_itable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
  assert( (oop*)(ime) == v, "wrong offset calculation (2)");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
// m must be a method in an interface
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1023
int klassItable::compute_itable_index(Method* m) {
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
  1024
  InstanceKlass* intf = m->method_holder();
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
  1025
  assert(intf->is_interface(), "sanity check");
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
  1026
  Array<Method*>* methods = intf->methods();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
  int index = 0;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1028
  while(methods->at(index) != m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
    index++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
    assert(index < methods->length(), "should find index for resolve_invoke");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1031
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1032
  // Adjust for <clinit>, which is left out of table if first method
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1033
  if (methods->length() > 0 && methods->at(0)->is_static_initializer()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1034
    index--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1035
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1036
  return index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1039
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1040
// inverse to compute_itable_index
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1041
Method* klassItable::method_for_itable_index(Klass* intf, int itable_index) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1042
  assert(InstanceKlass::cast(intf)->is_interface(), "sanity check");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1043
  Array<Method*>* methods = InstanceKlass::cast(intf)->methods();
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1044
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1045
  int index = itable_index;
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1046
  // Adjust for <clinit>, which is left out of table if first method
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1047
  if (methods->length() > 0 && methods->at(0)->is_static_initializer()) {
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1048
    index++;
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1049
  }
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1050
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1051
  if (itable_index < 0 || index >= methods->length())
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1052
    return NULL;                // help caller defend against bad indexes
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1053
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1054
  Method* m = methods->at(index);
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1055
  assert(compute_itable_index(m) == itable_index, "correct inverse");
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1056
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1057
  return m;
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1058
}
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 2136
diff changeset
  1059
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
void klassVtable::verify(outputStream* st, bool forced) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
  // make sure table is initialized
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
  if (!Universe::is_fully_initialized()) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
  // avoid redundant verifies
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
  if (!forced && _verify_count == Universe::verify_count()) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
  _verify_count = Universe::verify_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
  oop* end_of_obj = (oop*)_klass() + _klass()->size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
  oop* end_of_vtable = (oop *)&table()[_length];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
  if (end_of_vtable > end_of_obj) {
5403
6b0dd9c75dde 6888954: argument formatting for assert() and friends
jcoomes
parents: 5402
diff changeset
  1071
    fatal(err_msg("klass %s: klass object too short (vtable extends beyond "
6b0dd9c75dde 6888954: argument formatting for assert() and friends
jcoomes
parents: 5402
diff changeset
  1072
                  "end)", _klass->internal_name()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
  for (int i = 0; i < _length; i++) table()[i].verify(this, st);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
  // verify consistency with superKlass vtable
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1077
  Klass* super = _klass->super();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
  if (super != NULL) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1079
    InstanceKlass* sk = InstanceKlass::cast(super);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
    klassVtable* vt = sk->vtable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
    for (int i = 0; i < vt->length(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
      verify_against(st, vt, i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
void klassVtable::verify_against(outputStream* st, klassVtable* vt, int index) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
  vtableEntry* vte = &vt->table()[index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
  if (vte->method()->name()      != table()[index].method()->name() ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
      vte->method()->signature() != table()[index].method()->signature()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
    fatal("mismatched name/signature of vtable entries");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1092
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1094
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1096
void klassVtable::print() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1097
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
  tty->print("klassVtable for klass %s (length %d):\n", _klass->internal_name(), length());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
  for (int i = 0; i < length(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
    table()[i].print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1101
    tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
void vtableEntry::verify(klassVtable* vt, outputStream* st) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
  NOT_PRODUCT(FlagSetting fs(IgnoreLockingAssertions, true));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
  assert(method() != NULL, "must have set method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
  method()->verify();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
  // we sub_type, because it could be a miranda method
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
  if (!vt->klass()->is_subtype_of(method()->method_holder())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
    print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
#endif
5403
6b0dd9c75dde 6888954: argument formatting for assert() and friends
jcoomes
parents: 5402
diff changeset
  1115
    fatal(err_msg("vtableEntry " PTR_FORMAT ": method is from subclass", this));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
void vtableEntry::print() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
  tty->print("vtableEntry %s: ", method()->name()->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
  if (Verbose) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
    tty->print("m %#lx ", (address)method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
class VtableStats : AllStatic {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
  static int no_klasses;                // # classes with vtables
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
  static int no_array_klasses;          // # array classes
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1133
  static int no_instance_klasses;       // # instanceKlasses
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
  static int sum_of_vtable_len;         // total # of vtable entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
  static int sum_of_array_vtable_len;   // total # of vtable entries in array klasses only
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
  static int fixed;                     // total fixed overhead in bytes
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1137
  static int filler;                    // overhead caused by filler bytes
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
  static int entries;                   // total bytes consumed by vtable entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
  static int array_entries;             // total bytes consumed by array vtable entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1141
  static void do_class(Klass* k) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9172
diff changeset
  1142
    Klass* kl = k;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
    klassVtable* vt = kl->vtable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
    if (vt == NULL) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
    no_klasses++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
    if (kl->oop_is_instance()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
      no_instance_klasses++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
      kl->array_klasses_do(do_class);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
    if (kl->oop_is_array()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
      no_array_klasses++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
      sum_of_array_vtable_len += vt->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
    sum_of_vtable_len += vt->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
  static void compute() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
    SystemDictionary::classes_do(do_class);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
    fixed  = no_klasses * oopSize;      // vtable length
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
    // filler size is a conservative approximation
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
  1161
    filler = oopSize * (no_klasses - no_instance_klasses) * (sizeof(InstanceKlass) - sizeof(ArrayKlass) - 1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
    entries = sizeof(vtableEntry) * sum_of_vtable_len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
    array_entries = sizeof(vtableEntry) * sum_of_array_vtable_len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
int VtableStats::no_klasses = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1168
int VtableStats::no_array_klasses = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
int VtableStats::no_instance_klasses = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
int VtableStats::sum_of_vtable_len = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
int VtableStats::sum_of_array_vtable_len = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
int VtableStats::fixed = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1173
int VtableStats::filler = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
int VtableStats::entries = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
int VtableStats::array_entries = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
void klassVtable::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
  HandleMark hm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
  VtableStats::compute();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
  tty->print_cr("vtable statistics:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1182
  tty->print_cr("%6d classes (%d instance, %d array)", VtableStats::no_klasses, VtableStats::no_instance_klasses, VtableStats::no_array_klasses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
  int total = VtableStats::fixed + VtableStats::filler + VtableStats::entries;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1184
  tty->print_cr("%6d bytes fixed overhead (refs + vtable object header)", VtableStats::fixed);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
  tty->print_cr("%6d bytes filler overhead", VtableStats::filler);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
  tty->print_cr("%6d bytes for vtable entries (%d for arrays)", VtableStats::entries, VtableStats::array_entries);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
  tty->print_cr("%6d bytes total", total);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
int  klassItable::_total_classes;   // Total no. of classes with itables
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
long klassItable::_total_size;      // Total no. of bytes used for itables
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
void klassItable::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
 tty->print_cr("itable statistics:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
 tty->print_cr("%6d classes with itables", _total_classes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
 tty->print_cr("%6d K uses for itables (average by class: %d bytes)", _total_size / K, _total_size / _total_classes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
#endif // PRODUCT