Hi Stan,<br><br><div class="gmail_quote">On Thu, Sep 29, 2011 at 1:17 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">
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?<br></blockquote><div> </div><div>That's a workaround rather than a fix.</div><div> </div><div>There are 2 problems here:</div><div> </div><div>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">http://www.mail-archive.com/linux-rdma@vger.kernel.org/msg08391.html</a> email thread.</div>
<div> </div><div>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.</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">
<br>
Thanks,<br>
<br>
Stan.<br>
<br>
<br>
_______________________________________________<br>
ofw mailing list<br>
<a href="mailto:ofw@lists.openfabrics.org">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><br>
</blockquote></div><br>