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

[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 <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