[ofa-general] Re: [PATCH] opensm/osm_qos_policy.c : set SL for IPoIB broadcast groups

[Yevgeny Kliteynik]

Or Gerlitz wrote:
> Yevgeny Kliteynik wrote:
>> OpenSM re-reads the policy file every heavy sweep and creates new rules.
> OK, got it.
>>
>> The explanation of what the patch is in the beginning of the mail:
>>
>>   "QoS manager should set the right SL in the IPoIB broadcast
>>    groups to enforce the right SL for IPoIB traffic."
> yes, but when the patch lands in git, what remains is only the 
> change-log comment and not anything else, so one someone looks on it 
> without this info, what can he/she do? please add it when you resubmit 
> the patch.
>>
>> QoS manager finds the IPoIB broadcast group that was created for
>> pkey that is specified in the qos policy file, and updates the
>> broadcast (multicast) group's SL to match the SL that was specified
>> for this partition in the qos policy file.
>>
> OK, got it, so just to make sure I got it, without this patch, the QoS 
> (SL) for the per partition ipoib broadcast has to be specified through 
> the partition manager config file, correct?

Correct.

-- Yevgeny

> Or.
> 
> 
> 
>