src/hotspot/share/jfr/recorder/service/jfrPostBox.cpp
author egahlin
Fri, 17 May 2019 16:02:27 +0200
branchJEP-349-branch
changeset 57360 5d043a159d5c
parent 54645 05aaccf7d558
child 58159 892527a70da9
permissions -rw-r--r--
Preview
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 50429
diff changeset
     2
 * Copyright (c) 2013, 2019, Oracle and/or its affiliates. All rights reserved.
50113
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/service/jfrPostBox.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
#include "jfr/utilities/jfrTryLock.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
#include "runtime/atomic.hpp"
50429
83aec1d357d4 8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents: 50113
diff changeset
    29
#include "runtime/orderAccess.hpp"
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
#include "runtime/thread.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
#define MSG_IS_SYNCHRONOUS ( (MSGBIT(MSG_ROTATE)) |          \
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
                             (MSGBIT(MSG_STOP))   |          \
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
                             (MSGBIT(MSG_START))  |          \
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
                             (MSGBIT(MSG_CLONE_IN_MEMORY)) | \
57360
5d043a159d5c Preview
egahlin
parents: 54645
diff changeset
    36
                             (MSGBIT(MSG_VM_ERROR))        | \
5d043a159d5c Preview
egahlin
parents: 54645
diff changeset
    37
                             (MSGBIT(MSG_FLUSHPOINT))      | \
5d043a159d5c Preview
egahlin
parents: 54645
diff changeset
    38
                             (MSGBIT(MSG_FLUSHPOINT_METADATA)) \
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
                           )
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
static JfrPostBox* _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
JfrPostBox& JfrPostBox::instance() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
  return *_instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
JfrPostBox* JfrPostBox::create() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
  assert(_instance == NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
  _instance = new JfrPostBox();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
  return _instance;
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
void JfrPostBox::destroy() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
  assert(_instance != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
  delete _instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
  _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
JfrPostBox::JfrPostBox() :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
  _msg_read_serial(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
  _msg_handled_serial(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
  _messages(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
  _has_waiters(false) {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
static bool is_thread_lock_aversive() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
  Thread* const thread = Thread::current();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
  return (thread->is_Java_thread() && ((JavaThread*)thread)->thread_state() != _thread_in_vm) || thread->is_VM_thread();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
static bool is_synchronous(int messages) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
  return ((messages & MSG_IS_SYNCHRONOUS) != 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
void JfrPostBox::post(JFR_Msg msg) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
  const int the_message = MSGBIT(msg);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
  if (is_thread_lock_aversive()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
    deposit(the_message);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
  if (!is_synchronous(the_message)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
    asynchronous_post(the_message);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
  synchronous_post(the_message);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
void JfrPostBox::deposit(int new_messages) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
  while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
    const int current_msgs = OrderAccess::load_acquire(&_messages);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
    // OR the new message
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
    const int exchange_value = current_msgs | new_messages;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
    const int result = Atomic::cmpxchg(exchange_value, &_messages, current_msgs);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
    if (result == current_msgs) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
      return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
    /* Some other thread just set exactly what this thread wanted */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
    if ((result & new_messages) == new_messages) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
      return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
    }
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
void JfrPostBox::asynchronous_post(int msg) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
  assert(!is_synchronous(msg), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
  deposit(msg);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
  JfrMonitorTryLock try_msg_lock(JfrMsg_lock);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
  if (try_msg_lock.acquired()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
    JfrMsg_lock->notify_all();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
  }
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
void JfrPostBox::synchronous_post(int msg) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
  assert(is_synchronous(msg), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
  assert(!JfrMsg_lock->owned_by_self(), "should not hold JfrMsg_lock here!");
54645
05aaccf7d558 8222988: Use MonitorLocker rather than MutexLocker when wait/notify used
coleenp
parents: 54623
diff changeset
   115
  MonitorLocker msg_lock(JfrMsg_lock);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
  deposit(msg);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
  // serial_id is used to check when what we send in has been processed.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
  // _msg_read_serial is read under JfrMsg_lock protection.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
  const uintptr_t serial_id = OrderAccess::load_acquire(&_msg_read_serial) + 1;
54645
05aaccf7d558 8222988: Use MonitorLocker rather than MutexLocker when wait/notify used
coleenp
parents: 54623
diff changeset
   120
  msg_lock.notify_all();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
  while (!is_message_processed(serial_id)) {
54645
05aaccf7d558 8222988: Use MonitorLocker rather than MutexLocker when wait/notify used
coleenp
parents: 54623
diff changeset
   122
    msg_lock.wait();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
  }
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
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
 * Check if a synchronous message has been processed.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
 * We avoid racing on _msg_handled_serial by ensuring
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
 * that we are holding the JfrMsg_lock when checking
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
 * completion status.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
bool JfrPostBox::is_message_processed(uintptr_t serial_id) const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
  assert(JfrMsg_lock->owned_by_self(), "_msg_handled_serial must be read under JfrMsg_lock protection");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
  return serial_id <= OrderAccess::load_acquire(&_msg_handled_serial);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
bool JfrPostBox::is_empty() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
  assert(JfrMsg_lock->owned_by_self(), "not holding JfrMsg_lock!");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
  return OrderAccess::load_acquire(&_messages) == 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
int JfrPostBox::collect() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
  // get pending and reset to 0
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
  const int messages = Atomic::xchg(0, &_messages);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
  if (check_waiters(messages)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
    _has_waiters = true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
    assert(JfrMsg_lock->owned_by_self(), "incrementing _msg_read_serial is protected by JfrMsg_lock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
    // Update made visible on release of JfrMsg_lock via fence instruction in Monitor::IUnlock.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
    ++_msg_read_serial;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
  return messages;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
bool JfrPostBox::check_waiters(int messages) const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
  assert(JfrMsg_lock->owned_by_self(), "not holding JfrMsg_lock!");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
  assert(!_has_waiters, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
  return is_synchronous(messages);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
void JfrPostBox::notify_waiters() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
  if (!_has_waiters) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
  _has_waiters = false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
  assert(JfrMsg_lock->owned_by_self(), "incrementing _msg_handled_serial is protected by JfrMsg_lock.");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
  // Update made visible on release of JfrMsg_lock via fence instruction in Monitor::IUnlock.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
  ++_msg_handled_serial;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
  JfrMsg_lock->notify();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
// safeguard to ensure no threads are left waiting
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
void JfrPostBox::notify_collection_stop() {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 50429
diff changeset
   173
  MutexLocker msg_lock(JfrMsg_lock);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
  JfrMsg_lock->notify_all();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
}