[ofa-general] Re: [PATCH] opensm: Add infrastructure support for MulticastFDBTop

2009-08-30 Thread Sasha Khapyorsky
On 10:04 Wed 26 Aug , Hal Rosenstock wrote: @@ -5899,6 +5899,8 @@ typedef struct _ib_switch_info { ib_net16_t lids_per_port; ib_net16_t enforce_cap; uint8_t flags; + uint8_t resvd; + ib_net16_t mcast_top; } PACK_SUFFIX ib_switch_info_t; #include

Re: [ofa-general] Re: [PATCH] opensm: Add infrastructure support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
On 8/30/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 10:04 Wed 26 Aug , Hal Rosenstock wrote: @@ -5899,6 +5899,8 @@ typedef struct _ib_switch_info { ib_net16_t lids_per_port; ib_net16_t enforce_cap; uint8_t flags; + uint8_t resvd; + ib_net16_t

Re: [ofa-general] Re: [PATCH] opensm: Add infrastructure support for MulticastFDBTop

2009-08-30 Thread Sasha Khapyorsky
On 08:22 Sun 30 Aug , Hal Rosenstock wrote: Why should be pad[1] here? In struct switch_info you are adding three bytes (resvd - 1 and mcast_top - 2), no? 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 ?