src/hotspot/share/jfr/periodic/sampling/jfrThreadSampler.cpp
author mgronlun
Wed, 30 Oct 2019 19:43:52 +0100
changeset 58863 c16ac7a2eba4
parent 57699 4aea554692aa
permissions -rw-r--r--
8226511: Implement JFR Event Streaming Reviewed-by: egahlin, mseledtsov, mgronlun Contributed-by: erik.gahlin@oracle.com, mikhailo.seledtsov@oracle.com, markus.gronlund@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
53255
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
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/jfrEvents.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/periodic/sampling/jfrCallTrace.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
#include "jfr/periodic/sampling/jfrThreadSampler.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
#include "jfr/recorder/service/jfrOptionSet.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
#include "jfr/recorder/stacktrace/jfrStackTraceRepository.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
#include "jfr/support/jfrThreadId.hpp"
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
    33
#include "jfr/support/jfrThreadLocal.hpp"
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
#include "jfr/utilities/jfrTime.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
#include "logging/log.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
#include "runtime/frame.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
#include "runtime/os.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
#include "runtime/semaphore.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
#include "runtime/thread.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
#include "runtime/threadSMR.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
enum JfrSampleType {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
  NO_SAMPLE = 0,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
  JAVA_SAMPLE = 1,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
  NATIVE_SAMPLE = 2
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    48
static bool thread_state_in_java(JavaThread* thread) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    49
  assert(thread != NULL, "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    50
  switch(thread->thread_state()) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    51
    case _thread_new:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    52
    case _thread_uninitialized:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    53
    case _thread_new_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    54
    case _thread_in_vm_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    55
    case _thread_blocked_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    56
    case _thread_in_native_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    57
    case _thread_blocked:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    58
    case _thread_in_vm:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    59
    case _thread_in_native:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    60
    case _thread_in_Java_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    61
      break;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    62
    case _thread_in_Java:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    63
      return true;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    64
    default:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    65
      ShouldNotReachHere();
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    66
      break;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
  return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    71
static bool thread_state_in_native(JavaThread* thread) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    72
  assert(thread != NULL, "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    73
  switch(thread->thread_state()) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    74
    case _thread_new:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    75
    case _thread_uninitialized:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    76
    case _thread_new_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    77
    case _thread_blocked_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    78
    case _thread_blocked:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    79
    case _thread_in_vm:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    80
    case _thread_in_vm_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    81
    case _thread_in_Java_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    82
    case _thread_in_Java:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    83
    case _thread_in_native_trans:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    84
      break;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    85
    case _thread_in_native:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    86
      return true;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    87
    default:
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    88
      ShouldNotReachHere();
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
    89
      break;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
  return false;
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
class JfrThreadSampleClosure {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
 public:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
  JfrThreadSampleClosure(EventExecutionSample* events, EventNativeMethodSample* events_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
  ~JfrThreadSampleClosure() {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
  EventExecutionSample* next_event() { return &_events[_added_java++]; }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
  EventNativeMethodSample* next_event_native() { return &_events_native[_added_native++]; }
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   100
  void commit_events(JfrSampleType type);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   101
  bool do_sample_thread(JavaThread* thread, JfrStackFrame* frames, u4 max_frames, JfrSampleType type);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   102
  uint java_entries() { return _added_java; }
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   103
  uint native_entries() { return _added_native; }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
 private:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
  bool sample_thread_in_java(JavaThread* thread, JfrStackFrame* frames, u4 max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
  bool sample_thread_in_native(JavaThread* thread, JfrStackFrame* frames, u4 max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
  EventExecutionSample* _events;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
  EventNativeMethodSample* _events_native;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
  Thread* _self;
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   111
  uint _added_java;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   112
  uint _added_native;
50113
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
class OSThreadSampler : public os::SuspendedThreadTask {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
 public:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
  OSThreadSampler(JavaThread* thread,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
                  JfrThreadSampleClosure& closure,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
                  JfrStackFrame *frames,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
                  u4 max_frames) : os::SuspendedThreadTask((Thread*)thread),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
    _success(false),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
    _stacktrace(frames, max_frames),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
    _closure(closure),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
    _suspend_time() {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
  void take_sample();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
  void do_task(const os::SuspendedThreadTaskContext& context);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
  void protected_task(const os::SuspendedThreadTaskContext& context);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
  bool success() const { return _success; }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
  const JfrStackTrace& stacktrace() const { return _stacktrace; }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
 private:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
  bool _success;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
  JfrStackTrace _stacktrace;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
  JfrThreadSampleClosure& _closure;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
  JfrTicks _suspend_time;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
class OSThreadSamplerCallback : public os::CrashProtectionCallback {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
 public:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
  OSThreadSamplerCallback(OSThreadSampler& sampler, const os::SuspendedThreadTaskContext &context) :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
    _sampler(sampler), _context(context) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
  virtual void call() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
    _sampler.protected_task(_context);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
 private:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
  OSThreadSampler& _sampler;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
  const os::SuspendedThreadTaskContext& _context;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
void OSThreadSampler::do_task(const os::SuspendedThreadTaskContext& context) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
#ifndef ASSERT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
  guarantee(JfrOptionSet::sample_protection(), "Sample Protection should be on in product builds");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
#endif
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
  assert(_suspend_time.value() == 0, "already timestamped!");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
  _suspend_time = JfrTicks::now();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
  if (JfrOptionSet::sample_protection()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
    OSThreadSamplerCallback cb(*this, context);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
    os::ThreadCrashProtection crash_protection;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
    if (!crash_protection.call(cb)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
      log_error(jfr)("Thread method sampler crashed");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
  } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
    protected_task(context);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
}
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
* From this method and down the call tree we attempt to protect against crashes
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
* using a signal handler / __try block. Don't take locks, rely on destructors or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
* leave memory (in case of signal / exception) in an inconsistent state. */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
void OSThreadSampler::protected_task(const os::SuspendedThreadTaskContext& context) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
  JavaThread* jth = (JavaThread*)context.thread();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
  // Skip sample if we signaled a thread that moved to other state
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   177
  if (!thread_state_in_java(jth)) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
  JfrGetCallTrace trace(true, jth);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
  frame topframe;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
  if (trace.get_topframe(context.ucontext(), topframe)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
    if (_stacktrace.record_thread(*jth, topframe)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
      /* If we managed to get a topframe and a stacktrace, create an event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
      * and put it into our array. We can't call Jfr::_stacktraces.add()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
      * here since it would allocate memory using malloc. Doing so while
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
      * the stopped thread is inside malloc would deadlock. */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
      _success = true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
      EventExecutionSample *ev = _closure.next_event();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
      ev->set_starttime(_suspend_time);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
      ev->set_endtime(_suspend_time); // fake to not take an end time
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
      ev->set_sampledThread(JFR_THREAD_ID(jth));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
      ev->set_state(java_lang_Thread::get_thread_status(jth->threadObj()));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
void OSThreadSampler::take_sample() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
  run();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
class JfrNativeSamplerCallback : public os::CrashProtectionCallback {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
 public:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
  JfrNativeSamplerCallback(JfrThreadSampleClosure& closure, JavaThread* jt, JfrStackFrame* frames, u4 max_frames) :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
    _closure(closure), _jt(jt), _stacktrace(frames, max_frames), _success(false) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
  virtual void call();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
  bool success() { return _success; }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
  JfrStackTrace& stacktrace() { return _stacktrace; }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
 private:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
  JfrThreadSampleClosure& _closure;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
  JavaThread* _jt;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
  JfrStackTrace _stacktrace;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
  bool _success;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
static void write_native_event(JfrThreadSampleClosure& closure, JavaThread* jt) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
  EventNativeMethodSample *ev = closure.next_event_native();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
  ev->set_starttime(JfrTicks::now());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
  ev->set_sampledThread(JFR_THREAD_ID(jt));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
  ev->set_state(java_lang_Thread::get_thread_status(jt->threadObj()));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
void JfrNativeSamplerCallback::call() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
  // When a thread is only attach it will be native without a last java frame
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
  if (!_jt->has_last_Java_frame()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
  frame topframe = _jt->last_frame();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
  frame first_java_frame;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
  Method* method = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
  JfrGetCallTrace gct(false, _jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
  if (!gct.find_top_frame(topframe, &method, first_java_frame)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
  if (method == NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
  topframe = first_java_frame;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
  _success = _stacktrace.record_thread(*_jt, topframe);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
  if (_success) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
    write_native_event(_closure, _jt);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
bool JfrThreadSampleClosure::sample_thread_in_java(JavaThread* thread, JfrStackFrame* frames, u4 max_frames) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
  OSThreadSampler sampler(thread, *this, frames, max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
  sampler.take_sample();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
  /* We don't want to allocate any memory using malloc/etc while the thread
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
  * is stopped, so everything is stored in stack allocated memory until this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
  * point where the thread has been resumed again, if the sampling was a success
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
  * we need to store the stacktrace in the stacktrace repository and update
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
  * the event with the id that was returned. */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
  if (!sampler.success()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
    return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
  EventExecutionSample *event = &_events[_added_java - 1];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
  traceid id = JfrStackTraceRepository::add(sampler.stacktrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
  assert(id != 0, "Stacktrace id should not be 0");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
  event->set_stackTrace(id);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
  return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
bool JfrThreadSampleClosure::sample_thread_in_native(JavaThread* thread, JfrStackFrame* frames, u4 max_frames) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
  JfrNativeSamplerCallback cb(*this, thread, frames, max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
  if (JfrOptionSet::sample_protection()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
    os::ThreadCrashProtection crash_protection;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
    if (!crash_protection.call(cb)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
      log_error(jfr)("Thread method sampler crashed for native");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
  } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
    cb.call();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
  if (!cb.success()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
    return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
  EventNativeMethodSample *event = &_events_native[_added_native - 1];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
  traceid id = JfrStackTraceRepository::add(cb.stacktrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
  assert(id != 0, "Stacktrace id should not be 0");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
  event->set_stackTrace(id);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
  return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   284
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   286
static const uint MAX_NR_OF_JAVA_SAMPLES = 5;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   287
static const uint MAX_NR_OF_NATIVE_SAMPLES = 1;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   288
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   289
void JfrThreadSampleClosure::commit_events(JfrSampleType type) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   290
  if (JAVA_SAMPLE == type) {
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   291
    assert(_added_java > 0 && _added_java <= MAX_NR_OF_JAVA_SAMPLES, "invariant");
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   292
    for (uint i = 0; i < _added_java; ++i) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   293
      _events[i].commit();
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   294
    }
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   295
  } else {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   296
    assert(NATIVE_SAMPLE == type, "invariant");
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   297
    assert(_added_native > 0 && _added_native <= MAX_NR_OF_NATIVE_SAMPLES, "invariant");
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   298
    for (uint i = 0; i < _added_native; ++i) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   299
      _events_native[i].commit();
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   300
    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
JfrThreadSampleClosure::JfrThreadSampleClosure(EventExecutionSample* events, EventNativeMethodSample* events_native) :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
  _events(events),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
  _events_native(events_native),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
  _self(Thread::current()),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
  _added_java(0),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
  _added_native(0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
51548
35a6956f4243 8209976: Improve iteration over non-JavaThreads
kbarrett
parents: 50373
diff changeset
   312
class JfrThreadSampler : public NonJavaThread {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
  friend class JfrThreadSampling;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
 private:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
  Semaphore _sample;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
  Thread* _sampler_thread;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
  JfrStackFrame* const _frames;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
  JavaThread* _last_thread_java;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
  JavaThread* _last_thread_native;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   320
  size_t _interval_java;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   321
  size_t _interval_native;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   322
  int _cur_index;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   323
  const u4 _max_frames;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   324
  volatile bool _disenrolled;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   325
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   326
  JavaThread* next_thread(ThreadsList* t_list, JavaThread* first_sampled, JavaThread* current);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   327
  void task_stacktrace(JfrSampleType type, JavaThread** last_thread);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   328
  JfrThreadSampler(size_t interval_java, size_t interval_native, u4 max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   329
  ~JfrThreadSampler();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   330
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   331
  void start_thread();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   332
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   333
  void enroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   334
  void disenroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   335
  void set_java_interval(size_t interval) { _interval_java = interval; };
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   336
  void set_native_interval(size_t interval) { _interval_native = interval; };
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   337
  size_t get_java_interval() { return _interval_java; };
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   338
  size_t get_native_interval() { return _interval_native; };
53103
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   339
 protected:
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   340
  virtual void post_run();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   341
 public:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   342
  void run();
52448
bc5c7f63dbae 8213137: Remove static initialization of monitor/mutex instances
dholmes
parents: 52302
diff changeset
   343
  static Monitor* transition_block() { return JfrThreadSampler_lock; }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   344
  static void on_javathread_suspend(JavaThread* thread);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   345
};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   346
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   347
static void clear_transition_block(JavaThread* jt) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   348
  jt->clear_trace_flag();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   349
  JfrThreadLocal* const tl = jt->jfr_thread_local();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   350
  if (tl->is_trace_block()) {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 54562
diff changeset
   351
    MutexLocker ml(JfrThreadSampler::transition_block(), Mutex::_no_safepoint_check_flag);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   352
    JfrThreadSampler::transition_block()->notify_all();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   353
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   354
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   355
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   356
static bool is_excluded(JavaThread* thread) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   357
  assert(thread != NULL, "invariant");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   358
  return thread->is_hidden_from_external_view() || thread->in_deopt_handler() || thread->jfr_thread_local()->is_excluded();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   359
}
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   360
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   361
bool JfrThreadSampleClosure::do_sample_thread(JavaThread* thread, JfrStackFrame* frames, u4 max_frames, JfrSampleType type) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   362
  assert(Threads_lock->owned_by_self(), "Holding the thread table lock.");
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 57699
diff changeset
   363
  if (is_excluded(thread)) {
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   364
    return false;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   365
  }
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   366
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   367
  bool ret = false;
54562
1af5c0e68381 8218147: make_walkable asserts on multiple calls
rehn
parents: 53255
diff changeset
   368
  thread->set_trace_flag();  // Provides StoreLoad, needed to keep read of thread state from floating up.
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   369
  if (JAVA_SAMPLE == type) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   370
    if (thread_state_in_java(thread)) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   371
      ret = sample_thread_in_java(thread, frames, max_frames);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   372
    }
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   373
  } else {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   374
    assert(NATIVE_SAMPLE == type, "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   375
    if (thread_state_in_native(thread)) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   376
      ret = sample_thread_in_native(thread, frames, max_frames);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   377
    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   378
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   379
  clear_transition_block(thread);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   380
  return ret;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   381
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   382
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   383
JfrThreadSampler::JfrThreadSampler(size_t interval_java, size_t interval_native, u4 max_frames) :
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   384
  _sample(),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   385
  _sampler_thread(NULL),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   386
  _frames(JfrCHeapObj::new_array<JfrStackFrame>(max_frames)),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   387
  _last_thread_java(NULL),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   388
  _last_thread_native(NULL),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   389
  _interval_java(interval_java),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   390
  _interval_native(interval_native),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   391
  _cur_index(-1),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   392
  _max_frames(max_frames),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   393
  _disenrolled(true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   394
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   395
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   396
JfrThreadSampler::~JfrThreadSampler() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   397
  JfrCHeapObj::free(_frames, sizeof(JfrStackFrame) * _max_frames);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   398
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   399
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   400
void JfrThreadSampler::on_javathread_suspend(JavaThread* thread) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   401
  JfrThreadLocal* const tl = thread->jfr_thread_local();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   402
  tl->set_trace_block();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   403
  {
54645
05aaccf7d558 8222988: Use MonitorLocker rather than MutexLocker when wait/notify used
coleenp
parents: 54623
diff changeset
   404
    MonitorLocker ml(transition_block(), Mutex::_no_safepoint_check_flag);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   405
    while (thread->is_trace_suspend()) {
54645
05aaccf7d558 8222988: Use MonitorLocker rather than MutexLocker when wait/notify used
coleenp
parents: 54623
diff changeset
   406
      ml.wait();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   407
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   408
    tl->clear_trace_block();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   409
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   410
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   411
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   412
JavaThread* JfrThreadSampler::next_thread(ThreadsList* t_list, JavaThread* first_sampled, JavaThread* current) {
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   413
  assert(t_list != NULL, "invariant");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   414
  assert(Threads_lock->owned_by_self(), "Holding the thread table lock.");
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   415
  assert(_cur_index >= -1 && (uint)_cur_index + 1 <= t_list->length(), "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   416
  assert((current == NULL && -1 == _cur_index) || (t_list->find_index_of_JavaThread(current) == _cur_index), "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   417
  if ((uint)_cur_index + 1 == t_list->length()) {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   418
    // wrap
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   419
    _cur_index = 0;
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   420
  } else {
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   421
    _cur_index++;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   422
  }
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   423
  assert(_cur_index >= 0 && (uint)_cur_index < t_list->length(), "invariant");
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   424
  JavaThread* const next = t_list->thread_at(_cur_index);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   425
  return next != first_sampled ? next : NULL;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   426
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   427
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   428
void JfrThreadSampler::start_thread() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   429
  if (os::create_thread(this, os::os_thread)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   430
    os::start_thread(this);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   431
  } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   432
    log_error(jfr)("Failed to create thread for thread sampling");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   433
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   434
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   435
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   436
void JfrThreadSampler::enroll() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   437
  if (_disenrolled) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   438
    log_info(jfr)("Enrolling thread sampler");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   439
    _sample.signal();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   440
    _disenrolled = false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   441
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   442
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   443
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   444
void JfrThreadSampler::disenroll() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   445
  if (!_disenrolled) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   446
    _sample.wait();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   447
    _disenrolled = true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   448
    log_info(jfr)("Disenrolling thread sampler");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   449
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   450
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   451
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   452
static jlong get_monotonic_ms() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   453
  return os::javaTimeNanos() / 1000000;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   454
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   455
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   456
void JfrThreadSampler::run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   457
  assert(_sampler_thread == NULL, "invariant");
50222
911b913022ef 8203635: JFR sampler thread does not record stack info
zgu
parents: 50113
diff changeset
   458
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   459
  _sampler_thread = this;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   460
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   461
  jlong last_java_ms = get_monotonic_ms();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   462
  jlong last_native_ms = last_java_ms;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   463
  while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   464
    if (!_sample.trywait()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   465
      // disenrolled
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   466
      _sample.wait();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   467
      last_java_ms = get_monotonic_ms();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   468
      last_native_ms = last_java_ms;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   469
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   470
    _sample.signal();
55267
eabe64456156 8216283: Allow shorter method sampling interval than 10 ms
egahlin
parents: 54645
diff changeset
   471
    jlong java_interval = _interval_java == 0 ? max_jlong : MAX2<jlong>(_interval_java, 1);
eabe64456156 8216283: Allow shorter method sampling interval than 10 ms
egahlin
parents: 54645
diff changeset
   472
    jlong native_interval = _interval_native == 0 ? max_jlong : MAX2<jlong>(_interval_native, 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   473
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   474
    jlong now_ms = get_monotonic_ms();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   475
53255
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   476
    /*
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   477
     * Let I be java_interval or native_interval.
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   478
     * Let L be last_java_ms or last_native_ms.
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   479
     * Let N be now_ms.
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   480
     *
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   481
     * Interval, I, might be max_jlong so the addition
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   482
     * could potentially overflow without parenthesis (UB). Also note that
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   483
     * L - N < 0. Avoid UB, by adding parenthesis.
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   484
     */
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   485
    jlong next_j = java_interval + (last_java_ms - now_ms);
61a385765c9b 8216486: Possibility of integer overflow in JfrThreadSampler::run()
ysuenaga
parents: 53103
diff changeset
   486
    jlong next_n = native_interval + (last_native_ms - now_ms);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   487
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   488
    jlong sleep_to_next = MIN2<jlong>(next_j, next_n);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   489
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   490
    if (sleep_to_next > 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   491
      os::naked_short_sleep(sleep_to_next);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   492
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   493
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   494
    if ((next_j - sleep_to_next) <= 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   495
      task_stacktrace(JAVA_SAMPLE, &_last_thread_java);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   496
      last_java_ms = get_monotonic_ms();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   497
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   498
    if ((next_n - sleep_to_next) <= 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   499
      task_stacktrace(NATIVE_SAMPLE, &_last_thread_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   500
      last_native_ms = get_monotonic_ms();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   501
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   502
  }
53103
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   503
}
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   504
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   505
void JfrThreadSampler::post_run() {
67e3a8b3449c 8214097: Rework thread initialization and teardown logic
dholmes
parents: 53096
diff changeset
   506
  this->NonJavaThread::post_run();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   507
  delete this;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   508
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   509
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   510
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   511
void JfrThreadSampler::task_stacktrace(JfrSampleType type, JavaThread** last_thread) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   512
  ResourceMark rm;
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   513
  EventExecutionSample samples[MAX_NR_OF_JAVA_SAMPLES];
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   514
  EventNativeMethodSample samples_native[MAX_NR_OF_NATIVE_SAMPLES];
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   515
  JfrThreadSampleClosure sample_task(samples, samples_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   516
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   517
  const uint sample_limit = JAVA_SAMPLE == type ? MAX_NR_OF_JAVA_SAMPLES : MAX_NR_OF_NATIVE_SAMPLES;
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   518
  uint num_samples = 0;
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   519
  JavaThread* start = NULL;
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   520
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   521
  {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   522
    elapsedTimer sample_time;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   523
    sample_time.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   524
    {
57699
4aea554692aa 8226228: Make Threads_lock an always safepoint checked lock.
rehn
parents: 55267
diff changeset
   525
      MutexLocker tlock(Threads_lock);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   526
      ThreadsListHandle tlh;
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   527
      // Resolve a sample session relative start position index into the thread list array.
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   528
      // In cases where the last sampled thread is NULL or not-NULL but stale, find_index() returns -1.
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   529
      _cur_index = tlh.list()->find_index_of_JavaThread(*last_thread);
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   530
      JavaThread* current = _cur_index != -1 ? *last_thread : NULL;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   531
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   532
      while (num_samples < sample_limit) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   533
        current = next_thread(tlh.list(), start, current);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   534
        if (current == NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   535
          break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   536
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   537
        if (start == NULL) {
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   538
          start = current;  // remember the thread where we started to attempt sampling
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   539
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   540
        if (current->is_Compiler_thread()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   541
          continue;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   542
        }
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   543
        if (sample_task.do_sample_thread(current, _frames, _max_frames, type)) {
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   544
          num_samples++;
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   545
        }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   546
      }
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   547
      *last_thread = current;  // remember the thread we last attempted to sample
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   548
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   549
    sample_time.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   550
    log_trace(jfr)("JFR thread sampling done in %3.7f secs with %d java %d native samples",
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   551
                   sample_time.seconds(), sample_task.java_entries(), sample_task.native_entries());
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   552
  }
53096
58154bf80f90 8215727: Restore JFR thread sampler loop to old / previous behavior
mgronlun
parents: 52569
diff changeset
   553
  if (num_samples > 0) {
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   554
    sample_task.commit_events(type);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   555
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   556
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   557
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   558
static JfrThreadSampling* _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   559
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   560
JfrThreadSampling& JfrThreadSampling::instance() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   561
  return *_instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   562
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   563
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   564
JfrThreadSampling* JfrThreadSampling::create() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   565
  assert(_instance == NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   566
  _instance = new JfrThreadSampling();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   567
  return _instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   568
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   569
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   570
void JfrThreadSampling::destroy() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   571
  if (_instance != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   572
    delete _instance;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   573
    _instance = NULL;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   574
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   575
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   576
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   577
JfrThreadSampling::JfrThreadSampling() : _sampler(NULL) {}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   578
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   579
JfrThreadSampling::~JfrThreadSampling() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   580
  if (_sampler != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   581
    _sampler->disenroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   582
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   583
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   584
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   585
static void log(size_t interval_java, size_t interval_native) {
50373
22f611c395b3 8203921: JFR thread sampling is missing fixes from JDK-8194552
mgronlun
parents: 50222
diff changeset
   586
  log_info(jfr)("Updated thread sampler for java: " SIZE_FORMAT "  ms, native " SIZE_FORMAT " ms", interval_java, interval_native);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   587
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   588
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   589
void JfrThreadSampling::start_sampler(size_t interval_java, size_t interval_native) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   590
  assert(_sampler == NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   591
  log_info(jfr)("Enrolling thread sampler");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   592
  _sampler = new JfrThreadSampler(interval_java, interval_native, JfrOptionSet::stackdepth());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   593
  _sampler->start_thread();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   594
  _sampler->enroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   595
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   596
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   597
void JfrThreadSampling::set_sampling_interval(bool java_interval, size_t period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   598
  size_t interval_java = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   599
  size_t interval_native = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   600
  if (_sampler != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   601
    interval_java = _sampler->get_java_interval();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   602
    interval_native = _sampler->get_native_interval();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   603
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   604
  if (java_interval) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   605
    interval_java = period;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   606
  } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   607
    interval_native = period;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   608
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   609
  if (interval_java > 0 || interval_native > 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   610
    if (_sampler == NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   611
      log_info(jfr)("Creating thread sampler for java:%zu ms, native %zu ms", interval_java, interval_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   612
      start_sampler(interval_java, interval_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   613
    } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   614
      _sampler->set_java_interval(interval_java);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   615
      _sampler->set_native_interval(interval_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   616
      _sampler->enroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   617
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   618
    assert(_sampler != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   619
    log(interval_java, interval_native);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   620
  } else if (_sampler != NULL) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   621
    _sampler->disenroll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   622
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   623
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   624
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   625
void JfrThreadSampling::set_java_sample_interval(size_t period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   626
  if (_instance == NULL && 0 == period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   627
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   628
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   629
  instance().set_sampling_interval(true, period);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   630
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   631
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   632
void JfrThreadSampling::set_native_sample_interval(size_t period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   633
  if (_instance == NULL && 0 == period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   634
    return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   635
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   636
  instance().set_sampling_interval(false, period);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   637
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   638
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   639
void JfrThreadSampling::on_javathread_suspend(JavaThread* thread) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   640
  JfrThreadSampler::on_javathread_suspend(thread);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   641
}