src/hotspot/share/gc/shared/oopStorage.cpp
author coleenp
Wed, 01 May 2019 08:56:38 -0400
changeset 54663 f03d5a093093
parent 54623 1126f0607c70
child 55569 8e3a0ebf3497
child 58678 9cf78a70fa4f
permissions -rw-r--r--
8074355: make MutexLocker smarter about non-JavaThreads Summary: Use safepoint_check_always/safepoint_check_never instead of safepoint_check_sometimes for locks that are taken by JavaThreads and non-JavaThreads Reviewed-by: dholmes, rehn, dcubed, lmesnik
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     1
/*
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
     2
 * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     4
 *
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     7
 * published by the Free Software Foundation.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     8
 *
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    13
 * accompanied this code).
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    14
 *
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    18
 *
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    21
 * questions.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    22
 *
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    23
 */
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    24
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
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"
3495d6050efe 8194898: Move OopStorage inline definitions to an .inline.hpp
kbarrett
parents: 48808
diff changeset
    27
#include "gc/shared/oopStorageParState.inline.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    28
#include "logging/log.hpp"
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
    29
#include "logging/logStream.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    30
#include "memory/allocation.inline.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    31
#include "runtime/atomic.hpp"
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
    32
#include "runtime/globals.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    33
#include "runtime/handles.inline.hpp"
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
    34
#include "runtime/interfaceSupport.inline.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    35
#include "runtime/mutex.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    36
#include "runtime/mutexLocker.hpp"
50429
83aec1d357d4 8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents: 50332
diff changeset
    37
#include "runtime/orderAccess.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    38
#include "runtime/safepoint.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    39
#include "runtime/stubRoutines.hpp"
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
    40
