src/hotspot/share/jfr/recorder/repository/jfrRepository.cpp
author coleenp
Thu, 25 Apr 2019 10:56:31 -0400
changeset 54623 1126f0607c70
parent 54263 3cabb47758c9
child 57360 5d043a159d5c
child 58863 c16ac7a2eba4
permissions -rw-r--r--
8222811: Consolidate MutexLockerEx and MutexLocker Summary: Make MutexLocker be MutexLockerEx implementation, remove MutexLockerEx calls. Reviewed-by: dcubed, dholmes, pliden, rehn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
53762
65deccd64f3a 8218935: Make jfr strncpy uses GCC 8.x friendly
mikael
parents: 51106
diff changeset
     2
 * Copyright (c) 2011, 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/jfr.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
#include "jfr/jni/jfrJavaSupport.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
#include "jfr/recorder/jfrRecorder.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
#include "jfr/recorder/repository/jfrChunkState.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
#include "jfr/recorder/repository/jfrChunkWriter.hpp"
54263
3cabb47758c9 8217362: Emergency dump does not work when disk=false is set
mgronlun
parents: 53897
diff changeset
    31
#include "jfr/recorder/repository/jfrEmergencyDump.hpp"
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
#include "jfr/recorder/repository/jfrRepository.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
#include "jfr/recorder/service/jfrPostBox.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
#include "memory/resourceArea.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
#include "runtime/mutex.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
#include "runtime/thread.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
static JfrRepository* _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
JfrRepository& JfrRepository::instance() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
  return *_instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
static JfrChunkWriter* _chunkwriter = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
static bool initialize_chunkwriter() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
  assert(_chunkwriter == NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
  _chunkwriter = new JfrChunkWriter();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
  return _chunkwriter != NULL && _chunkwriter->initialize();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
JfrChunkWriter& JfrRepository::chunkwriter() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
  return *_chunkwriter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
JfrRepository::JfrRepository(JfrPostBox& post_box) : _path(NULL), _post_box(post_box) {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
bool JfrRepository::initialize() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
  return initialize_chunkwriter();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
JfrRepository::~JfrRepository() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
  if (_path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
    JfrCHeapObj::free(_path, strlen(_path) + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
    _path = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
  if (_chunkwriter != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
    delete _chunkwriter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
    _chunkwriter = NULL;
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
JfrRepository* JfrRepository::create(JfrPostBox& post_box) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
  assert(_instance == NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
  _instance = new JfrRepository(post_box);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
  return _instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
void JfrRepository::destroy() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
  assert(_instance != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
  delete _instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
  _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
void JfrRepository::on_vm_error() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
  assert(!JfrStream_lock->owned_by_self(), "invariant");
54263
3cabb47758c9 8217362: Emergency dump does not work when disk=false is set
mgronlun
parents: 53897
diff changeset
    88
  if (_path == NULL) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
    // completed already
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
  }
54263
3cabb47758c9 8217362: Emergency dump does not work when disk=false is set
mgronlun
parents: 53897
diff changeset
    92
  JfrEmergencyDump::on_vm_error(_path);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
bool JfrRepository::set_path(const char* path) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
  assert(path != NULL, "trying to set the repository path with a NULL string!");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
  if (_path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
    // delete existing
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
    JfrCHeapObj::free(_path, strlen(_path) + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
  const size_t path_len = strlen(path);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
  _path = JfrCHeapObj::new_array<char>(path_len + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
  if (_path == NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
    return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
  }
53762
65deccd64f3a 8218935: Make jfr strncpy uses GCC 8.x friendly
mikael
parents: 51106
diff changeset
   106
  strncpy(_path, path, path_len + 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
  return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
void JfrRepository::set_chunk_path(const char* path) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
  assert(JfrStream_lock->owned_by_self(), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
  chunkwriter().set_chunk_path(path);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
void JfrRepository::notify_on_new_chunk_path() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
  if (Jfr::is_recording()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
    instance()._post_box.post(MSG_ROTATE);
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
/**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
* Sets the file where data should be written.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
* Recording  Previous  Current  Action
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
* ==============================================
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
*   true     null      null     Ignore, keep recording in-memory
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
*   true     null      file1    Start disk recording
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
*   true     file      null     Copy out metadata to disk and continue in-memory recording
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
*   true     file1     file2    Copy out metadata and start with new File (file2)
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
*   false     *        null     Ignore, but start recording to memory
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
*   false     *        file     Ignore, but start recording to disk
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
*/
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
void JfrRepository::set_chunk_path(jstring path, JavaThread* jt) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
  DEBUG_ONLY(JfrJavaSupport::check_java_thread_in_vm(jt));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
  ResourceMark rm(jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
  const char* const canonical_chunk_path = JfrJavaSupport::c_str(path, jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
  {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 54263
diff changeset
   138
    MutexLocker stream_lock(JfrStream_lock, Mutex::_no_safepoint_check_flag);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
    if (NULL == canonical_chunk_path && !_chunkwriter->is_valid()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
      // new output is NULL and current output is NULL
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
      return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
    instance().set_chunk_path(canonical_chunk_path);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
  notify_on_new_chunk_path();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
void JfrRepository::set_path(jstring location, JavaThread* jt) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
  DEBUG_ONLY(JfrJavaSupport::check_java_thread_in_vm(jt));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
  ResourceMark rm(jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
  const char* const path = JfrJavaSupport::c_str(location, jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
  if (path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
    instance().set_path(path);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
bool JfrRepository::open_chunk(bool vm_error /* false */) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
  assert(JfrStream_lock->owned_by_self(), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
  if (vm_error) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
    ResourceMark rm;
54263
3cabb47758c9 8217362: Emergency dump does not work when disk=false is set
mgronlun
parents: 53897
diff changeset
   161
    _chunkwriter->set_chunk_path(JfrEmergencyDump::build_dump_path(_path));
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
  return _chunkwriter->open();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
53897
0abec72a3ac2 8217647: JFR: recordings on 32-bit systems unreadable
bulasevich
parents: 53762
diff changeset
   166
size_t JfrRepository::close_chunk(int64_t metadata_offset) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
  return _chunkwriter->close(metadata_offset);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
}