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

[Or Gerlitz]

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?

Or.