[ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-11 Thread Ales Musil
Add configuration option into NB global table called "mac_binding_removal_limit" defaulting to 0. This option allows to limit number of MAC bindings that can be removed by the aging mechanism in a single transaction. The 0 means that the mechanism is disabled. If the limit is reached next removal w

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-11 Thread 0-day Robot
Bleep bloop. Greetings Ales Musil, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 84 characters long (recommended limit is 79) #126 FILE: ovn-nb.xml:166:

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-11 Thread Mark Michelson
Acked-by: Mark Michelson On 8/11/22 10:44, 0-day Robot wrote: Bleep bloop. Greetings Ales Musil, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 84 characters

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-15 Thread Han Zhou
On Thu, Aug 11, 2022 at 7:20 AM Ales Musil wrote: > > Add configuration option into NB global table > called "mac_binding_removal_limit" defaulting to 0. > This option allows to limit number of MAC bindings > that can be removed by the aging mechanism in a single > transaction. The 0 means that th

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-15 Thread Ales Musil
On Mon, Aug 15, 2022 at 6:51 PM Han Zhou wrote: > > > On Thu, Aug 11, 2022 at 7:20 AM Ales Musil wrote: > > > > Add configuration option into NB global table > > called "mac_binding_removal_limit" defaulting to 0. > > This option allows to limit number of MAC bindings > > that can be removed by

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-17 Thread Han Zhou
On Mon, Aug 15, 2022 at 11:29 PM Ales Musil wrote: > > > > On Mon, Aug 15, 2022 at 6:51 PM Han Zhou wrote: >> >> >> >> On Thu, Aug 11, 2022 at 7:20 AM Ales Musil wrote: >> > >> > Add configuration option into NB global table >> > called "mac_binding_removal_limit" defaulting to 0. >> > This opti

Re: [ovs-dev] [PATCH ovn v5 5/5] northd: Add config to limit bulk removal of MAC binding

2022-08-17 Thread Ales Musil
On Wed, Aug 17, 2022 at 6:54 PM Han Zhou wrote: > > > On Mon, Aug 15, 2022 at 11:29 PM Ales Musil wrote: > > > > > > > > On Mon, Aug 15, 2022 at 6:51 PM Han Zhou wrote: > >> > >> > >> > >> On Thu, Aug 11, 2022 at 7:20 AM Ales Musil wrote: > >> > > >> > Add configuration option into NB global t