hotspot/src/share/vm/gc/shared/barrierSet.inline.hpp
author bpittore
Wed, 24 Jun 2015 12:12:25 -0400
changeset 31592 43f48e165466
parent 30764 fec48bf5a827
child 32596 8feecdee3156
permissions -rw-r--r--
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier" Summary: Need to add a space between macro identifier and string literal Reviewed-by: stefank, dholmes, kbarrett
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 22859
diff changeset
     2
 * Copyright (c) 2001, 2015, 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: 4886
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4886
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: 4886
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: 29325
diff changeset
    25
#ifndef SHARE_VM_GC_SHARED_BARRIERSET_INLINE_HPP
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29325
diff changeset
    26
#define SHARE_VM_GC_SHARED_BARRIERSET_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: 29325
diff changeset
    28
#include "gc/shared/barrierSet.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29325
diff changeset
    29
#include "gc/shared/cardTableModRefBS.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
// Inline functions of BarrierSet, which de-virtualize certain
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1388
diff changeset
    32
// performance-critical calls when the barrier is the most common
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
// card-table kind.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1388
diff changeset
    35
template <class T> void BarrierSet::write_ref_field_pre(T* field, oop new_val) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    36
  if (kind() == CardTableModRef) {
29325
0e86e64c66e5 8069016: Add BarrierSet downcast support
kbarrett
parents: 29081
diff changeset
    37
    barrier_set_cast<CardTableModRefBS>(this)->inline_write_ref_field_pre(field, new_val);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    38
  } else {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    39
    write_ref_field_pre_work(field, new_val);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    40
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    41
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    42
22859
7b88983393b7 8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents: 7397
diff changeset
    43
void BarrierSet::write_ref_field(void* field, oop new_val, bool release) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  if (kind() == CardTableModRef) {
29325
0e86e64c66e5 8069016: Add BarrierSet downcast support
kbarrett
parents: 29081
diff changeset
    45
    barrier_set_cast<CardTableModRefBS>(this)->inline_write_ref_field(field, new_val, release);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  } else {
22859
7b88983393b7 8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents: 7397
diff changeset
    47
    write_ref_field_work(field, new_val, release);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
4461
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    51
// count is number of array elements being written
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    52
void BarrierSet::write_ref_array(HeapWord* start, size_t count) {
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    53
  assert(count <= (size_t)max_intx, "count too large");
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    54
  HeapWord* end = (HeapWord*)((char*)start + (count*heapOopSize));
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    55
  // In the case of compressed oops, start and end may potentially be misaligned;
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    56
  // so we need to conservatively align the first downward (this is not
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    57
  // strictly necessary for current uses, but a case of good hygiene and,
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    58
  // if you will, aesthetics) and the second upward (this is essential for
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    59
  // current uses) to a HeapWord boundary, so we mark all cards overlapping
4886
a2f9a66475b3 6904516: More object array barrier fixes, following up on 6906727
ysr
parents: 4461
diff changeset
    60
  // this write. If this evolves in the future to calling a
4461
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    61
  // logging barrier of narrow oop granularity, like the pre-barrier for G1
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    62
  // (mentioned here merely by way of example), we will need to change this
4886
a2f9a66475b3 6904516: More object array barrier fixes, following up on 6906727
ysr
parents: 4461
diff changeset
    63
  // interface, so it is "exactly precise" (if i may be allowed the adverbial
a2f9a66475b3 6904516: More object array barrier fixes, following up on 6906727
ysr
parents: 4461
diff changeset
    64
  // redundancy for emphasis) and does not include narrow oop slots not
a2f9a66475b3 6904516: More object array barrier fixes, following up on 6906727
ysr
parents: 4461
diff changeset
    65
  // included in the original write interval.
4461
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    66
  HeapWord* aligned_start = (HeapWord*)align_size_down((uintptr_t)start, HeapWordSize);
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    67
  HeapWord* aligned_end   = (HeapWord*)align_size_up  ((uintptr_t)end,   HeapWordSize);
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    68
  // If compressed oops were not being used, these should already be aligned
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    69
  assert(UseCompressedOops || (aligned_start == start && aligned_end == end),
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    70
         "Expected heap word alignment of start and end");
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    71
#if 0
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
    72
  warning("Post:\t" INTPTR_FORMAT "[" SIZE_FORMAT "] : [" INTPTR_FORMAT "," INTPTR_FORMAT ")\t",
4461
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    73
                   start,            count,              aligned_start,   aligned_end);
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    74
#endif
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    75
  write_ref_array_work(MemRegion(aligned_start, aligned_end));
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    76
}
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    77
c17c526d36ef 6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents: 3262
diff changeset
    78
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 22859
diff changeset
    79
inline void BarrierSet::write_region(MemRegion mr) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  if (kind() == CardTableModRef) {
29325
0e86e64c66e5 8069016: Add BarrierSet downcast support
kbarrett
parents: 29081
diff changeset
    81
    barrier_set_cast<CardTableModRefBS>(this)->inline_write_region(mr);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    write_region_work(mr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
}
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    86
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29325
diff changeset
    87
#endif // SHARE_VM_GC_SHARED_BARRIERSET_INLINE_HPP