hotspot/src/share/vm/gc_implementation/concurrentMarkSweep/cmsPermGen.cpp
author ysr
Thu, 21 Oct 2010 17:29:24 -0700
changeset 6984 c6718f921eb6
parent 6765 f8d7d46aa4ff
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
/*
6765
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
     2
 * Copyright (c) 2007, 2010, 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: 386
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 386
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: 386
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
#include "incls/_precompiled.incl"
489c9b5090e2 Initial load
duke
parents:
diff changeset
    26
#include "incls/_cmsPermGen.cpp.incl"
489c9b5090e2 Initial load
duke
parents:
diff changeset
    27
489c9b5090e2 Initial load
duke
parents:
diff changeset
    28
CMSPermGen::CMSPermGen(ReservedSpace rs, size_t initial_byte_size,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    29
             CardTableRS* ct,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    30
             FreeBlockDictionary::DictionaryChoice dictionaryChoice) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
  CMSPermGenGen* g =
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
    new CMSPermGenGen(rs, initial_byte_size, -1, ct);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
  if (g == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
    vm_exit_during_initialization("Could not allocate a CompactingPermGen");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
  g->initialize_performance_counters();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
  _gen = g;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
HeapWord* CMSPermGen::mem_allocate(size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  Mutex* lock = _gen->freelistLock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  bool lock_owned = lock->owned_by_self();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  if (lock_owned) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
    MutexUnlocker mul(lock);
386
7f121b1192f2 6539517: CR 6186200 should be extended to perm gen allocation to prevent spurious OOM's from perm gen
apetrusenko
parents: 1
diff changeset
    47
    return mem_allocate_in_gen(size, _gen);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
  } else {
386
7f121b1192f2 6539517: CR 6186200 should be extended to perm gen allocation to prevent spurious OOM's from perm gen
apetrusenko
parents: 1
diff changeset
    49
    return mem_allocate_in_gen(size, _gen);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
6765
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    53
HeapWord* CMSPermGen::request_expand_and_allocate(Generation* gen,
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    54
                                                  size_t size,
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    55
                                                  GCCause::Cause prev_cause /* ignored */) {
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    56
  HeapWord* obj = gen->expand_and_allocate(size, false);
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    57
  if (gen->capacity() >= _capacity_expansion_limit) {
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    58
    set_capacity_expansion_limit(gen->capacity() + MaxPermHeapExpansion);
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    59
    assert(((ConcurrentMarkSweepGeneration*)gen)->should_concurrent_collect(),
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    60
           "Should kick off a collection if one not in progress");
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    61
  }
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    62
  return obj;
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    63
}
f8d7d46aa4ff 6794422: Perm gen expansion policy for concurrent collectors
ysr
parents: 5547
diff changeset
    64
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
void CMSPermGen::compute_new_size() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  _gen->compute_new_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
void CMSPermGenGen::initialize_performance_counters() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  const char* gen_name = "perm";
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  // Generation Counters - generation 2, 1 subspace
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  _gen_counters = new GenerationCounters(gen_name, 2, 1, &_virtual_space);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  _gc_counters = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  _space_counters = new GSpaceCounters(gen_name, 0,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
                                       _virtual_space.reserved_size(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
                                       this, _gen_counters);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
}