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