[ofa-general] Re: [PATCH V2] osm: QoS - adding CPI:CapabilityMask2 and turning on QOS_SUPPORTED bit

[Yevgeny Kliteynik]

Sasha Khapyorsky wrote:
> On 00:13 Thu 18 Oct     , Sasha Khapyorsky wrote:
>>> diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h
>>> index 0969755..3685007 100644
>>> --- a/opensm/include/iba/ib_types.h
>>> +++ b/opensm/include/iba/ib_types.h
>>> @@ -3247,8 +3247,7 @@ typedef struct _ib_class_port_info {
>>>  	uint8_t base_ver;
>>>  	uint8_t class_ver;
>>>  	ib_net16_t cap_mask;
>>> -	uint8_t reserved[3];
>>> -	uint8_t resp_time_val;
>>> +	ib_net32_t cap_mask2_resp_time;
> 
> This will break ibutils. We are in OFED already, so I think the patch
> for ibutils should be committed/pushed at same time.

Patch for ibutils is ready and waiting for the opensm patch to be applied.

-- Yevgeny

> Sasha
>