RFR (S): 8026775: nsk/jvmti/RedefineClasses/StressRedefine crashes due to EXCEPTION_ACCESS_VIOLATION

John Rose john.r.rose at oracle.com
Mon Nov 4 11:58:42 PST 2013


+   // Skip the first one because that was already touched in the above
+   // loop - the post decrement of temp means it's now a page below the
+   // last touch

The comments should say 'tmp' not 'temp'.  Also, the phrases 'first one' and 'it's now a page below' are hard to understand, and (to me) slightly misleading.

Suggest:
+   // At this point, (tmp-0) is the last address touched, so don't touch it again.
+   // (It was touched as (tmp-pagesize) but then tmp was post-decremented.)
+   // Skip this address by starting at i=1, and touch a few more pages below.
+   // N.B.  It is important to touch all the down to and including i=StackShadowPages.

— John


More information about the hotspot-compiler-dev mailing list