Re: [ovs-dev] [PATCH ovn v2] Learn the mac binding only if required

2019-09-24 Thread Numan Siddique
Thanks Han for the reviews. Please see below for some comments. Thanks Numan On Wed, Sep 18, 2019 at 4:56 AM Han Zhou wrote: > > On Mon, Sep 16, 2019 at 10:17 AM wrote: > > > > From: Numan Siddique > > > > OVN has the actions - put_arp and put_nd to learn the mac bindings from > the > > ARP

Re: [ovs-dev] [PATCH ovn v2] Learn the mac binding only if required

2019-09-17 Thread Han Zhou
On Mon, Sep 16, 2019 at 10:17 AM wrote: > > From: Numan Siddique > > OVN has the actions - put_arp and put_nd to learn the mac bindings from the > ARP/ND packets. These actions update the Southbound MAC_Binding table. > These actions translates to controller actions. Whenever pinctrl thread > rec

Re: [ovs-dev] [PATCH ovn v2] Learn the mac binding only if required

2019-09-16 Thread 0-day Robot
Bleep bloop. Greetings Numan Siddique, 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 91 characters long (recommended limit is 79) #1031 FILE: ovn-sb.xml:1401:

[ovs-dev] [PATCH ovn v2] Learn the mac binding only if required

2019-09-16 Thread nusiddiq
From: Numan Siddique OVN has the actions - put_arp and put_nd to learn the mac bindings from the ARP/ND packets. These actions update the Southbound MAC_Binding table. These actions translates to controller actions. Whenever pinctrl thread receives such packets, it wakes up the main ovn-controlle