src/hotspot/share/prims/jvmtiGetLoadedClasses.cpp
author poonam
Wed, 07 Feb 2018 21:53:38 +0000
changeset 48882 6b510cb0f14f
parent 47216 71c04702a3d5
child 49192 6734eeef4283
permissions -rw-r--r--
8187577: JVM crash during gc doing concurrent marking Summary: Inform G1's SATB that a klass has been resurrected and it should not be unloaded Reviewed-by: coleenp, tschatzl, kbarrett
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
48882
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
     2
 * Copyright (c) 2003, 2018, 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: 1
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1
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: 1
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"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29081
diff changeset
    27
#include "gc/shared/collectedHeap.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "memory/universe.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "prims/jvmtiGetLoadedClasses.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "runtime/thread.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 22234
diff changeset
    31
#include "utilities/stack.inline.hpp"
48882
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    32
#if INCLUDE_ALL_GCS
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    33
#include "gc/g1/g1SATBCardTableModRefBS.hpp"
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    34
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    37
// The closure for GetLoadedClasses
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    38
class LoadedClassesClosure : public KlassClosure {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    39
private:
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    40
  Stack<jclass, mtInternal> _classStack;
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    41
  JvmtiEnv* _env;
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
    42
  Thread*   _cur_thread;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
48882
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    44
// Tell the GC to keep this klass alive
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    45
static void ensure_klass_alive(oop o) {
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    46
  // A klass that was previously considered dead can be looked up in the
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    47
  // CLD/SD, and its _java_mirror or _class_loader can be stored in a root
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    48
  // or a reachable object making it alive again. The SATB part of G1 needs
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    49
  // to get notified about this potential resurrection, otherwise the marking
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    50
  // might not find the object.
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    51
#if INCLUDE_ALL_GCS
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    52
  if (UseG1GC && o != NULL) {
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    53
    G1SATBCardTableModRefBS::enqueue(o);
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    54
  }
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    55
#endif
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    56
}
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    57
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    58
public:
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
    59
  LoadedClassesClosure(Thread* thread, JvmtiEnv* env) : _cur_thread(thread), _env(env) {
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
    60
    assert(_cur_thread == Thread::current(), "must be current thread");
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    61
  }
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    62
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    63
  void do_klass(Klass* k) {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    64
    // Collect all jclasses
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
    65
    _classStack.push((jclass) _env->jni_reference(Handle(_cur_thread, k->java_mirror())));
48882
6b510cb0f14f 8187577: JVM crash during gc doing concurrent marking
poonam
parents: 47216
diff changeset
    66
    ensure_klass_alive(k->java_mirror());
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    67
  }
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    68
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    69
  int extract(jclass* result_list) {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    70
    // The size of the Stack will be 0 after extract, so get it here
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    71
    int count = (int)_classStack.size();
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    72
    int i = count;
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    73
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    74
    // Pop all jclasses, fill backwards
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    75
    while (!_classStack.is_empty()) {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    76
      result_list[--i] = _classStack.pop();
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    77
    }
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    78
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    79
    // Return the number of elements written
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    80
    return count;
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    81
  }
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    82
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    83
  // Return current size of the Stack
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    84
  int get_count() {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    85
    return (int)_classStack.size();
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    86
  }
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    87
};
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    88
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
    89
