src/hotspot/share/gc/shared/satbMarkQueue.cpp
author coleenp
Thu, 25 Apr 2019 10:56:31 -0400
changeset 54623 1126f0607c70
parent 54366 2b48cedce327
child 54713 040bf087ae39
permissions -rw-r--r--
8222811: Consolidate MutexLockerEx and MutexLocker Summary: Make MutexLocker be MutexLockerEx implementation, remove MutexLockerEx calls. Reviewed-by: dcubed, dholmes, pliden, rehn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     1
/*
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
     2
 * Copyright (c) 2001, 2019, Oracle and/or its affiliates. All rights reserved.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     4
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     7
 * published by the Free Software Foundation.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     8
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    13
 * accompanied this code).
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    14
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5082
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5082
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5082
diff changeset
    21
 * questions.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    22
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    23
 */
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
51441
2e91d927e00c 8154343: Make SATB related code available to other GCs
kbarrett
parents: 51403
diff changeset
    26
#include "gc/shared/satbMarkQueue.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30577
diff changeset
    27
#include "gc/shared/collectedHeap.hpp"
51393
cc8d309cd05a 8209345: Merge SATBMarkQueueFilter into SATBMarkQueueSet
kbarrett
parents: 51368
diff changeset
    28
#include "logging/log.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "memory/allocation.inline.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13195
diff changeset
    30
#include "oops/oop.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "runtime/mutexLocker.hpp"
51403
0f3bfea3acf1 8209347: SATBMarkQueue.cpp should not need jvm.h
kbarrett
parents: 51401
diff changeset
    32
#include "runtime/os.hpp"
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30279
diff changeset
    33
#include "runtime/safepoint.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "runtime/thread.hpp"
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47765
diff changeset
    35
#include "runtime/threadSMR.hpp"
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 7920
diff changeset
    36
#include "runtime/vmThread.hpp"
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    37
54255
c81fbf340ceb 8220240: Refactor shared dirty card queue
kbarrett
parents: 54006
diff changeset
    38
SATBMarkQueue::SATBMarkQueue(SATBMarkQueueSet* qset) :
33761
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    39
  // SATB queues are only active during marking cycles. We create
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    40
  // them with their active field set to false. If a thread is
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    41
  // created during a cycle and its SATB queue needs to be activated
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    42
  // before the thread starts running, we'll need to set its active
51401
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    43
  // field to true. This must be done in the collector-specific
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
    44
  // BarrierSet thread attachment protocol.
54255
c81fbf340ceb 8220240: Refactor shared dirty card queue
kbarrett
parents: 54006
diff changeset
    45
  PtrQueue(qset, false /* active */)
33761
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    46
{ }
329db4b51480 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents: 33105
diff changeset
    47
33792
2bae5bf9e431 8077571: ObjPtrQueue is poorly named
kbarrett
parents: 33786
diff changeset
    48
