<br><br><div class="gmail_quote">On Fri, Sep 30, 2011 at 12:09 PM, Smith, Stan <span dir="ltr"><<a href="mailto:stan.smith@intel.com">stan.smith@intel.com</a>></span> wrote:<br><blockquote style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;" class="gmail_quote">






<div lang="EN-US" vlink="purple" link="blue">
<div>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
<div style="border-width: medium medium medium 1.5pt; border-style: none none none solid; border-color: currentColor currentColor currentColor blue; padding: 0in 0in 0in 4pt;">
<div>
<div style="border-width: 1pt medium medium; border-style: solid none none; border-color: rgb(181, 196, 223) currentColor currentColor; padding: 3pt 0in 0in;">
<p class="MsoNormal"><b><span style="font-size: 10pt;">From:</span></b><span style="font-size: 10pt;"> Hal Rosenstock [mailto:<a href="mailto:hal.rosenstock@gmail.com" target="_blank">hal.rosenstock@gmail.com</a>]
<br>
<b>Sent:</b> Friday, September 30, 2011 4:09 AM<br>
<b>To:</b> Smith, Stan<br>
<b>Cc:</b> Alex Netes; Hal Rosenstock; <a href="mailto:ofw@lists.openfabrics.org" target="_blank">ofw@lists.openfabrics.org</a><br>
<b>Subject:</b> Re: [ofw] Opensm 3.3.11 (Debug) + 'osmtest -f m -M1' ASSERT() fires?<u></u><u></u></span></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<p style="margin-bottom: 12pt;" class="MsoNormal">Hi Stan,<u></u><u></u></p>
<div><div class="im">
<p class="MsoNormal">On Thu, Sep 29, 2011 at 1:17 PM, Smith, Stan <<a href="mailto:stan.smith@intel.com" target="_blank">stan.smith@intel.com</a>> wrote:<u></u><u></u></p>
<p class="MsoNormal">Hello again,<br>
 I forgot about another issue that was side-stepped via '#if 0' in order to make forward progress in the porting of OpenSM 3.3.11 to Windows.<br>
<br>
 opensm/osm_helper.c<br>
<br>
int ib_path_compare_rates(IN const int rate1, IN const int rate2)<br>
{<br>
       int orate1 = 0, orate2 = 0;<br>
<br>
       CL_ASSERT(rate1 >= IB_MIN_RATE && rate1 <= IB_MAX_RATE);<br>
       CL_ASSERT(rate2 >= IB_MIN_RATE && rate2 <= IB_MAX_RATE);<br>
<br>
When running a DEBUG version of opensm with 'osmtest -f m -M1', part of the test is to set invalid rates (rate1, rate2 == 1).<br>
When the rate1 or rate2 == 1 the CL_ASSERT() fires as IB_MIN_RATE == 2.<br>
I think the fix is to explicitly allow rate1 or rate2 == 1 for osmtest?<br>
<br>
       /* rate1 | rate2 == 1 comes from 'osmtest -f m -M1' (expected invalid rate) */<br>
       CL_ASSERT((rate1 >= IB_MIN_RATE || rate1 == 1) && rate1 <= IB_MAX_RATE);<br>
       CL_ASSERT((rate2 >= IB_MIN_RATE || rate2 == 1) && rate2 <= IB_MAX_RATE);<br>
<br>
Thoughts?<u></u><u></u></p>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
</div><div><div class="im">
<p class="MsoNormal">That's a workaround rather than a fix.<u></u><u></u></p>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
</div><p class="MsoNormal"><span style="color: rgb(0, 112, 192);">Agreed, which is why I did not submit this as a patch.</span><span style="color: rgb(31, 73, 125);"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
</div><div class="im">
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">There are 2 problems here:<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">1. Those particular tests in osmtest are not testing what they intended on testing and they need to be redesigned. The tests are using reserved values for rates. See
<a href="http://www.mail-archive.com/linux-rdma@vger.kernel.org/msg08391.html" target="_blank">http://www.mail-archive.com/linux-rdma@vger.kernel.org/msg08391.html</a> email thread.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">2. Rather than change this assert which is correct, the rate value should be validated and rejected by the SA if invalid. I'll work on a patch for this shortly.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
</div><div>
<p class="MsoNormal">-- Hal<u></u><u></u></p>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color: rgb(31, 73, 125); font-size: 11pt;"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color: rgb(0, 112, 192);">Sounds reasonable; standing by.</span></p><p class="MsoNormal"><span style="color: rgb(0, 112, 192);"></span> </p></div></div></div></div></div></blockquote><div>
 </div><div>Just sent the patch series on rate validation to fix this. You need patches 1, 3, 5, 6, and 7.</div><div> </div><div>-- Hal</div><div> </div><blockquote style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;" class="gmail_quote">
<div lang="EN-US" vlink="purple" link="blue"><div><div style="border-width: medium medium medium 1.5pt; border-style: none none none solid; border-color: currentColor currentColor currentColor blue; padding: 0in 0in 0in 4pt;">
<div><div><p class="MsoNormal"><span style="color: rgb(0, 112, 192);"></span> </p>
</div><div class="im">
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<blockquote style="border-width: medium medium medium 1pt; border-style: none none none solid; border-color: currentColor currentColor currentColor rgb(204, 204, 204); padding: 0in 0in 0in 6pt; margin-right: 0in; margin-left: 4.8pt;">

<p class="MsoNormal"><br>
Thanks,<br>
<br>
Stan.<br>
<br>
<br>
_______________________________________________<br>
ofw mailing list<br>
<a href="mailto:ofw@lists.openfabrics.org" target="_blank">ofw@lists.openfabrics.org</a><br>
<a href="http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw" target="_blank">http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw</a><u></u><u></u></p>
</blockquote>
</div></div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div>

</blockquote></div><br>