<div class="gmail_quote"><div>PTAL </div><div><a href="http://cr.openjdk.java.net/~rasbold/8061259/webrev.01/" target="_blank" style="line-height:19.7999992370605px">http://cr.openjdk.java.net/~rasbold/8061259/webrev.01/</a><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">I've looked a bit at the webrev. A couple of comments:<br></div><div bgcolor="#FFFFFF" text="#000000">
    <br>
    Why do you use OrderAccess methods for writing and reading the
    _has_promo_failed flag in has_promo_failed() and set_promot_failed()
    ?<br></div></blockquote><div><br></div></div><div class="gmail_quote"><div>I think that has no effect on x86, but I assumed that processors with weak memory model may want ordering of set/reset/has call.</div></div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">Can we write out the full word "promotion" instead of just "promo"
    in the variables and methods?<br></div></blockquote><div><br></div></div><div class="gmail_quote"><div>Done.</div></div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    Can we change the name of the flag from UseCMSFastPromotionFailure
    to CMSFastPromotionFailure? Most CMS flags start with CMS and I
    don't think we need the "Use" prefix.<br></div></blockquote><div><br></div></div><div class="gmail_quote"><div>Done.</div></div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">What do you think about making the flag true by default? At least
    for JDK 9. If we decide to backport to JDK 8 or 7 it might be a good
    idea to keep the default value as false.<br></div></blockquote><div><br></div></div><div class="gmail_quote"><div>Done.</div><div>Let me know if there is anything for me to do to backport to JDK8 and 7.</div></div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">Did you find the information provided by
    _fast_promo_failure_hitcount useful for debugging? If it not too
    useful I would consider removing it since it is cluttering up the
    code a bit.</div></blockquote><div><br></div></div><div class="gmail_quote"><div>I removed it.</div><div>It was useful to for development, but I think it is no longer needed.</div><div><br></div><div><br></div><div><br></div></div>