hotspot/src/share/vm/runtime/sweeper.cpp
author iveresov
Thu, 14 Apr 2011 00:02:21 -0700
changeset 9138 35e4b04eadf4
parent 8921 14bfe81f2a9d
child 9942 2e86734ba620
permissions -rw-r--r--
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ... Summary: Eliminate the race condition. Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
8921
14bfe81f2a9d 7010070: Update all 2010 Oracle-changed OpenJDK files to have the proper copyright dates - second pass
trims
parents: 8672
diff changeset
     2
 * Copyright (c) 1997, 2011, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    26
#include "code/codeCache.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    27
#include "code/nmethod.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    28
#include "compiler/compileBroker.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    29
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    30
#include "oops/methodOop.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    31
#include "runtime/atomic.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    32
#include "runtime/compilationPolicy.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    33
#include "runtime/mutexLocker.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    34
#include "runtime/os.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    35
#include "runtime/sweeper.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    36
#include "runtime/vm_operations.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    37
#include "utilities/events.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    38
#include "utilities/xmlstream.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
long      NMethodSweeper::_traversals = 0;   // No. of stack traversals performed
5895
7127e98012e3 6956958: assert(is_clean() || is_call_to_compiled() || is_call_to_interpreted() || is_optimized() || is_megam
never
parents: 5547
diff changeset
    41
nmethod*  NMethodSweeper::_current = NULL;   // Current nmethod
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
    42
int       NMethodSweeper::_seen = 0 ;        // No. of nmethods we have currently processed in current pass of CodeCache
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
    43
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
    44
volatile int NMethodSweeper::_invocations = 0;   // No. of invocations left until we are completed with this pass
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
    45
volatile int NMethodSweeper::_sweep_started = 0; // Whether a sweep is in progress.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
jint      NMethodSweeper::_locked_seen = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
jint      NMethodSweeper::_not_entrant_seen_on_stack = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
bool      NMethodSweeper::_rescan = false;
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    50
bool      NMethodSweeper::_do_sweep = false;
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
    51
bool      NMethodSweeper::_was_full = false;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
    52
jint      NMethodSweeper::_advise_to_sweep = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
    53
jlong     NMethodSweeper::_last_was_full = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
    54
uint      NMethodSweeper::_highest_marked = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
    55
long      NMethodSweeper::_was_full_traversal = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
3908
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    57
class MarkActivationClosure: public CodeBlobClosure {
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    58
public:
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    59
  virtual void do_code_blob(CodeBlob* cb) {
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    60
    // If we see an activation belonging to a non_entrant nmethod, we mark it.
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    61
    if (cb->is_nmethod() && ((nmethod*)cb)->is_not_entrant()) {
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    62
      ((nmethod*)cb)->mark_as_seen_on_stack();
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    63
    }
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    64
  }
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    65
};
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    66
static MarkActivationClosure mark_activation_closure;
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    67
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    68
void NMethodSweeper::scan_stacks() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  assert(SafepointSynchronize::is_at_safepoint(), "must be executed at a safepoint");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  if (!MethodFlushing) return;
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    71
  _do_sweep = true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  // No need to synchronize access, since this is always executed at a
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  // safepoint.  If we aren't in the middle of scan and a rescan
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    75
  // hasn't been requested then just return. If UseCodeCacheFlushing is on and
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    76
  // code cache flushing is in progress, don't skip sweeping to help make progress
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    77
  // clearing space in the code cache.
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    78
  if ((_current == NULL && !_rescan) && !(UseCodeCacheFlushing && !CompileBroker::should_compile_new_jobs())) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    79
    _do_sweep = false;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    80
    return;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    81
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  // Make sure CompiledIC_lock in unlocked, since we might update some
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  // inline caches. If it is, we just bail-out and try later.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  if (CompiledIC_lock->is_locked() || Patching_lock->is_locked()) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  // Check for restart
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  assert(CodeCache::find_blob_unsafe(_current) == _current, "Sweeper nmethod cached state invalid");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  if (_current == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    _seen        = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
    _invocations = NmethodSweepFraction;
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
    92
    _current     = CodeCache::first_nmethod();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
    _traversals  += 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    if (PrintMethodFlushing) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
      tty->print_cr("### Sweep: stack traversal %d", _traversals);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
    }
3908
24b55ad4c228 6863023: need non-perm oops in code cache for JSR 292
jrose
parents: 1
diff changeset
    97
    Threads::nmethods_do(&mark_activation_closure);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
    // reset the flags since we started a scan from the beginning.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
    _rescan = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
    _locked_seen = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
    _not_entrant_seen_on_stack = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   105
  if (UseCodeCacheFlushing) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   106
    if (!CodeCache::needs_flushing()) {
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   107
      // scan_stacks() runs during a safepoint, no race with setters
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   108
      _advise_to_sweep = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   109
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   110
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   111
    if (was_full()) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   112
      // There was some progress so attempt to restart the compiler
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   113
      jlong now           = os::javaTimeMillis();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   114
      jlong max_interval  = (jlong)MinCodeCacheFlushingInterval * (jlong)1000;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   115
      jlong curr_interval = now - _last_was_full;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   116
      if ((!CodeCache::needs_flushing()) && (curr_interval > max_interval)) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   117
        CompileBroker::set_should_compile_new_jobs(CompileBroker::run_compilation);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   118
        set_was_full(false);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   119
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   120
        // Update the _last_was_full time so we can tell how fast the
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   121
        // code cache is filling up
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   122
        _last_was_full = os::javaTimeMillis();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   123
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   124
        log_sweep("restart_compiler");
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   125
      }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   126
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   127
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   130
void NMethodSweeper::possibly_sweep() {
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   131
  assert(JavaThread::current()->thread_state() == _thread_in_vm, "must run in vm mode");
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   132
  if ((!MethodFlushing) || (!_do_sweep)) return;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   133
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   134
  if (_invocations > 0) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   135
    // Only one thread at a time will sweep
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   136
    jint old = Atomic::cmpxchg( 1, &_sweep_started, 0 );
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   137
    if (old != 0) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   138
      return;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   139
    }
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   140
    if (_invocations > 0) {
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   141
      sweep_code_cache();
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   142
      _invocations--;
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   143
    }
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   144
    _sweep_started = 0;
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   145
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   146
}
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   147
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   148
void NMethodSweeper::sweep_code_cache() {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   149
#ifdef ASSERT
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   150
  jlong sweep_start;
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   151
  if (PrintMethodFlushing) {
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   152
    sweep_start = os::javaTimeMillis();
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   153
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   154
#endif
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   155
  if (PrintMethodFlushing && Verbose) {
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   156
    tty->print_cr("### Sweep at %d out of %d. Invocations left: %d", _seen, CodeCache::nof_nmethods(), _invocations);
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   157
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   158
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   159
  // We want to visit all nmethods after NmethodSweepFraction
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   160
  // invocations so divide the remaining number of nmethods by the
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   161
  // remaining number of invocations.  This is only an estimate since
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   162
  // the number of nmethods changes during the sweep so the final
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   163
  // stage must iterate until it there are no more nmethods.
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   164
  int todo = (CodeCache::nof_nmethods() - _seen) / _invocations;
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   165
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   166
  assert(!SafepointSynchronize::is_at_safepoint(), "should not be in safepoint when we get here");
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   167
  assert(!CodeCache_lock->owned_by_self(), "just checking");
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   168
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   169
  {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   170
    MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   171
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   172
    // The last invocation iterates until there are no more nmethods
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   173
    for (int i = 0; (i < todo || _invocations == 1) && _current != NULL; i++) {
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   174
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   175
      // Since we will give up the CodeCache_lock, always skip ahead
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   176
      // to the next nmethod.  Other blobs can be deleted by other
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   177
      // threads but nmethods are only reclaimed by the sweeper.
5895
7127e98012e3 6956958: assert(is_clean() || is_call_to_compiled() || is_call_to_interpreted() || is_optimized() || is_megam
never
parents: 5547
diff changeset
   178
      nmethod* next = CodeCache::next_nmethod(_current);
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   179
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   180
      // Now ready to process nmethod and give up CodeCache_lock
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   181
      {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   182
        MutexUnlockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
5895
7127e98012e3 6956958: assert(is_clean() || is_call_to_compiled() || is_call_to_interpreted() || is_optimized() || is_megam
never
parents: 5547
diff changeset
   183
        process_nmethod(_current);
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   184
      }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   185
      _seen++;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   186
      _current = next;
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   187
    }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   188
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   189
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   190
  assert(_invocations > 1 || _current == NULL, "must have scanned the whole cache");
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   191
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   192
  if (_current == NULL && !_rescan && (_locked_seen || _not_entrant_seen_on_stack)) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   193
    // we've completed a scan without making progress but there were
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   194
    // nmethods we were unable to process either because they were
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   195
    // locked or were still on stack.  We don't have to aggresively
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   196
    // clean them up so just stop scanning.  We could scan once more
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   197
    // but that complicates the control logic and it's unlikely to
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   198
    // matter much.
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   199
    if (PrintMethodFlushing) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   200
      tty->print_cr("### Couldn't make progress on some nmethods so stopping sweep");
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   201
    }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   202
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   203
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   204
#ifdef ASSERT
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   205
  if(PrintMethodFlushing) {
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   206
    jlong sweep_end             = os::javaTimeMillis();
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   207
    tty->print_cr("### sweeper:      sweep time(%d): " INT64_FORMAT, _invocations, sweep_end - sweep_start);
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   208
  }
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   209
#endif
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   210
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   211
  if (_invocations == 1) {
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   212
    log_sweep("finished");
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   213
  }
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   214
}
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   215
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
void NMethodSweeper::process_nmethod(nmethod *nm) {
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   218
  assert(!CodeCache_lock->owned_by_self(), "just checking");
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   219
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  // Skip methods that are currently referenced by the VM
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  if (nm->is_locked_by_vm()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
    // But still remember to clean-up inline caches for alive nmethods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
    if (nm->is_alive()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
      // Clean-up all inline caches that points to zombie/non-reentrant methods
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   225
      MutexLocker cl(CompiledIC_lock);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
      nm->cleanup_inline_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
      _locked_seen++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
  if (nm->is_zombie()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
    // If it is first time, we see nmethod then we mark it. Otherwise,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
    // we reclame it. When we have seen a zombie method twice, we know that
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   236
    // there are no inline caches that refer to it.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
    if (nm->is_marked_for_reclamation()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      assert(!nm->is_locked_by_vm(), "must not flush locked nmethods");
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   239
      if (PrintMethodFlushing && Verbose) {
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   240
        tty->print_cr("### Nmethod %3d/" PTR_FORMAT " (marked for reclamation) being flushed", nm->compile_id(), nm);
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   241
      }
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   242
      MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
      nm->flush();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
    } else {
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   245
      if (PrintMethodFlushing && Verbose) {
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   246
        tty->print_cr("### Nmethod %3d/" PTR_FORMAT " (zombie) being marked for reclamation", nm->compile_id(), nm);
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   247
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
      nm->mark_for_reclamation();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
      _rescan = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  } else if (nm->is_not_entrant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
    // If there is no current activations of this method on the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    // stack we can safely convert it to a zombie method
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    if (nm->can_not_entrant_be_converted()) {
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   255
      if (PrintMethodFlushing && Verbose) {
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   256
        tty->print_cr("### Nmethod %3d/" PTR_FORMAT " (not entrant) being made zombie", nm->compile_id(), nm);
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   257
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
      nm->make_zombie();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
      _rescan = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
      // Still alive, clean up its inline caches
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   262
      MutexLocker cl(CompiledIC_lock);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
      nm->cleanup_inline_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
      // we coudn't transition this nmethod so don't immediately
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
      // request a rescan.  If this method stays on the stack for a
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   266
      // long time we don't want to keep rescanning the code cache.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
      _not_entrant_seen_on_stack++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  } else if (nm->is_unloaded()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
    // Unloaded code, just make it a zombie
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   271
    if (PrintMethodFlushing && Verbose)
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   272
      tty->print_cr("### Nmethod %3d/" PTR_FORMAT " (unloaded) being made zombie", nm->compile_id(), nm);
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 1
diff changeset
   273
    if (nm->is_osr_method()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
      // No inline caches will ever point to osr methods, so we can just remove it
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   275
      MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
      nm->flush();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
      nm->make_zombie();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
      _rescan = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
    assert(nm->is_alive(), "should be alive");
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   283
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   284
    if (UseCodeCacheFlushing) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   285
      if ((nm->method()->code() != nm) && !(nm->is_locked_by_vm()) && !(nm->is_osr_method()) &&
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   286
          (_traversals > _was_full_traversal+2) && (((uint)nm->compile_id()) < _highest_marked) &&
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   287
          CodeCache::needs_flushing()) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   288
        // This method has not been called since the forced cleanup happened
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   289
        nm->make_not_entrant();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   290
      }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   291
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   292
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
    // Clean-up all inline caches that points to zombie/non-reentrant methods
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   294
    MutexLocker cl(CompiledIC_lock);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
    nm->cleanup_inline_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
}
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   298
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   299
// Code cache unloading: when compilers notice the code cache is getting full,
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   300
// they will call a vm op that comes here. This code attempts to speculatively
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   301
// unload the oldest half of the nmethods (based on the compile job id) by
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   302
// saving the old code in a list in the CodeCache. Then
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   303
// execution resumes. If a method so marked is not called by the second sweeper
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   304
// stack traversal after the current one, the nmethod will be marked non-entrant and
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   305
// got rid of by normal sweeping. If the method is called, the methodOop's
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   306
// _code field is restored and the methodOop/nmethod
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   307
// go back to their normal state.
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   308
void NMethodSweeper::handle_full_code_cache(bool is_full) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   309
  // Only the first one to notice can advise us to start early cleaning
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   310
  if (!is_full){
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   311
    jint old = Atomic::cmpxchg( 1, &_advise_to_sweep, 0 );
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   312
    if (old != 0) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   313
      return;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   314
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   315
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   316
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   317
  if (is_full) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   318
    // Since code cache is full, immediately stop new compiles
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   319
    bool did_set = CompileBroker::set_should_compile_new_jobs(CompileBroker::stop_compilation);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   320
    if (!did_set) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   321
      // only the first to notice can start the cleaning,
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   322
      // others will go back and block
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   323
      return;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   324
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   325
    set_was_full(true);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   326
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   327
    // If we run out within MinCodeCacheFlushingInterval of the last unload time, give up
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   328
    jlong now = os::javaTimeMillis();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   329
    jlong max_interval = (jlong)MinCodeCacheFlushingInterval * (jlong)1000;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   330
    jlong curr_interval = now - _last_was_full;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   331
    if (curr_interval < max_interval) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   332
      _rescan = true;
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   333
      log_sweep("disable_compiler", "flushing_interval='" UINT64_FORMAT "'",
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   334
                           curr_interval/1000);
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   335
      return;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   336
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   337
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   338
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   339
  VM_HandleFullCodeCache op(is_full);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   340
  VMThread::execute(&op);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   341
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   342
  // rescan again as soon as possible
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   343
  _rescan = true;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   344
}
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   345
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   346
void NMethodSweeper::speculative_disconnect_nmethods(bool is_full) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   347
  // If there was a race in detecting full code cache, only run
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   348
  // one vm op for it or keep the compiler shut off
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   349
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   350
  debug_only(jlong start = os::javaTimeMillis();)
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   351
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   352
  if ((!was_full()) && (is_full)) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   353
    if (!CodeCache::needs_flushing()) {
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   354
      log_sweep("restart_compiler");
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   355
      CompileBroker::set_should_compile_new_jobs(CompileBroker::run_compilation);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   356
      return;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   357
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   358
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   359
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   360
  // Traverse the code cache trying to dump the oldest nmethods
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   361
  uint curr_max_comp_id = CompileBroker::get_compilation_id();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   362
  uint flush_target = ((curr_max_comp_id - _highest_marked) >> 1) + _highest_marked;
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   363
  log_sweep("start_cleaning");
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   364
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   365
  nmethod* nm = CodeCache::alive_nmethod(CodeCache::first());
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   366
  jint disconnected = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   367
  jint made_not_entrant  = 0;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   368
  while ((nm != NULL)){
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   369
    uint curr_comp_id = nm->compile_id();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   370
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   371
    // OSR methods cannot be flushed like this. Also, don't flush native methods
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   372
    // since they are part of the JDK in most cases
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   373
    if (nm->is_in_use() && (!nm->is_osr_method()) && (!nm->is_locked_by_vm()) &&
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   374
        (!nm->is_native_method()) && ((curr_comp_id < flush_target))) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   375
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   376
      if ((nm->method()->code() == nm)) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   377
        // This method has not been previously considered for
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   378
        // unloading or it was restored already
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   379
        CodeCache::speculatively_disconnect(nm);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   380
        disconnected++;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   381
      } else if (nm->is_speculatively_disconnected()) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   382
        // This method was previously considered for preemptive unloading and was not called since then
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5926
diff changeset
   383
        CompilationPolicy::policy()->delay_compilation(nm->method());
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   384
        nm->make_not_entrant();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   385
        made_not_entrant++;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   386
      }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   387
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   388
      if (curr_comp_id > _highest_marked) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   389
        _highest_marked = curr_comp_id;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   390
      }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   391
    }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   392
    nm = CodeCache::alive_nmethod(CodeCache::next(nm));
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   393
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   394
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   395
  log_sweep("stop_cleaning",
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   396
                       "disconnected='" UINT32_FORMAT "' made_not_entrant='" UINT32_FORMAT "'",
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   397
                       disconnected, made_not_entrant);
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   398
5533
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   399
  // Shut off compiler. Sweeper will start over with a new stack scan and
e8d9ff82ec62 6950075: nmethod sweeper should operate concurrently
never
parents: 4750
diff changeset
   400
  // traversal cycle and turn it back on if it clears enough space.
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   401
  if (was_full()) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   402
    _last_was_full = os::javaTimeMillis();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   403
    CompileBroker::set_should_compile_new_jobs(CompileBroker::stop_compilation);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   404
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   405
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   406
  // After two more traversals the sweeper will get rid of unrestored nmethods
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   407
  _was_full_traversal = _traversals;
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   408
#ifdef ASSERT
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   409
  jlong end = os::javaTimeMillis();
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   410
  if(PrintMethodFlushing && Verbose) {
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   411
    tty->print_cr("### sweeper: unload time: " INT64_FORMAT, end-start);
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   412
  }
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   413
#endif
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 3919
diff changeset
   414
}
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   415
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   416
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   417
// Print out some state information about the current sweep and the
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   418
// state of the code cache if it's requested.
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   419
void NMethodSweeper::log_sweep(const char* msg, const char* format, ...) {
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   420
  if (PrintMethodFlushing) {
9138
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   421
    stringStream s;
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   422
    // Dump code cache state into a buffer before locking the tty,
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   423
    // because log_state() will use locks causing lock conflicts.
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   424
    CodeCache::log_state(&s);
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   425
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   426
    ttyLocker ttyl;
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   427
    tty->print("### sweeper: %s ", msg);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   428
    if (format != NULL) {
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   429
      va_list ap;
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   430
      va_start(ap, format);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   431
      tty->vprint(format, ap);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   432
      va_end(ap);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   433
    }
9138
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   434
    tty->print_cr(s.as_string());
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   435
  }
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   436
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   437
  if (LogCompilation && (xtty != NULL)) {
9138
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   438
    stringStream s;
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   439
    // Dump code cache state into a buffer before locking the tty,
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   440
    // because log_state() will use locks causing lock conflicts.
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   441
    CodeCache::log_state(&s);
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   442
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   443
    ttyLocker ttyl;
5926
a36f90d986b6 6968385: malformed xml in sweeper logging
never
parents: 5924
diff changeset
   444
    xtty->begin_elem("sweeper state='%s' traversals='" INTX_FORMAT "' ", msg, (intx)traversal_count());
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   445
    if (format != NULL) {
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   446
      va_list ap;
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   447
      va_start(ap, format);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   448
      xtty->vprint(format, ap);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   449
      va_end(ap);
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   450
    }
9138
35e4b04eadf4 7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents: 8921
diff changeset
   451
    xtty->print(s.as_string());
5924
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   452
    xtty->stamp();
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   453
    xtty->end_elem();
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   454
  }
dc9d04930c82 6965184: possible races in make_not_entrant_or_zombie
never
parents: 5895
diff changeset
   455
}