src/hotspot/share/gc/g1/g1HeapVerifier.cpp
author tschatzl
Tue, 14 May 2019 15:36:26 +0200
changeset 54843 25c329958c70
parent 54786 ebf733a324d4
child 54923 23837d614c17
permissions -rw-r--r--
8200545: Improve filter for enqueued deferred cards Reviewed-by: kbarrett, sangheki
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     1
/*
53536
482109fae02b 8213229: Investigate treating StringTable as weak in young collections
tschatzl
parents: 53121
diff changeset
     2
 * Copyright (c) 2016, 2019, Oracle and/or its affiliates. All rights reserved.
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     4
 *
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     7
 * published by the Free Software Foundation.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     8
 *
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    13
 * accompanied this code).
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    14
 *
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    18
 *
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    21
 * questions.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    22
 *
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    23
 */
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    24
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    25
#include "precompiled.hpp"
52673
61b3b58a1d1d 8214181: safepoint header cleanup
rehn
parents: 52200
diff changeset
    26
#include "code/nmethod.hpp"
46285
5b673a9fa682 8171235: Move archive object code from G1MarkSweep into G1ArchiveAllocator
sjohanss
parents: 39698
diff changeset
    27
#include "gc/g1/g1Allocator.inline.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    28
#include "gc/g1/g1CollectedHeap.inline.hpp"
49659
0ed1370f52bb 8200371: In g1, rename ConcurrentMarkThread to G1ConcurrentMarkThread
lkorinth
parents: 49643
diff changeset
    29
#include "gc/g1/g1ConcurrentMarkThread.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    30
#include "gc/g1/g1HeapVerifier.hpp"
37985
539c597ee0fa 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents: 37242
diff changeset
    31
#include "gc/g1/g1Policy.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    32
#include "gc/g1/g1RemSet.hpp"
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    33
#include "gc/g1/g1RootProcessor.hpp"
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    34
#include "gc/g1/heapRegion.inline.hpp"
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    35
#include "gc/g1/heapRegionRemSet.hpp"
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    36
#include "gc/g1/g1StringDedup.hpp"
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
    37
#include "logging/log.hpp"
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
    38
#include "logging/logStream.hpp"
50752
9d62da00bf15 8204540: Automatic oop closure devirtualization
stefank
parents: 50565
diff changeset
    39
#include "memory/iterator.inline.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    40
#include "memory/resourceArea.hpp"
54786
ebf733a324d4 8223624: Cleanup includes of universe.hpp
stefank
parents: 54465
diff changeset
    41
#include "memory/universe.hpp"
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
    42
#include "oops/access.inline.hpp"
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
    43
#include "oops/compressedOops.inline.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    44
#include "oops/oop.inline.hpp"
49361
1956d0ec092a 8199319: Remove handles.inline.hpp include from reflectionUtils.hpp
stefank
parents: 49164
diff changeset
    45
#include "runtime/handles.inline.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    46
49602
db050c11c3b5 8199850: Move parsing of VerifyGCType to G1
pliden
parents: 49592
diff changeset
    47
