src/hotspot/share/gc/z/zVerify.cpp
author pliden
Tue, 06 Aug 2019 15:49:53 +0200
changeset 57656 9429ecaee2e0
parent 55603 3868dde58ebb
child 57658 0022b39ae5ae
permissions -rw-r--r--
8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong() Reviewed-by: stefank, eosterlund
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
55603
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     1
/*
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     2
 * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     4
 *
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     7
 * published by the Free Software Foundation.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     8
 *
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    13
 * accompanied this code).
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    14
 *
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    18
 *
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    21
 * questions.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    22
 */
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    23
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    24
#include "precompiled.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    25
#include "classfile/classLoaderData.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    26
#include "classfile/classLoaderDataGraph.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    27
#include "gc/z/zAddress.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    28
#include "gc/z/zHeap.inline.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    29
#include "gc/z/zOop.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    30
#include "gc/z/zResurrection.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    31
#include "gc/z/zRootsIterator.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    32
#include "gc/z/zStat.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    33
#include "gc/z/zVerify.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    34
#include "memory/allocation.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    35
#include "memory/iterator.inline.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    36
#include "oops/oop.inline.hpp"
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    37
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    38
#define BAD_OOP_REPORT(addr)                                                \
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    39
    "Bad oop " PTR_FORMAT " found at " PTR_FORMAT ", expected " PTR_FORMAT, \
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    40
    addr, p2i(p), ZAddress::good(addr)
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    41
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    42
class ZVerifyRootsClosure : public ZRootsIteratorClosure {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    43
public:
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    44
  virtual void do_oop(oop* p) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    45
    uintptr_t value = ZOop::to_address(*p);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    46
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    47
    if (value == 0) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    48
      return;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    49
    }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    50
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    51
    guarantee(!ZAddress::is_finalizable(value), BAD_OOP_REPORT(value));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    52
    guarantee(ZAddress::is_good(value), BAD_OOP_REPORT(value));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    53
    guarantee(oopDesc::is_oop(ZOop::from_address(value)), BAD_OOP_REPORT(value));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    54
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    55
  virtual void do_oop(narrowOop*) { ShouldNotReachHere(); }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    56
};
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    57
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    58
template <bool VisitReferents>
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    59
class ZVerifyOopClosure : public ClaimMetadataVisitingOopIterateClosure, public ZRootsIteratorClosure  {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    60
public:
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    61
  ZVerifyOopClosure() :
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    62
      ClaimMetadataVisitingOopIterateClosure(ClassLoaderData::_claim_other) {}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    63
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    64
  virtual void do_oop(oop* p);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    65
  virtual void do_oop(narrowOop* p) { ShouldNotReachHere(); }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    66
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    67
  virtual ReferenceIterationMode reference_iteration_mode() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    68
    return VisitReferents ? DO_FIELDS : DO_FIELDS_EXCEPT_REFERENT;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    69
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    70
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    71
#ifdef ASSERT
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    72
  // Verification handled by the closure itself
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    73
  virtual bool should_verify_oops() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    74
    return false;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    75
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    76
#endif
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    77
};
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    78
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    79
class ZVerifyObjectClosure : public ObjectClosure {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    80
private:
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    81
  bool _visit_referents;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    82
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    83
public:
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    84
  ZVerifyObjectClosure(bool visit_referents) : _visit_referents(visit_referents) {}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    85
  virtual void do_object(oop o);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    86
};
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    87
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    88
template <typename RootsIterator>
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    89
void ZVerify::roots_impl() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    90
  if (ZVerifyRoots) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    91
    ZVerifyRootsClosure cl;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    92
    RootsIterator iter;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    93
    iter.oops_do(&cl);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    94
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    95
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    96
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    97
void ZVerify::roots_strong() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    98
  roots_impl<ZRootsIterator>();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
    99
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   100
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   101
class ZVerifyConcurrentRootsIterator : public ZConcurrentRootsIterator {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   102
public:
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   103
  ZVerifyConcurrentRootsIterator()
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   104
      : ZConcurrentRootsIterator(ClassLoaderData::_claim_none) {}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   105
};
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   106
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   107
void ZVerify::roots_concurrent() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   108
  roots_impl<ZVerifyConcurrentRootsIterator>();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   109
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   110
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   111
void ZVerify::roots_weak() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   112
  assert(!ZResurrection::is_blocked(), "Invalid phase");
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   113
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   114
  roots_impl<ZWeakRootsIterator>();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   115
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   116
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   117
void ZVerify::roots(bool verify_weaks) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   118
  roots_strong();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   119
  roots_concurrent();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   120
  if (verify_weaks) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   121
    roots_weak();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   122
    roots_concurrent_weak();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   123
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   124
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   125
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   126
void ZVerify::objects(bool verify_weaks) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   127
  if (ZVerifyObjects) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   128
    ZVerifyObjectClosure cl(verify_weaks);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   129
    ZHeap::heap()->object_iterate(&cl, verify_weaks);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   130
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   131
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   132
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   133
void ZVerify::roots_concurrent_weak() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   134
  assert(!ZResurrection::is_blocked(), "Invalid phase");
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   135
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   136
  roots_impl<ZConcurrentWeakRootsIterator>();
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   137
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   138
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   139
void ZVerify::roots_and_objects(bool verify_weaks) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   140
  ZStatTimerDisable  _disable;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   141
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   142
  roots(verify_weaks);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   143
  objects(verify_weaks);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   144
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   145
57656
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   146
void ZVerify::before_zoperation() {
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   147
  // Verify strong roots
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   148
  ZStatTimerDisable disable;
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   149
  roots_strong();
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   150
}
9429ecaee2e0 8229135: ZGC: Adding missing ZStatTimerDisable before call to ZVerify::roots_strong()
pliden
parents: 55603
diff changeset
   151
