src/hotspot/share/gc/shared/space.inline.hpp
author rkennke
Thu, 05 Apr 2018 10:54:53 +0200
changeset 49722 a47d1e21b3f1
parent 47216 71c04702a3d5
child 49982 9042ffe5b7fe
permissions -rw-r--r--
8199735: Mark word updates need to use Access API Reviewed-by: shade, eosterlund
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
49722
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
     2
 * Copyright (c) 2000, 2018, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1374
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1374
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: 1374
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    25
#ifndef SHARE_VM_GC_SHARED_SPACE_INLINE_HPP
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    26
#define SHARE_VM_GC_SHARED_SPACE_INLINE_HPP
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    28
#include "gc/serial/markSweep.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    29
#include "gc/shared/collectedHeap.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    30
#include "gc/shared/generation.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    31
#include "gc/shared/space.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
    32
#include "gc/shared/spaceDecorator.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "memory/universe.hpp"
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    34
#include "oops/oopsHierarchy.hpp"
49722
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
    35
#include "oops/oop.inline.hpp"
24487
71ff0bd674eb 8042737: Introduce umbrella header prefetch.inline.hpp
goetz
parents: 13963
diff changeset
    36
#include "runtime/prefetch.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "runtime/safepoint.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    39
inline HeapWord* Space::block_start(const void* p) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    40
  return block_start_const(p);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    41
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    42
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
inline HeapWord* OffsetTableContigSpace::allocate(size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  HeapWord* res = ContiguousSpace::allocate(size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  if (res != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
    _offsets.alloc_block(res, size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
  return res;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
// Because of the requirement of keeping "_offsets" up to date with the
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
// allocations, we sequentialize these with a lock.  Therefore, best if
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
// this is used for larger LAB allocations only.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
inline HeapWord* OffsetTableContigSpace::par_allocate(size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  MutexLocker x(&_par_alloc_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  // This ought to be just "allocate", because of the lock above, but that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  // ContiguousSpace::allocate asserts that either the allocating thread
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  // holds the heap lock or it is the VM thread and we're at a safepoint.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  // The best I (dld) could figure was to put a field in ContiguousSpace
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  // meaning "locking at safepoint taken care of", and set/reset that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  // here.  But this will do for now, especially in light of the comment
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  // above.  Perhaps in the future some lock-free manner of keeping the
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  // coordination.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  HeapWord* res = ContiguousSpace::par_allocate(size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  if (res != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
    _offsets.alloc_block(res, size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  return res;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    71
inline HeapWord*
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 1
diff changeset
    72
OffsetTableContigSpace::block_start_const(const void* p) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  return _offsets.block_start(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
    76
size_t CompactibleSpace::obj_size(const HeapWord* addr) const {
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
    77
  return oop(addr)->size();
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
    78
}
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
    79
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    80
class DeadSpacer : StackObj {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    81
  size_t _allowed_deadspace_words;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    82
  bool _active;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    83
  CompactibleSpace* _space;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    84
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    85
public:
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    86
  DeadSpacer(CompactibleSpace* space) : _space(space), _allowed_deadspace_words(0) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    87
    size_t ratio = _space->allowed_dead_ratio();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    88
    _active = ratio > 0;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    89
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    90
    if (_active) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    91
      assert(!UseG1GC, "G1 should not be using dead space");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    92
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    93
      // We allow some amount of garbage towards the bottom of the space, so
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    94
      // we don't start compacting before there is a significant gain to be made.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    95
      // Occasionally, we want to ensure a full compaction, which is determined
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    96
      // by the MarkSweepAlwaysCompactCount parameter.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    97
      if ((MarkSweep::total_invocations() % MarkSweepAlwaysCompactCount) != 0) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    98
        _allowed_deadspace_words = (space->capacity() * ratio / 100) / HeapWordSize;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
    99
      } else {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   100
        _active = false;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   101
      }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   102
    }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   103
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   104
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   105
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   106
  bool insert_deadspace(HeapWord* dead_start, HeapWord* dead_end) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   107
    if (!_active) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   108
      return false;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   109
    }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   110
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   111
    size_t dead_length = pointer_delta(dead_end, dead_start);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   112
    if (_allowed_deadspace_words >= dead_length) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   113
      _allowed_deadspace_words -= dead_length;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   114
      CollectedHeap::fill_with_object(dead_start, dead_length);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   115
      oop obj = oop(dead_start);
49722
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
   116
      obj->set_mark_raw(obj->mark_raw()->set_marked());
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   117
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   118
      assert(dead_length == (size_t)obj->size(), "bad filler object size");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   119
      log_develop_trace(gc, compaction)("Inserting object to dead space: " PTR_FORMAT ", " PTR_FORMAT ", " SIZE_FORMAT "b",
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   120
          p2i(dead_start), p2i(dead_end), dead_length * HeapWordSize);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   121
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   122
      return true;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   123
    } else {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   124
      _active = false;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   125
      return false;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   126
    }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   127
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   128
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   129
};
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   130
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   131
template <class SpaceType>
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   132
inline void CompactibleSpace::scan_and_forward(SpaceType* space, CompactPoint* cp) {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   133
  // Compute the new addresses for the live objects and store it in the mark
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   134
  // Used by universe::mark_sweep_phase2()
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   135
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   136
  // We're sure to be here before any objects are compacted into this
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   137
  // space, so this is a good time to initialize this:
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   138
  space->set_compaction_top(space->bottom());
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   139
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   140
  if (cp->space == NULL) {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   141
    assert(cp->gen != NULL, "need a generation");
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   142
    assert(cp->threshold == NULL, "just checking");
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   143
    assert(cp->gen->first_compaction_space() == space, "just checking");
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   144
    cp->space = cp->gen->first_compaction_space();
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   145
    cp->threshold = cp->space->initialize_threshold();
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   146
    cp->space->set_compaction_top(cp->space->bottom());
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   147
  }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   148
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   149
  HeapWord* compact_top = cp->space->compaction_top(); // This is where we are currently compacting to.
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   150
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   151
  DeadSpacer dead_spacer(space);
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   152
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   153
  HeapWord*  end_of_live = space->bottom();  // One byte beyond the last byte of the last live object.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   154
  HeapWord*  first_dead = NULL; // The first dead object.
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   155
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   156
  const intx interval = PrefetchScanIntervalInBytes;
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   157
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   158
  HeapWord* cur_obj = space->bottom();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   159
  HeapWord* scan_limit = space->scan_limit();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   160
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   161
  while (cur_obj < scan_limit) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   162
    assert(!space->scanned_block_is_obj(cur_obj) ||
49722
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
   163
           oop(cur_obj)->mark_raw()->is_marked() || oop(cur_obj)->mark_raw()->is_unlocked() ||
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
   164
           oop(cur_obj)->mark_raw()->has_bias_pattern(),
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   165
           "these are the only valid states during a mark sweep");
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   166
    if (space->scanned_block_is_obj(cur_obj) && oop(cur_obj)->is_gc_marked()) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   167
      // prefetch beyond cur_obj
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   168
      Prefetch::write(cur_obj, interval);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   169
      size_t size = space->scanned_block_size(cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   170
      compact_top = cp->space->forward(oop(cur_obj), size, cp, compact_top);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   171
      cur_obj += size;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   172
      end_of_live = cur_obj;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   173
    } else {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   174
      // run over all the contiguous dead objects
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   175
      HeapWord* end = cur_obj;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   176
      do {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   177
        // prefetch beyond end
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   178
        Prefetch::write(end, interval);
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   179
        end += space->scanned_block_size(end);
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   180
      } while (end < scan_limit && (!space->scanned_block_is_obj(end) || !oop(end)->is_gc_marked()));
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   181
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   182
      // see if we might want to pretend this object is alive so that
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   183
      // we don't have to compact quite as often.
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   184
      if (cur_obj == compact_top && dead_spacer.insert_deadspace(cur_obj, end)) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   185
        oop obj = oop(cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   186
        compact_top = cp->space->forward(obj, obj->size(), cp, compact_top);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   187
        end_of_live = end;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   188
      } else {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   189
        // otherwise, it really is a free region.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   190
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   191
        // cur_obj is a pointer to a dead object. Use this dead memory to store a pointer to the next live object.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   192
        *(HeapWord**)cur_obj = end;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   193
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   194
        // see if this is the first dead region.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   195
        if (first_dead == NULL) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   196
          first_dead = cur_obj;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   197
        }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   198
      }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   199
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   200
      // move on to the next object
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   201
      cur_obj = end;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   202
    }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   203
  }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   204
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   205
  assert(cur_obj == scan_limit, "just checking");
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   206
  space->_end_of_live = end_of_live;
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   207
  if (first_dead != NULL) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   208
    space->_first_dead = first_dead;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   209
  } else {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   210
    space->_first_dead = end_of_live;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   211
  }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   212
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   213
  // save the compaction_top of the compaction space.
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   214
  cp->space->set_compaction_top(compact_top);
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   215
}
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   216
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   217
template <class SpaceType>
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   218
inline void CompactibleSpace::scan_and_adjust_pointers(SpaceType* space) {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   219
  // adjust all the interior pointers to point at the new locations of objects
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   220
  // Used by MarkSweep::mark_sweep_phase3()
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   221
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   222
  HeapWord* cur_obj = space->bottom();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   223
  HeapWord* const end_of_live = space->_end_of_live;  // Established by "scan_and_forward".
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   224
  HeapWord* const first_dead = space->_first_dead;    // Established by "scan_and_forward".
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   225
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   226
  assert(first_dead <= end_of_live, "Stands to reason, no?");
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   227
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   228
  const intx interval = PrefetchScanIntervalInBytes;
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   229
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   230
  debug_only(HeapWord* prev_obj = NULL);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   231
  while (cur_obj < end_of_live) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   232
    Prefetch::write(cur_obj, interval);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   233
    if (cur_obj < first_dead || oop(cur_obj)->is_gc_marked()) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   234
      // cur_obj is alive
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   235
      // point all the oops to the new location
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   236
      size_t size = MarkSweep::adjust_pointers(oop(cur_obj));
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   237
      size = space->adjust_obj_size(size);
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   238
      debug_only(prev_obj = cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   239
      cur_obj += size;
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   240
    } else {
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   241
      debug_only(prev_obj = cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   242
      // cur_obj is not a live object, instead it points at the next live object
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   243
      cur_obj = *(HeapWord**)cur_obj;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   244
      assert(cur_obj > prev_obj, "we should be moving forward through memory, cur_obj: " PTR_FORMAT ", prev_obj: " PTR_FORMAT, p2i(cur_obj), p2i(prev_obj));
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   245
    }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   246
  }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   247
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   248
  assert(cur_obj == end_of_live, "just checking");
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   249
}
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   250
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   251
#ifdef ASSERT
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   252
template <class SpaceType>
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   253
inline void CompactibleSpace::verify_up_to_first_dead(SpaceType* space) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   254
  HeapWord* cur_obj = space->bottom();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   255
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   256
  if (cur_obj < space->_end_of_live && space->_first_dead > cur_obj && !oop(cur_obj)->is_gc_marked()) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   257
     // we have a chunk of the space which hasn't moved and we've reinitialized
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   258
     // the mark word during the previous pass, so we can't use is_gc_marked for
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   259
     // the traversal.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   260
     HeapWord* prev_obj = NULL;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   261
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   262
     while (cur_obj < space->_first_dead) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   263
       size_t size = space->obj_size(cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   264
       assert(!oop(cur_obj)->is_gc_marked(), "should be unmarked (special dense prefix handling)");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   265
       prev_obj = cur_obj;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   266
       cur_obj += size;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   267
     }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   268
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   269
}
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   270
#endif
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   271
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   272
template <class SpaceType>
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   273
inline void CompactibleSpace::clear_empty_region(SpaceType* space) {
27624
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   274
  // Let's remember if we were empty before we did the compaction.
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   275
  bool was_empty = space->used_region().is_empty();
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   276
  // Reset space after compaction is complete
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   277
  space->reset_after_compaction();
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   278
  // We do this clear, below, since it has overloaded meanings for some
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   279
  // space subtypes.  For example, OffsetTableContigSpace's that were
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   280
  // compacted into will have had their offset table thresholds updated
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   281
  // continuously, but those that weren't need to have their thresholds
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   282
  // re-initialized.  Also mangles unused area for debugging.
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   283
  if (space->used_region().is_empty()) {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   284
    if (!was_empty) space->clear(SpaceDecorator::Mangle);
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   285
  } else {
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   286
    if (ZapUnusedHeapArea) space->mangle_unused_area();
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   287
  }
fe43edc5046d 8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents: 24487
diff changeset
   288
}
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
   289
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   290
template <class SpaceType>
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   291
inline void CompactibleSpace::scan_and_compact(SpaceType* space) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   292
  // Copy all live objects to their new location
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   293
  // Used by MarkSweep::mark_sweep_phase4()
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   294
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   295
  verify_up_to_first_dead(space);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   296