// The closure for GetClassLoaderClasses
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
class JvmtiGetLoadedClassesClosure : public StackObj {
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
    91
  // Since the ClassLoaderDataGraph::dictionary_all_entries_do callback
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  // doesn't pass a closureData pointer,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  // we use a thread-local slot to hold a pointer to
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  // a stack allocated instance of this structure.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  jobject _initiatingLoader;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  int     _count;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  Handle* _list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  int     _index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  // Getting and setting the thread local pointer
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  static JvmtiGetLoadedClassesClosure* get_this() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
    JvmtiGetLoadedClassesClosure* result = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
    JavaThread* thread = JavaThread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
    result = thread->get_jvmti_get_loaded_classes_closure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
    return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  static void set_this(JvmtiGetLoadedClassesClosure* that) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
    JavaThread* thread = JavaThread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
    thread->set_jvmti_get_loaded_classes_closure(that);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  // Constructor/Destructor
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  JvmtiGetLoadedClassesClosure() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
    JvmtiGetLoadedClassesClosure* that = get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
    assert(that == NULL, "JvmtiGetLoadedClassesClosure in use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
    _initiatingLoader = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
    _count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
    _list = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
    _index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
    set_this(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
  JvmtiGetLoadedClassesClosure(jobject initiatingLoader) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    JvmtiGetLoadedClassesClosure* that = get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
    assert(that == NULL, "JvmtiGetLoadedClassesClosure in use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    _initiatingLoader = initiatingLoader;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
    _count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
    _list = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
    _index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
    set_this(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
  ~JvmtiGetLoadedClassesClosure() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
    JvmtiGetLoadedClassesClosure* that = get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
    assert(that != NULL, "JvmtiGetLoadedClassesClosure not found");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
    set_this(NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
    _initiatingLoader = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
    _count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
    if (_list != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      FreeHeap(_list);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
      _list = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
    _index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
  // Accessors.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
  jobject get_initiatingLoader() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
    return _initiatingLoader;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  int get_count() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
    return _count;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  void set_count(int value) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
    _count = value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
  Handle* get_list() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
    return _list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
  void set_list(Handle* value) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
    _list = value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
  int get_index() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
    return _index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
  void set_index(int value) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
    _index = value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  Handle get_element(int index) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
    if ((_list != NULL) && (index < _count)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
      return _list[index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
      assert(false, "empty get_element");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
      return Handle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
  void set_element(int index, Handle value) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
    if ((_list != NULL) && (index < _count)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      _list[index] = value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
      assert(false, "bad set_element");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
  // Other predicates
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
  bool available() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
    return (_list != NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
  // For debugging.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  void check(int limit) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
    for (int i = 0; i < limit; i += 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
      assert(Universe::heap()->is_in(get_element(i)()), "check fails");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  // Public methods that get called within the scope of the closure
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
  void allocate() {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 7397
diff changeset
   211
    _list = NEW_C_HEAP_ARRAY(Handle, _count, mtInternal);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
    assert(_list != NULL, "Out of memory");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
    if (_list == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
      _count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  void extract(JvmtiEnv *env, jclass* result) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
    for (int index = 0; index < _count; index += 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
      result[index] = (jclass) env->jni_reference(get_element(index));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   224
  static void increment_with_loader(InstanceKlass* k, ClassLoaderData* loader_data) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
    JvmtiGetLoadedClassesClosure* that = JvmtiGetLoadedClassesClosure::get_this();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   226
    oop class_loader = loader_data->class_loader();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   227
    if (class_loader == JNIHandles::resolve(that->get_initiatingLoader())) {
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 13728
diff changeset
   228
      for (Klass* l = k; l != NULL; l = l->array_klass_or_null()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
        that->set_count(that->get_count() + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   234
  static void add_with_loader(InstanceKlass* k, ClassLoaderData* loader_data) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
    JvmtiGetLoadedClassesClosure* that = JvmtiGetLoadedClassesClosure::get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
    if (that->available()) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   237
      oop class_loader = loader_data->class_loader();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   238
      if (class_loader == JNIHandles::resolve(that->get_initiatingLoader())) {
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
   239
        Thread *thread = Thread::current();
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 13728
diff changeset
   240
        for (Klass* l = k; l != NULL; l = l->array_klass_or_null()) {
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
   241
          Handle mirror(thread, l->java_mirror());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
          that->set_element(that->get_index(), mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
          that->set_index(that->get_index() + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
  // increment the count for the given basic type array class (and any
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  // multi-dimensional arrays). For example, for [B we check for
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  // [[B, [[[B, .. and the count is incremented for each one that exists.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   252
  static void increment_for_basic_type_arrays(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    JvmtiGetLoadedClassesClosure* that = JvmtiGetLoadedClassesClosure::get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    assert(that != NULL, "no JvmtiGetLoadedClassesClosure");
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 13728
diff changeset
   255
    for (Klass* l = k; l != NULL; l = l->array_klass_or_null()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
      that->set_count(that->get_count() + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  // add the basic type array class and its multi-dimensional array classes to the list
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
   261
  static void add_for_basic_type_arrays(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
    JvmtiGetLoadedClassesClosure* that = JvmtiGetLoadedClassesClosure::get_this();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
    assert(that != NULL, "no JvmtiGetLoadedClassesClosure");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
    assert(that->available(), "no list");
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
   265
    Thread *thread = Thread::current();
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 13728
diff changeset
   266
    for (Klass* l = k; l != NULL; l = l->array_klass_or_null()) {
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
   267
      Handle mirror(thread, l->java_mirror());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
      that->set_element(that->get_index(), mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
      that->set_index(that->get_index() + 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
jvmtiError
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
JvmtiGetLoadedClasses::getLoadedClasses(JvmtiEnv *env, jint* classCountPtr, jclass** classesPtr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 30764
diff changeset
   278
  LoadedClassesClosure closure(Thread::current(), env);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    // To get a consistent list of classes we need MultiArray_lock to ensure
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   281
    // array classes aren't created.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
    MutexLocker ma(MultiArray_lock);
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   283
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   284
    // Iterate through all classes in ClassLoaderDataGraph
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   285
    // and collect them using the LoadedClassesClosure
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   286
    ClassLoaderDataGraph::loaded_classes_do(&closure);
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   287
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   289
  // Return results by extracting the collected contents into a list
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   290
  // allocated via JvmtiEnv
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   291
  jclass* result_list;
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   292
  jvmtiError error = env->Allocate(closure.get_count() * sizeof(jclass),
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   293
                               (unsigned char**)&result_list);
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   294
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   295
  if (error == JVMTI_ERROR_NONE) {
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   296
    int count = closure.extract(result_list);
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   297
    *classCountPtr = count;
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   298
    *classesPtr = result_list;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  }
21183
e148e499e5cd 8024423: JVMTI: GetLoadedClasses doesn't enumerate anonymous classes
farvidsson
parents: 14488
diff changeset
   300
  return error;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
jvmtiError
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
JvmtiGetLoadedClasses::getClassLoaderClasses(JvmtiEnv *env, jobject initiatingLoader,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
                                             jint* classCountPtr, jclass** classesPtr) {
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   306
  // Since ClassLoaderDataGraph::dictionary_all_entries_do only takes a function pointer
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
  // and doesn't call back with a closure data pointer,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  // we can only pass static methods.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
  JvmtiGetLoadedClassesClosure closure(initiatingLoader);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
    // To get a consistent list of classes we need MultiArray_lock to ensure
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
    // array classes aren't created, and SystemDictionary_lock to ensure that
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   313
    // classes aren't added to the class loader data dictionaries.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
    MutexLocker ma(MultiArray_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
    MutexLocker sd(SystemDictionary_lock);
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   316
    // First, count the classes in the class loader data dictionaries which have this loader recorded
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
    // as an initiating loader. For basic type arrays this information is not recorded
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
    // so GetClassLoaderClasses will return all of the basic type arrays. This is okay
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
    // because the defining loader for basic type arrays is always the boot class loader
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
    // and these classes are "visible" to all loaders.
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   321
    ClassLoaderDataGraph::dictionary_all_entries_do(&JvmtiGetLoadedClassesClosure::increment_with_loader);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
    Universe::basic_type_classes_do(&JvmtiGetLoadedClassesClosure::increment_for_basic_type_arrays);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
    // Next, fill in the classes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
    closure.allocate();
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46380
diff changeset
   325
    ClassLoaderDataGraph::dictionary_all_entries_do(&JvmtiGetLoadedClassesClosure::add_with_loader);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
    Universe::basic_type_classes_do(&JvmtiGetLoadedClassesClosure::add_for_basic_type_arrays);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
    // Drop the SystemDictionary_lock, so the results could be wrong from here,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
    // but we still have a snapshot.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
  // Post results
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
  jclass* result_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
  jvmtiError err = env->Allocate(closure.get_count() * sizeof(jclass),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
                                 (unsigned char**)&result_list);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
  if (err != JVMTI_ERROR_NONE) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
    return err;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
  closure.extract(env, result_list);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
  *classCountPtr = closure.get_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
  *classesPtr = result_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  return JVMTI_ERROR_NONE;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
}