55603
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   152
void ZVerify::after_mark() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   153
  // Only verify strong roots and references.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   154
  roots_and_objects(false /* verify_weaks */);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   155
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   156
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   157
void ZVerify::after_weak_processing() {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   158
  // Also verify weaks - all should have been processed at this point.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   159
  roots_and_objects(true /* verify_weaks */);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   160
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   161
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   162
template <bool VisitReferents>
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   163
void ZVerifyOopClosure<VisitReferents>::do_oop(oop* p) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   164
  guarantee(SafepointSynchronize::is_at_safepoint(), "Must be at a safepoint");
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   165
  guarantee(ZGlobalPhase == ZPhaseMarkCompleted, "Invalid phase");
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   166
  guarantee(!ZResurrection::is_blocked(), "Invalid phase");
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   167
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   168
  const oop o = RawAccess<>::oop_load(p);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   169
  if (o == NULL) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   170
    return;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   171
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   172
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   173
  const uintptr_t addr = ZOop::to_address(o);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   174
  if (VisitReferents) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   175
    guarantee(ZAddress::is_good(addr) || ZAddress::is_finalizable_good(addr), BAD_OOP_REPORT(addr));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   176
  } else {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   177
    // Should not encounter finalizable oops through strong-only paths. Assumes only strong roots are visited.
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   178
    guarantee(ZAddress::is_good(addr), BAD_OOP_REPORT(addr));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   179
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   180
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   181
  const uintptr_t good_addr = ZAddress::good(addr);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   182
  guarantee(oopDesc::is_oop(ZOop::from_address(good_addr)), BAD_OOP_REPORT(addr));
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   183
}
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   184
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   185
void ZVerifyObjectClosure::do_object(oop o) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   186
  if (_visit_referents) {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   187
    ZVerifyOopClosure<true /* VisitReferents */> cl;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   188
    o->oop_iterate((OopIterateClosure*)&cl);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   189
  } else {
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   190
    ZVerifyOopClosure<false /* VisitReferents */> cl;
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   191
    o->oop_iterate(&cl);
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   192
  }
3868dde58ebb 8227175: ZGC: ZHeapIterator visits potentially dead objects
stefank
parents:
diff changeset
   193
}