annotate src/share/vm/gc_implementation/g1/g1StringDedupThread.cpp @ 17976:8e20ef014b08

8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects Summary: Removed all write barriers during reference processing and added explicit write barriers when iterating through the discovered list. Reviewed-by: pliden, jmasa, tschatzl
author brutisso
date Wed, 11 Jun 2014 10:46:47 +0200
parents 1772223a25a2
children 581e70386ec9
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
17764
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
1 /*
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
2 * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
4 *
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
7 * published by the Free Software Foundation.
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
8 *
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
13 * accompanied this code).
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
14 *
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
18 *
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
21 * questions.
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
22 *
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
23 */
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
24
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
25 #include "precompiled.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
26 #include "gc_implementation/g1/g1Log.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
27 #include "gc_implementation/g1/g1StringDedup.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
28 #include "gc_implementation/g1/g1StringDedupTable.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
29 #include "gc_implementation/g1/g1StringDedupThread.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
30 #include "gc_implementation/g1/g1StringDedupQueue.hpp"
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
31
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
32 G1StringDedupThread* G1StringDedupThread::_thread = NULL;
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
33
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
34 G1StringDedupThread::G1StringDedupThread() :
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
35 ConcurrentGCThread() {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
36 set_name("String Deduplication Thread");
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
37 create_and_start();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
38 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
39
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
40 G1StringDedupThread::~G1StringDedupThread() {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
41 ShouldNotReachHere();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
42 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
43
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
44 void G1StringDedupThread::create() {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
45 assert(G1StringDedup::is_enabled(), "String deduplication not enabled");
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
46 assert(_thread == NULL, "One string deduplication thread allowed");
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
47 _thread = new G1StringDedupThread();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
48 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
49
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
50 G1StringDedupThread* G1StringDedupThread::thread() {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
51 assert(G1StringDedup::is_enabled(), "String deduplication not enabled");
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
52 assert(_thread != NULL, "String deduplication thread not created");
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
53 return _thread;
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
54 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
55
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
56 void G1StringDedupThread::print_on(outputStream* st) const {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
57 st->print("\"%s\" ", name());
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
58 Thread::print_on(st);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
59 st->cr();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
60 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
61
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
62 void G1StringDedupThread::run() {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
63 G1StringDedupStat total_stat;
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
64
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
65 initialize_in_thread();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
66 wait_for_universe_init();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
67
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
68 // Main loop
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
69 for (;;) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
70 G1StringDedupStat stat;
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
71
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
72 stat.mark_idle();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
73
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
74 // Wait for the queue to become non-empty
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
75 G1StringDedupQueue::wait();
17947
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
76 if (_should_terminate) {
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
77 break;
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
78 }
17764
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
79
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
80 // Include this thread in safepoints
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
81 stsJoin();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
82
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
83 stat.mark_exec();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
84
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
85 // Process the queue
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
86 for (;;) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
87 oop java_string = G1StringDedupQueue::pop();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
88 if (java_string == NULL) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
89 break;
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
90 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
91
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
92 G1StringDedupTable::deduplicate(java_string, stat);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
93
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
94 // Safepoint this thread if needed
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
95 if (stsShouldYield()) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
96 stat.mark_block();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
97 stsYield(NULL);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
98 stat.mark_unblock();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
99 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
100 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
101
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
102 G1StringDedupTable::trim_entry_cache();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
103
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
104 stat.mark_done();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
105
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
106 // Print statistics
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
107 total_stat.add(stat);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
108 print(gclog_or_tty, stat, total_stat);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
109
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
110 // Exclude this thread from safepoints
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
111 stsLeave();
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
112 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
113
17947
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
114 terminate();
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
115 }
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
116
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
117 void G1StringDedupThread::stop() {
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
118 {
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
119 MonitorLockerEx ml(Terminator_lock);
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
120 _thread->_should_terminate = true;
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
121 }
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
122
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
123 G1StringDedupQueue::cancel_wait();
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
124
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
125 {
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
126 MonitorLockerEx ml(Terminator_lock);
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
127 while (!_thread->_has_terminated) {
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
128 ml.wait();
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
129 }
1772223a25a2 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 17764
diff changeset
130 }
17764
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
131 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
132
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
133 void G1StringDedupThread::print(outputStream* st, const G1StringDedupStat& last_stat, const G1StringDedupStat& total_stat) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
134 if (G1Log::fine() || PrintStringDeduplicationStatistics) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
135 G1StringDedupStat::print_summary(st, last_stat, total_stat);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
136 if (PrintStringDeduplicationStatistics) {
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
137 G1StringDedupStat::print_statistics(st, last_stat, false);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
138 G1StringDedupStat::print_statistics(st, total_stat, true);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
139 G1StringDedupTable::print_statistics(st);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
140 G1StringDedupQueue::print_statistics(st);
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
141 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
142 }
595c0f60d50d 8029075: String deduplication in G1
pliden
parents:
diff changeset
143 }