<font size=2 face="sans-serif">Hi Thomas,</font><br><br><font size=2 face="sans-serif">>   we in the gc team have discussed
this change quite a bit internally.</font><br><font size=2 face="sans-serif">> Overall, we think this change seems
far too risky from both a</font><br><font size=2 face="sans-serif">> functional and performance perspective
to go into 9 at this time.</font><br><br><font size=2 face="sans-serif">Thank you for your comments and giving
a decision.</font><br><font size=2 face="sans-serif">I completely agree with the decision
and would like to keep contributing</font><br><font size=2 face="sans-serif">to this change for future releases.</font><br><br><font size=2 face="sans-serif">> We also think the testing needs
to include both functional and</font><br><font size=2 face="sans-serif">> performance testing, and the performance
testing ought to be using some</font><br><font size=2 face="sans-serif">> well-chosen benchmarks. (It was
pointed out very early in the</font><br><font size=2 face="sans-serif">> discussion of this change that
specjbb2013 is deprecated, yet that is</font><br><font size=2 face="sans-serif">> the only benchmark that's been
reported out.)</font><br><br><font size=2 face="sans-serif">I see. I will try other workloads and
evaluate effects of this change.</font><br><br><font size=2 face="sans-serif">> The most recent change also penalizes
current platforms that do not</font><br><font size=2 face="sans-serif">> implement the release-CAS with
an additional acquire. That might be not</font><br><font size=2 face="sans-serif">> an issue for TSO platforms, but
others will be affected.</font><br><font size=2 face="sans-serif">> </font><br><font size=2 face="sans-serif">> While we think other platforms
could quickly adapt to this, this would</font><br><font size=2 face="sans-serif">> force that the developer that implements
this for other platforms</font><br><font size=2 face="sans-serif">> (arm/aarch64) to be stuck with
re-analyzing these issues. We</font><br><font size=2 face="sans-serif">> do not think this is fair. We think
this is a change (or set of</font><br><font size=2 face="sans-serif">> changes) that needs to be pushed
for all platforms at the same time.</font><br><br><font size=2 face="sans-serif">Sure. I would like to ask developers
for the other platforms to consider</font><br><font size=2 face="sans-serif">this change.</font><br><br><font size=2 face="sans-serif">> There also one (minor) question
about the change: why isn't the CAS</font><br><font size=2 face="sans-serif">> result value being used for the
failing paths of the CAS, rather than</font><br><font size=2 face="sans-serif">> reloaded in copy_to_survivor_space?</font><br><br><font size=2 face="sans-serif">I believe, the original code also doesn't
use the CAS result because</font><br><font size=2 face="sans-serif">the current cas_forward_to doesn't return
the CAS result value.</font><br><br><font size=2 face="sans-serif">bool oopDesc::cas_forward_to(oop p,
markOop compare, cmpxchg_memory_order order)</font><br><br><font size=2 face="sans-serif">I guess, reloading a forwardee is not
expensive because CAS fails are rare,</font><br><font size=2 face="sans-serif">then maintenanceability was emphasized.</font><br><br><font size=2 face="sans-serif">"Doerr, Martin" <martin.doerr@sap.com>
wrote on 10/21/2016 21:57:42:</font><br><font size=2 face="sans-serif">> The webrev also contains a logging
change in </font><br><font size=2 face="sans-serif">> psPromotionManager.inline.hpp which
I'm not sure if it's still wanted.</font><br><br><font size=2 face="sans-serif">For the future discussion, I would like
to inform a webrev that doesn't</font><br><font size=2 face="sans-serif">have any changes of log formats.</font><br><a href=http://cr.openjdk.java.net/~horii/8154736/webrev.06/><font size=2 color=blue face="sans-serif">http://cr.openjdk.java.net/~horii/8154736/webrev.06/</font></a><br><br><font size=2 face="sans-serif">Regards,<br>Hiroshi<br>-----------------------<br>Hiroshi Horii, Ph.D.<br>IBM Research - Tokyo</font><tt><font size=2><br></font></tt><BR>