Thu, 22 May 2014 15:52:41 -0400
8037816: Fix for 8036122 breaks build with Xcode5/clang
8043029: Change 8037816 breaks HS build with older GCC versions which don't support diagnostic pragmas
8043164: Format warning in traceStream.hpp
Summary: Backport of main fix + two corrections, enables clang compilation, turns on format attributes, corrects/mutes warnings
Reviewed-by: kvn, coleenp, iveresov, twisti
duke@435 | 1 | /* |
drchase@6680 | 2 | * Copyright (c) 2001, 2014, Oracle and/or its affiliates. All rights reserved. |
duke@435 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@435 | 4 | * |
duke@435 | 5 | * This code is free software; you can redistribute it and/or modify it |
duke@435 | 6 | * under the terms of the GNU General Public License version 2 only, as |
duke@435 | 7 | * published by the Free Software Foundation. |
duke@435 | 8 | * |
duke@435 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
duke@435 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@435 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@435 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
duke@435 | 13 | * accompanied this code). |
duke@435 | 14 | * |
duke@435 | 15 | * You should have received a copy of the GNU General Public License version |
duke@435 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
duke@435 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@435 | 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. |
duke@435 | 22 | * |
duke@435 | 23 | */ |
duke@435 | 24 | |
stefank@2314 | 25 | #include "precompiled.hpp" |
jmasa@3730 | 26 | #include "gc_implementation/concurrentMarkSweep/freeChunk.hpp" |
jmasa@3730 | 27 | #include "memory/freeBlockDictionary.hpp" |
stefank@2314 | 28 | #include "utilities/copy.hpp" |
duke@435 | 29 | |
drchase@6680 | 30 | PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC |
drchase@6680 | 31 | |
duke@435 | 32 | #ifndef PRODUCT |
duke@435 | 33 | |
duke@435 | 34 | #define baadbabeHeapWord badHeapWordVal |
duke@435 | 35 | #define deadbeefHeapWord 0xdeadbeef |
duke@435 | 36 | |
duke@435 | 37 | size_t const FreeChunk::header_size() { |
duke@435 | 38 | return sizeof(FreeChunk)/HeapWordSize; |
duke@435 | 39 | } |
duke@435 | 40 | |
duke@435 | 41 | void FreeChunk::mangleAllocated(size_t size) { |
duke@435 | 42 | // mangle all but the header of a just-allocated block |
duke@435 | 43 | // of storage |
duke@435 | 44 | assert(size >= MinChunkSize, "smallest size of object"); |
duke@435 | 45 | // we can't assert that _size == size because this may be an |
duke@435 | 46 | // allocation out of a linear allocation block |
duke@435 | 47 | assert(sizeof(FreeChunk) % HeapWordSize == 0, |
duke@435 | 48 | "shouldn't write beyond chunk"); |
duke@435 | 49 | HeapWord* addr = (HeapWord*)this; |
duke@435 | 50 | size_t hdr = header_size(); |
duke@435 | 51 | Copy::fill_to_words(addr + hdr, size - hdr, baadbabeHeapWord); |
duke@435 | 52 | } |
duke@435 | 53 | |
coleenp@622 | 54 | void FreeChunk::mangleFreed(size_t sz) { |
duke@435 | 55 | assert(baadbabeHeapWord != deadbeefHeapWord, "Need distinct patterns"); |
duke@435 | 56 | // mangle all but the header of a just-freed block of storage |
duke@435 | 57 | // just prior to passing it to the storage dictionary |
coleenp@622 | 58 | assert(sz >= MinChunkSize, "smallest size of object"); |
coleenp@622 | 59 | assert(sz == size(), "just checking"); |
duke@435 | 60 | HeapWord* addr = (HeapWord*)this; |
duke@435 | 61 | size_t hdr = header_size(); |
coleenp@622 | 62 | Copy::fill_to_words(addr + hdr, sz - hdr, deadbeefHeapWord); |
duke@435 | 63 | } |
duke@435 | 64 | |
duke@435 | 65 | void FreeChunk::verifyList() const { |
duke@435 | 66 | FreeChunk* nextFC = next(); |
duke@435 | 67 | if (nextFC != NULL) { |
duke@435 | 68 | assert(this == nextFC->prev(), "broken chain"); |
duke@435 | 69 | assert(size() == nextFC->size(), "wrong size"); |
duke@435 | 70 | nextFC->verifyList(); |
duke@435 | 71 | } |
duke@435 | 72 | } |
duke@435 | 73 | #endif |
ysr@1580 | 74 | |
ysr@1580 | 75 | void FreeChunk::print_on(outputStream* st) { |
ysr@1580 | 76 | st->print_cr("Next: " PTR_FORMAT " Prev: " PTR_FORMAT " %s", |
ysr@1580 | 77 | next(), prev(), cantCoalesce() ? "[can't coalesce]" : ""); |
ysr@1580 | 78 | } |