author | alanb |
Thu, 01 Dec 2016 08:56:41 +0000 | |
changeset 42307 | cefc81dc1d52 |
parent 40664 | 1ec65b303bb7 |
child 42650 | 1f304d0c888b |
permissions | -rw-r--r-- |
1 | 1 |
/* |
36818
b40330c06dea
8136458: Remove "marked for reclamation" nmethod state
thartmann
parents:
36802
diff
changeset
|
2 |
* Copyright (c) 1997, 2016, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
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 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "code/codeCache.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
27 |
#include "code/compiledIC.hpp" |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
28 |
#include "code/icBuffer.hpp" |
7397 | 29 |
#include "code/nmethod.hpp" |
30 |
#include "compiler/compileBroker.hpp" |
|
31 |
#include "memory/resourceArea.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
32 |
#include "oops/method.hpp" |
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
38133
diff
changeset
|
33 |
#include "runtime/atomic.hpp" |
7397 | 34 |
#include "runtime/compilationPolicy.hpp" |
35 |
#include "runtime/mutexLocker.hpp" |
|
24351
61b33cc6d3cf
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
22872
diff
changeset
|
36 |
#include "runtime/orderAccess.inline.hpp" |
7397 | 37 |
#include "runtime/os.hpp" |
38 |
#include "runtime/sweeper.hpp" |
|
24351
61b33cc6d3cf
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
22872
diff
changeset
|
39 |
#include "runtime/thread.inline.hpp" |
7397 | 40 |
#include "runtime/vm_operations.hpp" |
18025 | 41 |
#include "trace/tracing.hpp" |
7397 | 42 |
#include "utilities/events.hpp" |
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
21727
diff
changeset
|
43 |
#include "utilities/ticks.inline.hpp" |
7397 | 44 |
#include "utilities/xmlstream.hpp" |
1 | 45 |
|
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
46 |
#ifdef ASSERT |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
47 |
|
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
48 |
#define SWEEP(nm) record_sweep(nm, __LINE__) |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
49 |
// Sweeper logging code |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
50 |
class SweeperRecord { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
51 |
public: |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
52 |
int traversal; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
53 |
int compile_id; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
54 |
long traversal_mark; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
55 |
int state; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
56 |
const char* kind; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
57 |
address vep; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
58 |
address uep; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
59 |
int line; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
60 |
|
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
61 |
void print() { |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
62 |
tty->print_cr("traversal = %d compile_id = %d %s uep = " PTR_FORMAT " vep = " |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
63 |
PTR_FORMAT " state = %d traversal_mark %ld line = %d", |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
64 |
traversal, |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
65 |
compile_id, |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
66 |
kind == NULL ? "" : kind, |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
67 |
p2i(uep), |
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
68 |
p2i(vep), |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
69 |
state, |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
70 |
traversal_mark, |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
71 |
line); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
72 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
73 |
}; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
74 |
|
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
75 |
static int _sweep_index = 0; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
76 |
static SweeperRecord* _records = NULL; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
77 |
|
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
78 |
void NMethodSweeper::report_events(int id, address entry) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
79 |
if (_records != NULL) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
80 |
for (int i = _sweep_index; i < SweeperLogEntries; i++) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
81 |
if (_records[i].uep == entry || |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
82 |
_records[i].vep == entry || |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
83 |
_records[i].compile_id == id) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
84 |
_records[i].print(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
85 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
86 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
87 |
for (int i = 0; i < _sweep_index; i++) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
88 |
if (_records[i].uep == entry || |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
89 |
_records[i].vep == entry || |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
90 |
_records[i].compile_id == id) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
91 |
_records[i].print(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
92 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
93 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
94 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
95 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
96 |
|
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
97 |
void NMethodSweeper::report_events() { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
98 |
if (_records != NULL) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
99 |
for (int i = _sweep_index; i < SweeperLogEntries; i++) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
100 |
// skip empty records |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
101 |
if (_records[i].vep == NULL) continue; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
102 |
_records[i].print(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
103 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
104 |
for (int i = 0; i < _sweep_index; i++) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
105 |
// skip empty records |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
106 |
if (_records[i].vep == NULL) continue; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
107 |
_records[i].print(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
108 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
109 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
110 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
111 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
112 |
void NMethodSweeper::record_sweep(CompiledMethod* nm, int line) { |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
113 |
if (_records != NULL) { |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
114 |
_records[_sweep_index].traversal = _traversals; |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
115 |
_records[_sweep_index].traversal_mark = nm->is_nmethod() ? ((nmethod*)nm)->_stack_traversal_mark : 0; |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
116 |
_records[_sweep_index].compile_id = nm->compile_id(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
117 |
_records[_sweep_index].kind = nm->compile_kind(); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
118 |
_records[_sweep_index].state = nm->get_state(); |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
119 |
_records[_sweep_index].vep = nm->verified_entry_point(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
120 |
_records[_sweep_index].uep = nm->entry_point(); |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
121 |
_records[_sweep_index].line = line; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
122 |
_sweep_index = (_sweep_index + 1) % SweeperLogEntries; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
123 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
124 |
} |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
125 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
126 |
void NMethodSweeper::init_sweeper_log() { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
127 |
if (LogSweeper && _records == NULL) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
128 |
// Create the ring buffer for the logging code |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
129 |
_records = NEW_C_HEAP_ARRAY(SweeperRecord, SweeperLogEntries, mtGC); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
130 |
memset(_records, 0, sizeof(SweeperRecord) * SweeperLogEntries); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
131 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
132 |
} |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
133 |
#else |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
134 |
#define SWEEP(nm) |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
135 |
#endif |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
136 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
137 |
CompiledMethodIterator NMethodSweeper::_current; // Current compiled method |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
138 |
long NMethodSweeper::_traversals = 0; // Stack scan count, also sweep ID. |
22210
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
139 |
long NMethodSweeper::_total_nof_code_cache_sweeps = 0; // Total number of full sweeps of the code cache |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
140 |
long NMethodSweeper::_time_counter = 0; // Virtual time used to periodically invoke sweeper |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
141 |
long NMethodSweeper::_last_sweep = 0; // Value of _time_counter when the last sweep happened |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
142 |
int NMethodSweeper::_seen = 0; // Nof. nmethod we have currently processed in current pass of CodeCache |
1 | 143 |
|
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
144 |
volatile bool NMethodSweeper::_should_sweep = true; // Indicates if we should invoke the sweeper |
30205 | 145 |
volatile bool NMethodSweeper::_force_sweep = false;// Indicates if we should force a sweep |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
146 |
volatile int NMethodSweeper::_bytes_changed = 0; // Counts the total nmethod size if the nmethod changed from: |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
147 |
// 1) alive -> not_entrant |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
148 |
// 2) not_entrant -> zombie |
22210
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
149 |
int NMethodSweeper::_hotness_counter_reset_val = 0; |
1 | 150 |
|
26796 | 151 |
long NMethodSweeper::_total_nof_methods_reclaimed = 0; // Accumulated nof methods flushed |
152 |
long NMethodSweeper::_total_nof_c2_methods_reclaimed = 0; // Accumulated nof methods flushed |
|
153 |
size_t NMethodSweeper::_total_flushed_size = 0; // Total number of bytes flushed from the code cache |
|
154 |
Tickspan NMethodSweeper::_total_time_sweeping; // Accumulated time sweeping |
|
155 |
Tickspan NMethodSweeper::_total_time_this_sweep; // Total time this sweep |
|
156 |
Tickspan NMethodSweeper::_peak_sweep_time; // Peak time for a full sweep |
|
157 |
Tickspan NMethodSweeper::_peak_sweep_fraction_time; // Peak time sweeping one fraction |
|
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
158 |
|
28163
322d55d167be
8047290: Make Mutex::_no_safepoint_check_flag locks verify that this lock never checks for safepoint
coleenp
parents:
27917
diff
changeset
|
159 |
Monitor* NMethodSweeper::_stat_lock = new Monitor(Mutex::special, "Sweeper::Statistics", true, Monitor::_safepoint_check_sometimes); |
18025 | 160 |
|
3908 | 161 |
class MarkActivationClosure: public CodeBlobClosure { |
162 |
public: |
|
163 |
virtual void do_code_blob(CodeBlob* cb) { |
|
26796 | 164 |
assert(cb->is_nmethod(), "CodeBlob should be nmethod"); |
165 |
nmethod* nm = (nmethod*)cb; |
|
166 |
nm->set_hotness_counter(NMethodSweeper::hotness_counter_reset_val()); |
|
167 |
// If we see an activation belonging to a non_entrant nmethod, we mark it. |
|
168 |
if (nm->is_not_entrant()) { |
|
169 |
nm->mark_as_seen_on_stack(); |
|
3908 | 170 |
} |
171 |
} |
|
172 |
}; |
|
173 |
static MarkActivationClosure mark_activation_closure; |
|
174 |
||
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
175 |
class SetHotnessClosure: public CodeBlobClosure { |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
176 |
public: |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
177 |
virtual void do_code_blob(CodeBlob* cb) { |
26796 | 178 |
assert(cb->is_nmethod(), "CodeBlob should be nmethod"); |
179 |
nmethod* nm = (nmethod*)cb; |
|
180 |
nm->set_hotness_counter(NMethodSweeper::hotness_counter_reset_val()); |
|
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
181 |
} |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
182 |
}; |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
183 |
static SetHotnessClosure set_hotness_closure; |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
184 |
|
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
185 |
|
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
186 |
int NMethodSweeper::hotness_counter_reset_val() { |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
187 |
if (_hotness_counter_reset_val == 0) { |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
188 |
_hotness_counter_reset_val = (ReservedCodeCacheSize < M) ? 1 : (ReservedCodeCacheSize / M) * 2; |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
189 |
} |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
190 |
return _hotness_counter_reset_val; |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
191 |
} |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
192 |
bool NMethodSweeper::wait_for_stack_scanning() { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
193 |
return _current.end(); |
17132
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
194 |
} |
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
195 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
196 |
/** |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
197 |
* Scans the stacks of all Java threads and marks activations of not-entrant methods. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
198 |
* No need to synchronize access, since 'mark_active_nmethods' is always executed at a |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
199 |
* safepoint. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
200 |
*/ |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
201 |
void NMethodSweeper::mark_active_nmethods() { |
1 | 202 |
assert(SafepointSynchronize::is_at_safepoint(), "must be executed at a safepoint"); |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
203 |
// If we do not want to reclaim not-entrant or zombie methods there is no need |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
204 |
// to scan stacks |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
205 |
if (!MethodFlushing) { |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
206 |
return; |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
207 |
} |
1 | 208 |
|
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
209 |
// Increase time so that we can estimate when to invoke the sweeper again. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
210 |
_time_counter++; |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
211 |
|
1 | 212 |
// Check for restart |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
213 |
if (_current.method() != NULL) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
214 |
if (_current.method()->is_nmethod()) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
215 |
assert(CodeCache::find_blob_unsafe(_current.method()) == _current.method(), "Sweeper nmethod cached state invalid"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
216 |
} else { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
217 |
ShouldNotReachHere(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
218 |
} |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
219 |
} |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
220 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
221 |
if (wait_for_stack_scanning()) { |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
222 |
_seen = 0; |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
223 |
_current = CompiledMethodIterator(); |
26796 | 224 |
// Initialize to first nmethod |
225 |
_current.next(); |
|
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
226 |
_traversals += 1; |
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
21727
diff
changeset
|
227 |
_total_time_this_sweep = Tickspan(); |
18025 | 228 |
|
1 | 229 |
if (PrintMethodFlushing) { |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
230 |
tty->print_cr("### Sweep: stack traversal %ld", _traversals); |
1 | 231 |
} |
3908 | 232 |
Threads::nmethods_do(&mark_activation_closure); |
1 | 233 |
|
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
234 |
} else { |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
235 |
// Only set hotness counter |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
236 |
Threads::nmethods_do(&set_hotness_closure); |
1 | 237 |
} |
238 |
||
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
239 |
OrderAccess::storestore(); |
1 | 240 |
} |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
241 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
242 |
/** |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
243 |
* This function triggers a VM operation that does stack scanning of active |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
244 |
* methods. Stack scanning is mandatory for the sweeper to make progress. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
245 |
*/ |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
246 |
void NMethodSweeper::do_stack_scanning() { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
247 |
assert(!CodeCache_lock->owned_by_self(), "just checking"); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
248 |
if (wait_for_stack_scanning()) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
249 |
VM_MarkActiveNMethods op; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
250 |
VMThread::execute(&op); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
251 |
_should_sweep = true; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
252 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
253 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
254 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
255 |
void NMethodSweeper::sweeper_loop() { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
256 |
bool timeout; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
257 |
while (true) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
258 |
{ |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
259 |
ThreadBlockInVM tbivm(JavaThread::current()); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
260 |
MutexLockerEx waiter(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
261 |
const long wait_time = 60*60*24 * 1000; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
262 |
timeout = CodeCache_lock->wait(Mutex::_no_safepoint_check_flag, wait_time); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
263 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
264 |
if (!timeout) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
265 |
possibly_sweep(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
266 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
267 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
268 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
269 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
270 |
/** |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
271 |
* Wakes up the sweeper thread to possibly sweep. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
272 |
*/ |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
273 |
void NMethodSweeper::notify(int code_blob_type) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
274 |
// Makes sure that we do not invoke the sweeper too often during startup. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
275 |
double start_threshold = 100.0 / (double)StartAggressiveSweepingAt; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
276 |
double aggressive_sweep_threshold = MIN2(start_threshold, 1.1); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
277 |
if (CodeCache::reverse_free_ratio(code_blob_type) >= aggressive_sweep_threshold) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
278 |
assert_locked_or_safepoint(CodeCache_lock); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
279 |
CodeCache_lock->notify(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
280 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
281 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
282 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
283 |
/** |
30205 | 284 |
* Wakes up the sweeper thread and forces a sweep. Blocks until it finished. |
285 |
*/ |
|
286 |
void NMethodSweeper::force_sweep() { |
|
287 |
ThreadBlockInVM tbivm(JavaThread::current()); |
|
288 |
MutexLockerEx waiter(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
|
289 |
// Request forced sweep |
|
290 |
_force_sweep = true; |
|
291 |
while (_force_sweep) { |
|
292 |
// Notify sweeper that we want to force a sweep and wait for completion. |
|
293 |
// In case a sweep currently takes place we timeout and try again because |
|
294 |
// we want to enforce a full sweep. |
|
295 |
CodeCache_lock->notify(); |
|
296 |
CodeCache_lock->wait(Mutex::_no_safepoint_check_flag, 1000); |
|
297 |
} |
|
298 |
} |
|
299 |
||
300 |
/** |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
301 |
* Handle a safepoint request |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
302 |
*/ |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
303 |
void NMethodSweeper::handle_safepoint_request() { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
304 |
if (SafepointSynchronize::is_synchronizing()) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
305 |
if (PrintMethodFlushing && Verbose) { |
34158
1f8d643b02d5
8067378: Add segmented code heaps info into jfr events: vm/code_cache/stats and vm/code_cache/config
thartmann
parents:
33148
diff
changeset
|
306 |
tty->print_cr("### Sweep at %d out of %d, yielding to safepoint", _seen, CodeCache::nmethod_count()); |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
307 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
308 |
MutexUnlockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
309 |
|
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
310 |
JavaThread* thread = JavaThread::current(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
311 |
ThreadBlockInVM tbivm(thread); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
312 |
thread->java_suspend_self(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
313 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
314 |
} |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
315 |
|
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
316 |
/** |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
317 |
* This function invokes the sweeper if at least one of the three conditions is met: |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
318 |
* (1) The code cache is getting full |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
319 |
* (2) There are sufficient state changes in/since the last sweep. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
320 |
* (3) We have not been sweeping for 'some time' |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
321 |
*/ |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
322 |
void NMethodSweeper::possibly_sweep() { |
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
323 |
assert(JavaThread::current()->thread_state() == _thread_in_vm, "must run in vm mode"); |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
324 |
// If there was no state change while nmethod sweeping, 'should_sweep' will be false. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
325 |
// This is one of the two places where should_sweep can be set to true. The general |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
326 |
// idea is as follows: If there is enough free space in the code cache, there is no |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
327 |
// need to invoke the sweeper. The following formula (which determines whether to invoke |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
328 |
// the sweeper or not) depends on the assumption that for larger ReservedCodeCacheSizes |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
329 |
// we need less frequent sweeps than for smaller ReservedCodecCacheSizes. Furthermore, |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
330 |
// the formula considers how much space in the code cache is currently used. Here are |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
331 |
// some examples that will (hopefully) help in understanding. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
332 |
// |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
333 |
// Small ReservedCodeCacheSizes: (e.g., < 16M) We invoke the sweeper every time, since |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
334 |
// the result of the division is 0. This |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
335 |
// keeps the used code cache size small |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
336 |
// (important for embedded Java) |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
337 |
// Large ReservedCodeCacheSize : (e.g., 256M + code cache is 10% full). The formula |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
338 |
// computes: (256 / 16) - 1 = 15 |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
339 |
// As a result, we invoke the sweeper after |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
340 |
// 15 invocations of 'mark_active_nmethods. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
341 |
// Large ReservedCodeCacheSize: (e.g., 256M + code Cache is 90% full). The formula |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
342 |
// computes: (256 / 16) - 10 = 6. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
343 |
if (!_should_sweep) { |
22208
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
344 |
const int time_since_last_sweep = _time_counter - _last_sweep; |
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
345 |
// ReservedCodeCacheSize has an 'unsigned' type. We need a 'signed' type for max_wait_time, |
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
346 |
// since 'time_since_last_sweep' can be larger than 'max_wait_time'. If that happens using |
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
347 |
// an unsigned type would cause an underflow (wait_until_next_sweep becomes a large positive |
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
348 |
// value) that disables the intended periodic sweeps. |
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
349 |
const int max_wait_time = ReservedCodeCacheSize / (16 * M); |
26796 | 350 |
double wait_until_next_sweep = max_wait_time - time_since_last_sweep - |
351 |
MAX2(CodeCache::reverse_free_ratio(CodeBlobType::MethodProfiled), |
|
352 |
CodeCache::reverse_free_ratio(CodeBlobType::MethodNonProfiled)); |
|
22208
fb1e20bd389f
8029091: Bug in calculation of code cache sweeping interval
anoll
parents:
21767
diff
changeset
|
353 |
assert(wait_until_next_sweep <= (double)max_wait_time, "Calculation of code cache sweeper interval is incorrect"); |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
354 |
|
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
355 |
if ((wait_until_next_sweep <= 0.0) || !CompileBroker::should_compile_new_jobs()) { |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
356 |
_should_sweep = true; |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
357 |
} |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
358 |
} |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
359 |
|
30205 | 360 |
// Remember if this was a forced sweep |
361 |
bool forced = _force_sweep; |
|
362 |
||
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
363 |
// Force stack scanning if there is only 10% free space in the code cache. |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
364 |
// We force stack scanning only if the non-profiled code heap gets full, since critical |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
365 |
// allocations go to the non-profiled heap and we must be make sure that there is |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
366 |
// enough space. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
367 |
double free_percent = 1 / CodeCache::reverse_free_ratio(CodeBlobType::MethodNonProfiled) * 100; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
368 |
if (free_percent <= StartAggressiveSweepingAt) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
369 |
do_stack_scanning(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
370 |
} |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
371 |
|
30205 | 372 |
if (_should_sweep || forced) { |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
373 |
init_sweeper_log(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
374 |
sweep_code_cache(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
375 |
} |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
376 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
377 |
// We are done with sweeping the code cache once. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
378 |
_total_nof_code_cache_sweeps++; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
379 |
_last_sweep = _time_counter; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
380 |
// Reset flag; temporarily disables sweeper |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
381 |
_should_sweep = false; |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
382 |
// If there was enough state change, 'possibly_enable_sweeper()' |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
383 |
// sets '_should_sweep' to true |
30205 | 384 |
possibly_enable_sweeper(); |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
385 |
// Reset _bytes_changed only if there was enough state change. _bytes_changed |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
386 |
// can further increase by calls to 'report_state_change'. |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
387 |
if (_should_sweep) { |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
388 |
_bytes_changed = 0; |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
389 |
} |
30205 | 390 |
|
391 |
if (forced) { |
|
392 |
// Notify requester that forced sweep finished |
|
393 |
assert(_force_sweep, "Should be a forced sweep"); |
|
394 |
MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
|
395 |
_force_sweep = false; |
|
396 |
CodeCache_lock->notify(); |
|
397 |
} |
|
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
398 |
} |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
399 |
|
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
400 |
void NMethodSweeper::sweep_code_cache() { |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
401 |
ResourceMark rm; |
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
21727
diff
changeset
|
402 |
Ticks sweep_start_counter = Ticks::now(); |
18025 | 403 |
|
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
404 |
int flushed_count = 0; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
405 |
int zombified_count = 0; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
406 |
int flushed_c2_count = 0; |
18025 | 407 |
|
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
408 |
if (PrintMethodFlushing && Verbose) { |
34158
1f8d643b02d5
8067378: Add segmented code heaps info into jfr events: vm/code_cache/stats and vm/code_cache/config
thartmann
parents:
33148
diff
changeset
|
409 |
tty->print_cr("### Sweep at %d out of %d", _seen, CodeCache::nmethod_count()); |
17132
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
410 |
} |
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
411 |
|
20019
3e9220f93c61
8022883: Assertion failed: sweptCount >= flushedCount + markedCount + zombifiedCount
anoll
parents:
18025
diff
changeset
|
412 |
int swept_count = 0; |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
413 |
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
|
414 |
assert(!CodeCache_lock->owned_by_self(), "just checking"); |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
415 |
|
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
416 |
int freed_memory = 0; |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
417 |
{ |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
418 |
MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
419 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
420 |
while (!_current.end()) { |
20019
3e9220f93c61
8022883: Assertion failed: sweptCount >= flushedCount + markedCount + zombifiedCount
anoll
parents:
18025
diff
changeset
|
421 |
swept_count++; |
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
422 |
// 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
|
423 |
// 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
|
424 |
// threads but nmethods are only reclaimed by the sweeper. |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
425 |
CompiledMethod* nm = _current.method(); |
26796 | 426 |
_current.next(); |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
427 |
|
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
428 |
// Now ready to process nmethod and give up CodeCache_lock |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
429 |
{ |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
430 |
MutexUnlockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
431 |
// Save information before potentially flushing the nmethod |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
432 |
// Only flushing nmethods so size only matters for them. |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
433 |
int size = nm->is_nmethod() ? ((nmethod*)nm)->total_size() : 0; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
434 |
bool is_c2_method = nm->is_compiled_by_c2(); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
435 |
bool is_osr = nm->is_osr_method(); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
436 |
int compile_id = nm->compile_id(); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
437 |
intptr_t address = p2i(nm); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
438 |
const char* state_before = nm->state(); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
439 |
const char* state_after = ""; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
440 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
441 |
MethodStateChange type = process_compiled_method(nm); |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
442 |
switch (type) { |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
443 |
case Flushed: |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
444 |
state_after = "flushed"; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
445 |
freed_memory += size; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
446 |
++flushed_count; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
447 |
if (is_c2_method) { |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
448 |
++flushed_c2_count; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
449 |
} |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
450 |
break; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
451 |
case MadeZombie: |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
452 |
state_after = "made zombie"; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
453 |
++zombified_count; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
454 |
break; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
455 |
case None: |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
456 |
break; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
457 |
default: |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
458 |
ShouldNotReachHere(); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
459 |
} |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
460 |
if (PrintMethodFlushing && Verbose && type != None) { |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
461 |
tty->print_cr("### %s nmethod %3d/" PTR_FORMAT " (%s) %s", is_osr ? "osr" : "", compile_id, address, state_before, state_after); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
462 |
} |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
463 |
} |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
464 |
|
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
465 |
_seen++; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
466 |
handle_safepoint_request(); |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
467 |
} |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
468 |
} |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
469 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
470 |
assert(_current.end(), "must have scanned the whole cache"); |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
471 |
|
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
21727
diff
changeset
|
472 |
const Ticks sweep_end_counter = Ticks::now(); |
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
21727
diff
changeset
|
473 |
const Tickspan sweep_time = sweep_end_counter - sweep_start_counter; |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
474 |
{ |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
475 |
MutexLockerEx mu(_stat_lock, Mutex::_no_safepoint_check_flag); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
476 |
_total_time_sweeping += sweep_time; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
477 |
_total_time_this_sweep += sweep_time; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
478 |
_peak_sweep_fraction_time = MAX2(sweep_time, _peak_sweep_fraction_time); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
479 |
_total_flushed_size += freed_memory; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
480 |
_total_nof_methods_reclaimed += flushed_count; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
481 |
_total_nof_c2_methods_reclaimed += flushed_c2_count; |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
482 |
_peak_sweep_time = MAX2(_peak_sweep_time, _total_time_this_sweep); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
483 |
} |
18025 | 484 |
EventSweepCodeCache event(UNTIMED); |
485 |
if (event.should_commit()) { |
|
486 |
event.set_starttime(sweep_start_counter); |
|
487 |
event.set_endtime(sweep_end_counter); |
|
40664
1ec65b303bb7
8164523: Clean up metadata for event based tracing
egahlin
parents:
40655
diff
changeset
|
488 |
event.set_sweepId(_traversals); |
20019
3e9220f93c61
8022883: Assertion failed: sweptCount >= flushedCount + markedCount + zombifiedCount
anoll
parents:
18025
diff
changeset
|
489 |
event.set_sweptCount(swept_count); |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
490 |
event.set_flushedCount(flushed_count); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
491 |
event.set_zombifiedCount(zombified_count); |
18025 | 492 |
event.commit(); |
493 |
} |
|
494 |
||
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
495 |
#ifdef ASSERT |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
496 |
if(PrintMethodFlushing) { |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
497 |
tty->print_cr("### sweeper: sweep time(" JLONG_FORMAT "): ", sweep_time.value()); |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
498 |
} |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
499 |
#endif |
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
500 |
|
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
501 |
log_sweep("finished"); |
17132
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
502 |
|
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
503 |
// Sweeper is the only case where memory is released, check here if it |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
504 |
// is time to restart the compiler. Only checking if there is a certain |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
505 |
// amount of free memory in the code cache might lead to re-enabling |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
506 |
// compilation although no memory has been released. For example, there are |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
507 |
// cases when compilation was disabled although there is 4MB (or more) free |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
508 |
// memory in the code cache. The reason is code cache fragmentation. Therefore, |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
509 |
// it only makes sense to re-enable compilation if we have actually freed memory. |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
510 |
// Note that typically several kB are released for sweeping 16MB of the code |
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
511 |
// cache. As a result, 'freed_memory' > 0 to restart the compiler. |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
512 |
if (!CompileBroker::should_compile_new_jobs() && (freed_memory > 0)) { |
17132
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
513 |
CompileBroker::set_should_compile_new_jobs(CompileBroker::run_compilation); |
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
514 |
log_sweep("restart_compiler"); |
dffd513b2a8c
8012547: Code cache flushing can get stuck reclaming of memory
neliasso
parents:
13728
diff
changeset
|
515 |
} |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
516 |
} |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
517 |
|
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
518 |
/** |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
519 |
* This function updates the sweeper statistics that keep track of nmethods |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
520 |
* state changes. If there is 'enough' state change, the sweeper is invoked |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
521 |
* as soon as possible. There can be data races on _bytes_changed. The data |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
522 |
* races are benign, since it does not matter if we loose a couple of bytes. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
523 |
* In the worst case we call the sweeper a little later. Also, we are guaranteed |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
524 |
* to invoke the sweeper if the code cache gets full. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
525 |
*/ |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
526 |
void NMethodSweeper::report_state_change(nmethod* nm) { |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
527 |
_bytes_changed += nm->total_size(); |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
528 |
possibly_enable_sweeper(); |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
529 |
} |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
530 |
|
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
531 |
/** |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
532 |
* Function determines if there was 'enough' state change in the code cache to invoke |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
533 |
* the sweeper again. Currently, we determine 'enough' as more than 1% state change in |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
534 |
* the code cache since the last sweep. |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
535 |
*/ |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
536 |
void NMethodSweeper::possibly_enable_sweeper() { |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
537 |
double percent_changed = ((double)_bytes_changed / (double)ReservedCodeCacheSize) * 100; |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
538 |
if (percent_changed > 1.0) { |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
539 |
_should_sweep = true; |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
540 |
} |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
541 |
} |
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
542 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
543 |
class CompiledMethodMarker: public StackObj { |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
544 |
private: |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
545 |
CodeCacheSweeperThread* _thread; |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
546 |
public: |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
547 |
CompiledMethodMarker(CompiledMethod* cm) { |
27420
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
548 |
JavaThread* current = JavaThread::current(); |
04e6f914cce1
8046809: vm/mlvm/meth/stress/compiler/deoptimize CodeCache is full.
anoll
parents:
27017
diff
changeset
|
549 |
assert (current->is_Code_cache_sweeper_thread(), "Must be"); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
550 |
_thread = (CodeCacheSweeperThread*)current; |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
551 |
if (!cm->is_zombie() && !cm->is_unloaded()) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
552 |
// Only expose live nmethods for scanning |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
553 |
_thread->set_scanned_compiled_method(cm); |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
554 |
} |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
555 |
} |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
556 |
~CompiledMethodMarker() { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
557 |
_thread->set_scanned_compiled_method(NULL); |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
558 |
} |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
559 |
}; |
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
560 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
561 |
void NMethodSweeper::release_compiled_method(CompiledMethod* nm) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
562 |
// Make sure the released nmethod is no longer referenced by the sweeper thread |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
563 |
CodeCacheSweeperThread* thread = (CodeCacheSweeperThread*)JavaThread::current(); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
564 |
thread->set_scanned_compiled_method(NULL); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
565 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
566 |
// Clean up any CompiledICHolders |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
567 |
{ |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
568 |
ResourceMark rm; |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
569 |
MutexLocker ml_patch(CompiledIC_lock); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
570 |
RelocIterator iter(nm); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
571 |
while (iter.next()) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
572 |
if (iter.type() == relocInfo::virtual_call_type) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
573 |
CompiledIC::cleanup_call_site(iter.virtual_call_reloc()); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
574 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
575 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
576 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
577 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
578 |
MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
579 |
nm->flush(); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
580 |
} |
1 | 581 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
582 |
NMethodSweeper::MethodStateChange NMethodSweeper::process_compiled_method(CompiledMethod* cm) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
583 |
assert(cm != NULL, "sanity"); |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
584 |
assert(!CodeCache_lock->owned_by_self(), "just checking"); |
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
585 |
|
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
586 |
MethodStateChange result = None; |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
587 |
// Make sure this nmethod doesn't get unloaded during the scan, |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
588 |
// since safepoints may happen during acquired below locks. |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
589 |
CompiledMethodMarker nmm(cm); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
590 |
SWEEP(cm); |
9942
2e86734ba620
6996747: SIGSEGV in nmethod::cleanup_inline_caches / CompiledIC::verify
never
parents:
9138
diff
changeset
|
591 |
|
1 | 592 |
// Skip methods that are currently referenced by the VM |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
593 |
if (cm->is_locked_by_vm()) { |
1 | 594 |
// But still remember to clean-up inline caches for alive nmethods |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
595 |
if (cm->is_alive()) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
596 |
// Clean inline caches that point to zombie/non-entrant/unloaded nmethods |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
597 |
MutexLocker cl(CompiledIC_lock); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
598 |
cm->cleanup_inline_caches(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
599 |
SWEEP(cm); |
1 | 600 |
} |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
601 |
return result; |
1 | 602 |
} |
603 |
||
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
604 |
if (cm->is_zombie()) { |
36818
b40330c06dea
8136458: Remove "marked for reclamation" nmethod state
thartmann
parents:
36802
diff
changeset
|
605 |
// All inline caches that referred to this nmethod were cleaned in the |
b40330c06dea
8136458: Remove "marked for reclamation" nmethod state
thartmann
parents:
36802
diff
changeset
|
606 |
// previous sweeper cycle. Now flush the nmethod from the code cache. |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
607 |
assert(!cm->is_locked_by_vm(), "must not flush locked Compiled Methods"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
608 |
release_compiled_method(cm); |
36818
b40330c06dea
8136458: Remove "marked for reclamation" nmethod state
thartmann
parents:
36802
diff
changeset
|
609 |
assert(result == None, "sanity"); |
b40330c06dea
8136458: Remove "marked for reclamation" nmethod state
thartmann
parents:
36802
diff
changeset
|
610 |
result = Flushed; |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
611 |
} else if (cm->is_not_entrant()) { |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
612 |
// If there are no current activations of this method on the |
1 | 613 |
// stack we can safely convert it to a zombie method |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
614 |
if (cm->can_convert_to_zombie()) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
615 |
// Clear ICStubs to prevent back patching stubs of zombie or flushed |
26912
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26799
diff
changeset
|
616 |
// nmethods during the next safepoint (see ICStub::finalize). |
27017
bae5d661dd4b
8059735: make_not_entrant_or_zombie sees zombies
thartmann
parents:
26912
diff
changeset
|
617 |
{ |
bae5d661dd4b
8059735: make_not_entrant_or_zombie sees zombies
thartmann
parents:
26912
diff
changeset
|
618 |
MutexLocker cl(CompiledIC_lock); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
619 |
cm->clear_ic_stubs(); |
27017
bae5d661dd4b
8059735: make_not_entrant_or_zombie sees zombies
thartmann
parents:
26912
diff
changeset
|
620 |
} |
32401
cc58aeaec340
8075805: Crash while trying to release CompiledICHolder
thartmann
parents:
31348
diff
changeset
|
621 |
// Code cache state change is tracked in make_zombie() |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
622 |
cm->make_zombie(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
623 |
SWEEP(cm); |
36816
987a22271911
8152861: VM crash on assert: locked methods shouldn't be flushed
thartmann
parents:
36802
diff
changeset
|
624 |
// The nmethod may have been locked by JVMTI after being made zombie (see |
987a22271911
8152861: VM crash on assert: locked methods shouldn't be flushed
thartmann
parents:
36802
diff
changeset
|
625 |
// JvmtiDeferredEvent::compiled_method_unload_event()). If so, we cannot |
987a22271911
8152861: VM crash on assert: locked methods shouldn't be flushed
thartmann
parents:
36802
diff
changeset
|
626 |
// flush the osr nmethod directly but have to wait for a later sweeper cycle. |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
627 |
if (cm->is_osr_method() && !cm->is_locked_by_vm()) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
628 |
// No inline caches will ever point to osr methods, so we can just remove it. |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
629 |
// Make sure that we unregistered the nmethod with the heap and flushed all |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
630 |
// dependencies before removing the nmethod (done in make_zombie()). |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
631 |
assert(cm->is_zombie(), "nmethod must be unregistered"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
632 |
release_compiled_method(cm); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
633 |
assert(result == None, "sanity"); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
634 |
result = Flushed; |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
635 |
} else { |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
636 |
assert(result == None, "sanity"); |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
637 |
result = MadeZombie; |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
638 |
assert(cm->is_zombie(), "nmethod must be zombie"); |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
639 |
} |
1 | 640 |
} else { |
641 |
// Still alive, clean up its inline caches |
|
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
642 |
MutexLocker cl(CompiledIC_lock); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
643 |
cm->cleanup_inline_caches(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
644 |
SWEEP(cm); |
1 | 645 |
} |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
646 |
} else if (cm->is_unloaded()) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
647 |
// Code is unloaded, so there are no activations on the stack. |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
648 |
// Convert the nmethod to zombie or flush it directly in the OSR case. |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
649 |
{ |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
650 |
// Clean ICs of unloaded nmethods as well because they may reference other |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
651 |
// unloaded nmethods that may be flushed earlier in the sweeper cycle. |
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
652 |
MutexLocker cl(CompiledIC_lock); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
653 |
cm->cleanup_inline_caches(); |
20290
2127dc70bce9
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
20019
diff
changeset
|
654 |
} |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
655 |
if (cm->is_osr_method()) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
656 |
SWEEP(cm); |
1 | 657 |
// No inline caches will ever point to osr methods, so we can just remove it |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
658 |
release_compiled_method(cm); |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
659 |
assert(result == None, "sanity"); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
660 |
result = Flushed; |
1 | 661 |
} else { |
21575
6a9645992cee
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
20290
diff
changeset
|
662 |
// Code cache state change is tracked in make_zombie() |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
663 |
cm->make_zombie(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
664 |
SWEEP(cm); |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
665 |
assert(result == None, "sanity"); |
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
666 |
result = MadeZombie; |
1 | 667 |
} |
668 |
} else { |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
669 |
if (cm->is_nmethod()) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
670 |
possibly_flush((nmethod*)cm); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
671 |
} |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
672 |
// Clean inline caches that point to zombie/non-entrant/unloaded nmethods |
5533
e8d9ff82ec62
6950075: nmethod sweeper should operate concurrently
never
parents:
4750
diff
changeset
|
673 |
MutexLocker cl(CompiledIC_lock); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
674 |
cm->cleanup_inline_caches(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36844
diff
changeset
|
675 |
SWEEP(cm); |
1 | 676 |
} |
27917
c5937f7b4e8b
8064669: compiler/whitebox/AllocationCodeBlobTest.java crashes / asserts
iignatyev
parents:
27420
diff
changeset
|
677 |
return result; |
4750 | 678 |
} |
679 |
||
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
680 |
|
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
681 |
void NMethodSweeper::possibly_flush(nmethod* nm) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
682 |
if (UseCodeCacheFlushing) { |
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
683 |
if (!nm->is_locked_by_vm() && !nm->is_native_method()) { |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
684 |
bool make_not_entrant = false; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
685 |
|
36802
18b1db5a7e70
8023191: OSR nmethods should be flushed to free space in CodeCache
thartmann
parents:
34158
diff
changeset
|
686 |
// Do not make native methods not-entrant |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
687 |
nm->dec_hotness_counter(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
688 |
// Get the initial value of the hotness counter. This value depends on the |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
689 |
// ReservedCodeCacheSize |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
690 |
int reset_val = hotness_counter_reset_val(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
691 |
int time_since_reset = reset_val - nm->hotness_counter(); |
31348
c28f02c7abb5
8077279: assert(ic->is_clean()) failed: IC should be clean
sjohanss
parents:
30205
diff
changeset
|
692 |
int code_blob_type = CodeCache::get_code_blob_type(nm); |
26796 | 693 |
double threshold = -reset_val + (CodeCache::reverse_free_ratio(code_blob_type) * NmethodSweepActivity); |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
694 |
// The less free space in the code cache we have - the bigger reverse_free_ratio() is. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
695 |
// I.e., 'threshold' increases with lower available space in the code cache and a higher |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
696 |
// NmethodSweepActivity. If the current hotness counter - which decreases from its initial |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
697 |
// value until it is reset by stack walking - is smaller than the computed threshold, the |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
698 |
// corresponding nmethod is considered for removal. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
699 |
if ((NmethodSweepActivity > 0) && (nm->hotness_counter() < threshold) && (time_since_reset > MinPassesBeforeFlush)) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
700 |
// A method is marked as not-entrant if the method is |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
701 |
// 1) 'old enough': nm->hotness_counter() < threshold |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
702 |
// 2) The method was in_use for a minimum amount of time: (time_since_reset > MinPassesBeforeFlush) |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
703 |
// The second condition is necessary if we are dealing with very small code cache |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
704 |
// sizes (e.g., <10m) and the code cache size is too small to hold all hot methods. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
705 |
// The second condition ensures that methods are not immediately made not-entrant |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
706 |
// after compilation. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
707 |
make_not_entrant = true; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
708 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
709 |
|
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
710 |
// The stack-scanning low-cost detection may not see the method was used (which can happen for |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
711 |
// flat profiles). Check the age counter for possible data. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
712 |
if (UseCodeAging && make_not_entrant && (nm->is_compiled_by_c2() || nm->is_compiled_by_c1())) { |
26799
0c349e16bbe8
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
26796
diff
changeset
|
713 |
MethodCounters* mc = nm->method()->get_method_counters(Thread::current()); |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
714 |
if (mc != NULL) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
715 |
// Snapshot the value as it's changed concurrently |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
716 |
int age = mc->nmethod_age(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
717 |
if (MethodCounters::is_nmethod_hot(age)) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
718 |
// The method has gone through flushing, and it became relatively hot that it deopted |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
719 |
// before we could take a look at it. Give it more time to appear in the stack traces, |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
720 |
// proportional to the number of deopts. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
721 |
MethodData* md = nm->method()->method_data(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
722 |
if (md != NULL && time_since_reset > (int)(MinPassesBeforeFlush * (md->tenure_traps() + 1))) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
723 |
// It's been long enough, we still haven't seen it on stack. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
724 |
// Try to flush it, but enable counters the next time. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
725 |
mc->reset_nmethod_age(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
726 |
} else { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
727 |
make_not_entrant = false; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
728 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
729 |
} else if (MethodCounters::is_nmethod_warm(age)) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
730 |
// Method has counters enabled, and the method was used within |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
731 |
// previous MinPassesBeforeFlush sweeps. Reset the counter. Stay in the existing |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
732 |
// compiled state. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
733 |
mc->reset_nmethod_age(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
734 |
// delay the next check |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
735 |
nm->set_hotness_counter(NMethodSweeper::hotness_counter_reset_val()); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
736 |
make_not_entrant = false; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
737 |
} else if (MethodCounters::is_nmethod_age_unset(age)) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
738 |
// No counters were used before. Set the counters to the detection |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
739 |
// limit value. If the method is going to be used again it will be compiled |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
740 |
// with counters that we're going to use for analysis the the next time. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
741 |
mc->reset_nmethod_age(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
742 |
} else { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
743 |
// Method was totally idle for 10 sweeps |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
744 |
// The counter already has the initial value, flush it and may be recompile |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
745 |
// later with counters |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
746 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
747 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
748 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
749 |
|
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
750 |
if (make_not_entrant) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
751 |
nm->make_not_entrant(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
752 |
|
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
753 |
// Code cache state change is tracked in make_not_entrant() |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
754 |
if (PrintMethodFlushing && Verbose) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
755 |
tty->print_cr("### Nmethod %d/" PTR_FORMAT "made not-entrant: hotness counter %d/%d threshold %f", |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
32466
diff
changeset
|
756 |
nm->compile_id(), p2i(nm), nm->hotness_counter(), reset_val, threshold); |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
757 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
758 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
759 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
760 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
761 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
762 |
|
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
763 |
// 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
|
764 |
// state of the code cache if it's requested. |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
765 |
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
|
766 |
if (PrintMethodFlushing) { |
29339
f0b1b7788a51
8073257: compiler/codecache/stress/RandomAllocationTest.java + fastdebug + -XX:+LogCompilation, "allocating without ResourceMark"
thartmann
parents:
28163
diff
changeset
|
767 |
ResourceMark rm; |
9138
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
768 |
stringStream s; |
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
769 |
// 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
|
770 |
// 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
|
771 |
CodeCache::log_state(&s); |
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
772 |
|
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
773 |
ttyLocker ttyl; |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
774 |
tty->print("### sweeper: %s ", msg); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
775 |
if (format != NULL) { |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
776 |
va_list ap; |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
777 |
va_start(ap, format); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
778 |
tty->vprint(format, ap); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
779 |
va_end(ap); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
780 |
} |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24351
diff
changeset
|
781 |
tty->print_cr("%s", s.as_string()); |
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
782 |
} |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
783 |
|
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
784 |
if (LogCompilation && (xtty != NULL)) { |
29339
f0b1b7788a51
8073257: compiler/codecache/stress/RandomAllocationTest.java + fastdebug + -XX:+LogCompilation, "allocating without ResourceMark"
thartmann
parents:
28163
diff
changeset
|
785 |
ResourceMark rm; |
9138
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
786 |
stringStream s; |
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
787 |
// 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
|
788 |
// 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
|
789 |
CodeCache::log_state(&s); |
35e4b04eadf4
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
8921
diff
changeset
|
790 |
|
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
791 |
ttyLocker ttyl; |
5926 | 792 |
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
|
793 |
if (format != NULL) { |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
794 |
va_list ap; |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
795 |
va_start(ap, format); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
796 |
xtty->vprint(format, ap); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
797 |
va_end(ap); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
798 |
} |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24351
diff
changeset
|
799 |
xtty->print("%s", s.as_string()); |
5924
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
800 |
xtty->stamp(); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
801 |
xtty->end_elem(); |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
802 |
} |
dc9d04930c82
6965184: possible races in make_not_entrant_or_zombie
never
parents:
5895
diff
changeset
|
803 |
} |
22210
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
804 |
|
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
805 |
void NMethodSweeper::print() { |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
806 |
ttyLocker ttyl; |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
807 |
tty->print_cr("Code cache sweeper statistics:"); |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
808 |
tty->print_cr(" Total sweep time: %1.0lfms", (double)_total_time_sweeping.value()/1000000); |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
809 |
tty->print_cr(" Total number of full sweeps: %ld", _total_nof_code_cache_sweeps); |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
810 |
tty->print_cr(" Total number of flushed methods: %ld(%ld C2 methods)", _total_nof_methods_reclaimed, |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
811 |
_total_nof_c2_methods_reclaimed); |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
812 |
tty->print_cr(" Total size of flushed methods: " SIZE_FORMAT "kB", _total_flushed_size/K); |
b0408ba029f6
8025277: Add -XX: flag to print code cache sweeper statistics
anoll
parents:
22208
diff
changeset
|
813 |
} |