JDK-6918185 : Remove unused code for lost card-marking optimization in BacktraceBuilder
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: hs17
  • Priority: P4
  • Status: Closed
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2010-01-20
  • Updated: 2011-11-25
  • Resolved: 2011-09-30
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.
JDK 7 JDK 8 Other
7u2Fixed 8Fixed hs22Fixed
Related Reports
Relates :  
Description
javaClasses.cpp'BacktraceBuilder appears to have infrastructure for
deferring card-marks until after a sequence of stores into its
_methods array. However, that optimization appears to have been
lost when compressed oops were implemented because the relevant raw
pointer store code was replaced with obj_at_put() which does
the (pre- and post-) barriers. The latter post-barrier for the
entire length of the array in the flush() is thus redundant.

I noticed this when fixing 6904516, as part of which I decided to
remove the redundant card-marks in the flush() method.

This CR is being filed to restore the original optimization
and replace the obj_at_put() with the appropriate raw [narrow]oop
stores and re-enable the deferred card-mark optimization, if it
is deemed important for performance on certain applications.

Lacking any knowledge of specific apps that would gain from
the optimization, I am filing this a P4. Please re-prioritize
and file subCR's as appropriate.

Comments
EVALUATION See main CR
12-09-2011

EVALUATION http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/74cd10898bea
08-07-2011

EVALUATION http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/74cd10898bea
08-07-2011

EVALUATION http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/74cd10898bea
14-06-2011

EVALUATION Removed the dead code.
14-06-2011