src/hotspot/share/gc/g1/g1FreeIdSet.cpp
author stefank
Mon, 25 Nov 2019 12:33:15 +0100
changeset 59252 623722a6aeb9
parent 53482 771b50dd0b08
permissions -rw-r--r--
8234740: Harmonize parameter order in Atomic - cmpxchg Reviewed-by: rehn, dholmes
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
53482
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     1
/*
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     2
 * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     4
 *
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     7
 * published by the Free Software Foundation.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     8
 *
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    13
 * accompanied this code).
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    14
 *
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    18
 *
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    21
 * questions.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    22
 *
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    23
 */
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    24
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    25
#include "precompiled.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    26
#include "gc/g1/g1FreeIdSet.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    27
#include "memory/allocation.inline.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    28
#include "runtime/atomic.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    29
#include "utilities/debug.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    30
#include "utilities/globalDefinitions.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    31
#include "utilities/macros.hpp"
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    32
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    33
G1FreeIdSet::G1FreeIdSet(uint start, uint size) :
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    34
  _sem(size),          // counting semaphore for available ids
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    35
  _next(NULL),         // array of "next" indices
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    36
  _start(start),       // first id value
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    37
  _size(size),         // number of available ids
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    38
  _head_index_mask(0), // mask for extracting index from a _head value.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    39
  _head(0)             // low part: index; high part: update counter
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    40
{
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    41
  assert(size != 0, "precondition");
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    42
  assert(start <= (UINT_MAX - size),
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    43
         "start (%u) + size (%u) overflow: ", start, size);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    44
  // 2^shift must be greater than size. Equal is not permitted, because
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    45
  // size is the "end of list" value, and can be the index part of _head.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    46
  uint shift = log2_intptr((uintptr_t)size) + 1;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    47
  assert(shift <= (BitsPerWord / 2), "excessive size %u", size);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    48
  _head_index_mask = (uintx(1) << shift) - 1;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    49
  assert(size <= _head_index_mask, "invariant");
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    50
  _next = NEW_C_HEAP_ARRAY(uint, size, mtGC);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    51
  for (uint i = 0; i < size; ++i) {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    52
    _next[i] = i + 1;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    53
  }
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    54
}
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    55
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    56
G1FreeIdSet::~G1FreeIdSet() {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    57
  FREE_C_HEAP_ARRAY(uint, _next);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    58
}
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    59
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    60
uint G1FreeIdSet::head_index(uintx head) const {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    61
  return head & _head_index_mask;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    62
}
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    63
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    64
uintx G1FreeIdSet::make_head(uint index, uintx old_head) const {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    65
  // Include incremented old update counter to avoid ABA problem.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    66
  return index | ((old_head & ~_head_index_mask) + 1 + _head_index_mask);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    67
}
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    68
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    69
const uint Claimed = UINT_MAX;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    70
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    71
uint G1FreeIdSet::claim_par_id() {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    72
  _sem.wait();
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    73
  // Semaphore gate permits passage by no more than the number of
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    74
  // available ids, so there must be one that we can claim.  But there
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    75
  // may be multiple threads trying to claim ids at the same time.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    76
  uintx old_head = Atomic::load(&_head);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    77
  uint index;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    78
  while (true) {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    79
    index = head_index(old_head);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    80
    assert(index < _size, "invariant");
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    81
    uintx new_head = make_head(_next[index], old_head);
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 53482
diff changeset
    82
    new_head = Atomic::cmpxchg(&_head, old_head, new_head);
53482
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    83
    if (new_head == old_head) break;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    84
    old_head = new_head;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    85
  }
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    86
  DEBUG_ONLY(_next[index] = Claimed;)
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    87
  return _start + index;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    88
}
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    89
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    90
void G1FreeIdSet::release_par_id(uint id) {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    91
  uint index = id - _start;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    92
  assert(index < _size, "invalid id %u", id);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    93
  assert(_next[index] == Claimed, "precondition");
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    94
  uintx old_head = Atomic::load(&_head);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    95
  while (true) {
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    96
    _next[index] = head_index(old_head);
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    97
    uintx new_head = make_head(index, old_head);
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 53482
diff changeset
    98
    new_head = Atomic::cmpxchg(&_head, old_head, new_head);
53482
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
    99
    if (new_head == old_head) break;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
   100
    old_head = new_head;
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
   101
  }
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
   102
  // Now that id has been released, permit another thread through the gate.
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
   103
  _sem.signal();
771b50dd0b08 8216258: Make FreeIdSet semaphore-based
kbarrett
parents:
diff changeset
   104
}