Mercurial > jdk8-mips64-public > hotspot / annotate
src/share/vm/gc_implementation/g1/heapRegionSeq.inline.hpp@063382f9b575 (annotated)
src/share/vm/gc_implementation/g1/heapRegionSeq.inline.hpp
Fri, 29 Apr 2011 14:59:04 -0400
- author
- tonyp
- date
- Fri, 29 Apr 2011 14:59:04 -0400
- changeset 2849
- 063382f9b575
- parent 2314
- f95d63e2154a
- child 2963
- c3f1170908be
- permissions
- -rw-r--r--
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
Summary: We should only undirty cards after we decide that they are not on a young region, not before. The fix also includes improvements to the verify_dirty_region() method which print out which cards were not found dirty.
Reviewed-by: johnc, brutisso
| ysr@777 | 1 | /* |
| stefank@2314 | 2 | * Copyright (c) 2001, 2010, Oracle and/or its affiliates. All rights reserved. |
| ysr@777 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
| ysr@777 | 4 | * |
| ysr@777 | 5 | * This code is free software; you can redistribute it and/or modify it |
| ysr@777 | 6 | * under the terms of the GNU General Public License version 2 only, as |
| ysr@777 | 7 | * published by the Free Software Foundation. |
| ysr@777 | 8 | * |
| ysr@777 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
| ysr@777 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
| ysr@777 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
| ysr@777 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
| ysr@777 | 13 | * accompanied this code). |
| ysr@777 | 14 | * |
| ysr@777 | 15 | * You should have received a copy of the GNU General Public License version |
| ysr@777 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
| ysr@777 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
| ysr@777 | 18 | * |
| trims@1907 | 19 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
| trims@1907 | 20 | * or visit www.oracle.com if you need additional information or have any |
| trims@1907 | 21 | * questions. |
| ysr@777 | 22 | * |
| ysr@777 | 23 | */ |
| ysr@777 | 24 | |
| stefank@2314 | 25 | #ifndef SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP |
| stefank@2314 | 26 | #define SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP |
| stefank@2314 | 27 | |
| stefank@2314 | 28 | #include "gc_implementation/g1/heapRegionSeq.hpp" |
| stefank@2314 | 29 | |
| ysr@777 | 30 | inline HeapRegion* HeapRegionSeq::addr_to_region(const void* addr) { |
| ysr@777 | 31 | assert(_seq_bottom != NULL, "bad _seq_bottom in addr_to_region"); |
| ysr@777 | 32 | if ((char*) addr >= _seq_bottom) { |
| ysr@777 | 33 | size_t diff = (size_t) pointer_delta((HeapWord*) addr, |
| ysr@777 | 34 | (HeapWord*) _seq_bottom); |
| ysr@777 | 35 | int index = (int) (diff >> HeapRegion::LogOfHRGrainWords); |
| ysr@777 | 36 | assert(index >= 0, "invariant / paranoia"); |
| ysr@777 | 37 | if (index < _regions.length()) { |
| ysr@777 | 38 | HeapRegion* hr = _regions.at(index); |
| ysr@777 | 39 | assert(hr->is_in_reserved(addr), |
| ysr@777 | 40 | "addr_to_region is wrong..."); |
| ysr@777 | 41 | return hr; |
| ysr@777 | 42 | } |
| ysr@777 | 43 | } |
| ysr@777 | 44 | return NULL; |
| ysr@777 | 45 | } |
| stefank@2314 | 46 | |
| stefank@2314 | 47 | #endif // SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP |
