<br><br>
<div class="gmail_quote">On Wed, Aug 12, 2009 at 1:09 PM, Sean Hefty <span dir="ltr"><<a href="mailto:sean.hefty@intel.com">sean.hefty@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">
<div class="im">>+      mad_recvq_size = roundup_pow_of_two(mad_recvq_size);<br>>+      mad_recvq_size = min(mad_recvq_size, IB_MAD_QP_MAX_SIZE);<br>>+      mad_recvq_size = max(mad_recvq_size, IB_MAD_QP_MIN_SIZE);<br>
>+<br>>+      mad_sendq_size = roundup_pow_of_two(mad_sendq_size);<br>>+      mad_sendq_size = min(mad_sendq_size, IB_MAD_QP_MAX_SIZE);<br>>+      mad_sendq_size = max(mad_sendq_size, IB_MAD_QP_MIN_SIZE);<br><br>
</div>Why round up to a power of two or have min/max restrictions?</blockquote>
<div> </div>
<div>power of two is arbitrary and could be removed. min is also somewhat arbitrary but didn't want to allow it too much smaller than it already is (default for this patch). max truly is a maximum as create QP fails with larger size (didn't try this across all HCAs though).</div>

<div> </div>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid"><span id=""></span><br>
<div>
<div></div>
<div class="h5"><br>_______________________________________________<br>general mailing list<br><a href="mailto:general@lists.openfabrics.org">general@lists.openfabrics.org</a><br><a href="http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general" target="_blank">http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general</a><br>
<br>To unsubscribe, please visit <a href="http://openib.org/mailman/listinfo/openib-general" target="_blank">http://openib.org/mailman/listinfo/openib-general</a><br></div></div></blockquote></div><br>