On 11/8/2014 9:29 AM, Weiny, Ira wrote:
>> --- a/src/fields.c
>> > +++ b/src/fields.c
>> > @@ -949,8 +949,42 @@ static const ib_field_t ib_mad_f[] = {
>> > {480, 32, "Counter14", mad_dump_uint},
>> > {0, 0}, /* IB_PSR_LAST_F */
>> >
>> > - {0, 0} /* IB_FIELD_LAST_ */
>> > + /*
>> > + * PortInfoExtended fields
>> > + */
>> > + {0, 32, "CapMask", mad_dump_hex},
> I'm not quite sure why the spec was written this way but technically CapMask
> is only 1 bit???
PortInfoExtended:CapMask is 32 bit field. Current 1.3 draft is
misformatted. The two rows are supposed to be combined into 1 and things
are misaligned. IsFECModeSupported is 1 bit (first bit out of those 32
bits).
> Therefore I'm not sure we can define this field as above. Can we get some
> assurances from the management WG
> that the 31 bit reserved field following the CapMask are only going to extend
> the CapMask?
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
