src/hotspot/share/gc/parallel/psVMOperations.cpp
author kbarrett
Wed, 31 Jul 2019 14:28:51 -0400
changeset 57607 8d3886985964
parent 52876 2d17750d41e7
permissions -rw-r--r--
8048556: Unnecessary GCLocker-initiated young GCs Summary: Fixed recognition of unnecessary GCLocker collections. Reviewed-by: pliden, tschatzl
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
52876
2d17750d41e7 8214791: Consistently name gc files containing VM operations
tschatzl
parents: 49982
diff changeset
     2
 * Copyright (c) 2007, 2018, 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
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30273
diff changeset
    26
#include "gc/parallel/parallelScavengeHeap.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30273
diff changeset
    27
#include "gc/parallel/psScavenge.hpp"
52876
2d17750d41e7 8214791: Consistently name gc files containing VM operations
tschatzl
parents: 49982
diff changeset
    28
#include "gc/parallel/psVMOperations.hpp"
49594
898ef81cbc0e 8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents: 47216
diff changeset
    29
#include "gc/shared/gcLocker.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "utilities/dtrace.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
// The following methods are used by the parallel scavenge collector
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    33
VM_ParallelGCFailedAllocation::VM_ParallelGCFailedAllocation(size_t word_size,
29078
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 26183
diff changeset
    34
                                                             uint gc_count) :
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    35
    VM_CollectForAllocation(word_size, gc_count, GCCause::_allocation_failure) {
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    36
  assert(word_size != 0, "An allocation should always be requested with this operation.");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
void VM_ParallelGCFailedAllocation::doit() {
7896
08aadd7aa3ee 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
kamg
parents: 7397
diff changeset
    40
  SvcGCMarker sgcm(SvcGCMarker::MINOR);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 29079
diff changeset
    42
  ParallelScavengeHeap* heap = ParallelScavengeHeap::heap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  GCCauseSetter gccs(heap, _gc_cause);
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    45
  _result = heap->failed_mem_allocate(_word_size);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
35492
c8c0273e6b91 8146690: Make all classes in GC follow the naming convention.
david
parents: 30764
diff changeset
    47
  if (_result == NULL && GCLocker::is_active_and_needs_gc()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
    set_gc_locked();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
57607
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    52
static bool is_cause_full(GCCause::Cause cause) {
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    53
  return (cause != GCCause::_gc_locker) && (cause != GCCause::_wb_young_gc)
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    54
         DEBUG_ONLY(&& (cause != GCCause::_scavenge_alot));
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    55
}
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    56
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
// Only used for System.gc() calls
29078
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 26183
diff changeset
    58
VM_ParallelGCSystemGC::VM_ParallelGCSystemGC(uint gc_count,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 26183
diff changeset
    59
                                             uint full_gc_count,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
                                             GCCause::Cause gc_cause) :
57607
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    61
  VM_GC_Operation(gc_count, gc_cause, full_gc_count, is_cause_full(gc_cause))
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
void VM_ParallelGCSystemGC::doit() {
7896
08aadd7aa3ee 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
kamg
parents: 7397
diff changeset
    66
  SvcGCMarker sgcm(SvcGCMarker::FULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 29079
diff changeset
    68
  ParallelScavengeHeap* heap = ParallelScavengeHeap::heap();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  GCCauseSetter gccs(heap, _gc_cause);
57607
8d3886985964 8048556: Unnecessary GCLocker-initiated young GCs
kbarrett
parents: 52876
diff changeset
    71
  if (!_full) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
    // If (and only if) the scavenge fails, this will invoke a full gc.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
    heap->invoke_scavenge();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 9997
diff changeset
    75
    heap->do_full_collection(false);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
}