src/hotspot/share/runtime/jniHandles.cpp
author coleenp
Wed, 14 Aug 2019 10:07:00 -0400
changeset 57745 789e967c2731
parent 57698 9dc92e89243a
child 57828 35db8fba55f9
permissions -rw-r--r--
5103339: Strengthen NoSafepointVerifier Summary: Add NSV check at possible safepoint transition or places that could take out locks. Consolidate with clearing unhandled oops. Reviewed-by: dholmes, rehn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 53075
diff changeset
     2
 * Copyright (c) 1998, 2019, 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: 4571
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4571
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: 4571
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: 5896
diff changeset
    25
#include "precompiled.hpp"
48816
3495d6050efe 8194898: Move OopStorage inline definitions to an .inline.hpp
kbarrett
parents: 48808
diff changeset
    26
#include "gc/shared/oopStorage.inline.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33148
diff changeset
    27
#include "logging/log.hpp"
37106
0856b64af754 8151539: Remove duplicate AlwaysTrueClosures
stefank
parents: 36398
diff changeset
    28
#include "memory/iterator.hpp"
54786
ebf733a324d4 8223624: Cleanup includes of universe.hpp
stefank
parents: 54623
diff changeset
    29
#include "memory/universe.hpp"
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
    30
#include "oops/access.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5896
diff changeset
    31
#include "oops/oop.inline.hpp"
49393
93fe2fc5c093 8199472: Fix non-PCH build after JDK-8199319
simonis
parents: 49347
diff changeset
    32
#include "runtime/handles.inline.hpp"
49192
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
    33
#include "runtime/jniHandles.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5896
diff changeset
    34
#include "runtime/mutexLocker.hpp"
14583
d70ee55535f4 8003935: Simplify the needed includes for using Thread::current()
stefank
parents: 13728
diff changeset
    35
#include "runtime/thread.inline.hpp"
46625
edefffab74e2 8183552: Move align functions to align.hpp
stefank
parents: 46619
diff changeset
    36