void SATBMarkQueue::flush() {
30255
f43e306ec51e 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 30154
diff changeset
    49
  // Filter now to possibly save work later.  If filtering empties the
f43e306ec51e 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 30154
diff changeset
    50
  // buffer then flush_impl can deallocate the buffer.
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    51
  filter();
28507
354ef83ee258 8048949: Requeue queue implementation
tschatzl
parents: 27251
diff changeset
    52
  flush_impl();
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    53
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    54
51278
d56dd9798d54 8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents: 49752
diff changeset
    55
// This method will first apply filtering to the buffer. If filtering
d56dd9798d54 8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents: 49752
diff changeset
    56
// retains a small enough collection in the buffer, we can continue to
d56dd9798d54 8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents: 49752
diff changeset
    57
// use the buffer as-is, instead of enqueueing and replacing it.
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    58
33792
2bae5bf9e431 8077571: ObjPtrQueue is poorly named
kbarrett
parents: 33786
diff changeset
    59
bool SATBMarkQueue::should_enqueue_buffer() {
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    60
  // This method should only be called if there is a non-NULL buffer
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    61
  // that is full.
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    62
  assert(index() == 0, "pre-condition");
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    63
  assert(_buf != NULL, "pre-condition");
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    64
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    65
  filter();
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    66
51401
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    67
  SATBMarkQueueSet* satb_qset = static_cast<SATBMarkQueueSet*>(qset());
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    68
  size_t threshold = satb_qset->buffer_enqueue_threshold();
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    69
  // Ensure we'll enqueue completely full buffers.
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    70
  assert(threshold > 0, "enqueue threshold = 0");
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    71
  // Ensure we won't enqueue empty buffers.
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    72
  assert(threshold <= capacity(),
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    73
         "enqueue threshold " SIZE_FORMAT " exceeds capacity " SIZE_FORMAT,
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    74
         threshold, capacity());
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
    75
  return index() < threshold;
7920
298df61588a2 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 7397
diff changeset
    76
}
298df61588a2 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 7397
diff changeset
    77
33792
2bae5bf9e431 8077571: ObjPtrQueue is poorly named
kbarrett
parents: 33786
diff changeset
    78
void SATBMarkQueue::apply_closure_and_empty(SATBBufferClosure* cl) {
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30279
diff changeset
    79
  assert(SafepointSynchronize::is_at_safepoint(),
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30279
diff changeset
    80
         "SATB queues must only be processed at safepoints");
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    81
  if (_buf != NULL) {
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    82
    cl->do_buffer(&_buf[index()], size());
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    83
    reset();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    84
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    85
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    86
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    87
#ifndef PRODUCT
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    88
// Helpful for debugging
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    89
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    90
static void print_satb_buffer(const char* name,
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    91
                              void** buf,
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    92
                              size_t index,
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    93
                              size_t capacity) {
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    94
  tty->print_cr("  SATB BUFFER [%s] buf: " PTR_FORMAT " index: " SIZE_FORMAT
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    95
                " capacity: " SIZE_FORMAT,
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    96
                name, p2i(buf), index, capacity);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    97
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
    98
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
    99
void SATBMarkQueue::print(const char* name) {
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
   100
  print_satb_buffer(name, _buf, index(), capacity());
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   101
}
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
   102
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   103
#endif // PRODUCT
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   104
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   105
SATBMarkQueueSet::SATBMarkQueueSet() :
30268
d28a9aa10804 8078021: SATB apply_closure_to_completed_buffer should have closure argument
kbarrett
parents: 30255
diff changeset
   106
  PtrQueueSet(),
51401
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   107
  _buffer_enqueue_threshold(0)
51278
d56dd9798d54 8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents: 49752
diff changeset
   108
{}
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   109
52582
6df094be7f58 8213352: Separate BufferNode allocation from PtrQueueSet
kbarrett
parents: 51441
diff changeset
   110
void SATBMarkQueueSet::initialize(Monitor* cbl_mon,
6df094be7f58 8213352: Separate BufferNode allocation from PtrQueueSet
kbarrett
parents: 51441
diff changeset
   111
                                  BufferNode::Allocator* allocator,
52726
9cfa2e273b77 8214278: Cleanup process_completed_threshold and related state
kbarrett
parents: 52582
diff changeset
   112
                                  size_t process_completed_buffers_threshold,
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   113
                                  uint buffer_enqueue_threshold_percentage) {
52726
9cfa2e273b77 8214278: Cleanup process_completed_threshold and related state
kbarrett
parents: 52582
diff changeset
   114
  PtrQueueSet::initialize(cbl_mon, allocator);
9cfa2e273b77 8214278: Cleanup process_completed_threshold and related state
kbarrett
parents: 52582
diff changeset
   115
  set_process_completed_buffers_threshold(process_completed_buffers_threshold);
51401
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   116
  assert(buffer_size() != 0, "buffer size not initialized");
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   117
  // Minimum threshold of 1 ensures enqueuing of completely full buffers.
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   118
  size_t size = buffer_size();
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   119
  size_t enqueue_qty = (size * buffer_enqueue_threshold_percentage) / 100;
58113ce90caf 8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents: 51393
diff changeset
   120
  _buffer_enqueue_threshold = MAX2(size - enqueue_qty, (size_t)1);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   121
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   122
5082
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   123
#ifdef ASSERT
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   124
void SATBMarkQueueSet::dump_active_states(bool expected_active) {
36090
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35061
diff changeset
   125
  log_error(gc, verify)("Expected SATB active state: %s", expected_active ? "ACTIVE" : "INACTIVE");
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35061
diff changeset
   126
  log_error(gc, verify)("Actual SATB active states:");
cffc1dd98258 8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents: 35061
diff changeset
   127
  log_error(gc, verify)("  Queue set: %s", is_active() ? "ACTIVE" : "INACTIVE");
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   128
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   129
  class DumpThreadStateClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   130
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   131
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   132
    DumpThreadStateClosure(SATBMarkQueueSet* qset) : _qset(qset) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   133
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   134
      SATBMarkQueue& queue = _qset->satb_queue_for_thread(t);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   135
      log_error(gc, verify)("  Thread \"%s\" queue: %s",
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   136
                            t->name(),
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   137
                            queue.is_active() ? "ACTIVE" : "INACTIVE");
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   138
    }
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   139
  } closure(this);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   140
  Threads::threads_do(&closure);
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   141
}
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   142
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   143
void SATBMarkQueueSet::verify_active_states(bool expected_active) {
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   144
  // Verify queue set state
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   145
  if (is_active() != expected_active) {
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   146
    dump_active_states(expected_active);
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   147
    fatal("SATB queue set has an unexpected active state");
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   148
  }
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   149
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   150
  // Verify thread queue states
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   151
  class VerifyThreadStatesClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   152
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   153
    bool _expected_active;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   154
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   155
    VerifyThreadStatesClosure(SATBMarkQueueSet* qset, bool expected_active) :
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   156
      _qset(qset), _expected_active(expected_active) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   157
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   158
      if (_qset->satb_queue_for_thread(t).is_active() != _expected_active) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   159
        _qset->dump_active_states(_expected_active);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   160
        fatal("Thread SATB queue has an unexpected active state");
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   161
      }
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   162
    }
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   163
  } closure(this, expected_active);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   164
  Threads::threads_do(&closure);
5082
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   165
}
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   166
#endif // ASSERT
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   167
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   168
void SATBMarkQueueSet::set_active_all_threads(bool active, bool expected_active) {
5082
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   169
  assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   170
#ifdef ASSERT
22497
b2e1896e9616 8029162: G1: Shared SATB queue never enabled
pliden
parents: 13728
diff changeset
   171
  verify_active_states(expected_active);
5082
19e725a3d2eb 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 4481
diff changeset
   172
#endif // ASSERT
54366
2b48cedce327 8220671: Initialization race for non-JavaThread PtrQueues
kbarrett
parents: 54255
diff changeset
   173
  // Update the global state, synchronized with threads list management.
2b48cedce327 8220671: Initialization race for non-JavaThread PtrQueues
kbarrett
parents: 54255
diff changeset
   174
  {
54623
1126f0607c70 8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents: 54366
diff changeset
   175
    MutexLocker ml(NonJavaThreadsList_lock, Mutex::_no_safepoint_check_flag);
54366
2b48cedce327 8220671: Initialization race for non-JavaThread PtrQueues
kbarrett
parents: 54255
diff changeset
   176
    _all_active = active;
2b48cedce327 8220671: Initialization race for non-JavaThread PtrQueues
kbarrett
parents: 54255
diff changeset
   177
  }
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   178
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   179
  class SetThreadActiveClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   180
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   181
    bool _active;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   182
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   183
    SetThreadActiveClosure(SATBMarkQueueSet* qset, bool active) :
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   184
      _qset(qset), _active(active) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   185
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   186
      _qset->satb_queue_for_thread(t).set_active(_active);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   187
    }
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   188
  } closure(this, active);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   189
  Threads::threads_do(&closure);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   190
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   191
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   192
void SATBMarkQueueSet::filter_thread_buffers() {
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   193
  class FilterThreadBufferClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   194
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   195
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   196
    FilterThreadBufferClosure(SATBMarkQueueSet* qset) : _qset(qset) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   197
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   198
      _qset->satb_queue_for_thread(t).filter();
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   199
    }
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   200
  } closure(this);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   201
  Threads::threads_do(&closure);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   202
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   203
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30279
diff changeset
   204