int G1HeapVerifier::_enabled_verification_types = G1HeapVerifier::G1VerifyAll;
db050c11c3b5 8199850: Move parsing of VerifyGCType to G1
pliden
parents: 49592
diff changeset
    48
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    49
class VerifyRootsClosure: public OopClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    50
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    51
  G1CollectedHeap* _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    52
  VerifyOption     _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    53
  bool             _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    54
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    55
  // _vo == UsePrevMarking -> use "prev" marking information,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    56
  // _vo == UseNextMarking -> use "next" marking information,
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
    57
  // _vo == UseFullMarking -> use "next" marking bitmap but no TAMS
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    58
  VerifyRootsClosure(VerifyOption vo) :
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    59
    _g1h(G1CollectedHeap::heap()),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    60
    _vo(vo),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    61
    _failures(false) { }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    62
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    63
  bool failures() { return _failures; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    64
50752
9d62da00bf15 8204540: Automatic oop closure devirtualization
stefank
parents: 50565
diff changeset
    65
  template <class T> void do_oop_work(T* p) {
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
    66
    T heap_oop = RawAccess<>::oop_load(p);
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
    67
    if (!CompressedOops::is_null(heap_oop)) {
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
    68
      oop obj = CompressedOops::decode_not_null(heap_oop);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    69
      if (_g1h->is_obj_dead_cond(obj, _vo)) {
37242
91e5f98fff6f 8152632: Rename LogHandle(...) to Log(...)
stefank
parents: 36090
diff changeset
    70
        Log(gc, verify) log;
47088
03cfedd40b53 8185624: G1HeapVerifier's VerifyRootsClosure prints important information on info log level
dbuck
parents: 46810
diff changeset
    71
        log.error("Root location " PTR_FORMAT " points to dead obj " PTR_FORMAT, p2i(p), p2i(obj));
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    72
        ResourceMark rm;
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
    73
        LogStream ls(log.error());
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
    74
        obj->print_on(&ls);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    75
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    76
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    77
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    78
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    79
50752
9d62da00bf15 8204540: Automatic oop closure devirtualization
stefank
parents: 50565
diff changeset
    80
  void do_oop(oop* p)       { do_oop_work(p); }
9d62da00bf15 8204540: Automatic oop closure devirtualization
stefank
parents: 50565
diff changeset
    81
  void do_oop(narrowOop* p) { do_oop_work(p); }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    82
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    83
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    84
class G1VerifyCodeRootOopClosure: public OopClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    85
  G1CollectedHeap* _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    86
  OopClosure* _root_cl;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    87
  nmethod* _nm;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    88
  VerifyOption _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    89
  bool _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    90
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    91
  template <class T> void do_oop_work(T* p) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    92
    // First verify that this root is live
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    93
    _root_cl->do_oop(p);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    94
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    95
    if (!G1VerifyHeapRegionCodeRoots) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    96
      // We're not verifying the code roots attached to heap region.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    97
      return;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    98
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
    99
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   100
    // Don't check the code roots during marking verification in a full GC
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   101
    if (_vo == VerifyOption_G1UseFullMarking) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   102
      return;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   103
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   104
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   105
    // Now verify that the current nmethod (which contains p) is
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   106
    // in the code root list of the heap region containing the
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   107
    // object referenced by p.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   108
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
   109
    T heap_oop = RawAccess<>::oop_load(p);
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
   110
    if (!CompressedOops::is_null(heap_oop)) {
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
   111
      oop obj = CompressedOops::decode_not_null(heap_oop);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   112
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   113
      // Now fetch the region containing the object
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   114
      HeapRegion* hr = _g1h->heap_region_containing(obj);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   115
      HeapRegionRemSet* hrrs = hr->rem_set();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   116
      // Verify that the strong code root list for this region
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   117
      // contains the nmethod
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   118
      if (!hrrs->strong_code_roots_list_contains(_nm)) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   119
        log_error(gc, verify)("Code root location " PTR_FORMAT " "
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   120
                              "from nmethod " PTR_FORMAT " not in strong "
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   121
                              "code roots for region [" PTR_FORMAT "," PTR_FORMAT ")",
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   122
                              p2i(p), p2i(_nm), p2i(hr->bottom()), p2i(hr->end()));
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   123
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   124
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   125
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   126
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   127
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   128
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   129
  G1VerifyCodeRootOopClosure(G1CollectedHeap* g1h, OopClosure* root_cl, VerifyOption vo):
51332
c25572739e7c 8208669: GC changes to allow enabling -Wreorder
tschatzl
parents: 50800
diff changeset
   130
    _g1h(g1h), _root_cl(root_cl), _nm(NULL), _vo(vo), _failures(false) {}
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   131
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   132
  void do_oop(oop* p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   133
  void do_oop(narrowOop* p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   134
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   135
  void set_nmethod(nmethod* nm) { _nm = nm; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   136
  bool failures() { return _failures; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   137
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   138
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   139
class G1VerifyCodeRootBlobClosure: public CodeBlobClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   140
  G1VerifyCodeRootOopClosure* _oop_cl;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   141
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   142
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   143
  G1VerifyCodeRootBlobClosure(G1VerifyCodeRootOopClosure* oop_cl):
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   144
    _oop_cl(oop_cl) {}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   145
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   146
  void do_code_blob(CodeBlob* cb) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   147
    nmethod* nm = cb->as_nmethod_or_null();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   148
    if (nm != NULL) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   149
      _oop_cl->set_nmethod(nm);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   150
      nm->oops_do(_oop_cl);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   151
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   152
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   153
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   154
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   155
class YoungRefCounterClosure : public OopClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   156
  G1CollectedHeap* _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   157
  int              _count;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   158
 public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   159
  YoungRefCounterClosure(G1CollectedHeap* g1h) : _g1h(g1h), _count(0) {}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   160
  void do_oop(oop* p)       { if (_g1h->is_in_young(*p)) { _count++; } }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   161
  void do_oop(narrowOop* p) { ShouldNotReachHere(); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   162
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   163
  int count() { return _count; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   164
  void reset_count() { _count = 0; };
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   165
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   166
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
   167
class VerifyCLDClosure: public CLDClosure {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   168
  YoungRefCounterClosure _young_ref_counter_closure;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   169
  OopClosure *_oop_closure;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   170
 public:
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
   171
  VerifyCLDClosure(G1CollectedHeap* g1h, OopClosure* cl) : _young_ref_counter_closure(g1h), _oop_closure(cl) {}
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
   172
  void do_cld(ClassLoaderData* cld) {
52141
de6dc206a92b 8210330: Make CLD claiming allow multiple claim bits
eosterlund
parents: 52062
diff changeset
   173
    cld->oops_do(_oop_closure, ClassLoaderData::_claim_none);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   174
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   175
    _young_ref_counter_closure.reset_count();
52141
de6dc206a92b 8210330: Make CLD claiming allow multiple claim bits
eosterlund
parents: 52062
diff changeset
   176
    cld->oops_do(&_young_ref_counter_closure, ClassLoaderData::_claim_none);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   177
    if (_young_ref_counter_closure.count() > 0) {
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
   178
      guarantee(cld->has_modified_oops(), "CLD " PTR_FORMAT ", has young %d refs but is not dirty.", p2i(cld), _young_ref_counter_closure.count());
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   179
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   180
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   181
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   182
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   183
class VerifyLivenessOopClosure: public BasicOopIterateClosure {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   184
  G1CollectedHeap* _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   185
  VerifyOption _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   186
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   187
  VerifyLivenessOopClosure(G1CollectedHeap* g1h, VerifyOption vo):
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   188
    _g1h(g1h), _vo(vo)
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   189
  { }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   190
  void do_oop(narrowOop *p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   191
  void do_oop(      oop *p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   192
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   193
  template <class T> void do_oop_work(T *p) {
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
   194
    oop obj = RawAccess<>::oop_load(p);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   195
    guarantee(obj == NULL || !_g1h->is_obj_dead_cond(obj, _vo),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   196
              "Dead object referenced by a not dead object");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   197
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   198
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   199
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   200
class VerifyObjsInRegionClosure: public ObjectClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   201
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   202
  G1CollectedHeap* _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   203
  size_t _live_bytes;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   204
  HeapRegion *_hr;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   205
  VerifyOption _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   206
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   207
  // _vo == UsePrevMarking -> use "prev" marking information,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   208
  // _vo == UseNextMarking -> use "next" marking information,
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   209
  // _vo == UseFullMarking -> use "next" marking bitmap but no TAMS.
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   210
  VerifyObjsInRegionClosure(HeapRegion *hr, VerifyOption vo)
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   211
    : _live_bytes(0), _hr(hr), _vo(vo) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   212
    _g1h = G1CollectedHeap::heap();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   213
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   214
  void do_object(oop o) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   215
    VerifyLivenessOopClosure isLive(_g1h, _vo);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   216
    assert(o != NULL, "Huh?");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   217
    if (!_g1h->is_obj_dead_cond(o, _vo)) {
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   218
      // If the object is alive according to the full gc mark,
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   219
      // then verify that the marking information agrees.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   220
      // Note we can't verify the contra-positive of the
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   221
      // above: if the object is dead (according to the mark
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   222
      // word), it may not be marked, or may have been marked
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   223
      // but has since became dead, or may have been allocated
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   224
      // since the last marking.
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   225
      if (_vo == VerifyOption_G1UseFullMarking) {
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   226
        guarantee(!_g1h->is_obj_dead(o), "Full GC marking and concurrent mark mismatch");
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   227
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   228
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   229
      o->oop_iterate(&isLive);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   230
      if (!_hr->obj_allocated_since_prev_marking(o)) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   231
        size_t obj_size = o->size();    // Make sure we don't overflow
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   232
        _live_bytes += (obj_size * HeapWordSize);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   233
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   234
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   235
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   236
  size_t live_bytes() { return _live_bytes; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   237
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   238
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   239
class VerifyArchiveOopClosure: public BasicOopIterateClosure {
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   240
  HeapRegion* _hr;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   241
public:
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   242
  VerifyArchiveOopClosure(HeapRegion *hr)
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   243
    : _hr(hr) { }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   244
  void do_oop(narrowOop *p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   245
  void do_oop(      oop *p) { do_oop_work(p); }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   246
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   247
  template <class T> void do_oop_work(T *p) {
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49361
diff changeset
   248
    oop obj = RawAccess<>::oop_load(p);
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   249
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   250
    if (_hr->is_open_archive()) {
52062
8dbf1a13af49 8206009: Move CDS java heap object archiving code to heapShared.hpp and heapShared.cpp
jiangli
parents: 51494
diff changeset
   251
      guarantee(obj == NULL || G1ArchiveAllocator::is_archived_object(obj),
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   252
                "Archive object at " PTR_FORMAT " references a non-archive object at " PTR_FORMAT,
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   253
                p2i(p), p2i(obj));
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   254
    } else {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   255
      assert(_hr->is_closed_archive(), "should be closed archive region");
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   256
      guarantee(obj == NULL || G1ArchiveAllocator::is_closed_archive_object(obj),
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   257
                "Archive object at " PTR_FORMAT " references a non-archive object at " PTR_FORMAT,
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   258
                p2i(p), p2i(obj));
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   259
    }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   260
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   261
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   262
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   263
class VerifyObjectInArchiveRegionClosure: public ObjectClosure {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   264
  HeapRegion* _hr;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   265
public:
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   266
  VerifyObjectInArchiveRegionClosure(HeapRegion *hr, bool verbose)
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   267
    : _hr(hr) { }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   268
  // Verify that all object pointers are to archive regions.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   269
  void do_object(oop o) {
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   270
    VerifyArchiveOopClosure checkOop(_hr);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   271
    assert(o != NULL, "Should not be here for NULL oops");
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   272
    o->oop_iterate(&checkOop);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   273
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   274
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   275
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   276
// Should be only used at CDS dump time
52811
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   277
class VerifyReadyForArchivingRegionClosure : public HeapRegionClosure {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   278
  bool _seen_free;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   279
  bool _has_holes;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   280
  bool _has_unexpected_holes;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   281
  bool _has_humongous;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   282
public:
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   283
  bool has_holes() {return _has_holes;}
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   284
  bool has_unexpected_holes() {return _has_unexpected_holes;}
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   285
  bool has_humongous() {return _has_humongous;}
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   286
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   287
  VerifyReadyForArchivingRegionClosure() : HeapRegionClosure() {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   288
    _seen_free = false;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   289
    _has_holes = false;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   290
    _has_unexpected_holes = false;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   291
    _has_humongous = false;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   292
  }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   293
  virtual bool do_heap_region(HeapRegion* hr) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   294
    const char* hole = "";
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   295
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   296
    if (hr->is_free()) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   297
      _seen_free = true;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   298
    } else {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   299
      if (_seen_free) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   300
        _has_holes = true;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   301
        if (hr->is_humongous()) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   302
          hole = " hole";
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   303
        } else {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   304
          _has_unexpected_holes = true;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   305
          hole = " hole **** unexpected ****";
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   306
        }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   307
      }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   308
    }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   309
    if (hr->is_humongous()) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   310
      _has_humongous = true;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   311
    }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   312
    log_info(gc, region, cds)("HeapRegion " INTPTR_FORMAT " %s%s", p2i(hr->bottom()), hr->get_type_str(), hole);
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   313
    return false;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   314
  }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   315
};
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   316
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   317
// We want all used regions to be moved to the bottom-end of the heap, so we have
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   318
// a contiguous range of free regions at the top end of the heap. This way, we can
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   319
// avoid fragmentation while allocating the archive regions.
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   320
//
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   321
// Before calling this, a full GC should have been executed with a single worker thread,
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   322
// so that no old regions would be moved to the middle of the heap.
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   323
void G1HeapVerifier::verify_ready_for_archiving() {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   324
  VerifyReadyForArchivingRegionClosure cl;
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   325
  G1CollectedHeap::heap()->heap_region_iterate(&cl);
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   326
  if (cl.has_holes()) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   327
    log_warning(gc, verify)("All free regions should be at the top end of the heap, but"
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   328
                            " we found holes. This is probably caused by (unmovable) humongous"
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   329
                            " allocations, and may lead to fragmentation while"
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   330
                            " writing archive heap memory regions.");
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   331
  }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   332
  if (cl.has_humongous()) {
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   333
    log_warning(gc, verify)("(Unmovable) humongous regions have been found and"
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   334
                            " may lead to fragmentation while"
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   335
                            " writing archive heap memory regions.");
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   336
  }
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   337
  assert(!cl.has_unexpected_holes(), "all holes should have been caused by humongous regions");
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   338
}
ff04b71bf6f1 8214388: CDS dumping fails with java heap fragmentation
iklam
parents: 52673
diff changeset
   339
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   340
class VerifyArchivePointerRegionClosure: public HeapRegionClosure {
48969
jwilhelm
parents: 48890
diff changeset
   341
  virtual bool do_heap_region(HeapRegion* r) {
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   342
   if (r->is_archive()) {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   343
      VerifyObjectInArchiveRegionClosure verify_oop_pointers(r, false);
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   344
      r->object_iterate(&verify_oop_pointers);
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   345
    }
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   346
    return false;
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   347
  }
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   348
};
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   349
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   350
void G1HeapVerifier::verify_archive_regions() {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   351
  G1CollectedHeap*  g1h = G1CollectedHeap::heap();
53034
de99beff5c0e 8214946: G1: Initialize all class members on construction
lkorinth
parents: 52811
diff changeset
   352
  VerifyArchivePointerRegionClosure cl;
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   353
  g1h->heap_region_iterate(&cl);
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   354
}
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   355
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   356
class VerifyRegionClosure: public HeapRegionClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   357
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   358
  bool             _par;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   359
  VerifyOption     _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   360
  bool             _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   361
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   362
  // _vo == UsePrevMarking -> use "prev" marking information,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   363
  // _vo == UseNextMarking -> use "next" marking information,
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   364
  // _vo == UseFullMarking -> use "next" marking bitmap but no TAMS
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   365
  VerifyRegionClosure(bool par, VerifyOption vo)
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   366
    : _par(par),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   367
      _vo(vo),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   368
      _failures(false) {}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   369
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   370
  bool failures() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   371
    return _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   372
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   373
48969
jwilhelm
parents: 48890
diff changeset
   374
  bool do_heap_region(HeapRegion* r) {
54465
c4f16445675a 8218668: Clean up evacuation of optional collection set
tschatzl
parents: 53986
diff changeset
   375
    guarantee(!r->has_index_in_opt_cset(), "Region %u still has opt collection set index %u", r->hrm_index(), r->index_in_opt_cset());
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49602
diff changeset
   376
    guarantee(!r->is_young() || r->rem_set()->is_complete(), "Remembered set for Young region %u must be complete, is %s", r->hrm_index(), r->rem_set()->get_state_str());
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49602
diff changeset
   377
    // Humongous and old regions regions might be of any state, so can't check here.
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49602
diff changeset
   378
    guarantee(!r->is_free() || !r->rem_set()->is_tracked(), "Remembered set for free region %u must be untracked, is %s", r->hrm_index(), r->rem_set()->get_state_str());
50565
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   379
    // Verify that the continues humongous regions' remembered set state matches the
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   380
    // one from the starts humongous region.
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   381
    if (r->is_continues_humongous()) {
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   382
      if (r->rem_set()->get_state_str() != r->humongous_start_region()->rem_set()->get_state_str()) {
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   383
         log_error(gc, verify)("Remset states differ: Region %u (%s) remset %s with starts region %u (%s) remset %s",
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   384
                               r->hrm_index(),
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   385
                               r->get_short_type_str(),
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   386
                               r->rem_set()->get_state_str(),
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   387
                               r->humongous_start_region()->hrm_index(),
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   388
                               r->humongous_start_region()->get_short_type_str(),
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   389
                               r->humongous_start_region()->rem_set()->get_state_str());
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   390
         _failures = true;
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   391
      }
69e82329ad01 8204169: Humongous continues region remembered set states do not match the one from the corresponding humongous start region
tschatzl
parents: 49664
diff changeset
   392
    }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   393
    // For archive regions, verify there are no heap pointers to
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   394
    // non-pinned regions. For all others, verify liveness info.
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   395
    if (r->is_closed_archive()) {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   396
      VerifyObjectInArchiveRegionClosure verify_oop_pointers(r, false);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   397
      r->object_iterate(&verify_oop_pointers);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   398
      return true;
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   399
    } else if (r->is_open_archive()) {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   400
      VerifyObjsInRegionClosure verify_open_archive_oop(r, _vo);
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   401
      r->object_iterate(&verify_open_archive_oop);
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   402
      return true;
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46750
diff changeset
   403
    } else if (!r->is_continues_humongous()) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   404
      bool failures = false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   405
      r->verify(_vo, &failures);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   406
      if (failures) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   407
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   408
      } else if (!r->is_starts_humongous()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   409
        VerifyObjsInRegionClosure not_dead_yet_cl(r, _vo);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   410
        r->object_iterate(&not_dead_yet_cl);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   411
        if (_vo != VerifyOption_G1UseNextMarking) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   412
          if (r->max_live_bytes() < not_dead_yet_cl.live_bytes()) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   413
            log_error(gc, verify)("[" PTR_FORMAT "," PTR_FORMAT "] max_live_bytes " SIZE_FORMAT " < calculated " SIZE_FORMAT,
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   414
                                  p2i(r->bottom()), p2i(r->end()), r->max_live_bytes(), not_dead_yet_cl.live_bytes());
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   415
            _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   416
          }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   417
        } else {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   418
          // When vo == UseNextMarking we cannot currently do a sanity
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   419
          // check on the live bytes as the calculation has not been
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   420
          // finalized yet.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   421
        }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   422
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   423
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   424
    return false; // stop the region iteration if we hit a failure
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   425
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   426
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   427
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   428
// This is the task used for parallel verification of the heap regions
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   429
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   430
class G1ParVerifyTask: public AbstractGangTask {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   431
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   432
  G1CollectedHeap*  _g1h;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   433
  VerifyOption      _vo;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   434
  bool              _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   435
  HeapRegionClaimer _hrclaimer;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   436
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   437
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   438
  // _vo == UsePrevMarking -> use "prev" marking information,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   439
  // _vo == UseNextMarking -> use "next" marking information,
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   440
  // _vo == UseFullMarking -> use "next" marking bitmap but no TAMS
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   441
  G1ParVerifyTask(G1CollectedHeap* g1h, VerifyOption vo) :
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   442
      AbstractGangTask("Parallel verify task"),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   443
      _g1h(g1h),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   444
      _vo(vo),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   445
      _failures(false),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   446
      _hrclaimer(g1h->workers()->active_workers()) {}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   447
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   448
  bool failures() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   449
    return _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   450
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   451
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   452
  void work(uint worker_id) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   453
    HandleMark hm;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   454
    VerifyRegionClosure blk(true, _vo);
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47678
diff changeset
   455
    _g1h->heap_region_par_iterate_from_worker_offset(&blk, &_hrclaimer, worker_id);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   456
    if (blk.failures()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   457
      _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   458
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   459
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   460
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   461
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   462
void G1HeapVerifier::enable_verification_type(G1VerifyType type) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   463
  // First enable will clear _enabled_verification_types.
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   464
  if (_enabled_verification_types == G1VerifyAll) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   465
    _enabled_verification_types = type;
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   466
  } else {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   467
    _enabled_verification_types |= type;
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   468
  }
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   469
}
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   470
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   471
bool G1HeapVerifier::should_verify(G1VerifyType type) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   472
  return (_enabled_verification_types & type) == type;
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   473
}
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   474
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   475
void G1HeapVerifier::verify(VerifyOption vo) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   476
  if (!SafepointSynchronize::is_at_safepoint()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   477
    log_info(gc, verify)("Skipping verification. Not at safepoint.");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   478
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   479
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   480
  assert(Thread::current()->is_VM_thread(),
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   481
         "Expected to be executed serially by the VM thread at this point");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   482
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   483
  log_debug(gc, verify)("Roots");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   484
  VerifyRootsClosure rootsCl(vo);
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
   485
  VerifyCLDClosure cldCl(_g1h, &rootsCl);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   486
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   487
  // We apply the relevant closures to all the oops in the
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   488
  // system dictionary, class loader data graph, the string table
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   489
  // and the nmethods in the code cache.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   490
  G1VerifyCodeRootOopClosure codeRootsCl(_g1h, &rootsCl, vo);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   491
  G1VerifyCodeRootBlobClosure blobsCl(&codeRootsCl);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   492
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   493
  {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   494
    G1RootProcessor root_processor(_g1h, 1);
53536
482109fae02b 8213229: Investigate treating StringTable as weak in young collections
tschatzl
parents: 53121
diff changeset
   495
    root_processor.process_all_roots(&rootsCl, &cldCl, &blobsCl);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   496
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   497
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   498
  bool failures = rootsCl.failures() || codeRootsCl.failures();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   499
53985
0da5c17f5e43 8219747: Remove g1_ prefix to g1_remset and g1_policy members in G1CollectedHeap
tschatzl
parents: 53536
diff changeset
   500
  if (!_g1h->policy()->collector_state()->in_full_gc()) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   501
    // If we're verifying during a full GC then the region sets
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   502
    // will have been torn down at the start of the GC. Therefore
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   503
    // verifying the region sets will fail. So we only verify
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   504
    // the region sets when not in a full GC.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   505
    log_debug(gc, verify)("HeapRegionSets");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   506
    verify_region_sets();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   507
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   508
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   509
  log_debug(gc, verify)("HeapRegions");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   510
  if (GCParallelVerificationEnabled && ParallelGCThreads > 1) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   511
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   512
    G1ParVerifyTask task(_g1h, vo);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   513
    _g1h->workers()->run_task(&task);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   514
    if (task.failures()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   515
      failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   516
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   517
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   518
  } else {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   519
    VerifyRegionClosure blk(false, vo);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   520
    _g1h->heap_region_iterate(&blk);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   521
    if (blk.failures()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   522
      failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   523
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   524
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   525
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   526
  if (G1StringDedup::is_enabled()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   527
    log_debug(gc, verify)("StrDedup");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   528
    G1StringDedup::verify();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   529
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   530
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   531
  if (failures) {
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49602
diff changeset
   532
    log_error(gc, verify)("Heap after failed verification (kind %d):", vo);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   533
    // It helps to have the per-region information in the output to
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   534
    // help us track down what went wrong. This is why we call
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   535
    // print_extended_on() instead of print_on().
37242
91e5f98fff6f 8152632: Rename LogHandle(...) to Log(...)
stefank
parents: 36090
diff changeset
   536
    Log(gc, verify) log;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   537
    ResourceMark rm;
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
   538
    LogStream ls(log.error());
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46611
diff changeset
   539
    _g1h->print_extended_on(&ls);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   540
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   541
  guarantee(!failures, "there should not have been any failures");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   542
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   543
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   544
// Heap region set verification
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   545
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   546
class VerifyRegionListsClosure : public HeapRegionClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   547
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   548
  HeapRegionSet*   _old_set;
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   549
  HeapRegionSet*   _archive_set;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   550
  HeapRegionSet*   _humongous_set;
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   551
  HeapRegionManager* _hrm;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   552
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   553
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   554
  uint _old_count;
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   555
  uint _archive_count;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   556
  uint _humongous_count;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   557
  uint _free_count;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   558
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   559
  VerifyRegionListsClosure(HeapRegionSet* old_set,
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   560
                           HeapRegionSet* archive_set,
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   561
                           HeapRegionSet* humongous_set,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   562
                           HeapRegionManager* hrm) :
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   563
    _old_set(old_set), _archive_set(archive_set), _humongous_set(humongous_set), _hrm(hrm),
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   564
    _old_count(), _archive_count(), _humongous_count(), _free_count(){ }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   565
48969
jwilhelm
parents: 48890
diff changeset
   566
  bool do_heap_region(HeapRegion* hr) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   567
    if (hr->is_young()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   568
      // TODO
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   569
    } else if (hr->is_humongous()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   570
      assert(hr->containing_set() == _humongous_set, "Heap region %u is humongous but not in humongous set.", hr->hrm_index());
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   571
      _humongous_count++;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   572
    } else if (hr->is_empty()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   573
      assert(_hrm->is_free(hr), "Heap region %u is empty but not on the free list.", hr->hrm_index());
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   574
      _free_count++;
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   575
    } else if (hr->is_archive()) {
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   576
      assert(hr->containing_set() == _archive_set, "Heap region %u is archive but not in the archive set.", hr->hrm_index());
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   577
      _archive_count++;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   578
    } else if (hr->is_old()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   579
      assert(hr->containing_set() == _old_set, "Heap region %u is old but not in the old set.", hr->hrm_index());
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   580
      _old_count++;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   581
    } else {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   582
      // There are no other valid region types. Check for one invalid
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   583
      // one we can identify: pinned without old or humongous set.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   584
      assert(!hr->is_pinned(), "Heap region %u is pinned but not old (archive) or humongous.", hr->hrm_index());
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   585
      ShouldNotReachHere();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   586
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   587
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   588
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   589
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   590
  void verify_counts(HeapRegionSet* old_set, HeapRegionSet* archive_set, HeapRegionSet* humongous_set, HeapRegionManager* free_list) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   591
    guarantee(old_set->length() == _old_count, "Old set count mismatch. Expected %u, actual %u.", old_set->length(), _old_count);
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   592
    guarantee(archive_set->length() == _archive_count, "Archive set count mismatch. Expected %u, actual %u.", archive_set->length(), _archive_count);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   593
    guarantee(humongous_set->length() == _humongous_count, "Hum set count mismatch. Expected %u, actual %u.", humongous_set->length(), _humongous_count);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   594
    guarantee(free_list->num_free_regions() == _free_count, "Free list count mismatch. Expected %u, actual %u.", free_list->num_free_regions(), _free_count);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   595
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   596
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   597
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   598
void G1HeapVerifier::verify_region_sets() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   599
  assert_heap_locked_or_at_safepoint(true /* should_be_vm_thread */);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   600
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   601
  // First, check the explicit lists.
53116
bb03098c4dde 8211425: Allocation of old generation of java heap on alternate memory devices - G1 GC
sangheki
parents: 52811
diff changeset
   602
  _g1h->_hrm->verify();
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   603
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   604
  // Finally, make sure that the region accounting in the lists is
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   605
  // consistent with what we see in the heap.
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   606
53116
bb03098c4dde 8211425: Allocation of old generation of java heap on alternate memory devices - G1 GC
sangheki
parents: 52811
diff changeset
   607
  VerifyRegionListsClosure cl(&_g1h->_old_set, &_g1h->_archive_set, &_g1h->_humongous_set, _g1h->_hrm);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   608
  _g1h->heap_region_iterate(&cl);
53116
bb03098c4dde 8211425: Allocation of old generation of java heap on alternate memory devices - G1 GC
sangheki
parents: 52811
diff changeset
   609
  cl.verify_counts(&_g1h->_old_set, &_g1h->_archive_set, &_g1h->_humongous_set, _g1h->_hrm);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   610
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   611
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   612
void G1HeapVerifier::prepare_for_verify() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   613
  if (SafepointSynchronize::is_at_safepoint() || ! UseTLAB) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   614
    _g1h->ensure_parsability(false);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   615
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   616
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   617
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   618
double G1HeapVerifier::verify(G1VerifyType type, VerifyOption vo, const char* msg) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   619
  double verify_time_ms = 0.0;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   620
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   621
  if (should_verify(type) && _g1h->total_collections() >= VerifyGCStartAt) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   622
    double verify_start = os::elapsedTime();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   623
    HandleMark hm;  // Discard invalid handles created during verification
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   624
    prepare_for_verify();
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   625
    Universe::verify(vo, msg);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   626
    verify_time_ms = (os::elapsedTime() - verify_start) * 1000;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   627
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   628
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   629
  return verify_time_ms;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   630
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   631
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   632
void G1HeapVerifier::verify_before_gc(G1VerifyType type) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   633
  if (VerifyBeforeGC) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   634
    double verify_time_ms = verify(type, VerifyOption_G1UsePrevMarking, "Before GC");
53986
9ad20fd52b19 8219748: Add and use getter for the timing object in G1
tschatzl
parents: 53985
diff changeset
   635
    _g1h->phase_times()->record_verify_before_time_ms(verify_time_ms);
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   636
  }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   637
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   638
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   639
void G1HeapVerifier::verify_after_gc(G1VerifyType type) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   640
  if (VerifyAfterGC) {
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   641
    double verify_time_ms = verify(type, VerifyOption_G1UsePrevMarking, "After GC");
53986
9ad20fd52b19 8219748: Add and use getter for the timing object in G1
tschatzl
parents: 53985
diff changeset
   642
    _g1h->phase_times()->record_verify_after_time_ms(verify_time_ms);
48179
34fe70d22e9c 8191821: Finer granularity for GC verification
sjohanss
parents: 47885
diff changeset
   643
  }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   644
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   645
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   646
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   647
#ifndef PRODUCT
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   648
class G1VerifyCardTableCleanup: public HeapRegionClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   649
  G1HeapVerifier* _verifier;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   650
public:
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   651
  G1VerifyCardTableCleanup(G1HeapVerifier* verifier)
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   652
    : _verifier(verifier) { }
48969
jwilhelm
parents: 48890
diff changeset
   653
  virtual bool do_heap_region(HeapRegion* r) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   654
    if (r->is_survivor()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   655
      _verifier->verify_dirty_region(r);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   656
    } else {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   657
      _verifier->verify_not_dirty_region(r);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   658
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   659
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   660
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   661
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   662
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   663
void G1HeapVerifier::verify_card_table_cleanup() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   664
  if (G1VerifyCTCleanup || VerifyAfterGC) {
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   665
    G1VerifyCardTableCleanup cleanup_verifier(this);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   666
    _g1h->heap_region_iterate(&cleanup_verifier);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   667
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   668
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   669
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   670
void G1HeapVerifier::verify_not_dirty_region(HeapRegion* hr) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   671
  // All of the region should be clean.
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   672
  G1CardTable* ct = _g1h->card_table();
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   673
  MemRegion mr(hr->bottom(), hr->end());
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   674
  ct->verify_not_dirty_region(mr);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   675
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   676
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   677
void G1HeapVerifier::verify_dirty_region(HeapRegion* hr) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   678
  // We cannot guarantee that [bottom(),end()] is dirty.  Threads
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   679
  // dirty allocated blocks as they allocate them. The thread that
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   680
  // retires each region and replaces it with a new one will do a
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   681
  // maximal allocation to fill in [pre_dummy_top(),end()] but will
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   682
  // not dirty that area (one less thing to have to do while holding
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   683
  // a lock). So we can only verify that [bottom(),pre_dummy_top()]
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   684
  // is dirty.
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   685
  G1CardTable* ct = _g1h->card_table();
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   686
  MemRegion mr(hr->bottom(), hr->pre_dummy_top());
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   687
  if (hr->is_young()) {
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   688
    ct->verify_g1_young_region(mr);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   689
  } else {
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 48969
diff changeset
   690
    ct->verify_dirty_region(mr);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   691
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   692
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   693
39698
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   694
class G1VerifyDirtyYoungListClosure : public HeapRegionClosure {
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   695
private:
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   696
  G1HeapVerifier* _verifier;
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   697
public:
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   698
  G1VerifyDirtyYoungListClosure(G1HeapVerifier* verifier) : HeapRegionClosure(), _verifier(verifier) { }
48969
jwilhelm
parents: 48890
diff changeset
   699
  virtual bool do_heap_region(HeapRegion* r) {
39698
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   700
    _verifier->verify_dirty_region(r);
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   701
    return false;
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   702
  }
39698
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   703
};
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   704
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   705
void G1HeapVerifier::verify_dirty_young_regions() {
39698
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   706
  G1VerifyDirtyYoungListClosure cl(this);
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38186
diff changeset
   707
  _g1h->collection_set()->iterate(&cl);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   708
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   709
46750
bcab0128a16f 8184346: Clean up G1CMBitmap
tschatzl
parents: 46701
diff changeset
   710
bool G1HeapVerifier::verify_no_bits_over_tams(const char* bitmap_name, const G1CMBitMap* const bitmap,
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   711
                                               HeapWord* tams, HeapWord* end) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   712
  guarantee(tams <= end,
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   713
            "tams: " PTR_FORMAT " end: " PTR_FORMAT, p2i(tams), p2i(end));
46750
bcab0128a16f 8184346: Clean up G1CMBitmap
tschatzl
parents: 46701
diff changeset
   714
  HeapWord* result = bitmap->get_next_marked_addr(tams, end);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   715
  if (result < end) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   716
    log_error(gc, verify)("## wrong marked address on %s bitmap: " PTR_FORMAT, bitmap_name, p2i(result));
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   717
    log_error(gc, verify)("## %s tams: " PTR_FORMAT " end: " PTR_FORMAT, bitmap_name, p2i(tams), p2i(end));
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   718
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   719
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   720
  return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   721
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   722
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   723
bool G1HeapVerifier::verify_bitmaps(const char* caller, HeapRegion* hr) {
47678
c84eeb55c55e 8184667: Clean up G1ConcurrentMark files
tschatzl
parents: 47580
diff changeset
   724
  const G1CMBitMap* const prev_bitmap = _g1h->concurrent_mark()->prev_mark_bitmap();
c84eeb55c55e 8184667: Clean up G1ConcurrentMark files
tschatzl
parents: 47580
diff changeset
   725
  const G1CMBitMap* const next_bitmap = _g1h->concurrent_mark()->next_mark_bitmap();
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   726
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   727
  HeapWord* ptams  = hr->prev_top_at_mark_start();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   728
  HeapWord* ntams  = hr->next_top_at_mark_start();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   729
  HeapWord* end    = hr->end();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   730
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   731
  bool res_p = verify_no_bits_over_tams("prev", prev_bitmap, ptams, end);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   732
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   733
  bool res_n = true;
49664
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   734
  // We cannot verify the next bitmap while we are about to clear it.
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   735
  if (!_g1h->collector_state()->clearing_next_bitmap()) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   736
    res_n = verify_no_bits_over_tams("next", next_bitmap, ntams, end);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   737
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   738
  if (!res_p || !res_n) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   739
    log_error(gc, verify)("#### Bitmap verification failed for " HR_FORMAT, HR_FORMAT_PARAMS(hr));
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   740
    log_error(gc, verify)("#### Caller: %s", caller);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   741
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   742
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   743
  return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   744
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   745
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   746
void G1HeapVerifier::check_bitmaps(const char* caller, HeapRegion* hr) {
49664
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   747
  if (!G1VerifyBitmaps) {
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   748
    return;
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   749
  }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   750
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   751
  guarantee(verify_bitmaps(caller, hr), "bitmap verification");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   752
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   753
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   754
class G1VerifyBitmapClosure : public HeapRegionClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   755
private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   756
  const char* _caller;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   757
  G1HeapVerifier* _verifier;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   758
  bool _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   759
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   760
public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   761
  G1VerifyBitmapClosure(const char* caller, G1HeapVerifier* verifier) :
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   762
    _caller(caller), _verifier(verifier), _failures(false) { }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   763
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   764
  bool failures() { return _failures; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   765
48969
jwilhelm
parents: 48890
diff changeset
   766
  virtual bool do_heap_region(HeapRegion* hr) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   767
    bool result = _verifier->verify_bitmaps(_caller, hr);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   768
    if (!result) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   769
      _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   770
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   771
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   772
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   773
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   774
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   775
void G1HeapVerifier::check_bitmaps(const char* caller) {
49664
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   776
  if (!G1VerifyBitmaps) {
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   777
    return;
9a04cc89dde0 8200234: Cleanup Remark and Cleanup pause code
tschatzl
parents: 49659
diff changeset
   778
  }
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   779
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   780
  G1VerifyBitmapClosure cl(caller, this);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   781
  _g1h->heap_region_iterate(&cl);
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   782
  guarantee(!cl.failures(), "bitmap verification");
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   783
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   784
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   785
class G1CheckCSetFastTableClosure : public HeapRegionClosure {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   786
 private:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   787
  bool _failures;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   788
 public:
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   789
  G1CheckCSetFastTableClosure() : HeapRegionClosure(), _failures(false) { }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   790
48969
jwilhelm
parents: 48890
diff changeset
   791
  virtual bool do_heap_region(HeapRegion* hr) {
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   792
    uint i = hr->hrm_index();
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   793
    G1HeapRegionAttr region_attr = (G1HeapRegionAttr) G1CollectedHeap::heap()->_region_attr.get_by_index(i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   794
    if (hr->is_humongous()) {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   795
      if (hr->in_collection_set()) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35943
diff changeset
   796
        log_error(gc, verify)("## humongous region %u in CSet", i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   797
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   798
        return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   799
      }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   800
      if (region_attr.is_in_cset()) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   801
        log_error(gc, verify)("## inconsistent region attr type %s for humongous region %u", region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   802
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   803
        return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   804
      }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   805
      if (hr->is_continues_humongous() && region_attr.is_humongous()) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   806
        log_error(gc, verify)("## inconsistent region attr type %s for continues humongous region %u", region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   807
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   808
        return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   809
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   810
    } else {
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   811
      if (region_attr.is_humongous()) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   812
        log_error(gc, verify)("## inconsistent region attr type %s for non-humongous region %u", region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   813
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   814
        return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   815
      }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   816
      if (hr->in_collection_set() != region_attr.is_in_cset()) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   817
        log_error(gc, verify)("## in CSet %d / region attr type %s inconsistency for region %u",
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   818
                             hr->in_collection_set(), region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   819
        _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   820
        return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   821
      }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   822
      if (region_attr.is_in_cset()) {
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   823
        if (hr->is_archive()) {
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   824
          log_error(gc, verify)("## is_archive in collection set for region %u", i);
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   825
          _failures = true;
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   826
          return true;
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 51332
diff changeset
   827
        }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   828
        if (hr->is_young() != (region_attr.is_young())) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   829
          log_error(gc, verify)("## is_young %d / region attr type %s inconsistency for region %u",
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   830
                               hr->is_young(), region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   831
          _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   832
          return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   833
        }
54843
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   834
        if (hr->is_old() != (region_attr.is_old())) {
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   835
          log_error(gc, verify)("## is_old %d / region attr type %s inconsistency for region %u",
25c329958c70 8200545: Improve filter for enqueued deferred cards
tschatzl
parents: 54786
diff changeset
   836
                               hr->is_old(), region_attr.get_type_str(), i);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   837
          _failures = true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   838
          return true;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   839
        }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   840
      }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   841
    }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   842
    return false;
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   843
  }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   844
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   845
  bool failures() const { return _failures; }
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   846
};
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   847
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   848
bool G1HeapVerifier::check_cset_fast_test() {
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   849
  G1CheckCSetFastTableClosure cl;
53116
bb03098c4dde 8211425: Allocation of old generation of java heap on alternate memory devices - G1 GC
sangheki
parents: 52811
diff changeset
   850
  _g1h->_hrm->iterate(&cl);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   851
  return !cl.failures();
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   852
}
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents:
diff changeset
   853
#endif // PRODUCT