src/share/vm/oops/markOop.cpp

Wed, 30 Mar 2011 07:47:19 -0700

author
never
date
Wed, 30 Mar 2011 07:47:19 -0700
changeset 2693
63997f575155
parent 2314
f95d63e2154a
child 3156
f08d439fab8c
permissions
-rw-r--r--

7031614: jmap -permstat fails with java.lang.InternalError in sun.jvm.hotspot.oops.OopField.getValue
Reviewed-by: kvn, dcubed

duke@435 1 /*
stefank@2314 2 * Copyright (c) 1997, 2010, 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"
stefank@2314 26 #include "oops/markOop.hpp"
stefank@2314 27 #ifdef TARGET_OS_FAMILY_linux
stefank@2314 28 # include "thread_linux.inline.hpp"
stefank@2314 29 #endif
stefank@2314 30 #ifdef TARGET_OS_FAMILY_solaris
stefank@2314 31 # include "thread_solaris.inline.hpp"
stefank@2314 32 #endif
stefank@2314 33 #ifdef TARGET_OS_FAMILY_windows
stefank@2314 34 # include "thread_windows.inline.hpp"
stefank@2314 35 #endif
duke@435 36
duke@435 37
duke@435 38 void markOopDesc::print_on(outputStream* st) const {
duke@435 39 if (is_locked()) {
duke@435 40 st->print("locked(0x%lx)->", value());
duke@435 41 markOop(*(markOop*)value())->print_on(st);
duke@435 42 } else {
jrose@1474 43 assert(is_unlocked() || has_bias_pattern(), "just checking");
duke@435 44 st->print("mark(");
jrose@1474 45 if (has_bias_pattern()) st->print("biased,");
duke@435 46 st->print("hash %#lx,", hash());
duke@435 47 st->print("age %d)", age());
duke@435 48 }
duke@435 49 }
dcubed@483 50
dcubed@483 51
dcubed@483 52 // Give advice about whether the oop that contains this markOop
dcubed@483 53 // should be cached or not.
dcubed@483 54 bool markOopDesc::should_not_be_cached() const {
dcubed@483 55 // the cast is because decode_pointer() isn't marked const
dcubed@483 56 if (is_marked() && ((markOopDesc *)this)->decode_pointer() != NULL) {
dcubed@483 57 // If the oop containing this markOop is being forwarded, then
dcubed@483 58 // we are in the middle of GC and we do not want the containing
dcubed@483 59 // oop to be added to a cache. We have no way of knowing whether
dcubed@483 60 // the cache has already been visited by the current GC phase so
dcubed@483 61 // we don't know whether the forwarded oop will be properly
dcubed@483 62 // processed in this phase. If the forwarded oop is not properly
dcubed@483 63 // processed, then we'll see strange crashes or asserts during
dcubed@483 64 // the next GC run because the markOop will contain an unexpected
dcubed@483 65 // value.
dcubed@483 66 //
dcubed@483 67 // This situation has been seen when we are GC'ing a methodOop
dcubed@483 68 // because we use the methodOop while we're GC'ing it. Scary
dcubed@483 69 // stuff. Some of the uses the methodOop cause the methodOop to
dcubed@483 70 // be added to the OopMapCache in the instanceKlass as a side
dcubed@483 71 // effect. This check lets the cache maintainer know when a
dcubed@483 72 // cache addition would not be safe.
dcubed@483 73 return true;
dcubed@483 74 }
dcubed@483 75
dcubed@483 76 // caching the containing oop should be just fine
dcubed@483 77 return false;
dcubed@483 78 }

mercurial