#include "utilities/align.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    37
#include "utilities/debug.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    39
OopStorage* JNIHandles::_global_handles = NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    40
OopStorage* JNIHandles::_weak_global_handles = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    42
OopStorage* JNIHandles::global_handles() {
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    43
  assert(_global_handles != NULL, "Uninitialized JNI global handles");
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    44
  return _global_handles;
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    45
}
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    46
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    47
OopStorage* JNIHandles::weak_global_handles() {
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    48
  assert(_weak_global_handles != NULL, "Uninitialized JNI weak global handles");
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    49
  return _weak_global_handles;
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    50
}
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
    51
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
jobject JNIHandles::make_local(oop obj) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
  if (obj == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
    return NULL;                // ignore null handles
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
    Thread* thread = Thread::current();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    58
    assert(oopDesc::is_oop(obj), "not an oop");
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
    59
    assert(!current_thread_in_native(), "must not be in native");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
    return thread->active_handles()->allocate_handle(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
// optimized versions
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
jobject JNIHandles::make_local(Thread* thread, oop obj) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  if (obj == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
    return NULL;                // ignore null handles
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  } else {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    71
    assert(oopDesc::is_oop(obj), "not an oop");
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
    72
    assert(thread->is_Java_thread(), "not a Java thread");
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
    73
    assert(!current_thread_in_native(), "must not be in native");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
    return thread->active_handles()->allocate_handle(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
jobject JNIHandles::make_local(JNIEnv* env, oop obj) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  if (obj == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
    return NULL;                // ignore null handles
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    JavaThread* thread = JavaThread::thread_from_jni_environment(env);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    84
    assert(oopDesc::is_oop(obj), "not an oop");
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
    85
    assert(!current_thread_in_native(), "must not be in native");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
    return thread->active_handles()->allocate_handle(obj);
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
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    91
static void report_handle_allocation_failure(AllocFailType alloc_failmode,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    92
                                             const char* handle_kind) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    93
  if (alloc_failmode == AllocFailStrategy::EXIT_OOM) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    94
    // Fake size value, since we don't know the min allocation size here.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    95
    vm_exit_out_of_memory(sizeof(oop), OOM_MALLOC_ERROR,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    96
                          "Cannot create %s JNI handle", handle_kind);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    97
  } else {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    98
    assert(alloc_failmode == AllocFailStrategy::RETURN_NULL, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
    99
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   100
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   101
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   102
jobject JNIHandles::make_global(Handle obj, AllocFailType alloc_failmode) {
5896
c60924a89622 6656830: assert((*p)->is_oop(),"expected an oop while scanning weak refs")
never
parents: 5547
diff changeset
   103
  assert(!Universe::heap()->is_gc_active(), "can't extend the root set during GC");
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   104
  assert(!current_thread_in_native(), "must not be in native");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  jobject res = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  if (!obj.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
    // ignore null handles
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   108
    assert(oopDesc::is_oop(obj()), "not an oop");
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   109
    oop* ptr = global_handles()->allocate();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   110
    // Return NULL on allocation failure.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   111
    if (ptr != NULL) {
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   112
      assert(*ptr == NULL, "invariant");
50803
45c1fde86050 8205559: Remove IN_CONCURRENT_ROOT Access decorator
kbarrett
parents: 50728
diff changeset
   113
      NativeAccess<>::oop_store(ptr, obj());
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   114
      res = reinterpret_cast<jobject>(ptr);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   115
    } else {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   116
      report_handle_allocation_failure(alloc_failmode, "global");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   117
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
  return res;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   124
jobject JNIHandles::make_weak_global(Handle obj, AllocFailType alloc_failmode) {
5896
c60924a89622 6656830: assert((*p)->is_oop(),"expected an oop while scanning weak refs")
never
parents: 5547
diff changeset
   125
  assert(!Universe::heap()->is_gc_active(), "can't extend the root set during GC");
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   126
  assert(!current_thread_in_native(), "must not be in native");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
  jobject res = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
  if (!obj.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    // ignore null handles
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   130
    assert(oopDesc::is_oop(obj()), "not an oop");
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   131
    oop* ptr = weak_global_handles()->allocate();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   132
    // Return NULL on allocation failure.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   133
    if (ptr != NULL) {
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   134
      assert(*ptr == NULL, "invariant");
50599
ecc2af326b5f 8204939: Change Access nomenclature: root to native
kbarrett
parents: 50525
diff changeset
   135
      NativeAccess<ON_PHANTOM_OOP_REF>::oop_store(ptr, obj());
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   136
      char* tptr = reinterpret_cast<char*>(ptr) + weak_tag_value;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   137
      res = reinterpret_cast<jobject>(tptr);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   138
    } else {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   139
      report_handle_allocation_failure(alloc_failmode, "weak global");
44406
a46a6c4d1dd9 8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents: 44093
diff changeset
   140
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  return res;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
49192
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   145
// Resolve some erroneous cases to NULL, rather than treating them as
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   146
// possibly unchecked errors.  In particular, deleted handles are
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   147
// treated as NULL (though a deleted and later reallocated handle
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   148
// isn't detected).
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   149
oop JNIHandles::resolve_external_guard(jobject handle) {
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   150
  oop result = NULL;
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   151
  if (handle != NULL) {
53075
747d29313e5a 8215492: Rename INTERNAL_EMPTY to something less "internal"
pliden
parents: 53060
diff changeset
   152
    result = resolve_impl<DECORATORS_NONE, true /* external_guard */>(handle);
49192
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   153
  }
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   154
  return result;
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   155
}
6734eeef4283 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents: 48816
diff changeset
   156
47794
e84aa2c71241 8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents: 47779
diff changeset
   157
bool JNIHandles::is_global_weak_cleared(jweak handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   158
  assert(handle != NULL, "precondition");
47794
e84aa2c71241 8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents: 47779
diff changeset
   159
  assert(is_jweak(handle), "not a weak handle");
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   160
  oop* oop_ptr = jweak_ptr(handle);
50599
ecc2af326b5f 8204939: Change Access nomenclature: root to native
kbarrett
parents: 50525
diff changeset
   161
  oop value = NativeAccess<ON_PHANTOM_OOP_REF | AS_NO_KEEPALIVE>::oop_load(oop_ptr);
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   162
  return value == NULL;
47794
e84aa2c71241 8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents: 47779
diff changeset
   163
}
e84aa2c71241 8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents: 47779
diff changeset
   164
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
void JNIHandles::destroy_global(jobject handle) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
  if (handle != NULL) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   167
    assert(!is_jweak(handle), "wrong method for detroying jweak");
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   168
    oop* oop_ptr = jobject_ptr(handle);
50803
45c1fde86050 8205559: Remove IN_CONCURRENT_ROOT Access decorator
kbarrett
parents: 50728
diff changeset
   169
    NativeAccess<>::oop_store(oop_ptr, (oop)NULL);
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   170
    global_handles()->release(oop_ptr);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
void JNIHandles::destroy_weak_global(jobject handle) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
  if (handle != NULL) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   177
    assert(is_jweak(handle), "JNI handle not jweak");
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   178
    oop* oop_ptr = jweak_ptr(handle);
50599
ecc2af326b5f 8204939: Change Access nomenclature: root to native
kbarrett
parents: 50525
diff changeset
   179
    NativeAccess<ON_PHANTOM_OOP_REF>::oop_store(oop_ptr, (oop)NULL);
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   180
    weak_global_handles()->release(oop_ptr);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
void JNIHandles::oops_do(OopClosure* f) {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   186
  global_handles()->oops_do(f);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
32817
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32607
diff changeset
   190
void JNIHandles::weak_oops_do(BoolObjectClosure* is_alive, OopClosure* f) {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   191
  weak_global_handles()->weak_oops_do(is_alive, f);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
37106
0856b64af754 8151539: Remove duplicate AlwaysTrueClosures
stefank
parents: 36398
diff changeset
   195
void JNIHandles::weak_oops_do(OopClosure* f) {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   196
  weak_global_handles()->weak_oops_do(f);
37106
0856b64af754 8151539: Remove duplicate AlwaysTrueClosures
stefank
parents: 36398
diff changeset
   197
}
0856b64af754 8151539: Remove duplicate AlwaysTrueClosures
stefank
parents: 36398
diff changeset
   198
0856b64af754 8151539: Remove duplicate AlwaysTrueClosures
stefank
parents: 36398
diff changeset
   199
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
void JNIHandles::initialize() {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   201
  _global_handles = new OopStorage("JNI Global",
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   202
                                   JNIGlobalAlloc_lock,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   203
                                   JNIGlobalActive_lock);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   204
  _weak_global_handles = new OopStorage("JNI Weak",
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   205
                                        JNIWeakAlloc_lock,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   206
                                        JNIWeakActive_lock);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   207
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   208
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   209
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   210
inline bool is_storage_handle(const OopStorage* storage, const oop* ptr) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   211
  return storage->allocation_status(ptr) == OopStorage::ALLOCATED_ENTRY;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   212
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   213
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   214
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   215
jobjectRefType JNIHandles::handle_type(Thread* thread, jobject handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   216
  assert(handle != NULL, "precondition");
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   217
  jobjectRefType result = JNIInvalidRefType;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   218
  if (is_jweak(handle)) {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   219
    if (is_storage_handle(weak_global_handles(), jweak_ptr(handle))) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   220
      result = JNIWeakGlobalRefType;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   221
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   222
  } else {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   223
    switch (global_handles()->allocation_status(jobject_ptr(handle))) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   224
    case OopStorage::ALLOCATED_ENTRY:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   225
      result = JNIGlobalRefType;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   226
      break;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   227
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   228
    case OopStorage::UNALLOCATED_ENTRY:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   229
      break;                    // Invalid global handle
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   230
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   231
    case OopStorage::INVALID_ENTRY:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   232
      // Not in global storage.  Might be a local handle.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   233
      if (is_local_handle(thread, handle) ||
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   234
          (thread->is_Java_thread() &&
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   235
           is_frame_handle((JavaThread*)thread, handle))) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   236
        result = JNILocalRefType;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   237
      }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   238
      break;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   239
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   240
    default:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   241
      ShouldNotReachHere();
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   242
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   243
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   244
  return result;
1
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
bool JNIHandles::is_local_handle(Thread* thread, jobject handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   249
  assert(handle != NULL, "precondition");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  JNIHandleBlock* block = thread->active_handles();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  // Look back past possible native calls to jni_PushLocalFrame.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  while (block != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    if (block->chain_contains(handle)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
    block = block->pop_frame_link();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
// Determine if the handle is somewhere in the current thread's stack.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
// We easily can't isolate any particular stack frame the handle might
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
// come from, so we'll check the whole stack.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   267
bool JNIHandles::is_frame_handle(JavaThread* thr, jobject handle) {
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   268
  assert(handle != NULL, "precondition");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  // If there is no java frame, then this must be top level code, such
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  // as the java command executable, in which case, this type of handle
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  // is not permitted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  return (thr->has_last_Java_frame() &&
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   273
         (void*)handle < (void*)thr->stack_base() &&
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   274
         (void*)handle >= (void*)thr->last_Java_sp());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
bool JNIHandles::is_global_handle(jobject handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   279
  assert(handle != NULL, "precondition");
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   280
  return !is_jweak(handle) && is_storage_handle(global_handles(), jobject_ptr(handle));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
bool JNIHandles::is_weak_global_handle(jobject handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   285
  assert(handle != NULL, "precondition");
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   286
  return is_jweak(handle) && is_storage_handle(weak_global_handles(), jweak_ptr(handle));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   289
size_t JNIHandles::global_handle_memory_usage() {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   290
  return global_handles()->total_memory_usage();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   291
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   293
size_t JNIHandles::weak_global_handle_memory_usage() {
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   294
  return weak_global_handles()->total_memory_usage();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   295
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   296
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
// We assume this is called at a safepoint: no lock is needed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
void JNIHandles::print_on(outputStream* st) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   302
  st->print_cr("JNI global refs: " SIZE_FORMAT ", weak refs: " SIZE_FORMAT,
49909
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   303
               global_handles()->allocation_count(),
f276b348ec14 8202230: Provide accessors for JNIHandles storage objects
kbarrett
parents: 49656
diff changeset
   304
               weak_global_handles()->allocation_count());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
  st->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
  st->flush();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
54807
33fe50b6d707 8223626: move print() functions to cpp files
coleenp
parents: 54786
diff changeset
   309
void JNIHandles::print() { print_on(tty); }
33fe50b6d707 8223626: move print() functions to cpp files
coleenp
parents: 54786
diff changeset
   310
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   311
class VerifyJNIHandles: public OopClosure {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
public:
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 1
diff changeset
   313
  virtual void do_oop(oop* root) {
53057
c7ab0761ef88 8215395: Allow null oops in Dictionary and JNIHandle verification
pliden
parents: 52851
diff changeset
   314
    guarantee(oopDesc::is_oop_or_null(RawAccess<>::oop_load(root)), "Invalid oop");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
  }
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 1
diff changeset
   316
  virtual void do_oop(narrowOop* root) { ShouldNotReachHere(); }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
void JNIHandles::verify() {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   320
  VerifyJNIHandles verify_handle;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
  oops_do(&verify_handle);
52851
f0c62b8f73c0 8214784: Adjust Dictionary and JNIHandle verification
pliden
parents: 51268
diff changeset
   323
  weak_oops_do(&verify_handle);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
48465
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   326
// This method is implemented here to avoid circular includes between
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   327
// jniHandles.hpp and thread.hpp.
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   328
bool JNIHandles::current_thread_in_native() {
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   329
  Thread* thread = Thread::current();
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   330
  return (thread->is_Java_thread() &&
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   331
          JavaThread::current()->thread_state() == _thread_in_native);
f82e79958beb 8167372: Add code to check for getting oops while thread is in native
hseigel
parents: 48147
diff changeset
   332
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
void jni_handles_init() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
  JNIHandles::initialize();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
int             JNIHandleBlock::_blocks_allocated     = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
JNIHandleBlock* JNIHandleBlock::_block_free_list      = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
JNIHandleBlock* JNIHandleBlock::_block_list           = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   346
static inline bool is_tagged_free_list(uintptr_t value) {
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   347
  return (value & 1u) != 0;
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   348
}
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   349
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   350
static inline uintptr_t tag_free_list(uintptr_t value) {
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   351
  return value | 1u;
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   352
}
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   353
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   354
static inline uintptr_t untag_free_list(uintptr_t value) {
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   355
  return value & ~(uintptr_t)1u;
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   356
}
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   357
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   358
// There is a freelist of handles running through the JNIHandleBlock
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   359
// with a tagged next pointer, distinguishing these next pointers from
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   360
// oops. The freelist handling currently relies on the size of oops
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   361
// being the same as a native pointer. If this ever changes, then
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   362
// this freelist handling must change too.
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   363
STATIC_ASSERT(sizeof(oop) == sizeof(uintptr_t));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   365
#ifdef ASSERT
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
void JNIHandleBlock::zap() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  // Zap block values
46703
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   368
  _top = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  for (int index = 0; index < block_size_in_oops; index++) {
49656
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   370
    // NOT using Access here; just bare clobbering to NULL, since the
be608cad0b2a 8195972: Refactor oops in JNI to use the Access API
kbarrett
parents: 49393
diff changeset
   371
    // block no longer contains valid oops.
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   372
    _handles[index] = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
}
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   375
#endif // ASSERT
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
JNIHandleBlock* JNIHandleBlock::allocate_block(Thread* thread)  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
  assert(thread == NULL || thread == Thread::current(), "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  JNIHandleBlock* block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
  // Check the thread-local free list for a block so we don't
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
  // have to acquire a mutex.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  if (thread != NULL && thread->free_handle_block() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
    block = thread->free_handle_block();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
    thread->set_free_handle_block(block->_next);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
  else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
    // locking with safepoint checking introduces a potential deadlock:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
    // - we would hold JNIHandleBlockFreeList_lock and then Threads_lock
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
    // - another would hold Threads_lock (jni_AttachCurrentThread) and then
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
    //   JNIHandleBlockFreeList_lock (JNIHandleBlock::allocate_block)
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 53075
diff changeset
   391
    MutexLocker ml(JNIHandleBlockFreeList_lock,
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 53075
diff changeset
   392
                   Mutex::_no_safepoint_check_flag);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
    if (_block_free_list == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
      // Allocate new block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
      block = new JNIHandleBlock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
      _blocks_allocated++;
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   397
      block->zap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
      #ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
      // Link new block to list of all allocated blocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
      block->_block_list_link = _block_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
      _block_list = block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
      #endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
      // Get block from free list
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
      block = _block_free_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
      _block_free_list = _block_free_list->_next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
  }
46703
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   409
  block->_top = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
  block->_next = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
  block->_pop_frame_link = NULL;
25058
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   412
  block->_planned_capacity = block_size_in_oops;
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   413
  // _last, _free_list & _allocate_before_rebuild initialized in allocate_handle
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
  debug_only(block->_last = NULL);
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   415
  debug_only(block->_free_list = NULL);
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   416
  debug_only(block->_allocate_before_rebuild = -1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
  return block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
void JNIHandleBlock::release_block(JNIHandleBlock* block, Thread* thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  assert(thread == NULL || thread == Thread::current(), "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  JNIHandleBlock* pop_frame_link = block->pop_frame_link();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
  // Put returned block at the beginning of the thread-local free list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
  // Note that if thread == NULL, we use it as an implicit argument that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
  // we _don't_ want the block to be kept on the free_handle_block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
  // See for instance JavaThread::exit().
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
  if (thread != NULL ) {
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   429
    block->zap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
    JNIHandleBlock* freelist = thread->free_handle_block();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
    block->_pop_frame_link = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
    thread->set_free_handle_block(block);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
    // Add original freelist to end of chain
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
    if ( freelist != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
      while ( block->_next != NULL ) block = block->_next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
      block->_next = freelist;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
    block = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
  if (block != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
    // Return blocks to free list
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
    // locking with safepoint checking introduces a potential deadlock:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
    // - we would hold JNIHandleBlockFreeList_lock and then Threads_lock
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
    // - another would hold Threads_lock (jni_AttachCurrentThread) and then
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
    //   JNIHandleBlockFreeList_lock (JNIHandleBlock::allocate_block)
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 53075
diff changeset
   447
    MutexLocker ml(JNIHandleBlockFreeList_lock,
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 53075
diff changeset
   448
                   Mutex::_no_safepoint_check_flag);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
    while (block != NULL) {
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   450
      block->zap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
      JNIHandleBlock* next = block->_next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
      block->_next = _block_free_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
      _block_free_list = block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
      block = next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
  if (pop_frame_link != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
    // As a sanity check we release blocks pointed to by the pop_frame_link.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
    // This should never happen (only if PopLocalFrame is not called the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
    // correct number of times).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
    release_block(pop_frame_link, thread);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
void JNIHandleBlock::oops_do(OopClosure* f) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  JNIHandleBlock* current_chain = this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
  // Iterate over chain of blocks, followed by chains linked through the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
  // pop frame links.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  while (current_chain != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
    for (JNIHandleBlock* current = current_chain; current != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
         current = current->_next) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
      assert(current == current_chain || current->pop_frame_link() == NULL,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
        "only blocks first in chain should have pop frame link set");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
      for (int index = 0; index < current->_top; index++) {
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   476
        uintptr_t* addr = &(current->_handles)[index];
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   477
        uintptr_t value = *addr;
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   478
        // traverse heap pointers only, not deleted handles or free list
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   479
        // pointers
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   480
        if (value != 0 && !is_tagged_free_list(value)) {
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   481
          oop* root = (oop*)addr;
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   482
          f->do_oop(root);
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   483
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
      // the next handle block is valid only if current block is full
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
      if (current->_top < block_size_in_oops) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
    current_chain = current_chain->pop_frame_link();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
jobject JNIHandleBlock::allocate_handle(oop obj) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
  assert(Universe::heap()->is_in_reserved(obj), "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
  if (_top == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
    // This is the first allocation or the initial block got zapped when
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
    // entering a native function. If we have any following blocks they are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
    // not valid anymore.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
    for (JNIHandleBlock* current = _next; current != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
         current = current->_next) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
      assert(current->_last == NULL, "only first block should have _last set");
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   504
      assert(current->_free_list == NULL,
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   505
             "only first block should have _free_list set");
46703
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   506
      if (current->_top == 0) {
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   507
        // All blocks after the first clear trailing block are already cleared.
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   508
#ifdef ASSERT
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   509
        for (current = current->_next; current != NULL; current = current->_next) {
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   510
          assert(current->_top == 0, "trailing blocks must already be cleared");
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   511
        }
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   512
#endif
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   513
        break;
f152b500376e 8175318: Performance issue regarding local JNI references
kbarrett
parents: 46625
diff changeset
   514
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
      current->_top = 0;
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   516
      current->zap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
    // Clear initial block
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   519
    _free_list = NULL;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   520
    _allocate_before_rebuild = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
    _last = this;
48147
1b57d99567ed 8191870: Remove badJNIHandle
kbarrett
parents: 47794
diff changeset
   522
    zap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
  // Try last block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
  if (_last->_top < block_size_in_oops) {
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   527
    oop* handle = (oop*)&(_last->_handles)[_last->_top++];
50728
9375184cec98 8205459: Rename Access API flag decorators
kbarrett
parents: 50599
diff changeset
   528
    NativeAccess<IS_DEST_UNINITIALIZED>::oop_store(handle, obj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
    return (jobject) handle;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   532
  // Try free list
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   533
  if (_free_list != NULL) {
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   534
    oop* handle = (oop*)_free_list;
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   535
    _free_list = (uintptr_t*) untag_free_list(*_free_list);
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   536
    NativeAccess<IS_DEST_UNINITIALIZED>::oop_store(handle, obj);
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   537
    return (jobject) handle;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   538
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
  // Check if unused block follow last
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
  if (_last->_next != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
    // update last and retry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
    _last = _last->_next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    return allocate_handle(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   546
  // No space available, we have to rebuild free list or expand
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   547
  if (_allocate_before_rebuild == 0) {
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   548
      rebuild_free_list();        // updates _allocate_before_rebuild counter
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   549
  } else {
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   550
    // Append new block
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   551
    Thread* thread = Thread::current();
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   552
    Handle obj_handle(thread, obj);
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   553
    // This can block, so we need to preserve obj across call.
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   554
    _last->_next = JNIHandleBlock::allocate_block(thread);
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   555
    _last = _last->_next;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   556
    _allocate_before_rebuild--;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   557
    obj = obj_handle();
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   558
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  return allocate_handle(obj);  // retry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   562
void JNIHandleBlock::rebuild_free_list() {
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   563
  assert(_allocate_before_rebuild == 0 && _free_list == NULL, "just checking");
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   564
  int free = 0;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   565
  int blocks = 0;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   566
  for (JNIHandleBlock* current = this; current != NULL; current = current->_next) {
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   567
    for (int index = 0; index < current->_top; index++) {
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   568
      uintptr_t* handle = &(current->_handles)[index];
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   569
      if (*handle == 0) {
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   570
        // this handle was cleared out by a delete call, reuse it
57698
9dc92e89243a 8229027: Improve how JNIHandleBlock::oops_do distinguishes oops from non-oops
eosterlund
parents: 55213
diff changeset
   571
        *handle = _free_list == NULL ? 0 : tag_free_list((uintptr_t)_free_list);
55213
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   572
        _free_list = handle;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   573
        free++;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   574
      }
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   575
    }
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   576
    // we should not rebuild free list if there are unused handles at the end
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   577
    assert(current->_top == block_size_in_oops, "just checking");
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   578
    blocks++;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   579
  }
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   580
  // Heuristic: if more than half of the handles are free we rebuild next time
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   581
  // as well, otherwise we append a corresponding number of new blocks before
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   582
  // attempting a free list rebuild again.
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   583
  int total = blocks * block_size_in_oops;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   584
  int extra = total - 2*free;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   585
  if (extra > 0) {
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   586
    // Not as many free handles as we would like - compute number of new blocks to append
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   587
    _allocate_before_rebuild = (extra + block_size_in_oops - 1) / block_size_in_oops;
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   588
  }
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   589
}
b78597cfcced 8225217: Backout: JDK-8224814: Remove dead JNIHandleBlock freelist code
dholmes
parents: 55153
diff changeset
   590
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
bool JNIHandleBlock::contains(jobject handle) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
  return ((jobject)&_handles[0] <= handle && handle<(jobject)&_handles[_top]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
bool JNIHandleBlock::chain_contains(jobject handle) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
  for (JNIHandleBlock* current = (JNIHandleBlock*) this; current != NULL; current = current->_next) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    if (current->contains(handle)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   607
size_t JNIHandleBlock::length() const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   608
  size_t result = 1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
  for (JNIHandleBlock* current = _next; current != NULL; current = current->_next) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
    result++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
  return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   615
class CountJNIHandleClosure: public OopClosure {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   616
private:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   617
  int _count;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   618
public:
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   619
  CountJNIHandleClosure(): _count(0) {}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   620
  virtual void do_oop(oop* ooph) { _count++; }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   621
  virtual void do_oop(narrowOop* unused) { ShouldNotReachHere(); }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   622
  int count() { return _count; }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   623
};
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   624
25058
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   625
const size_t JNIHandleBlock::get_number_of_live_handles() {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   626
  CountJNIHandleClosure counter;
25058
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   627
  oops_do(&counter);
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   628
  return counter.count();
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   629
}
4542f853c2ac 8043224: -Xcheck:jni improvements to exception checking and excessive local refs
dsimms
parents: 24424
diff changeset
   630
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 22234
diff changeset
   631
// This method is not thread-safe, i.e., must be called while holding a lock on the
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
// structure.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   633
size_t JNIHandleBlock::memory_usage() const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
  return length() * sizeof(JNIHandleBlock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   640
bool JNIHandles::is_local_handle(jobject handle) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   641
  return JNIHandleBlock::any_contains(handle);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   642
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents: 48465
diff changeset
   643
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
bool JNIHandleBlock::any_contains(jobject handle) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   645
  assert(handle != NULL, "precondition");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
  for (JNIHandleBlock* current = _block_list; current != NULL; current = current->_block_list_link) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
    if (current->contains(handle)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
void JNIHandleBlock::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
  int used_blocks = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
  int free_blocks = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
  int used_handles = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
  int free_handles = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
  JNIHandleBlock* block = _block_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
  while (block != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
    if (block->_top > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
      used_blocks++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
      free_blocks++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
    used_handles += block->_top;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
    free_handles += (block_size_in_oops - block->_top);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
    block = block->_block_list_link;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
  tty->print_cr("JNIHandleBlocks statistics");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
  tty->print_cr("- blocks allocated: %d", used_blocks + free_blocks);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
  tty->print_cr("- blocks in use:    %d", used_blocks);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
  tty->print_cr("- blocks free:      %d", free_blocks);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
  tty->print_cr("- handles in use:   %d", used_handles);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
  tty->print_cr("- handles free:     %d", free_handles);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
#endif