[ofa-general] Re: [PATCH] opensm: Add infrastructure support for MulticastFDBTop
Sasha Khapyorsky
sashak at voltaire.com
Sun Aug 30 05:04:55 PDT 2009
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 <complib/cl_packoff.h>
> /************/
> @@ -5908,7 +5910,7 @@ typedef struct _ib_switch_info_record {
> ib_net16_t lid;
> uint16_t resv0;
> ib_switch_info_t switch_info;
> - uint8_t pad[3];
> + uint8_t pad[1];
Why should be pad[1] here? In struct switch_info you are adding three
bytes (resvd - 1 and mcast_top - 2), no?
Sasha
More information about the general
mailing list