<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;"><br><div><div>On Jul 13, 2018, at 1:02 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 12/07/2018 23:20, Brian Burkhalter 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;"><a href="https://bugs.openjdk.java.net/browse/JDK-8206448">https://bugs.openjdk.java.net/browse/JDK-8206448</a><br><a href="http://cr.openjdk.java.net/~bpb/8206448/webrev.00/">http://cr.openjdk.java.net/~bpb/8206448/webrev.00/</a><br><br>Modified<br><br>1) not to assume that extended attributes are enabled by default on ext3, and<br>2) to assume that they are enabled by default on ext4 if and only if the kernel version is greater than or equal to 2.6.39.<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 mostly looks okay to me. It would be good to move getKernelVersion closer to the end of the source file as this area puts the fields at the top.</span></blockquote><div><br></div>I was going by the old coding style of putting static things at the top of the class but I agree it would be better near where it is used.</div><div><br><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;">We should probably clean up the xattrChecked/xattrEnabled handling at some point but what you have is fine. Also I assume we'll drop the kernel check at some point soon as 2.6.39 is really old and unlikely there will be any supported systems with these older kernels soon.</span></blockquote></div><br><div>Is it worthwhile to file an issue to track that?</div><div><br></div><div>Thanks,</div><div><br></div><div>Brian</div></body></html>