src/share/vm/gc_implementation/g1/heapRegionManager.inline.hpp

Fri, 10 Oct 2014 15:51:58 +0200

author
tschatzl
date
Fri, 10 Oct 2014 15:51:58 +0200
changeset 7257
e7d0505c8a30
parent 7091
a8ea2f110d87
child 7535
7ae4e26cb1e0
child 9327
f96fcd9e1e1b
permissions
-rw-r--r--

8059758: Footprint regressions with JDK-8038423
Summary: Changes in JDK-8038423 always initialize (zero out) virtual memory used for auxiliary data structures. This causes a footprint regression for G1 in startup benchmarks. This is because they do not touch that memory at all, so the operating system does not actually commit these pages. The fix is to, if the initialization value of the data structures matches the default value of just committed memory (=0), do not do anything.
Reviewed-by: jwilhelm, brutisso

ysr@777 1 /*
drchase@6680 2 * Copyright (c) 2001, 2014, 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
tschatzl@7091 25 #ifndef SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONMANAGER_INLINE_HPP
tschatzl@7091 26 #define SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONMANAGER_INLINE_HPP
stefank@2314 27
tonyp@2963 28 #include "gc_implementation/g1/heapRegion.hpp"
tschatzl@7091 29 #include "gc_implementation/g1/heapRegionManager.hpp"
tschatzl@7051 30 #include "gc_implementation/g1/heapRegionSet.inline.hpp"
stefank@2314 31
tschatzl@7091 32 inline HeapRegion* HeapRegionManager::addr_to_region(HeapWord* addr) const {
brutisso@7049 33 assert(addr < heap_end(),
brutisso@7049 34 err_msg("addr: "PTR_FORMAT" end: "PTR_FORMAT, p2i(addr), p2i(heap_end())));
brutisso@7049 35 assert(addr >= heap_bottom(),
brutisso@7049 36 err_msg("addr: "PTR_FORMAT" bottom: "PTR_FORMAT, p2i(addr), p2i(heap_bottom())));
brutisso@7049 37
tschatzl@5773 38 HeapRegion* hr = _regions.get_by_address(addr);
tonyp@2963 39 return hr;
tonyp@2963 40 }
tonyp@2963 41
tschatzl@7091 42 inline HeapRegion* HeapRegionManager::at(uint index) const {
tschatzl@7050 43 assert(is_available(index), "pre-condition");
tschatzl@5773 44 HeapRegion* hr = _regions.get_by_index(index);
tonyp@2963 45 assert(hr != NULL, "sanity");
tschatzl@7091 46 assert(hr->hrm_index() == index, "sanity");
tonyp@2963 47 return hr;
tonyp@2963 48 }
tonyp@2963 49
tschatzl@7091 50 inline void HeapRegionManager::insert_into_free_list(HeapRegion* hr) {
tschatzl@7050 51 _free_list.add_ordered(hr);
tschatzl@7050 52 }
tschatzl@7050 53
tschatzl@7091 54 inline void HeapRegionManager::allocate_free_regions_starting_at(uint first, uint num_regions) {
tschatzl@7050 55 _free_list.remove_starting_at(at(first), num_regions);
tschatzl@7050 56 }
tschatzl@7050 57
tschatzl@7091 58 #endif // SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONMANAGER_INLINE_HPP

mercurial