Thanks for the fix! Acked-by: Ihar Hrachyshka <[email protected]>
On Tue, Apr 4, 2023 at 5:01 PM Lorenzo Bianconi <[email protected]> wrote: > > Similar to qos_max_rate and qos_burst, take into account qos_min_rate in > port_has_qos_params routine. > > Fixes: dbf12e5fe1f7 ("qos: add support for port minimum bandwidth guarantee") > Signed-off-by: Lorenzo Bianconi <[email protected]> > --- > northd/northd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/northd/northd.c b/northd/northd.c > index 25b27607a..a8914e8df 100644 > --- a/northd/northd.c > +++ b/northd/northd.c > @@ -559,7 +559,7 @@ free_chassis_queueid(struct hmap *set, const struct uuid > *uuid, > static inline bool > port_has_qos_params(const struct smap *opts) > { > - return (smap_get(opts, "qos_max_rate") || > + return (smap_get(opts, "qos_max_rate") || smap_get(opts, "qos_min_rate") > || > smap_get(opts, "qos_burst")); > } > > -- > 2.39.2 > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
