Re: [ovs-dev] [PATCH v3 ovn 5/7] controller: use unsigned long long int fro qos_max_rate/qos_min_rate/qos_burst

2023-04-18 Thread Lorenzo Bianconi
> The commit message headline should be fixed: fro -> for ack, I will fix it. > > Should a test case be adjusted to confirm that this works? An existing > test case can be reused. sure, will do. Regards, Lorenzo > > > On Thu, Apr 13, 2023 at 10:45 AM Lorenzo Bianconi > wrote: > > > > This

Re: [ovs-dev] [PATCH v3 ovn 5/7] controller: use unsigned long long int fro qos_max_rate/qos_min_rate/qos_burst

2023-04-14 Thread Ihar Hrachyshka
The commit message headline should be fixed: fro -> for Should a test case be adjusted to confirm that this works? An existing test case can be reused. On Thu, Apr 13, 2023 at 10:45 AM Lorenzo Bianconi wrote: > > This patch allow to configure max/min rate greater than 4Gbps > > https://bugzilla

[ovs-dev] [PATCH v3 ovn 5/7] controller: use unsigned long long int fro qos_max_rate/qos_min_rate/qos_burst

2023-04-13 Thread Lorenzo Bianconi
This patch allow to configure max/min rate greater than 4Gbps https://bugzilla.redhat.com/show_bug.cgi?id=2129742 Tested-by: Rodolfo Alonso Signed-off-by: Lorenzo Bianconi --- controller/binding.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git