JDK-8234508 : VM_HeapWalkOperation::iterate_over_object reads non-strong fields with an on-strong load barrier
- Type: Bug
- Component: hotspot
- Sub-Component: gc
- Affected Version: 11,14
- Priority: P2
- Status: Resolved
- Resolution: Fixed
- Submitted: 2019-11-20
- Updated: 2021-02-08
- Resolved: 2019-12-09
The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
To download the current JDK release, click here.
Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
To download the current JDK release, click here.
| JDK 11 | JDK 14 |
|---|---|
| 11.0.11Fixed | 14 b27Fixed  |
Related Reports
|
Blocks :
|
Description
I added some verification code to check that we never apply on-strong load barriers to non-strong fields.
The verification triggered at this point:
// an object references a class and its instance fields
// (static fields are ignored here as we report these as
// references from the class).
inline bool VM_HeapWalkOperation::iterate_over_object(oop o) {
// reference to the class
if (!CallbackInvoker::report_class_reference(o, o->klass()->java_mirror())) {
return false;
}
// iterate over instance fields
ClassFieldMap* field_map = JvmtiCachedClassFieldMap::get_map_of_instance_fields(o);
for (int i=0; i<field_map->field_count(); i++) {
ClassFieldDescriptor* field = field_map->field_at(i);
char type = field->field_type();
if (!is_primitive_field_type(type)) {
oop fld_o = o->obj_field(field->field_offset()); <<< Here
Comments
|