src/hotspot/share/code/dependencyContext.cpp
author stefank
Tue, 26 Nov 2019 10:47:46 +0100
changeset 59290 97d13893ec3c
parent 59252 623722a6aeb9
permissions -rw-r--r--
8234748: Clean up atomic and orderAccess includes Reviewed-by: dholmes
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     1
/*
48955
e22914003cf0 8194691: Cleanup unnecessary casts in Atomic/OrderAccess uses
kbarrett
parents: 47216
diff changeset
     2
 * Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved.
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     4
 *
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     7
 * published by the Free Software Foundation.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     8
 *
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    13
 * accompanied this code).
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    14
 *
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    18
 *
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    21
 * questions.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    22
 *
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    23
 */
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    24
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    25
#include "precompiled.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    26
#include "code/nmethod.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    27
#include "code/dependencies.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    28
#include "code/dependencyContext.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    29
#include "memory/resourceArea.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    30
#include "runtime/atomic.hpp"
59290
97d13893ec3c 8234748: Clean up atomic and orderAccess includes
stefank
parents: 59252
diff changeset
    31
#include "runtime/orderAccess.hpp"
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    32
#include "runtime/perfData.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    33
#include "utilities/exceptions.hpp"
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    34
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    35
PerfCounter* DependencyContext::_perf_total_buckets_allocated_count   = NULL;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    36
PerfCounter* DependencyContext::_perf_total_buckets_deallocated_count = NULL;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    37
PerfCounter* DependencyContext::_perf_total_buckets_stale_count       = NULL;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    38
PerfCounter* DependencyContext::_perf_total_buckets_stale_acc_count   = NULL;
52819
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
    39
nmethodBucket* volatile DependencyContext::_purge_list                = NULL;
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
    40
volatile uint64_t DependencyContext::_cleaning_epoch                  = 0;
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
    41
uint64_t  DependencyContext::_cleaning_epoch_monotonic                = 0;
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    42
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    43
void dependencyContext_init() {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    44
  DependencyContext::init();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    45
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    46
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    47
void DependencyContext::init() {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    48
  if (UsePerfData) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    49
    EXCEPTION_MARK;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    50
    _perf_total_buckets_allocated_count =
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    51
        PerfDataManager::create_counter(SUN_CI, "nmethodBucketsAllocated", PerfData::U_Events, CHECK);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    52
    _perf_total_buckets_deallocated_count =
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    53
        PerfDataManager::create_counter(SUN_CI, "nmethodBucketsDeallocated", PerfData::U_Events, CHECK);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    54
    _perf_total_buckets_stale_count =
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    55
        PerfDataManager::create_counter(SUN_CI, "nmethodBucketsStale", PerfData::U_Events, CHECK);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    56
    _perf_total_buckets_stale_acc_count =
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    57
        PerfDataManager::create_counter(SUN_CI, "nmethodBucketsStaleAccumulated", PerfData::U_Events, CHECK);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    58
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    59
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    60
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    61
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    62
// Walk the list of dependent nmethods searching for nmethods which
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    63
// are dependent on the changes that were passed in and mark them for
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    64
// deoptimization.  Returns the number of nmethods found.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    65
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    66
int DependencyContext::mark_dependent_nmethods(DepChange& changes) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    67
  int found = 0;
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
    68
  for (nmethodBucket* b = dependencies_not_unloading(); b != NULL; b = b->next_not_unloading()) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    69
    nmethod* nm = b->get_nmethod();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    70
    // since dependencies aren't removed until an nmethod becomes a zombie,
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    71
    // the dependency list may contain nmethods which aren't alive.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    72
    if (b->count() > 0 && nm->is_alive() && !nm->is_marked_for_deoptimization() && nm->check_dependency_on(changes)) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    73
      if (TraceDependencies) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    74
        ResourceMark rm;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    75
        tty->print_cr("Marked for deoptimization");
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    76
        changes.print();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    77
        nm->print();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    78
        nm->print_dependencies();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    79
      }
36300
5b47f168b948 7177745: JSR292: Many Callsite relinkages cause target method to always run in interpreter mode
vlivanov
parents: 34221
diff changeset
    80
      changes.mark_for_deoptimization(nm);
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    81
      found++;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    82
    }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    83
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    84
  return found;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    85
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    86
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    87
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    88
// Add an nmethod to the dependency context.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    89
// It's possible that an nmethod has multiple dependencies on a klass
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    90
// so a count is kept for each bucket to guarantee that creation and
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    91
// deletion of dependencies is consistent.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    92
//
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
    93