40888
f09b053be364 8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents: 37495
diff changeset
   297
  HeapWord* const bottom = space->bottom();
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   298
  HeapWord* const end_of_live = space->_end_of_live;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   299
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   300
  assert(space->_first_dead <= end_of_live, "Invariant. _first_dead: " PTR_FORMAT " <= end_of_live: " PTR_FORMAT, p2i(space->_first_dead), p2i(end_of_live));
40888
f09b053be364 8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents: 37495
diff changeset
   301
  if (space->_first_dead == end_of_live && (bottom == end_of_live || !oop(bottom)->is_gc_marked())) {
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   302
    // Nothing to compact. The space is either empty or all live object should be left in place.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   303
    clear_empty_region(space);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   304
    return;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   305
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   306
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   307
  const intx scan_interval = PrefetchScanIntervalInBytes;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   308
  const intx copy_interval = PrefetchCopyIntervalInBytes;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   309
40888
f09b053be364 8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents: 37495
diff changeset
   310
  assert(bottom < end_of_live, "bottom: " PTR_FORMAT " should be < end_of_live: " PTR_FORMAT, p2i(bottom), p2i(end_of_live));
f09b053be364 8155917: Memory access in free regions during G1 full gc causes regressions in SPECjvm2008 scimark.fft,lu,sor,sparse with 9+116 on Linux-x64
tschatzl
parents: 37495
diff changeset
   311
  HeapWord* cur_obj = bottom;
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   312
  if (space->_first_dead > cur_obj && !oop(cur_obj)->is_gc_marked()) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   313
    // All object before _first_dead can be skipped. They should not be moved.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   314
    // A pointer to the first live object is stored at the memory location for _first_dead.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   315
    cur_obj = *(HeapWord**)(space->_first_dead);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   316
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   317
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   318
  debug_only(HeapWord* prev_obj = NULL);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   319
  while (cur_obj < end_of_live) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   320
    if (!oop(cur_obj)->is_gc_marked()) {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   321
      debug_only(prev_obj = cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   322
      // The first word of the dead object contains a pointer to the next live object or end of space.
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   323
      cur_obj = *(HeapWord**)cur_obj;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   324
      assert(cur_obj > prev_obj, "we should be moving forward through memory");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   325
    } else {
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   326
      // prefetch beyond q
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   327
      Prefetch::read(cur_obj, scan_interval);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   328
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   329
      // size and destination
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   330
      size_t size = space->obj_size(cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   331
      HeapWord* compaction_top = (HeapWord*)oop(cur_obj)->forwardee();
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   332
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   333
      // prefetch beyond compaction_top
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   334
      Prefetch::write(compaction_top, copy_interval);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   335
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   336
      // copy object and reinit its mark
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   337
      assert(cur_obj != compaction_top, "everything in this pass should be moving");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   338
      Copy::aligned_conjoint_words(cur_obj, compaction_top, size);
49722
a47d1e21b3f1 8199735: Mark word updates need to use Access API
rkennke
parents: 47216
diff changeset
   339
      oop(compaction_top)->init_mark_raw();
37495
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   340
      assert(oop(compaction_top)->klass() != NULL, "should have a class");
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   341
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   342
      debug_only(prev_obj = cur_obj);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   343
      cur_obj += size;
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   344
    }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   345
  }
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   346
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   347
  clear_empty_region(space);
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   348
}
7ef1f23dfc9c 8073321: assert(q > prev_q) failed: we should be moving forward through memory
brutisso
parents: 37238
diff changeset
   349
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
   350
size_t ContiguousSpace::scanned_block_size(const HeapWord* addr) const {
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
   351
  return oop(addr)->size();
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
   352
}
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 30764
diff changeset
   353
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29792
diff changeset
   354
#endif // SHARE_VM_GC_SHARED_SPACE_INLINE_HPP