<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>Hi Pavel,</div><div><br></div>+1<div><br></div><div>Brian</div><div><br><div><div>On Feb 7, 2018, at 4:07 AM, Alan Bateman <<a href="mailto:Alan.Bateman@oracle.com">Alan.Bateman@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">On 07/02/2018 11:29, Pavel Rappo wrote:</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Hello,<br><br>Please review the following change:<br><br> <a href="http://cr.openjdk.java.net/~prappo/8196908/webrev.00/">http://cr.openjdk.java.net/~prappo/8196908/webrev.00/</a><br><br>This change addresses a number of issues with safe publication and covers some<br>trivial typos. It seems that there are more similar issues in that area.<br>However, I believe it is better to be rather conservative and firstly address<br>issues that are pretty straightforward.<br><br></blockquote><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">This look good, the only thing to check is the new asserts in SelectorImpl.implRegister and whether they push the method size over an inlining threshold. As they are useful when changing this area then you could add them commented out if you want.</span></blockquote></div><div><br></div><div><div>On Feb 7, 2018, at 4:33 AM, Daniel Fuchs <<a href="mailto:daniel.fuchs@oracle.com">daniel.fuchs@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi Pavel,<br><br>KQueueArrayWrapper.java: I think you can make is64bit final.<br><br>cheers,<br><br>-- daniel</blockquote><div><br></div></div><br></div></body></html>