<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">I had wanted to say that such a bug would be caught by compiling with some reasonalbe warning level; but I think I was wrong.<div><br></div><div>I compiled</div><div><span style="white-space:pre">    </span>if(1==1);<br></div><div><br></div><div>with some wrapper and got nothing with whatever gcc I have on this laptop, until </div><div>gcc -Wextra  </div><div><br></div><div>which is more persnickety than -Wall, and just got</div><div><div>mynoop.c: In function 'main':</div><div>mynoop.c:4:10: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]</div><div>  if(1==1);</div><div>          ^</div></div><div><br></div><div>So I guess I have to forgive the software engineer who fat-fingered that semicolon. Of course I've done worse.</div><div><br></div><div>Peter</div><div><br></div><div><br></div><div><br></div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 10, 2018 at 4:22 AM, Chris Samuel <span dir="ltr"><<a href="mailto:chris@csamuel.org" target="_blank">chris@csamuel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Friday, 17 August 2018 2:47:37 PM AEST Chris Samuel wrote:<br>
<br>
</span><span class="">> Just a heads up that the 3.10.0-862.11.6.el7.x86_64 kernel from RHEL/CentOS<br>
> that was released to address the most recent Intel CPU problem "L1TF" seems<br>
> to break RDMA (found by a colleague here at Swinburne).<br>
<br>
</span>So this CentOS bug has a one line bug fix for this problem!<br>
<br>
<a href="https://bugs.centos.org/view.php?id=15193" rel="noreferrer" target="_blank">https://bugs.centos.org/view.<wbr>php?id=15193</a><br>
<br>
It's a corker - basically it looks like someone typo'd a ; into an if <br>
statement, the fix is:<br>
<br>
-       if (!rdma_is_port_valid_nospec(<wbr>device, &ah_attr->port_num));<br>
+       if (!rdma_is_port_valid_nospec(<wbr>device, &ah_attr->port_num))<br>
                return -EINVAL;<br>
<br>
So it always returns -EINVAL when checking the port as the if becomes a noop.. <br>
:-(<br>
<br>
Patch attached...<br>
<div class="HOEnZb"><div class="h5"><br>
-- <br>
 Chris Samuel  :  <a href="http://www.csamuel.org/" rel="noreferrer" target="_blank">http://www.csamuel.org/</a>  :  Melbourne, VIC<br>
</div></div><br>______________________________<wbr>_________________<br>
Beowulf mailing list, <a href="mailto:Beowulf@beowulf.org">Beowulf@beowulf.org</a> sponsored by Penguin Computing<br>
To change your subscription (digest mode or unsubscribe) visit <a href="http://www.beowulf.org/mailman/listinfo/beowulf" rel="noreferrer" target="_blank">http://www.beowulf.org/<wbr>mailman/listinfo/beowulf</a><br>
<br></blockquote></div><br></div>