src/hotspot/share/jfr/recorder/repository/jfrChunkState.cpp
author mgronlun
Sat, 14 Sep 2019 14:40:09 +0200
changeset 58132 caa25ab47aca
parent 53897 0abec72a3ac2
permissions -rw-r--r--
8225797: OldObjectSample event creates unexpected amount of checkpoint data Reviewed-by: egahlin
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: 50113
diff changeset
     2
 * Copyright (c) 2012, 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/dcmd/jfrDcmds.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
#include "jfr/recorder/jfrRecorder.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
#include "jfr/recorder/repository/jfrChunkState.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
#include "jfr/recorder/repository/jfrChunkWriter.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
#include "jfr/utilities/jfrTime.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
#include "jfr/utilities/jfrTimeConverter.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
#include "logging/log.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
#include "runtime/os.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
#include "runtime/thread.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
JfrChunkState::JfrChunkState() :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
  _path(NULL),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
  _start_ticks(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
  _start_nanos(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
  _previous_start_ticks(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
  _previous_start_nanos(0),
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    42
  _last_checkpoint_offset(0) {}
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
JfrChunkState::~JfrChunkState() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
  reset();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
void JfrChunkState::reset() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
  if (_path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
    JfrCHeapObj::free(_path, strlen(_path) + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
    _path = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
  }
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    53
  set_last_checkpoint_offset(0);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    56
void JfrChunkState::set_last_checkpoint_offset(int64_t offset) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    57
  _last_checkpoint_offset = offset;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    60
int64_t JfrChunkState::last_checkpoint_offset() const {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53897
diff changeset
    61
  return _last_checkpoint_offset;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
53897
0abec72a3ac2 8217647: JFR: recordings on 32-bit systems unreadable
bulasevich
parents: 53762
diff changeset
    64
int64_t JfrChunkState::previous_start_ticks() const {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
  return _previous_start_ticks;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
53897
0abec72a3ac2 8217647: JFR: recordings on 32-bit systems unreadable
bulasevich
parents: 53762
diff changeset
    68
int64_t JfrChunkState::previous_start_nanos() const {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
  return _previous_start_nanos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
void JfrChunkState::update_start_ticks() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
  _start_ticks = JfrTicks::now();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
void JfrChunkState::update_start_nanos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
  _start_nanos = os::javaTimeMillis() * JfrTimeConverter::NANOS_PER_MILLISEC;
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 JfrChunkState::save_current_and_update_start_ticks() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
  _previous_start_ticks = _start_ticks;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
  update_start_ticks();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
void JfrChunkState::save_current_and_update_start_nanos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
  _previous_start_nanos = _start_nanos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
  update_start_nanos();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
void JfrChunkState::update_time_to_now() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
  save_current_and_update_start_nanos();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
  save_current_and_update_start_ticks();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
53897
0abec72a3ac2 8217647: JFR: recordings on 32-bit systems unreadable
bulasevich
parents: 53762
diff changeset
    95
int64_t JfrChunkState::last_chunk_duration() const {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
  return _start_nanos - _previous_start_nanos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
static char* copy_path(const char* path) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
  assert(path != NULL, "invariant");
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
  char* new_path = JfrCHeapObj::new_array<char>(path_len + 1);
53762
65deccd64f3a 8218935: Make jfr strncpy uses GCC 8.x friendly
mikael
parents: 50113
diff changeset
   103
  strncpy(new_path, path, path_len + 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
  return new_path;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
void JfrChunkState::set_path(const char* path) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
  assert(JfrStream_lock->owned_by_self(), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
  if (_path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
    JfrCHeapObj::free(_path, strlen(_path) + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
    _path = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
  if (path != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
    _path = copy_path(path);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
const char* JfrChunkState::path() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
  return _path;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
}