<br><br>
<div><span class="gmail_quote">On 8/30/09, <b class="gmail_sendername">Sasha Khapyorsky</b> <<a href="mailto:sashak@voltaire.com">sashak@voltaire.com</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">On 10:04 Wed 26 Aug , Hal Rosenstock wrote:<br>> @@ -5899,6 +5899,8 @@ typedef struct _ib_switch_info {<br>
> ib_net16_t lids_per_port;<br>> ib_net16_t enforce_cap;<br>> uint8_t flags;<br>> + uint8_t resvd;<br>> + ib_net16_t mcast_top;<br>> } PACK_SUFFIX ib_switch_info_t;<br>> #include <complib/cl_packoff.h><br>
> /************/<br>> @@ -5908,7 +5910,7 @@ typedef struct _ib_switch_info_record {<br>> ib_net16_t lid;<br>> uint16_t resv0;<br>> ib_switch_info_t switch_info;<br>> - uint8_t pad[3];<br>
> + uint8_t pad[1];<br><br>Why should be pad[1] here? In struct switch_info you are adding three<br>bytes (resvd - 1 and mcast_top - 2), no?</blockquote>
<div> </div>
<div>Good catch. It was due to an initial version which didn't have the 16 bit MFTTop alignment. Do you want a v2 patch for this ?</div>
<div> </div>
<div>-- Hal</div><br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Sasha<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">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">http://openib.org/mailman/listinfo/openib-general</a><br>
</blockquote></div><br>