annotate src/share/vm/gc_implementation/concurrentMarkSweep/cmsLockVerifier.cpp @ 1145:e018e6884bd8

6631166: CMS: better heuristics when combatting fragmentation Summary: Autonomic per-worker free block cache sizing, tunable coalition policies, fixes to per-size block statistics, retuned gain and bandwidth of some feedback loop filters to allow quicker reactivity to abrupt changes in ambient demand, and other heuristics to reduce fragmentation of the CMS old gen. Also tightened some assertions, including those related to locking. Reviewed-by: jmasa
author ysr
date Wed, 23 Dec 2009 09:23:54 -0800
parents a61af66fc99e
children c18cbe5936b8
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
2 * Copyright 2002-2005 Sun Microsystems, Inc. All Rights Reserved.
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
a61af66fc99e Initial load
duke
parents:
diff changeset
19 * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
a61af66fc99e Initial load
duke
parents:
diff changeset
20 * CA 95054 USA or visit www.sun.com if you need additional information or
a61af66fc99e Initial load
duke
parents:
diff changeset
21 * have any questions.
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
a61af66fc99e Initial load
duke
parents:
diff changeset
25 # include "incls/_precompiled.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
26 # include "incls/_cmsLockVerifier.cpp.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
27
a61af66fc99e Initial load
duke
parents:
diff changeset
28 ///////////// Locking verification specific to CMS //////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
29 // Much like "assert_lock_strong()", except that it relaxes the
a61af66fc99e Initial load
duke
parents:
diff changeset
30 // assertion somewhat for the parallel GC case, where VM thread
a61af66fc99e Initial load
duke
parents:
diff changeset
31 // or the CMS thread might hold the lock on behalf of the parallel
a61af66fc99e Initial load
duke
parents:
diff changeset
32 // threads. The second argument is in support of an extra locking
a61af66fc99e Initial load
duke
parents:
diff changeset
33 // check for CFL spaces' free list locks.
a61af66fc99e Initial load
duke
parents:
diff changeset
34 #ifndef PRODUCT
1145
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
35 void CMSLockVerifier::assert_locked(const Mutex* lock,
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
36 const Mutex* p_lock1,
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
37 const Mutex* p_lock2) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
38 if (!Universe::is_fully_initialized()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
39 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
40 }
a61af66fc99e Initial load
duke
parents:
diff changeset
41
a61af66fc99e Initial load
duke
parents:
diff changeset
42 Thread* myThread = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 if (lock == NULL) { // a "lock-free" structure, e.g. MUT, protected by CMS token
1145
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
45 assert(p_lock1 == NULL && p_lock2 == NULL, "Unexpected caller error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
46 if (myThread->is_ConcurrentGC_thread()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
47 // This test might have to change in the future, if there can be
a61af66fc99e Initial load
duke
parents:
diff changeset
48 // multiple peer CMS threads. But for now, if we're testing the CMS
a61af66fc99e Initial load
duke
parents:
diff changeset
49 assert(myThread == ConcurrentMarkSweepThread::cmst(),
a61af66fc99e Initial load
duke
parents:
diff changeset
50 "In CMS, CMS thread is the only Conc GC thread.");
a61af66fc99e Initial load
duke
parents:
diff changeset
51 assert(ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
52 "CMS thread should have CMS token");
a61af66fc99e Initial load
duke
parents:
diff changeset
53 } else if (myThread->is_VM_thread()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
54 assert(ConcurrentMarkSweepThread::vm_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
55 "VM thread should have CMS token");
a61af66fc99e Initial load
duke
parents:
diff changeset
56 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
57 // Token should be held on our behalf by one of the other
a61af66fc99e Initial load
duke
parents:
diff changeset
58 // of CMS or VM thread; not enough easily testable
a61af66fc99e Initial load
duke
parents:
diff changeset
59 // state info to test which here.
a61af66fc99e Initial load
duke
parents:
diff changeset
60 assert(myThread->is_GC_task_thread(), "Unexpected thread type");
a61af66fc99e Initial load
duke
parents:
diff changeset
61 }
a61af66fc99e Initial load
duke
parents:
diff changeset
62 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
63 }
a61af66fc99e Initial load
duke
parents:
diff changeset
64
1145
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
65 if (myThread->is_VM_thread()
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
66 || myThread->is_ConcurrentGC_thread()
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
67 || myThread->is_Java_thread()) {
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
68 // Make sure that we are holding the associated lock.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
69 assert_lock_strong(lock);
1145
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
70 // The checking of p_lock is a spl case for CFLS' free list
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
71 // locks: we make sure that none of the parallel GC work gang
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
72 // threads are holding "sub-locks" of freeListLock(). We check only
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
73 // the parDictionaryAllocLock because the others are too numerous.
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
74 // This spl case code is somewhat ugly and any improvements
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
75 // are welcome.
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
76 assert(p_lock1 == NULL || !p_lock1->is_locked() || p_lock1->owned_by_self(),
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
77 "Possible race between this and parallel GC threads");
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
78 assert(p_lock2 == NULL || !p_lock2->is_locked() || p_lock2->owned_by_self(),
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
79 "Possible race between this and parallel GC threads");
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
80 } else if (myThread->is_GC_task_thread()) {
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
81 // Make sure that the VM or CMS thread holds lock on our behalf
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
82 // XXX If there were a concept of a gang_master for a (set of)
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
83 // gang_workers, we could have used the identity of that thread
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
84 // for checking ownership here; for now we just disjunct.
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
85 assert(lock->owner() == VMThread::vm_thread() ||
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
86 lock->owner() == ConcurrentMarkSweepThread::cmst(),
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
87 "Should be locked by VM thread or CMS thread on my behalf");
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
88 if (p_lock1 != NULL) {
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
89 assert_lock_strong(p_lock1);
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
90 }
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
91 if (p_lock2 != NULL) {
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
92 assert_lock_strong(p_lock2);
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
93 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
94 } else {
1145
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
95 // Make sure we didn't miss some other thread type calling into here;
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
96 // perhaps as a result of future VM evolution.
e018e6884bd8 6631166: CMS: better heuristics when combatting fragmentation
ysr
parents: 0
diff changeset
97 ShouldNotReachHere();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
98 }
a61af66fc99e Initial load
duke
parents:
diff changeset
99 }
a61af66fc99e Initial load
duke
parents:
diff changeset
100 #endif