void DependencyContext::add_dependent_nmethod(nmethod* nm) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    94
  assert_lock_strong(CodeCache_lock);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
    95
  for (nmethodBucket* b = dependencies_not_unloading(); b != NULL; b = b->next_not_unloading()) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    96
    if (nm == b->get_nmethod()) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    97
      b->increment();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    98
      return;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
    99
    }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   100
  }
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   101
  nmethodBucket* new_head = new nmethodBucket(nm, NULL);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   102
  for (;;) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   103
    nmethodBucket* head = Atomic::load(_dependency_context_addr);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   104
    new_head->set_next(head);
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 59250
diff changeset
   105
    if (Atomic::cmpxchg(_dependency_context_addr, head, new_head) == head) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   106
      break;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   107
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   108
  }
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   109
  if (UsePerfData) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   110
    _perf_total_buckets_allocated_count->inc();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   111
  }
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   112
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   113
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   114
void DependencyContext::release(nmethodBucket* b) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   115
  bool expunge = Atomic::load(&_cleaning_epoch) == 0;
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   116
  if (expunge) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   117
    assert_locked_or_safepoint(CodeCache_lock);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   118
    delete b;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   119
    if (UsePerfData) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   120
      _perf_total_buckets_deallocated_count->inc();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   121
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   122
  } else {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   123
    // Mark the context as having stale entries, since it is not safe to
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   124
    // expunge the list right now.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   125
    for (;;) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   126
      nmethodBucket* purge_list_head = Atomic::load(&_purge_list);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   127
      b->set_purge_list_next(purge_list_head);
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 59250
diff changeset
   128
      if (Atomic::cmpxchg(&_purge_list, purge_list_head, b) == purge_list_head) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   129
        break;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   130
      }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   131
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   132
    if (UsePerfData) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   133
      _perf_total_buckets_stale_count->inc();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   134
      _perf_total_buckets_stale_acc_count->inc();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   135
    }
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   136
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   137
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   138
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   139
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   140
// Remove an nmethod dependency from the context.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   141
// Decrement count of the nmethod in the dependency list and, optionally, remove
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   142
// the bucket completely when the count goes to 0.  This method must find
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   143
// a corresponding bucket otherwise there's a bug in the recording of dependencies.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   144
// Can be called concurrently by parallel GC threads.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   145
//
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   146
void DependencyContext::remove_dependent_nmethod(nmethod* nm) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   147
  assert_locked_or_safepoint(CodeCache_lock);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   148
  nmethodBucket* first = dependencies_not_unloading();
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   149
  nmethodBucket* last = NULL;
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   150
  for (nmethodBucket* b = first; b != NULL; b = b->next_not_unloading()) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   151
    if (nm == b->get_nmethod()) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   152
      int val = b->decrement();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   153
      guarantee(val >= 0, "Underflow: %d", val);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   154
      if (val == 0) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   155
        if (last == NULL) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   156
          // If there was not a head that was not unloading, we can set a new
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   157
          // head without a CAS, because we know there is no contending cleanup.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   158
          set_dependencies(b->next_not_unloading());
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   159
        } else {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   160
          // Only supports a single inserting thread (protected by CodeCache_lock)
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   161
          // for now. Therefore, the next pointer only competes with another cleanup
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   162
          // operation. That interaction does not need a CAS.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   163
          last->set_next(b->next_not_unloading());
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   164
        }
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   165
        release(b);
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   166
      }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   167
      return;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   168
    }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   169
    last = b;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   170
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   171
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   172
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   173
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   174
// Reclaim all unused buckets.
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   175
//
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   176
void DependencyContext::purge_dependency_contexts() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   177
  int removed = 0;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   178
  for (nmethodBucket* b = _purge_list; b != NULL;) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   179
    nmethodBucket* next = b->purge_list_next();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   180
    removed++;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   181
    delete b;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   182
    b = next;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   183
  }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   184
  if (UsePerfData && removed > 0) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   185
    _perf_total_buckets_deallocated_count->inc(removed);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   186
  }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   187
  _purge_list = NULL;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   188
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   189
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   190
//
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   191
// Cleanup a dependency context by unlinking and placing all dependents corresponding
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   192
// to is_unloading nmethods on a purge list, which will be deleted later when it is safe.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   193
void DependencyContext::clean_unloading_dependents() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   194
  if (!claim_cleanup()) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   195
    // Somebody else is cleaning up this dependency context.
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   196
    return;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   197
  }
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   198
  // Walk the nmethodBuckets and move dead entries on the purge list, which will
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   199
  // be deleted during ClassLoaderDataGraph::purge().
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   200
  nmethodBucket* b = dependencies_not_unloading();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   201
  while (b != NULL) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   202
    nmethodBucket* next = b->next_not_unloading();
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   203
    b = next;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   204
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   205
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   206
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   207
//
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   208
// Invalidate all dependencies in the context
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   209
int DependencyContext::remove_all_dependents() {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   210
  nmethodBucket* b = dependencies_not_unloading();
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   211
  set_dependencies(NULL);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   212
  int marked = 0;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   213
  int removed = 0;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   214
  while (b != NULL) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   215
    nmethod* nm = b->get_nmethod();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   216
    if (b->count() > 0 && nm->is_alive() && !nm->is_marked_for_deoptimization()) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   217
      nm->mark_for_deoptimization();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   218
      marked++;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   219
    }
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   220
    nmethodBucket* next = b->next_not_unloading();
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   221
    removed++;
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   222
    release(b);
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   223
    b = next;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   224
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   225
  if (UsePerfData && removed > 0) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   226
    _perf_total_buckets_deallocated_count->inc(removed);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   227
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   228
  return marked;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   229
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   230
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   231
#ifndef PRODUCT
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   232
void DependencyContext::print_dependent_nmethods(bool verbose) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   233
  int idx = 0;
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   234
  for (nmethodBucket* b = dependencies_not_unloading(); b != NULL; b = b->next_not_unloading()) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   235
    nmethod* nm = b->get_nmethod();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   236
    tty->print("[%d] count=%d { ", idx++, b->count());
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   237
    if (!verbose) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   238
      nm->print_on(tty, "nmethod");
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   239
      tty->print_cr(" } ");
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   240
    } else {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   241
      nm->print();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   242
      nm->print_dependencies();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   243
      tty->print_cr("--- } ");
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   244
    }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   245
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   246
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   247
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   248
bool DependencyContext::is_dependent_nmethod(nmethod* nm) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   249
  for (nmethodBucket* b = dependencies_not_unloading(); b != NULL; b = b->next_not_unloading()) {
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   250
    if (nm == b->get_nmethod()) {
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   251
#ifdef ASSERT
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   252
      int count = b->count();
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   253
      assert(count >= 0, "count shouldn't be negative: %d", count);
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   254
#endif
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   255
      return true;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   256
    }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   257
  }
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   258
  return false;
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   259
}
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   260
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   261
#endif //PRODUCT
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   262
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   263
int nmethodBucket::decrement() {
59250
a6deb69743d4 8234738: Harmonize parameter order in Atomic - sub
stefank
parents: 59248
diff changeset
   264
  return Atomic::sub(&_count, 1);
34195
89011d12ebd3 8139595: MethodHandles::remove_dependent_nmethod is not MT safe
vlivanov
parents:
diff changeset
   265
}
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   266
52819
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   267
// We use a monotonically increasing epoch counter to track the last epoch a given
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   268
// dependency context was cleaned. GC threads claim cleanup tasks by performing
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   269
// a CAS on this value.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   270
bool DependencyContext::claim_cleanup() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   271
  uint64_t cleaning_epoch = Atomic::load(&_cleaning_epoch);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   272
  uint64_t last_cleanup = Atomic::load(_last_cleanup_addr);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   273
  if (last_cleanup >= cleaning_epoch) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   274
    return false;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   275
  }
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 59250
diff changeset
   276
  return Atomic::cmpxchg(_last_cleanup_addr, last_cleanup, cleaning_epoch) == last_cleanup;
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   277
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   278
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   279
// Retrieve the first nmethodBucket that has a dependent that does not correspond to
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   280
// an is_unloading nmethod. Any nmethodBucket entries observed from the original head
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   281
// that is_unloading() will be unlinked and placed on the purge list.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   282
nmethodBucket* DependencyContext::dependencies_not_unloading() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   283
  for (;;) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   284
    // Need acquire becase the read value could come from a concurrent insert.
