7130993: nsk/jdi/ReferenceType/instances/instances004 fails with JFR: assert(ServiceUtil::visible_oop(obj))

Skip reporting invisible refs in iterate_over_object to avoid assert(ServiceUtil::visible_oop(obj))

Reviewed-by: dcubed, mgronlun, rbackman
This commit is contained in:
Serguei Spitsyn 2012-02-14 17:04:40 -08:00 committed by Serguei Spitsyn
parent 8a9ec2c624
commit c25eebd321

View File

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved. * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
* *
* This code is free software; you can redistribute it and/or modify it * This code is free software; you can redistribute it and/or modify it
@ -2999,7 +2999,8 @@ inline bool VM_HeapWalkOperation::iterate_over_object(oop o) {
char type = field->field_type(); char type = field->field_type();
if (!is_primitive_field_type(type)) { if (!is_primitive_field_type(type)) {
oop fld_o = o->obj_field(field->field_offset()); oop fld_o = o->obj_field(field->field_offset());
if (fld_o != NULL) { // ignore any objects that aren't visible to profiler
if (fld_o != NULL && ServiceUtil::visible_oop(fld_o)) {
// reflection code may have a reference to a klassOop. // reflection code may have a reference to a klassOop.
// - see sun.reflect.UnsafeStaticFieldAccessorImpl and sun.misc.Unsafe // - see sun.reflect.UnsafeStaticFieldAccessorImpl and sun.misc.Unsafe
if (fld_o->is_klass()) { if (fld_o->is_klass()) {