author | stefank |
Thu, 22 Feb 2018 18:36:32 +0100 | |
changeset 49048 | 4e8c86b75428 |
parent 47216 | 71c04702a3d5 |
child 49722 | a47d1e21b3f1 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
2 |
* Copyright (c) 2000, 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:
1374
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1374
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:
1374
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
30764 | 25 |
#ifndef SHARE_VM_GC_SHARED_SPACE_INLINE_HPP |
26 |
#define SHARE_VM_GC_SHARED_SPACE_INLINE_HPP |
|
7397 | 27 |
|
30764 | 28 |
#include "gc/serial/markSweep.inline.hpp" |
29 |
#include "gc/shared/collectedHeap.hpp" |
|
30 |
#include "gc/shared/generation.hpp" |
|
31 |
#include "gc/shared/space.hpp" |
|
32 |
#include "gc/shared/spaceDecorator.hpp" |
|
7397 | 33 |
#include "memory/universe.hpp" |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
34 |
#include "oops/oopsHierarchy.hpp" |
24487
71ff0bd674eb
8042737: Introduce umbrella header prefetch.inline.hpp
goetz
parents:
13963
diff
changeset
|
35 |
#include "runtime/prefetch.inline.hpp" |
7397 | 36 |
#include "runtime/safepoint.hpp" |
37 |
||
1374 | 38 |
inline HeapWord* Space::block_start(const void* p) { |
39 |
return block_start_const(p); |
|
40 |
} |
|
41 |
||
1 | 42 |
inline HeapWord* OffsetTableContigSpace::allocate(size_t size) { |
43 |
HeapWord* res = ContiguousSpace::allocate(size); |
|
44 |
if (res != NULL) { |
|
45 |
_offsets.alloc_block(res, size); |
|
46 |
} |
|
47 |
return res; |
|
48 |
} |
|
49 |
||
50 |
// Because of the requirement of keeping "_offsets" up to date with the |
|
51 |
// allocations, we sequentialize these with a lock. Therefore, best if |
|
52 |
// this is used for larger LAB allocations only. |
|
53 |
inline HeapWord* OffsetTableContigSpace::par_allocate(size_t size) { |
|
54 |
MutexLocker x(&_par_alloc_lock); |
|
55 |
// This ought to be just "allocate", because of the lock above, but that |
|
56 |
// ContiguousSpace::allocate asserts that either the allocating thread |
|
57 |
// holds the heap lock or it is the VM thread and we're at a safepoint. |
|
58 |
// The best I (dld) could figure was to put a field in ContiguousSpace |
|
59 |
// meaning "locking at safepoint taken care of", and set/reset that |
|
60 |
// here. But this will do for now, especially in light of the comment |
|
61 |
// above. Perhaps in the future some lock-free manner of keeping the |
|
62 |
// coordination. |
|
63 |
HeapWord* res = ContiguousSpace::par_allocate(size); |
|
64 |
if (res != NULL) { |
|
65 |
_offsets.alloc_block(res, size); |
|
66 |
} |
|
67 |
return res; |
|
68 |
} |
|
69 |
||
1374 | 70 |
inline HeapWord* |
71 |
OffsetTableContigSpace::block_start_const(const void* p) const { |
|
1 | 72 |
return _offsets.block_start(p); |
73 |
} |
|
74 |
||
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
75 |
size_t CompactibleSpace::obj_size(const HeapWord* addr) const { |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
76 |
return oop(addr)->size(); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
77 |
} |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
78 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
79 |
class DeadSpacer : StackObj { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
80 |
size_t _allowed_deadspace_words; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
81 |
bool _active; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
82 |
CompactibleSpace* _space; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
83 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
84 |
public: |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
85 |
DeadSpacer(CompactibleSpace* space) : _space(space), _allowed_deadspace_words(0) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
86 |
size_t ratio = _space->allowed_dead_ratio(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
87 |
_active = ratio > 0; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
88 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
89 |
if (_active) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
90 |
assert(!UseG1GC, "G1 should not be using dead space"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
91 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
92 |
// We allow some amount of garbage towards the bottom of the space, so |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
93 |
// we don't start compacting before there is a significant gain to be made. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
94 |
// Occasionally, we want to ensure a full compaction, which is determined |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
95 |
// by the MarkSweepAlwaysCompactCount parameter. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
96 |
if ((MarkSweep::total_invocations() % MarkSweepAlwaysCompactCount) != 0) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
97 |
_allowed_deadspace_words = (space->capacity() * ratio / 100) / HeapWordSize; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
98 |
} else { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
99 |
_active = false; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
100 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
101 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
102 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
103 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
104 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
105 |
bool insert_deadspace(HeapWord* dead_start, HeapWord* dead_end) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
106 |
if (!_active) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
107 |
return false; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
108 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
109 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
110 |
size_t dead_length = pointer_delta(dead_end, dead_start); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
111 |
if (_allowed_deadspace_words >= dead_length) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
112 |
_allowed_deadspace_words -= dead_length; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
113 |
CollectedHeap::fill_with_object(dead_start, dead_length); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
114 |
oop obj = oop(dead_start); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
115 |
obj->set_mark(obj->mark()->set_marked()); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
116 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
117 |
assert(dead_length == (size_t)obj->size(), "bad filler object size"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
118 |
log_develop_trace(gc, compaction)("Inserting object to dead space: " PTR_FORMAT ", " PTR_FORMAT ", " SIZE_FORMAT "b", |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
119 |
p2i(dead_start), p2i(dead_end), dead_length * HeapWordSize); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
120 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
121 |
return true; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
122 |
} else { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
123 |
_active = false; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
124 |
return false; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
125 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
126 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
127 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
128 |
}; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
129 |
|
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
130 |
template <class SpaceType> |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
131 |
inline void CompactibleSpace::scan_and_forward(SpaceType* space, CompactPoint* cp) { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
132 |
// Compute the new addresses for the live objects and store it in the mark |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
133 |
// Used by universe::mark_sweep_phase2() |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
134 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
135 |
// We're sure to be here before any objects are compacted into this |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
136 |
// space, so this is a good time to initialize this: |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
137 |
space->set_compaction_top(space->bottom()); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
138 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
139 |
if (cp->space == NULL) { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
140 |
assert(cp->gen != NULL, "need a generation"); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
141 |
assert(cp->threshold == NULL, "just checking"); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
142 |
assert(cp->gen->first_compaction_space() == space, "just checking"); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
143 |
cp->space = cp->gen->first_compaction_space(); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
144 |
cp->threshold = cp->space->initialize_threshold(); |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
145 |
cp->space->set_compaction_top(cp->space->bottom()); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
146 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
147 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
148 |
HeapWord* compact_top = cp->space->compaction_top(); // This is where we are currently compacting to. |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
149 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
150 |
DeadSpacer dead_spacer(space); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
151 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
152 |
HeapWord* end_of_live = space->bottom(); // One byte beyond the last byte of the last live object. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
153 |
HeapWord* first_dead = NULL; // The first dead object. |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
154 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
155 |
const intx interval = PrefetchScanIntervalInBytes; |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
156 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
157 |
HeapWord* cur_obj = space->bottom(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
158 |
HeapWord* scan_limit = space->scan_limit(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
159 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
160 |
while (cur_obj < scan_limit) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
161 |
assert(!space->scanned_block_is_obj(cur_obj) || |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
162 |
oop(cur_obj)->mark()->is_marked() || oop(cur_obj)->mark()->is_unlocked() || |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
163 |
oop(cur_obj)->mark()->has_bias_pattern(), |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
164 |
"these are the only valid states during a mark sweep"); |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
165 |
if (space->scanned_block_is_obj(cur_obj) && oop(cur_obj)->is_gc_marked()) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
166 |
// prefetch beyond cur_obj |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
167 |
Prefetch::write(cur_obj, interval); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
168 |
size_t size = space->scanned_block_size(cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
169 |
compact_top = cp->space->forward(oop(cur_obj), size, cp, compact_top); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
170 |
cur_obj += size; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
171 |
end_of_live = cur_obj; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
172 |
} else { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
173 |
// run over all the contiguous dead objects |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
174 |
HeapWord* end = cur_obj; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
175 |
do { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
176 |
// prefetch beyond end |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
177 |
Prefetch::write(end, interval); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
178 |
end += space->scanned_block_size(end); |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
179 |
} while (end < scan_limit && (!space->scanned_block_is_obj(end) || !oop(end)->is_gc_marked())); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
180 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
181 |
// see if we might want to pretend this object is alive so that |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
182 |
// we don't have to compact quite as often. |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
183 |
if (cur_obj == compact_top && dead_spacer.insert_deadspace(cur_obj, end)) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
184 |
oop obj = oop(cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
185 |
compact_top = cp->space->forward(obj, obj->size(), cp, compact_top); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
186 |
end_of_live = end; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
187 |
} else { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
188 |
// otherwise, it really is a free region. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
189 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
190 |
// cur_obj is a pointer to a dead object. Use this dead memory to store a pointer to the next live object. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
191 |
*(HeapWord**)cur_obj = end; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
192 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
193 |
// see if this is the first dead region. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
194 |
if (first_dead == NULL) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
195 |
first_dead = cur_obj; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
196 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
197 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
198 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
199 |
// move on to the next object |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
200 |
cur_obj = end; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
201 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
202 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
203 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
204 |
assert(cur_obj == scan_limit, "just checking"); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
205 |
space->_end_of_live = end_of_live; |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
206 |
if (first_dead != NULL) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
207 |
space->_first_dead = first_dead; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
208 |
} else { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
209 |
space->_first_dead = end_of_live; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
210 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
211 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
212 |
// save the compaction_top of the compaction space. |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
213 |
cp->space->set_compaction_top(compact_top); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
214 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
215 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
216 |
template <class SpaceType> |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
217 |
inline void CompactibleSpace::scan_and_adjust_pointers(SpaceType* space) { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
218 |
// adjust all the interior pointers to point at the new locations of objects |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
219 |
// Used by MarkSweep::mark_sweep_phase3() |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
220 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
221 |
HeapWord* cur_obj = space->bottom(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
222 |
HeapWord* const end_of_live = space->_end_of_live; // Established by "scan_and_forward". |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
223 |
HeapWord* const first_dead = space->_first_dead; // Established by "scan_and_forward". |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
224 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
225 |
assert(first_dead <= end_of_live, "Stands to reason, no?"); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
226 |
|
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
227 |
const intx interval = PrefetchScanIntervalInBytes; |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
228 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
229 |
debug_only(HeapWord* prev_obj = NULL); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
230 |
while (cur_obj < end_of_live) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
231 |
Prefetch::write(cur_obj, interval); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
232 |
if (cur_obj < first_dead || oop(cur_obj)->is_gc_marked()) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
233 |
// cur_obj is alive |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
234 |
// point all the oops to the new location |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
235 |
size_t size = MarkSweep::adjust_pointers(oop(cur_obj)); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
236 |
size = space->adjust_obj_size(size); |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
237 |
debug_only(prev_obj = cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
238 |
cur_obj += size; |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
239 |
} else { |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
240 |
debug_only(prev_obj = cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
241 |
// cur_obj is not a live object, instead it points at the next live object |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
242 |
cur_obj = *(HeapWord**)cur_obj; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
243 |
assert(cur_obj > prev_obj, "we should be moving forward through memory, cur_obj: " PTR_FORMAT ", prev_obj: " PTR_FORMAT, p2i(cur_obj), p2i(prev_obj)); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
244 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
245 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
246 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
247 |
assert(cur_obj == end_of_live, "just checking"); |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
248 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
249 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
250 |
#ifdef ASSERT |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
251 |
template <class SpaceType> |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
252 |
inline void CompactibleSpace::verify_up_to_first_dead(SpaceType* space) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
253 |
HeapWord* cur_obj = space->bottom(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
254 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
255 |
if (cur_obj < space->_end_of_live && space->_first_dead > cur_obj && !oop(cur_obj)->is_gc_marked()) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
256 |
// we have a chunk of the space which hasn't moved and we've reinitialized |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
257 |
// the mark word during the previous pass, so we can't use is_gc_marked for |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
258 |
// the traversal. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
259 |
HeapWord* prev_obj = NULL; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
260 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
261 |
while (cur_obj < space->_first_dead) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
262 |
size_t size = space->obj_size(cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
263 |
assert(!oop(cur_obj)->is_gc_marked(), "should be unmarked (special dense prefix handling)"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
264 |
prev_obj = cur_obj; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
265 |
cur_obj += size; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
266 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
267 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
268 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
269 |
#endif |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
270 |
|
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
271 |
template <class SpaceType> |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
272 |
inline void CompactibleSpace::clear_empty_region(SpaceType* space) { |
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
273 |
// Let's remember if we were empty before we did the compaction. |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
274 |
bool was_empty = space->used_region().is_empty(); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
275 |
// Reset space after compaction is complete |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
276 |
space->reset_after_compaction(); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
277 |
// We do this clear, below, since it has overloaded meanings for some |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
278 |
// space subtypes. For example, OffsetTableContigSpace's that were |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
279 |
// compacted into will have had their offset table thresholds updated |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
280 |
// continuously, but those that weren't need to have their thresholds |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
281 |
// re-initialized. Also mangles unused area for debugging. |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
282 |
if (space->used_region().is_empty()) { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
283 |
if (!was_empty) space->clear(SpaceDecorator::Mangle); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
284 |
} else { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
285 |
if (ZapUnusedHeapArea) space->mangle_unused_area(); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
286 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
24487
diff
changeset
|
287 |
} |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
288 |
|
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
289 |
template <class SpaceType> |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
290 |
inline void CompactibleSpace::scan_and_compact(SpaceType* space) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
291 |
// Copy all live objects to their new location |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
292 |
// Used by MarkSweep::mark_sweep_phase4() |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
293 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
294 |
verify_up_to_first_dead(space); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
295 |
|
40888
f09b053be364
8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents:
37495
diff
changeset
|
296 |
HeapWord* const bottom = space->bottom(); |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
297 |
HeapWord* const end_of_live = space->_end_of_live; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
298 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
299 |
assert(space->_first_dead <= end_of_live, "Invariant. _first_dead: " PTR_FORMAT " <= end_of_live: " PTR_FORMAT, p2i(space->_first_dead), p2i(end_of_live)); |
40888
f09b053be364
8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents:
37495
diff
changeset
|
300 |
if (space->_first_dead == end_of_live && (bottom == end_of_live || !oop(bottom)->is_gc_marked())) { |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
301 |
// Nothing to compact. The space is either empty or all live object should be left in place. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
302 |
clear_empty_region(space); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
303 |
return; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
304 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
305 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
306 |
const intx scan_interval = PrefetchScanIntervalInBytes; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
307 |
const intx copy_interval = PrefetchCopyIntervalInBytes; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
308 |
|
40888
f09b053be364
8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents:
37495
diff
changeset
|
309 |
assert(bottom < end_of_live, "bottom: " PTR_FORMAT " should be < end_of_live: " PTR_FORMAT, p2i(bottom), p2i(end_of_live)); |
f09b053be364
8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents:
37495
diff
changeset
|
310 |
HeapWord* cur_obj = bottom; |
37495
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
311 |
if (space->_first_dead > cur_obj && !oop(cur_obj)->is_gc_marked()) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
312 |
// All object before _first_dead can be skipped. They should not be moved. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
313 |
// A pointer to the first live object is stored at the memory location for _first_dead. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
314 |
cur_obj = *(HeapWord**)(space->_first_dead); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
315 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
316 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
317 |
debug_only(HeapWord* prev_obj = NULL); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
318 |
while (cur_obj < end_of_live) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
319 |
if (!oop(cur_obj)->is_gc_marked()) { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
320 |
debug_only(prev_obj = cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
321 |
// The first word of the dead object contains a pointer to the next live object or end of space. |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
322 |
cur_obj = *(HeapWord**)cur_obj; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
323 |
assert(cur_obj > prev_obj, "we should be moving forward through memory"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
324 |
} else { |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
325 |
// prefetch beyond q |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
326 |
Prefetch::read(cur_obj, scan_interval); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
327 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
328 |
// size and destination |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
329 |
size_t size = space->obj_size(cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
330 |
HeapWord* compaction_top = (HeapWord*)oop(cur_obj)->forwardee(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
331 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
332 |
// prefetch beyond compaction_top |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
333 |
Prefetch::write(compaction_top, copy_interval); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
334 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
335 |
// copy object and reinit its mark |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
336 |
assert(cur_obj != compaction_top, "everything in this pass should be moving"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
337 |
Copy::aligned_conjoint_words(cur_obj, compaction_top, size); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
338 |
oop(compaction_top)->init_mark(); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
339 |
assert(oop(compaction_top)->klass() != NULL, "should have a class"); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
340 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
341 |
debug_only(prev_obj = cur_obj); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
342 |
cur_obj += size; |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
343 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
344 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
345 |
|
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
346 |
clear_empty_region(space); |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
347 |
} |
7ef1f23dfc9c
8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents:
37238
diff
changeset
|
348 |
|
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
349 |
size_t ContiguousSpace::scanned_block_size(const HeapWord* addr) const { |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
350 |
return oop(addr)->size(); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
351 |
} |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
30764
diff
changeset
|
352 |
|
30764 | 353 |
#endif // SHARE_VM_GC_SHARED_SPACE_INLINE_HPP |