hotspot/src/share/vm/gc_implementation/concurrentMarkSweep/cmsOopClosures.inline.hpp
author ysr
Thu, 21 Oct 2010 17:29:24 -0700
changeset 6984 c6718f921eb6
parent 5547 f4b087cbb361
child 7397 5b173b4ca846
permissions -rw-r--r--
6992998: CMSWaitDuration=0 causes hangs with +ExplicitGCInvokesConcurrent Summary: Closed a timing hole during which concurrent full gc requests can be missed. The hole can increase the latency of the response to a full gc request by up to the value of CMSWaitDuration. If CMSWaitDuration=0 is, as currently, interpreted as an unbounded wait, suitable in certain tuning scenarios, the application can potentially hang. Made two obscure tunables, including CMSWaitDuration, manageable. Reviewed-by: jcoomes, tonyp
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3696
diff changeset
     2
 * Copyright (c) 2007, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3696
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3696
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: 3696
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
489c9b5090e2 Initial load
duke
parents:
diff changeset
    25
// Trim our work_queue so its length is below max at return
489c9b5090e2 Initial load
duke
parents:
diff changeset
    26
inline void Par_MarkRefsIntoAndScanClosure::trim_queue(uint max) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    27
  while (_work_queue->size() > max) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    28
    oop newOop;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    29
    if (_work_queue->pop_local(newOop)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    30
      assert(newOop->is_oop(), "Expected an oop");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
      assert(_bit_map->isMarked((HeapWord*)newOop),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
             "only grey objects on this stack");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
      // iterate over the oops in this oop, marking and pushing
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
      // the ones in CMS heap (i.e. in _span).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
      newOop->oop_iterate(&_par_pushAndMarkClosure);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
3690
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    40
#ifndef PRODUCT
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    41
void KlassRememberingOopClosure::check_remember_klasses() const {
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    42
  assert(_should_remember_klasses == must_remember_klasses(),
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    43
    "Should remember klasses in this context.");
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    44
}
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    45
#endif
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    46
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    47
void KlassRememberingOopClosure::remember_klass(Klass* k) {
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    48
  if (!_revisit_stack->push(oop(k))) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
    fatal("Revisit stack overflow in PushOrMarkClosure");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  }
3690
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    51
  check_remember_klasses();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    54
inline void PushOrMarkClosure::remember_mdo(DataLayout* v) {
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    55
  // TBD
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    56
}
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    57
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    58
3690
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    59
void Par_KlassRememberingOopClosure::remember_klass(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  if (!_revisit_stack->par_push(oop(k))) {
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    61
    fatal("Revisit stack overflow in Par_KlassRememberingOopClosure");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  }
3690
dba50b88bd50 6798898: CMS: bugs related to class unloading
jmasa
parents: 1
diff changeset
    63
  check_remember_klasses();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
3696
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    66
inline void Par_PushOrMarkClosure::remember_mdo(DataLayout* v) {
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    67
  // TBD
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    68
}
9e5d9b5e1049 4957990: Perm heap bloat in JVM
ysr
parents: 3690
diff changeset
    69
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
inline void PushOrMarkClosure::do_yield_check() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  _parent->do_yield_check();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
inline void Par_PushOrMarkClosure::do_yield_check() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  _parent->do_yield_check();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
}