59247
56bf71d64d51 8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents: 52819
diff changeset
   285
    nmethodBucket* head = Atomic::load_acquire(_dependency_context_addr);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   286
    if (head == NULL || !head->get_nmethod()->is_unloading()) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   287
      return head;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   288
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   289
    nmethodBucket* head_next = head->next();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   290
    OrderAccess::loadload();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   291
    if (Atomic::load(_dependency_context_addr) != head) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   292
      // Unstable load of head w.r.t. head->next
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   293
      continue;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   294
    }
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 59250
diff changeset
   295
    if (Atomic::cmpxchg(_dependency_context_addr, head, head_next) == head) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   296
      // Release is_unloading entries if unlinking was claimed
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   297
      DependencyContext::release(head);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   298
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   299
  }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   300
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   301
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   302
// Relaxed accessors
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   303
void DependencyContext::set_dependencies(nmethodBucket* b) {
59248
e92153ed8bdc 8234736: Harmonize parameter order in Atomic - store
stefank
parents: 59247
diff changeset
   304
  Atomic::store(_dependency_context_addr, b);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   305
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   306
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   307
nmethodBucket* DependencyContext::dependencies() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   308
  return Atomic::load(_dependency_context_addr);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   309
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   310
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   311
// After the gc_prologue, the dependency contexts may be claimed by the GC
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   312
// and releasing of nmethodBucket entries will be deferred and placed on
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   313
// a purge list to be deleted later.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   314
void DependencyContext::cleaning_start() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   315
  assert(SafepointSynchronize::is_at_safepoint(), "must be");
52819
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   316
  uint64_t epoch = ++_cleaning_epoch_monotonic;
59248
e92153ed8bdc 8234736: Harmonize parameter order in Atomic - store
stefank
parents: 59247
diff changeset
   317
  Atomic::store(&_cleaning_epoch, epoch);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   318
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   319
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   320
// The epilogue marks the end of dependency context cleanup by the GC,
52819
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   321
// and also makes subsequent releases of nmethodBuckets cause immediate
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   322
// deletion. It is okay to delay calling of cleaning_end() to a concurrent
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   323
// phase, subsequent to the safepoint operation in which cleaning_start()
022420a4cc63 8214556: Crash in DependencyContext::remove_dependent_nmethod still happens
eosterlund
parents: 52781
diff changeset
   324
// was called. That allows dependency contexts to be cleaned concurrently.
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   325
void DependencyContext::cleaning_end() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   326
  uint64_t epoch = 0;
59248
e92153ed8bdc 8234736: Harmonize parameter order in Atomic - store
stefank
parents: 59247
diff changeset
   327
  Atomic::store(&_cleaning_epoch, epoch);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   328
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   329
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   330
// This function skips over nmethodBuckets in the list corresponding to
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   331
// nmethods that are is_unloading. This allows exposing a view of the
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   332
// dependents as-if they were already cleaned, despite being cleaned
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   333
// concurrently. Any entry observed that is_unloading() will be unlinked
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   334
// and placed on the purge list.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   335
nmethodBucket* nmethodBucket::next_not_unloading() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   336
  for (;;) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   337
    // Do not need acquire because the loaded entry can never be
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   338
    // concurrently inserted.
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   339
    nmethodBucket* next = Atomic::load(&_next);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   340
    if (next == NULL || !next->get_nmethod()->is_unloading()) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   341
      return next;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   342
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   343
    nmethodBucket* next_next = Atomic::load(&next->_next);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   344
    OrderAccess::loadload();
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   345
    if (Atomic::load(&_next) != next) {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   346
      // Unstable load of next w.r.t. next->next
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   347
      continue;
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   348
    }
59252
623722a6aeb9 8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents: 59250
diff changeset
   349
    if (Atomic::cmpxchg(&_next, next, next_next) == next) {
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   350
      // Release is_unloading entries if unlinking was claimed
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   351
      DependencyContext::release(next);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   352
    }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   353
  }
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   354
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   355
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   356
// Relaxed accessors
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   357
nmethodBucket* nmethodBucket::next() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   358
  return Atomic::load(&_next);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   359
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   360
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   361
void nmethodBucket::set_next(nmethodBucket* b) {
59248
e92153ed8bdc 8234736: Harmonize parameter order in Atomic - store
stefank
parents: 59247
diff changeset
   362
  Atomic::store(&_next, b);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   363
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   364
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   365
nmethodBucket* nmethodBucket::purge_list_next() {
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   366
  return Atomic::load(&_purge_list_next);
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   367
}
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   368
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   369
void nmethodBucket::set_purge_list_next(nmethodBucket* b) {
59248
e92153ed8bdc 8234736: Harmonize parameter order in Atomic - store
stefank
parents: 59247
diff changeset
   370
  Atomic::store(&_purge_list_next, b);
52781
436097b038a1 8213565: Crash in DependencyContext::remove_dependent_nmethod
eosterlund
parents: 51887
diff changeset
   371
}