src/hotspot/share/jfr/recorder/storage/jfrStorageControl.cpp
author mgronlun
Wed, 23 May 2018 15:21:54 +0200
changeset 50234 6ba3e32a9882
parent 50113 caf115bb98ad
child 50429 83aec1d357d4
permissions -rw-r--r--
8203457: Add back missing full buffer notification Reviewed-by: redestad, sjohanss
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     2
 * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
#include "precompiled.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
#include "jfr/recorder/storage/jfrStorageControl.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
#include "runtime/atomic.hpp"
50234
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    28
#include "runtime/mutexLocker.hpp"
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
#include "runtime/orderAccess.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
// returns the updated value
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
static jlong atomic_add(size_t value, size_t volatile* const dest) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
  size_t compare_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
  size_t exchange_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
  do {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
    compare_value = OrderAccess::load_acquire(dest);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
    exchange_value = compare_value + value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
  } while (Atomic::cmpxchg(exchange_value, dest, compare_value) != compare_value);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
  return exchange_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
static jlong atomic_dec(size_t volatile* const dest) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
  size_t compare_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
  size_t exchange_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
  do {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
    compare_value = OrderAccess::load_acquire(dest);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
    assert(compare_value >= 1, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
    exchange_value = compare_value - 1;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
  } while (Atomic::cmpxchg(exchange_value, dest, compare_value) != compare_value);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
  return exchange_value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
const size_t max_lease_factor = 2;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
JfrStorageControl::JfrStorageControl(size_t global_count_total, size_t in_memory_discard_threshold) :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
  _global_count_total(global_count_total),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
  _full_count(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
  _global_lease_count(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
  _dead_count(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
  _to_disk_threshold(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
  _in_memory_discard_threshold(in_memory_discard_threshold),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
  _global_lease_threshold(global_count_total / max_lease_factor),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
  _scavenge_threshold(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
  _to_disk(false) {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
bool JfrStorageControl::to_disk() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
  return _to_disk;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
void JfrStorageControl::set_to_disk(bool enable) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
  _to_disk = enable;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
size_t JfrStorageControl::full_count() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
  return _full_count;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
50234
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    77
// mutexed access
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
size_t JfrStorageControl::increment_full() {
50234
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    79
  assert(JfrBuffer_lock->owned_by_self(), "invariant");
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    80
  return ++_full_count;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
size_t JfrStorageControl::decrement_full() {
50234
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    84
  assert(JfrBuffer_lock->owned_by_self(), "invariant");
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    85
  assert(_full_count > 0, "invariant");
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    86
  return --_full_count;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
void JfrStorageControl::reset_full() {
50234
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    90
  assert(JfrBuffer_lock->owned_by_self(), "invariant");
6ba3e32a9882 8203457: Add back missing full buffer notification
mgronlun
parents: 50113
diff changeset
    91
  _full_count = 0;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
bool JfrStorageControl::should_post_buffer_full_message() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
  return to_disk() && (full_count() > _to_disk_threshold);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
bool JfrStorageControl::should_discard() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
  return !to_disk() && full_count() >= _in_memory_discard_threshold;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
// concurrent with accuracy requirement
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
size_t JfrStorageControl::global_lease_count() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
  return OrderAccess::load_acquire(&_global_lease_count);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
size_t JfrStorageControl::increment_leased() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
  return atomic_add(1, &_global_lease_count);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
size_t JfrStorageControl::decrement_leased() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
  return atomic_dec(&_global_lease_count);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
bool JfrStorageControl::is_global_lease_allowed() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
  return global_lease_count() <= _global_lease_threshold;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
// concurrent with lax requirement
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
size_t JfrStorageControl::dead_count() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
  return _dead_count;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
size_t JfrStorageControl::increment_dead() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
  return atomic_add(1, &_dead_count);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
size_t JfrStorageControl::decrement_dead() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
  return atomic_dec(&_dead_count);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
bool JfrStorageControl::should_scavenge() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
  return dead_count() >= _scavenge_threshold;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
void JfrStorageControl::set_scavenge_threshold(size_t number_of_dead_buffers) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
  _scavenge_threshold = number_of_dead_buffers;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141