#include "runtime/thread.hpp"
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    41
#include "utilities/align.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    42
#include "utilities/count_trailing_zeros.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    43
#include "utilities/debug.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    44
#include "utilities/globalDefinitions.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    45
#include "utilities/macros.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    46
#include "utilities/ostream.hpp"
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    47
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    48
OopStorage::AllocationListEntry::AllocationListEntry() : _prev(NULL), _next(NULL) {}
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    49
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    50
OopStorage::AllocationListEntry::~AllocationListEntry() {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    51
  assert(_prev == NULL, "deleting attached block");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    52
  assert(_next == NULL, "deleting attached block");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    53
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    54
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    55
OopStorage::AllocationList::AllocationList() : _head(NULL), _tail(NULL) {}
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    56
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    57
OopStorage::AllocationList::~AllocationList() {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    58
  // ~OopStorage() empties its lists before destroying them.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    59
  assert(_head == NULL, "deleting non-empty block list");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    60
  assert(_tail == NULL, "deleting non-empty block list");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    61
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    62
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    63
void OopStorage::AllocationList::push_front(const Block& block) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    64
  const Block* old = _head;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    65
  if (old == NULL) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    66
    assert(_tail == NULL, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    67
    _head = _tail = █
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    68
  } else {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    69
    block.allocation_list_entry()._next = old;
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    70
    old->allocation_list_entry()._prev = █
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    71
    _head = █
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    72
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    73
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    74
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    75
void OopStorage::AllocationList::push_back(const Block& block) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    76
  const Block* old = _tail;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    77
  if (old == NULL) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    78
    assert(_head == NULL, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    79
    _head = _tail = █
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    80
  } else {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    81
    old->allocation_list_entry()._next = █
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    82
    block.allocation_list_entry()._prev = old;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    83
    _tail = █
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    84
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    85
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    86
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    87
void OopStorage::AllocationList::unlink(const Block& block) {
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    88
  const AllocationListEntry& block_entry = block.allocation_list_entry();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    89
  const Block* prev_blk = block_entry._prev;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    90
  const Block* next_blk = block_entry._next;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    91
  block_entry._prev = NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    92
  block_entry._next = NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    93
  if ((prev_blk == NULL) && (next_blk == NULL)) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    94
    assert(_head == &block, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    95
    assert(_tail == &block, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    96
    _head = _tail = NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    97
  } else if (prev_blk == NULL) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
    98
    assert(_head == &block, "invariant");
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
    99
    next_blk->allocation_list_entry()._prev = NULL;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   100
    _head = next_blk;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   101
  } else if (next_blk == NULL) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   102
    assert(_tail == &block, "invariant");
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   103
    prev_blk->allocation_list_entry()._next = NULL;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   104
    _tail = prev_blk;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   105
  } else {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   106
    next_blk->allocation_list_entry()._prev = prev_blk;
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   107
    prev_blk->allocation_list_entry()._next = next_blk;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   108
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   109
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   110
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   111
OopStorage::ActiveArray::ActiveArray(size_t size) :
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   112
  _size(size),
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   113
  _block_count(0),
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   114
  _refcount(0)
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   115
{}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   116
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   117
OopStorage::ActiveArray::~ActiveArray() {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   118
  assert(_refcount == 0, "precondition");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   119
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   120
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   121
OopStorage::ActiveArray* OopStorage::ActiveArray::create(size_t size, AllocFailType alloc_fail) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   122
  size_t size_in_bytes = blocks_offset() + sizeof(Block*) * size;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   123
  void* mem = NEW_C_HEAP_ARRAY3(char, size_in_bytes, mtGC, CURRENT_PC, alloc_fail);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   124
  if (mem == NULL) return NULL;
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   125
  return new (mem) ActiveArray(size);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   126
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   127
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   128
void OopStorage::ActiveArray::destroy(ActiveArray* ba) {
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   129
  ba->~ActiveArray();
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   130
  FREE_C_HEAP_ARRAY(char, ba);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   131
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   132
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   133
size_t OopStorage::ActiveArray::size() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   134
  return _size;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   135
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   136
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   137
size_t OopStorage::ActiveArray::block_count() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   138
  return _block_count;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   139
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   140
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   141
size_t OopStorage::ActiveArray::block_count_acquire() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   142
  return OrderAccess::load_acquire(&_block_count);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   143
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   144
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   145
void OopStorage::ActiveArray::increment_refcount() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   146
  int new_value = Atomic::add(1, &_refcount);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   147
  assert(new_value >= 1, "negative refcount %d", new_value - 1);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   148
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   149
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   150
bool OopStorage::ActiveArray::decrement_refcount() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   151
  int new_value = Atomic::sub(1, &_refcount);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   152
  assert(new_value >= 0, "negative refcount %d", new_value);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   153
  return new_value == 0;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   154
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   155
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   156
bool OopStorage::ActiveArray::push(Block* block) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   157
  size_t index = _block_count;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   158
  if (index < _size) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   159
    block->set_active_index(index);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   160
    *block_ptr(index) = block;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   161
    // Use a release_store to ensure all the setup is complete before
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   162
    // making the block visible.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   163
    OrderAccess::release_store(&_block_count, index + 1);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   164
    return true;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   165
  } else {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   166
    return false;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   167
  }
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   168
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   169
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   170
void OopStorage::ActiveArray::remove(Block* block) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   171
  assert(_block_count > 0, "array is empty");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   172
  size_t index = block->active_index();
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   173
  assert(*block_ptr(index) == block, "block not present");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   174
  size_t last_index = _block_count - 1;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   175
  Block* last_block = *block_ptr(last_index);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   176
  last_block->set_active_index(index);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   177
  *block_ptr(index) = last_block;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   178
  _block_count = last_index;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   179
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   180
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   181
void OopStorage::ActiveArray::copy_from(const ActiveArray* from) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   182
  assert(_block_count == 0, "array must be empty");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   183
  size_t count = from->_block_count;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   184
  assert(count <= _size, "precondition");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   185
  Block* const* from_ptr = from->block_ptr(0);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   186
  Block** to_ptr = block_ptr(0);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   187
  for (size_t i = 0; i < count; ++i) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   188
    Block* block = *from_ptr++;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   189
    assert(block->active_index() == i, "invariant");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   190
    *to_ptr++ = block;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   191
  }
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   192
  _block_count = count;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   193
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   194
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   195
// Blocks start with an array of BitsPerWord oop entries.  That array
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   196
// is divided into conceptual BytesPerWord sections of BitsPerByte
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   197
// entries.  Blocks are allocated aligned on section boundaries, for
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   198
// the convenience of mapping from an entry to the containing block;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   199
// see block_for_ptr().  Aligning on section boundary rather than on
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   200
// the full _data wastes a lot less space, but makes for a bit more
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   201
// work in block_for_ptr().
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   202
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   203
const unsigned section_size = BitsPerByte;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   204
const unsigned section_count = BytesPerWord;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   205
const unsigned block_alignment = sizeof(oop) * section_size;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   206
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   207
OopStorage::Block::Block(const OopStorage* owner, void* memory) :
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   208
  _data(),
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   209
  _allocated_bitmask(0),
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   210
  _owner(owner),
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   211
  _memory(memory),
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   212
  _active_index(0),
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   213
  _allocation_list_entry(),
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   214
  _deferred_updates_next(NULL),
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   215
  _release_refcount(0)
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   216
{
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   217
  STATIC_ASSERT(_data_pos == 0);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   218
  STATIC_ASSERT(section_size * section_count == ARRAY_SIZE(_data));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   219
  assert(offset_of(Block, _data) == _data_pos, "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   220
  assert(owner != NULL, "NULL owner");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   221
  assert(is_aligned(this, block_alignment), "misaligned block");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   222
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   223
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   224
OopStorage::Block::~Block() {
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   225
  assert(_release_refcount == 0, "deleting block while releasing");
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   226
  assert(_deferred_updates_next == NULL, "deleting block with deferred update");
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   227
  // Clear fields used by block_for_ptr and entry validation, which
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   228
  // might help catch bugs.  Volatile to prevent dead-store elimination.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   229
  const_cast<uintx volatile&>(_allocated_bitmask) = 0;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   230
  const_cast<OopStorage* volatile&>(_owner) = NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   231
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   232
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   233
size_t OopStorage::Block::allocation_size() {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   234
  // _data must be first member, so aligning Block aligns _data.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   235
  STATIC_ASSERT(_data_pos == 0);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   236
  return sizeof(Block) + block_alignment - sizeof(void*);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   237
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   238
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   239
size_t OopStorage::Block::allocation_alignment_shift() {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   240
  return exact_log2(block_alignment);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   241
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   242
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   243
inline bool is_full_bitmask(uintx bitmask) { return ~bitmask == 0; }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   244
inline bool is_empty_bitmask(uintx bitmask) { return bitmask == 0; }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   245
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   246
bool OopStorage::Block::is_full() const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   247
  return is_full_bitmask(allocated_bitmask());
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   248
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   249
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   250
bool OopStorage::Block::is_empty() const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   251
  return is_empty_bitmask(allocated_bitmask());
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   252
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   253
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   254
uintx OopStorage::Block::bitmask_for_entry(const oop* ptr) const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   255
  return bitmask_for_index(get_index(ptr));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   256
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   257
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   258
// An empty block is not yet deletable if either:
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   259
// (1) There is a release() operation currently operating on it.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   260
// (2) It is in the deferred updates list.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   261
// For interaction with release(), these must follow the empty check,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   262
// and the order of these checks is important.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   263
bool OopStorage::Block::is_safe_to_delete() const {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   264
  assert(is_empty(), "precondition");
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   265
  OrderAccess::loadload();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   266
  return (OrderAccess::load_acquire(&_release_refcount) == 0) &&
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   267
         (OrderAccess::load_acquire(&_deferred_updates_next) == NULL);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   268
}
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   269
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   270
OopStorage::Block* OopStorage::Block::deferred_updates_next() const {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   271
  return _deferred_updates_next;
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   272
}
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   273
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   274
void OopStorage::Block::set_deferred_updates_next(Block* block) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   275
  _deferred_updates_next = block;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   276
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   277
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   278
bool OopStorage::Block::contains(const oop* ptr) const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   279
  const oop* base = get_pointer(0);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   280
  return (base <= ptr) && (ptr < (base + ARRAY_SIZE(_data)));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   281
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   282
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   283
size_t OopStorage::Block::active_index() const {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   284
  return _active_index;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   285
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   286
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   287
void OopStorage::Block::set_active_index(size_t index) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   288
  _active_index = index;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   289
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   290
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   291
size_t OopStorage::Block::active_index_safe(const Block* block) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   292
  STATIC_ASSERT(sizeof(intptr_t) == sizeof(block->_active_index));
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   293
  assert(CanUseSafeFetchN(), "precondition");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   294
  return SafeFetchN((intptr_t*)&block->_active_index, 0);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   295
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   296
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   297
unsigned OopStorage::Block::get_index(const oop* ptr) const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   298
  assert(contains(ptr), PTR_FORMAT " not in block " PTR_FORMAT, p2i(ptr), p2i(this));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   299
  return static_cast<unsigned>(ptr - get_pointer(0));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   300
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   301
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   302
oop* OopStorage::Block::allocate() {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   303
  // Use CAS loop because release may change bitmask outside of lock.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   304
  uintx allocated = allocated_bitmask();
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   305
  while (true) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   306
    assert(!is_full_bitmask(allocated), "attempt to allocate from full block");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   307
    unsigned index = count_trailing_zeros(~allocated);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   308
    uintx new_value = allocated | bitmask_for_index(index);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   309
    uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, allocated);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   310
    if (fetched == allocated) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   311
      return get_pointer(index); // CAS succeeded; return entry for index.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   312
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   313
    allocated = fetched;       // CAS failed; retry with latest value.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   314
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   315
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   316
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   317
OopStorage::Block* OopStorage::Block::new_block(const OopStorage* owner) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   318
  // _data must be first member: aligning block => aligning _data.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   319
  STATIC_ASSERT(_data_pos == 0);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   320
  size_t size_needed = allocation_size();
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   321
  void* memory = NEW_C_HEAP_ARRAY_RETURN_NULL(char, size_needed, mtGC);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   322
  if (memory == NULL) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   323
    return NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   324
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   325
  void* block_mem = align_up(memory, block_alignment);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   326
  assert(sizeof(Block) + pointer_delta(block_mem, memory, 1) <= size_needed,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   327
         "allocated insufficient space for aligned block");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   328
  return ::new (block_mem) Block(owner, memory);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   329
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   330
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   331
void OopStorage::Block::delete_block(const Block& block) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   332
  void* memory = block._memory;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   333
  block.Block::~Block();
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   334
  FREE_C_HEAP_ARRAY(char, memory);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   335
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   336
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   337
// This can return a false positive if ptr is not contained by some
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   338
// block.  For some uses, it is a precondition that ptr is valid,
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   339
// e.g. contained in some block in owner's _active_array.  Other uses
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   340
// require additional validation of the result.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   341
OopStorage::Block*
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   342
OopStorage::Block::block_for_ptr(const OopStorage* owner, const oop* ptr) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   343
  assert(CanUseSafeFetchN(), "precondition");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   344
  STATIC_ASSERT(_data_pos == 0);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   345
  // Const-ness of ptr is not related to const-ness of containing block.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   346
  // Blocks are allocated section-aligned, so get the containing section.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   347
  oop* section_start = align_down(const_cast<oop*>(ptr), block_alignment);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   348
  // Start with a guess that the containing section is the last section,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   349
  // so the block starts section_count-1 sections earlier.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   350
  oop* section = section_start - (section_size * (section_count - 1));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   351
  // Walk up through the potential block start positions, looking for
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   352
  // the owner in the expected location.  If we're below the actual block
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   353
  // start position, the value at the owner position will be some oop
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   354
  // (possibly NULL), which can never match the owner.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   355
  intptr_t owner_addr = reinterpret_cast<intptr_t>(owner);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   356
  for (unsigned i = 0; i < section_count; ++i, section += section_size) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   357
    Block* candidate = reinterpret_cast<Block*>(section);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   358
    intptr_t* candidate_owner_addr
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   359
      = reinterpret_cast<intptr_t*>(&candidate->_owner);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   360
    if (SafeFetchN(candidate_owner_addr, 0) == owner_addr) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   361
      return candidate;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   362
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   363
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   364
  return NULL;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   365
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   366
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   367
//////////////////////////////////////////////////////////////////////////////
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   368
// Allocation
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   369
//
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   370
// Allocation involves the _allocation_list, which contains a subset of the
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   371
// blocks owned by a storage object.  This is a doubly-linked list, linked
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   372
// through dedicated fields in the blocks.  Full blocks are removed from this
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   373
// list, though they are still present in the _active_array.  Empty blocks are
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   374
// kept at the end of the _allocation_list, to make it easy for empty block
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   375
// deletion to find them.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   376
//
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   377
// allocate(), and delete_empty_blocks() lock the
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   378
// _allocation_mutex while performing any list and array modifications.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   379
//
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   380
// allocate() and release() update a block's _allocated_bitmask using CAS
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   381
// loops.  This prevents loss of updates even though release() performs
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   382
// its updates without any locking.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   383
//
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   384
// allocate() obtains the entry from the first block in the _allocation_list,
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   385
// and updates that block's _allocated_bitmask to indicate the entry is in
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   386
// use.  If this makes the block full (all entries in use), the block is
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   387
// removed from the _allocation_list so it won't be considered by future
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   388
// allocations until some entries in it are released.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   389
//
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   390
// release() is performed lock-free. (Note: This means it can't notify the
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   391
// service thread of pending cleanup work.  It must be lock-free because
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   392
// it is called in all kinds of contexts where even quite low ranked locks
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   393
// may be held.)  release() first looks up the block for
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   394
// the entry, using address alignment to find the enclosing block (thereby
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   395
// avoiding iteration over the _active_array).  Once the block has been
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   396
// determined, its _allocated_bitmask needs to be updated, and its position in
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   397
// the _allocation_list may need to be updated.  There are two cases:
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   398
//
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   399
// (a) If the block is neither full nor would become empty with the release of
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   400
// the entry, only its _allocated_bitmask needs to be updated.  But if the CAS
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   401
// update fails, the applicable case may change for the retry.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   402
//
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   403
// (b) Otherwise, the _allocation_list also needs to be modified.  This requires
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   404
// locking the _allocation_mutex.  To keep the release() operation lock-free,
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   405
// rather than updating the _allocation_list itself, it instead performs a
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   406
// lock-free push of the block onto the _deferred_updates list.  Entries on
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   407
// that list are processed by allocate() and delete_empty_blocks(), while
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   408
// they already hold the necessary lock.  That processing makes the block's
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   409
// list state consistent with its current _allocated_bitmask.  The block is
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   410
// added to the _allocation_list if not already present and the bitmask is not
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   411
// full.  The block is moved to the end of the _allocation_list if the bitmask
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   412
// is empty, for ease of empty block deletion processing.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   413
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   414
oop* OopStorage::allocate() {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   415
  MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   416
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   417
  // Note: Without this we might never perform cleanup.  As it is,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   418
  // cleanup is only requested here, when completing a concurrent
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   419
  // iteration, or when someone entirely else wakes up the service
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   420
  // thread, which isn't ideal.  But we can't notify in release().
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   421
  if (reduce_deferred_updates()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   422
    notify_needs_cleanup();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   423
  }
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   424
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   425
  Block* block = block_for_allocation();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   426
  if (block == NULL) return NULL; // Block allocation failed.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   427
  assert(!block->is_full(), "invariant");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   428
  if (block->is_empty()) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   429
    // Transitioning from empty to not empty.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   430
    log_debug(oopstorage, blocks)("%s: block not empty " PTR_FORMAT, name(), p2i(block));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   431
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   432
  oop* result = block->allocate();
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   433
  assert(result != NULL, "allocation failed");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   434
  assert(!block->is_empty(), "postcondition");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   435
  Atomic::inc(&_allocation_count); // release updates outside lock.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   436
  if (block->is_full()) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   437
    // Transitioning from not full to full.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   438
    // Remove full blocks from consideration by future allocates.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   439
    log_debug(oopstorage, blocks)("%s: block full " PTR_FORMAT, name(), p2i(block));
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   440
    _allocation_list.unlink(*block);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   441
  }
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   442
  log_trace(oopstorage, ref)("%s: allocated " PTR_FORMAT, name(), p2i(result));
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   443
  return result;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   444
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   445
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   446
bool OopStorage::try_add_block() {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   447
  assert_lock_strong(_allocation_mutex);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   448
  Block* block;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   449
  {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   450
    MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   451
    block = Block::new_block(this);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   452
  }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   453
  if (block == NULL) return false;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   454
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   455
  // Add new block to the _active_array, growing if needed.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   456
  if (!_active_array->push(block)) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   457
    if (expand_active_array()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   458
      guarantee(_active_array->push(block), "push failed after expansion");
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   459
    } else {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   460
      log_debug(oopstorage, blocks)("%s: failed active array expand", name());
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   461
      Block::delete_block(*block);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   462
      return false;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   463
    }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   464
  }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   465
  // Add to end of _allocation_list.  The mutex release allowed other
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   466
  // threads to add blocks to the _allocation_list.  We prefer to
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   467
  // allocate from non-empty blocks, to allow empty blocks to be
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   468
  // deleted.  But we don't bother notifying about the empty block
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   469
  // because we're (probably) about to allocate an entry from it.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   470
  _allocation_list.push_back(*block);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   471
  log_debug(oopstorage, blocks)("%s: new block " PTR_FORMAT, name(), p2i(block));
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   472
  return true;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   473
}
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   474
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   475
OopStorage::Block* OopStorage::block_for_allocation() {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   476
  assert_lock_strong(_allocation_mutex);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   477
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   478
  while (true) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   479
    // Use the first block in _allocation_list for the allocation.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   480
    Block* block = _allocation_list.head();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   481
    if (block != NULL) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   482
      return block;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   483
    } else if (reduce_deferred_updates()) {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   484
      MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   485
      notify_needs_cleanup();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   486
    } else if (try_add_block()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   487
      block = _allocation_list.head();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   488
      assert(block != NULL, "invariant");
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   489
      return block;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   490
    } else if (reduce_deferred_updates()) { // Once more before failure.
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   491
      MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   492
      notify_needs_cleanup();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   493
    } else {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   494
      // Attempt to add a block failed, no other thread added a block,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   495
      // and no deferred updated added a block, then allocation failed.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   496
      log_debug(oopstorage, blocks)("%s: failed block allocation", name());
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   497
      return NULL;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   498
    }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   499
  }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   500
}
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   501
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   502
// Create a new, larger, active array with the same content as the
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   503
// current array, and then replace, relinquishing the old array.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   504
// Return true if the array was successfully expanded, false to
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   505
// indicate allocation failure.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   506
bool OopStorage::expand_active_array() {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   507
  assert_lock_strong(_allocation_mutex);
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   508
  ActiveArray* old_array = _active_array;
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   509
  size_t new_size = 2 * old_array->size();
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   510
  log_debug(oopstorage, blocks)("%s: expand active array " SIZE_FORMAT,
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   511
                                name(), new_size);
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   512
  ActiveArray* new_array = ActiveArray::create(new_size, AllocFailStrategy::RETURN_NULL);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   513
  if (new_array == NULL) return false;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   514
  new_array->copy_from(old_array);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   515
  replace_active_array(new_array);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   516
  relinquish_block_array(old_array);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   517
  return true;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   518
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   519
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   520
// Make new_array the _active_array.  Increments new_array's refcount
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   521
// to account for the new reference.  The assignment is atomic wrto
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   522
// obtain_active_array; once this function returns, it is safe for the
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   523
// caller to relinquish the old array.
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   524
void OopStorage::replace_active_array(ActiveArray* new_array) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   525
  // Caller has the old array that is the current value of _active_array.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   526
  // Update new_array refcount to account for the new reference.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   527
  new_array->increment_refcount();
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   528
  // Install new_array, ensuring its initialization is complete first.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   529
  OrderAccess::release_store(&_active_array, new_array);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   530
  // Wait for any readers that could read the old array from _active_array.
51511
eb8d5aeabab3 8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents: 50954
diff changeset
   531
  // Can't use GlobalCounter here, because this is called from allocate(),
eb8d5aeabab3 8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents: 50954
diff changeset
   532
  // which may be called in the scope of a GlobalCounter critical section
eb8d5aeabab3 8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents: 50954
diff changeset
   533
  // when inserting a StringTable entry.
eb8d5aeabab3 8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents: 50954
diff changeset
   534
  _protect_active.synchronize();
50332
d0d933d61610 8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents: 50300
diff changeset
   535
  // All obtain critical sections that could see the old array have
d0d933d61610 8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents: 50300
diff changeset
   536
  // completed, having incremented the refcount of the old array.  The
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   537
  // caller can now safely relinquish the old array.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   538
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   539
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   540
// Atomically (wrto replace_active_array) get the active array and
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   541
// increment its refcount.  This provides safe access to the array,
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   542
// even if an allocate operation expands and replaces the value of
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   543
// _active_array.  The caller must relinquish the array when done
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   544
// using it.
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   545
OopStorage::ActiveArray* OopStorage::obtain_active_array() const {
51511
eb8d5aeabab3 8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents: 50954
diff changeset
   546
  SingleWriterSynchronizer::CriticalSection cs(&_protect_active);
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   547
  ActiveArray* result = OrderAccess::load_acquire(&_active_array);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   548
  result->increment_refcount();
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   549
  return result;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   550
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   551
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   552
// Decrement refcount of array and destroy if refcount is zero.
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   553
void OopStorage::relinquish_block_array(ActiveArray* array) const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   554
  if (array->decrement_refcount()) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   555
    assert(array != _active_array, "invariant");
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   556
    ActiveArray::destroy(array);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   557
  }
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   558
}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   559
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   560
class OopStorage::WithActiveArray : public StackObj {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   561
  const OopStorage* _storage;
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   562
  ActiveArray* _active_array;
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   563
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   564
public:
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   565
  WithActiveArray(const OopStorage* storage) :
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   566
    _storage(storage),
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   567
    _active_array(storage->obtain_active_array())
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   568
  {}
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   569
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   570
  ~WithActiveArray() {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   571
    _storage->relinquish_block_array(_active_array);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   572
  }
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   573
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   574
  ActiveArray& active_array() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   575
    return *_active_array;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   576
  }
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   577
};
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   578
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   579
OopStorage::Block* OopStorage::find_block_or_null(const oop* ptr) const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   580
  assert(ptr != NULL, "precondition");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   581
  return Block::block_for_ptr(this, ptr);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   582
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   583
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   584
static void log_release_transitions(uintx releasing,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   585
                                    uintx old_allocated,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   586
                                    const OopStorage* owner,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   587
                                    const void* block) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   588
  Log(oopstorage, blocks) log;
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   589
  LogStream ls(log.debug());
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   590
  if (is_full_bitmask(old_allocated)) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   591
    ls.print_cr("%s: block not full " PTR_FORMAT, owner->name(), p2i(block));
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   592
  }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   593
  if (releasing == old_allocated) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   594
    ls.print_cr("%s: block empty " PTR_FORMAT, owner->name(), p2i(block));
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   595
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   596
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   597
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   598
void OopStorage::Block::release_entries(uintx releasing, OopStorage* owner) {
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   599
  assert(releasing != 0, "preconditon");
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   600
  // Prevent empty block deletion when transitioning to empty.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   601
  Atomic::inc(&_release_refcount);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   602
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   603
  // Atomically update allocated bitmask.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   604
  uintx old_allocated = _allocated_bitmask;
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   605
  while (true) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   606
    assert((releasing & ~old_allocated) == 0, "releasing unallocated entries");
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   607
    uintx new_value = old_allocated ^ releasing;
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   608
    uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, old_allocated);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   609
    if (fetched == old_allocated) break; // Successful update.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   610
    old_allocated = fetched;             // Retry with updated bitmask.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   611
  }
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   612
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   613
  // Now that the bitmask has been updated, if we have a state transition
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   614
  // (updated bitmask is empty or old bitmask was full), atomically push
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   615
  // this block onto the deferred updates list.  Some future call to
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   616
  // reduce_deferred_updates will make any needed changes related to this
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   617
  // block and _allocation_list.  This deferral avoids _allocation_list
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   618
  // updates and the associated locking here.
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   619
  if ((releasing == old_allocated) || is_full_bitmask(old_allocated)) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   620
    // Log transitions.  Both transitions are possible in a single update.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   621
    if (log_is_enabled(Debug, oopstorage, blocks)) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   622
      log_release_transitions(releasing, old_allocated, _owner, this);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   623
    }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   624
    // Attempt to claim responsibility for adding this block to the deferred
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   625
    // list, by setting the link to non-NULL by self-looping.  If this fails,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   626
    // then someone else has made such a claim and the deferred update has not
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   627
    // yet been processed and will include our change, so we don't need to do
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   628
    // anything further.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   629
    if (Atomic::replace_if_null(this, &_deferred_updates_next)) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   630
      // Successfully claimed.  Push, with self-loop for end-of-list.
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   631
      Block* head = owner->_deferred_updates;
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   632
      while (true) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   633
        _deferred_updates_next = (head == NULL) ? this : head;
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   634
        Block* fetched = Atomic::cmpxchg(this, &owner->_deferred_updates, head);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   635
        if (fetched == head) break; // Successful update.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   636
        head = fetched;             // Retry with updated head.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   637
      }
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   638
      owner->record_needs_cleanup();
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   639
      log_debug(oopstorage, blocks)("%s: deferred update " PTR_FORMAT,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   640
                                    _owner->name(), p2i(this));
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   641
    }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   642
  }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   643
  // Release hold on empty block deletion.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   644
  Atomic::dec(&_release_refcount);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   645
}
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   646
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   647
// Process one available deferred update.  Returns true if one was processed.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   648
bool OopStorage::reduce_deferred_updates() {
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   649
  assert_lock_strong(_allocation_mutex);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   650
  // Atomically pop a block off the list, if any available.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   651
  // No ABA issue because this is only called by one thread at a time.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   652
  // The atomicity is wrto pushes by release().
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   653
  Block* block = OrderAccess::load_acquire(&_deferred_updates);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   654
  while (true) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   655
    if (block == NULL) return false;
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   656
    // Try atomic pop of block from list.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   657
    Block* tail = block->deferred_updates_next();
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   658
    if (block == tail) tail = NULL; // Handle self-loop end marker.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   659
    Block* fetched = Atomic::cmpxchg(tail, &_deferred_updates, block);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   660
    if (fetched == block) break; // Update successful.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   661
    block = fetched;             // Retry with updated block.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   662
  }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   663
  block->set_deferred_updates_next(NULL); // Clear tail after updating head.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   664
  // Ensure bitmask read after pop is complete, including clearing tail, for
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   665
  // ordering with release().  Without this, we may be processing a stale
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   666
  // bitmask state here while blocking a release() operation from recording
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   667
  // the deferred update needed for its bitmask change.
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   668
  OrderAccess::fence();
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   669
  // Process popped block.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   670
  uintx allocated = block->allocated_bitmask();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   671
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   672
  // Make membership in list consistent with bitmask state.
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   673
  if ((_allocation_list.ctail() != NULL) &&
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   674
      ((_allocation_list.ctail() == block) ||
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   675
       (_allocation_list.next(*block) != NULL))) {
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   676
    // Block is in the _allocation_list.
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   677
    assert(!is_full_bitmask(allocated), "invariant");
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   678
  } else if (!is_full_bitmask(allocated)) {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   679
    // Block is not in the _allocation_list, but now should be.
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   680
    _allocation_list.push_front(*block);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   681
  } // Else block is full and not in list, which is correct.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   682
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   683
  // Move empty block to end of list, for possible deletion.
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   684
  if (is_empty_bitmask(allocated)) {
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   685
    _allocation_list.unlink(*block);
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   686
    _allocation_list.push_back(*block);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   687
    notify_needs_cleanup();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   688
  }
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   689
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   690
  log_debug(oopstorage, blocks)("%s: processed deferred update " PTR_FORMAT,
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   691
                                name(), p2i(block));
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   692
  return true;              // Processed one pending update.
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   693
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   694
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   695
inline void check_release_entry(const oop* entry) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   696
  assert(entry != NULL, "Releasing NULL");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   697
  assert(*entry == NULL, "Releasing uncleared entry: " PTR_FORMAT, p2i(entry));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   698
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   699
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   700
void OopStorage::release(const oop* ptr) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   701
  check_release_entry(ptr);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   702
  Block* block = find_block_or_null(ptr);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   703
  assert(block != NULL, "%s: invalid release " PTR_FORMAT, name(), p2i(ptr));
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   704
  log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(ptr));
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   705
  block->release_entries(block->bitmask_for_entry(ptr), this);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   706
  Atomic::dec(&_allocation_count);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   707
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   708
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   709
void OopStorage::release(const oop* const* ptrs, size_t size) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   710
  size_t i = 0;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   711
  while (i < size) {
48808
2b0b7f222800 8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents: 48787
diff changeset
   712
    check_release_entry(ptrs[i]);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   713
    Block* block = find_block_or_null(ptrs[i]);
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   714
    assert(block != NULL, "%s: invalid release " PTR_FORMAT, name(), p2i(ptrs[i]));
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   715
    log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(ptrs[i]));
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   716
    size_t count = 0;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   717
    uintx releasing = 0;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   718
    for ( ; i < size; ++i) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   719
      const oop* entry = ptrs[i];
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   720
      check_release_entry(entry);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   721
      // If entry not in block, finish block and resume outer loop with entry.
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   722
      if (!block->contains(entry)) break;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   723
      // Add entry to releasing bitmap.
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   724
      log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(entry));
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   725
      uintx entry_bitmask = block->bitmask_for_entry(entry);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   726
      assert((releasing & entry_bitmask) == 0,
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   727
             "Duplicate entry: " PTR_FORMAT, p2i(entry));
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   728
      releasing |= entry_bitmask;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   729
      ++count;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   730
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   731
    // Release the contiguous entries that are in block.
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   732
    block->release_entries(releasing, this);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   733
    Atomic::sub(count, &_allocation_count);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   734
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   735
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   736
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   737
const char* dup_name(const char* name) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   738
  char* dup = NEW_C_HEAP_ARRAY(char, strlen(name) + 1, mtGC);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   739
  strcpy(dup, name);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   740
  return dup;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   741
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   742
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   743
// Possible values for OopStorage::_needs_cleanup.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   744
const uint needs_cleanup_none = 0;     // No cleanup needed.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   745
const uint needs_cleanup_marked = 1;   // Requested, but no notification made.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   746
const uint needs_cleanup_notified = 2; // Requested and Service thread notified.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   747
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   748
const size_t initial_active_array_size = 8;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   749
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   750
OopStorage::OopStorage(const char* name,
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   751
                       Mutex* allocation_mutex,
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   752
                       Mutex* active_mutex) :
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   753
  _name(dup_name(name)),
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   754
  _active_array(ActiveArray::create(initial_active_array_size)),
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   755
  _allocation_list(),
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   756
  _deferred_updates(NULL),
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   757
  _allocation_mutex(allocation_mutex),
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   758
  _active_mutex(active_mutex),
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   759
  _allocation_count(0),
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   760
  _concurrent_iteration_count(0),
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   761
  _needs_cleanup(needs_cleanup_none)
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   762
{
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   763
  _active_array->increment_refcount();
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   764
  assert(_active_mutex->rank() < _allocation_mutex->rank(),
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   765
         "%s: active_mutex must have lower rank than allocation_mutex", _name);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   766
  assert(Service_lock->rank() < _active_mutex->rank(),
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   767
         "%s: active_mutex must have higher rank than Service_lock", _name);
54663
f03d5a093093 8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents: 54623
diff changeset
   768
  assert(_active_mutex->_safepoint_check_required == Mutex::_safepoint_check_never,
f03d5a093093 8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents: 54623
diff changeset
   769
         "%s: active mutex requires never safepoint check", _name);
f03d5a093093 8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents: 54623
diff changeset
   770
  assert(_allocation_mutex->_safepoint_check_required == Mutex::_safepoint_check_never,
f03d5a093093 8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents: 54623
diff changeset
   771
         "%s: allocation mutex requires never safepoint check", _name);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   772
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   773
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   774
void OopStorage::delete_empty_block(const Block& block) {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   775
  assert(block.is_empty(), "discarding non-empty block");
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   776
  log_debug(oopstorage, blocks)("%s: delete empty block " PTR_FORMAT, name(), p2i(&block));
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   777
  Block::delete_block(block);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   778
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   779
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   780
OopStorage::~OopStorage() {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   781
  Block* block;
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   782
  while ((block = _deferred_updates) != NULL) {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   783
    _deferred_updates = block->deferred_updates_next();
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   784
    block->set_deferred_updates_next(NULL);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   785
  }
50954
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   786
  while ((block = _allocation_list.head()) != NULL) {
f85092465b0c 8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents: 50513
diff changeset
   787
    _allocation_list.unlink(*block);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   788
  }
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   789
  bool unreferenced = _active_array->decrement_refcount();
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   790
  assert(unreferenced, "deleting storage while _active_array is referenced");
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   791
  for (size_t i = _active_array->block_count(); 0 < i; ) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   792
    block = _active_array->at(--i);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   793
    Block::delete_block(*block);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   794
  }
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   795
  ActiveArray::destroy(_active_array);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   796
  FREE_C_HEAP_ARRAY(char, _name);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   797
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   798
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   799
// Called by service thread to check for pending work.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   800
bool OopStorage::needs_delete_empty_blocks() const {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   801
  return Atomic::load(&_needs_cleanup) != needs_cleanup_none;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   802
}
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   803
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   804
// Record that cleanup is needed, without notifying the Service thread.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   805
// Used by release(), where we can't lock even Service_lock.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   806
void OopStorage::record_needs_cleanup() {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   807
  Atomic::cmpxchg(needs_cleanup_marked, &_needs_cleanup, needs_cleanup_none);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   808
}
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   809
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   810
// Record that cleanup is needed, and notify the Service thread.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   811
void OopStorage::notify_needs_cleanup() {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   812
  // Avoid re-notification if already notified.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   813
  const uint notified = needs_cleanup_notified;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   814
  if (Atomic::xchg(notified, &_needs_cleanup) != notified) {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   815
    MonitorLocker ml(Service_lock, Monitor::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   816
    ml.notify_all();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   817
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   818
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   819
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   820
bool OopStorage::delete_empty_blocks() {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   821
  MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   822
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   823
  // Clear the request before processing.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   824
  Atomic::store(needs_cleanup_none, &_needs_cleanup);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   825
  OrderAccess::fence();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   826
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   827
  // Other threads could be adding to the empty block count or the
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   828
  // deferred update list while we're working.  Set an upper bound on
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   829
  // how many updates we'll process and blocks we'll try to release,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   830
  // so other threads can't cause an unbounded stay in this function.
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   831
  size_t limit = block_count();
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   832
  if (limit == 0) return false; // Empty storage; nothing at all to do.
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   833
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   834
  for (size_t i = 0; i < limit; ++i) {
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   835
    // Process deferred updates, which might make empty blocks available.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   836
    // Continue checking once deletion starts, since additional updates
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   837
    // might become available while we're working.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   838
    if (reduce_deferred_updates()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   839
      // Be safepoint-polite while looping.
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   840
      MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   841
      ThreadBlockInVM tbiv(JavaThread::current());
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   842
    } else {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   843
      Block* block = _allocation_list.tail();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   844
      if ((block == NULL) || !block->is_empty()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   845
        return false;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   846
      } else if (!block->is_safe_to_delete()) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   847
        // Look for other work while waiting for block to be deletable.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   848
        break;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   849
      }
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   850
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   851
      // Try to delete the block.  First, try to remove from _active_array.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   852
      {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   853
        MutexLocker aml(_active_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   854
        // Don't interfere with an active concurrent iteration.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   855
        // Instead, give up immediately.  There is more work to do,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   856
        // but don't re-notify, to avoid useless spinning of the
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   857
        // service thread.  Instead, iteration completion notifies.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   858
        if (_concurrent_iteration_count > 0) return true;
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   859
        _active_array->remove(block);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   860
      }
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   861
      // Remove block from _allocation_list and delete it.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   862
      _allocation_list.unlink(*block);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   863
      // Be safepoint-polite while deleting and looping.
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   864
      MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   865
      delete_empty_block(*block);
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   866
      ThreadBlockInVM tbiv(JavaThread::current());
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   867
    }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   868
  }
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   869
  // Exceeded work limit or can't delete last block.  This will
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   870
  // cause the service thread to loop, giving other subtasks an
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   871
  // opportunity to run too.  There's no need for a notification,
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   872
  // because we are part of the service thread (unless gtesting).
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   873
  record_needs_cleanup();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   874
  return true;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   875
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   876
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   877
OopStorage::EntryStatus OopStorage::allocation_status(const oop* ptr) const {
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   878
  const Block* block = find_block_or_null(ptr);
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   879
  if (block != NULL) {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   880
    // Prevent block deletion and _active_array modification.
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   881
    MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   882
    // Block could be a false positive, so get index carefully.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   883
    size_t index = Block::active_index_safe(block);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   884
    if ((index < _active_array->block_count()) &&
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   885
        (block == _active_array->at(index)) &&
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   886
        block->contains(ptr)) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   887
      if ((block->allocated_bitmask() & block->bitmask_for_entry(ptr)) != 0) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   888
        return ALLOCATED_ENTRY;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   889
      } else {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   890
        return UNALLOCATED_ENTRY;
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   891
      }
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   892
    }
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   893
  }
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
   894
  return INVALID_ENTRY;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   895
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   896
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   897
size_t OopStorage::allocation_count() const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   898
  return _allocation_count;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   899
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   900
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   901
size_t OopStorage::block_count() const {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   902
  WithActiveArray wab(this);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   903
  // Count access is racy, but don't care.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   904
  return wab.active_array().block_count();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   905
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   906
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   907
size_t OopStorage::total_memory_usage() const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   908
  size_t total_size = sizeof(OopStorage);
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   909
  total_size += strlen(name()) + 1;
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   910
  total_size += sizeof(ActiveArray);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   911
  WithActiveArray wab(this);
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   912
  const ActiveArray& blocks = wab.active_array();
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   913
  // Count access is racy, but don't care.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   914
  total_size += blocks.block_count() * Block::allocation_size();
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   915
  total_size += blocks.size() * sizeof(Block*);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   916
  return total_size;
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   917
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   918
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   919
// Parallel iteration support
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   920
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   921
uint OopStorage::BasicParState::default_estimated_thread_count(bool concurrent) {
50298
188a87cbfac3 8203843: BasicParState::default_estimated_thread_count(false) can return 0 in gtest
kbarrett
parents: 50209
diff changeset
   922
  uint configured = concurrent ? ConcGCThreads : ParallelGCThreads;
188a87cbfac3 8203843: BasicParState::default_estimated_thread_count(false) can return 0 in gtest
kbarrett
parents: 50209
diff changeset
   923
  return MAX2(1u, configured);  // Never estimate zero threads.
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   924
}
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   925
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   926
OopStorage::BasicParState::BasicParState(const OopStorage* storage,
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   927
                                         uint estimated_thread_count,
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   928
                                         bool concurrent) :
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   929
  _storage(storage),
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   930
  _active_array(_storage->obtain_active_array()),
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   931
  _block_count(0),              // initialized properly below
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   932
  _next_block(0),
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   933
  _estimated_thread_count(estimated_thread_count),
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   934
  _concurrent(concurrent)
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   935
{
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   936
  assert(estimated_thread_count > 0, "estimated thread count must be positive");
52037
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
   937
  update_concurrent_iteration_count(1);
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   938
  // Get the block count *after* iteration state updated, so concurrent
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   939
  // empty block deletion is suppressed and can't reduce the count.  But
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   940
  // ensure the count we use was written after the block with that count
50209
2fdce199fcb9 8202863: Rename OopStorage inner collection classes
kbarrett
parents: 49977
diff changeset
   941
  // was fully initialized; see ActiveArray::push.
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   942
  _block_count = _active_array->block_count_acquire();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   943
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   944
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   945
OopStorage::BasicParState::~BasicParState() {
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   946
  _storage->relinquish_block_array(_active_array);
52037
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
   947
  update_concurrent_iteration_count(-1);
52421
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   948
  if (_concurrent) {
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   949
    // We may have deferred some work.
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   950
    const_cast<OopStorage*>(_storage)->notify_needs_cleanup();
3021c1ad958b 8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents: 52037
diff changeset
   951
  }
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   952
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   953
52037
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
   954
void OopStorage::BasicParState::update_concurrent_iteration_count(int value) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   955
  if (_concurrent) {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 52421
diff changeset
   956
    MutexLocker ml(_storage->_active_mutex, Mutex::_no_safepoint_check_flag);
52037
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
   957
    _storage->_concurrent_iteration_count += value;
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
   958
    assert(_storage->_concurrent_iteration_count >= 0, "invariant");
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   959
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   960
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   961
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   962
bool OopStorage::BasicParState::claim_next_segment(IterationData* data) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   963
  data->_processed += data->_segment_end - data->_segment_start;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   964
  size_t start = OrderAccess::load_acquire(&_next_block);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   965
  if (start >= _block_count) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   966
    return finish_iteration(data); // No more blocks available.
49333
489f1dd40582 8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents: 48886
diff changeset
   967
  }
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   968
  // Try to claim several at a time, but not *too* many.  We want to
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   969
  // avoid deciding there are many available and selecting a large
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   970
  // quantity, get delayed, and then end up claiming most or all of
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   971
  // the remaining largish amount of work, leaving nothing for other
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   972
  // threads to do.  But too small a step can lead to contention
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   973
  // over _next_block, esp. when the work per block is small.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   974
  size_t max_step = 10;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   975
  size_t remaining = _block_count - start;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   976
  size_t step = MIN2(max_step, 1 + (remaining / _estimated_thread_count));
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   977
  // Atomic::add with possible overshoot.  This can perform better
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   978
  // than a CAS loop on some platforms when there is contention.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   979
  // We can cope with the uncertainty by recomputing start/end from
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   980
  // the result of the add, and dealing with potential overshoot.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   981
  size_t end = Atomic::add(step, &_next_block);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   982
  // _next_block may have changed, so recompute start from result of add.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   983
  start = end - step;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   984
  // _next_block may have changed so much that end has overshot.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   985
  end = MIN2(end, _block_count);
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   986
  // _next_block may have changed so much that even start has overshot.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   987
  if (start < _block_count) {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   988
    // Record claimed segment for iteration.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   989
    data->_segment_start = start;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   990
    data->_segment_end = end;
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   991
    return true;                // Success.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   992
  } else {
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   993
    // No more blocks to claim.
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   994
    return finish_iteration(data);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   995
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   996
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
   997
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
   998
bool OopStorage::BasicParState::finish_iteration(const IterationData* data) const {
51752
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
   999
  log_info(oopstorage, blocks, stats)
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
  1000
          ("Parallel iteration on %s: blocks = " SIZE_FORMAT
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
  1001
           ", processed = " SIZE_FORMAT " (%2.f%%)",
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
  1002
           _storage->name(), _block_count, data->_processed,
43323ced5e40 8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents: 51511
diff changeset
  1003
           percent_of(data->_processed, _block_count));
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
  1004
  return false;
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1005
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1006
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1007
const char* OopStorage::name() const { return _name; }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1008
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1009
#ifndef PRODUCT
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1010
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1011
void OopStorage::print_on(outputStream* st) const {
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1012
  size_t allocations = _allocation_count;
49977
9f758f0bb058 8200557: OopStorage parallel iteration scales poorly
kbarrett
parents: 49711
diff changeset
  1013
  size_t blocks = _active_array->block_count();
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1014
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1015
  double data_size = section_size * section_count;
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
  1016
  double alloc_percentage = percent_of((double)allocations, blocks * data_size);
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1017
48886
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
  1018
  st->print("%s: " SIZE_FORMAT " entries in " SIZE_FORMAT " blocks (%.F%%), " SIZE_FORMAT " bytes",
e1d09bd56d2d 8196083: Avoid locking in OopStorage::release
kbarrett
parents: 48816
diff changeset
  1019
            name(), allocations, blocks, alloc_percentage, total_memory_usage());
52037
d2a6c3cbc110 8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents: 51752
diff changeset
  1020
  if (_concurrent_iteration_count > 0) {
48787
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1021
    st->print(", concurrent iteration active");
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1022
  }
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1023
}
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1024
7638bf98a312 8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff changeset
  1025
#endif // !PRODUCT