bool SATBMarkQueueSet::apply_closure_to_completed_buffer(SATBBufferClosure* cl) {
53102
35530ca3e0b2 8214201: Make PtrQueueSet completed buffer list private
kbarrett
parents: 52726
diff changeset
   205
  BufferNode* nd = get_completed_buffer();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   206
  if (nd != NULL) {
4481
de92ec484f5e 6862387: tune concurrent refinement further
iveresov
parents: 3262
diff changeset
   207
    void **buf = BufferNode::make_buffer_from_node(nd);
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
   208
    size_t index = nd->index();
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
   209
    size_t size = buffer_size();
37065
c00d1c2ffb7c 8150676: Use BufferNode index
kbarrett
parents: 36090
diff changeset
   210
    assert(index <= size, "invariant");
c00d1c2ffb7c 8150676: Use BufferNode index
kbarrett
parents: 36090
diff changeset
   211
    cl->do_buffer(buf + index, size - index);
c00d1c2ffb7c 8150676: Use BufferNode index
kbarrett
parents: 36090
diff changeset
   212
    deallocate_buffer(nd);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   213
    return true;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   214
  } else {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   215
    return false;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   216
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   217
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   218
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   219
#ifndef PRODUCT
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   220
// Helpful for debugging
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   221
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   222
#define SATB_PRINTER_BUFFER_SIZE 256
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   223
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   224
void SATBMarkQueueSet::print_all(const char* msg) {
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   225
  char buffer[SATB_PRINTER_BUFFER_SIZE];
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   226
  assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   227
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33792
diff changeset
   228
  tty->cr();
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33792
diff changeset
   229
  tty->print_cr("SATB BUFFERS [%s]", msg);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   230
53102
35530ca3e0b2 8214201: Make PtrQueueSet completed buffer list private
kbarrett
parents: 52726
diff changeset
   231
  BufferNode* nd = completed_buffers_head();
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   232
  int i = 0;
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   233
  while (nd != NULL) {
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   234
    void** buf = BufferNode::make_buffer_from_node(nd);
51403
0f3bfea3acf1 8209347: SATBMarkQueue.cpp should not need jvm.h
kbarrett
parents: 51401
diff changeset
   235
    os::snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Enqueued: %d", i);
46443
cdb638b5ec53 8178836: Improve PtrQueue index abstraction
kbarrett
parents: 37065
diff changeset
   236
    print_satb_buffer(buffer, buf, nd->index(), buffer_size());
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   237
    nd = nd->next();
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   238
    i += 1;
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   239
  }
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   240
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   241
  class PrintThreadClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   242
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   243
    char* _buffer;
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   244
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   245
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   246
    PrintThreadClosure(SATBMarkQueueSet* qset, char* buffer) :
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   247
      _qset(qset), _buffer(buffer) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   248
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   249
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   250
      os::snprintf(_buffer, SATB_PRINTER_BUFFER_SIZE, "Thread: %s", t->name());
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   251
      _qset->satb_queue_for_thread(t).print(_buffer);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   252
    }
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   253
  } closure(this, buffer);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   254
  Threads::threads_do(&closure);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   255
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33792
diff changeset
   256
  tty->cr();
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   257
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   258
#endif // PRODUCT
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 10670
diff changeset
   259
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   260
void SATBMarkQueueSet::abandon_partial_marking() {
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   261
  assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
53102
35530ca3e0b2 8214201: Make PtrQueueSet completed buffer list private
kbarrett
parents: 52726
diff changeset
   262
  abandon_completed_buffers();
54006
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   263
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   264
  class AbandonThreadQueueClosure : public ThreadClosure {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   265
    SATBMarkQueueSet* _qset;
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   266
  public:
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   267
    AbandonThreadQueueClosure(SATBMarkQueueSet* qset) : _qset(qset) {}
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   268
    virtual void do_thread(Thread* t) {
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   269
      _qset->satb_queue_for_thread(t).reset();
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   270
    }
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   271
  } closure(this);
a421bdf22394 8219613: Use NonJavaThread PtrQueues
kbarrett
parents: 53102
diff changeset
   272
  Threads::threads_do(&closure);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   273
}