mbox series

[ovs-dev,ovn,branch-20.06,0/7] Backport ARP/NS responder flow explosion fixes.

Message ID 20200810105317.25272.51568.stgit@dceara.remote.csb
Headers show
Series Backport ARP/NS responder flow explosion fixes. | expand

Message

Dumitru Ceara Aug. 10, 2020, 10:53 a.m. UTC
This series backports the recent ARP/NS responder fixes that reduce the number
of generated logical and OF flows.

This has been previously discussed in the OVN community meeting:

http://eavesdrop.openstack.org/meetings//ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-07-30-17.14.log.html#l-21

CC: Han Zhou <hzhou@ovn.org>
CC: Mark Michelson <mmichels@redhat.com>
CC: Numan Siddique <numans@ovn.org> 
Signed-off-by: Dumitru Ceara <dceara@redhat.com>

Dumitru Ceara (7):
      ovn-northd: Document OVS register usage in logical flows.
      ovn-northd: Store ETH address of router inport in xreg0.
      ovn-northd: Refactor ARP/NS responder in router pipeline.
      ovn-northd: Refactor NAT address parsing.
      ovn-northd: Minimize number of ARP/NS responder flows for DNAT.
      ovn-northd: Fix is_chassis_resident() match for DNAT.
      ovn-northd: Fix logical flows to limit ARP/NS broadcast domain.


 northd/ovn-northd.8.xml |   46 +-
 northd/ovn-northd.c     |  992 ++++++++++++++++++++++++++++++-----------------
 tests/ovn-northd.at     |  149 +++++++
 tests/ovn.at            |   82 ++++
 4 files changed, 893 insertions(+), 376 deletions(-)

Comments

Numan Siddique Aug. 10, 2020, 11:35 a.m. UTC | #1
On Mon, Aug 10, 2020 at 4:23 PM Dumitru Ceara <dceara@redhat.com> wrote:
>
> This series backports the recent ARP/NS responder fixes that reduce the number
> of generated logical and OF flows.
>
> This has been previously discussed in the OVN community meeting:
>
> http://eavesdrop.openstack.org/meetings//ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-07-30-17.14.log.html#l-21
>
> CC: Han Zhou <hzhou@ovn.org>
> CC: Mark Michelson <mmichels@redhat.com>
> CC: Numan Siddique <numans@ovn.org>
> Signed-off-by: Dumitru Ceara <dceara@redhat.com>

Thanks Dumitru. I applied these patches to branch-20.06.

Regards
Numan

>
> Dumitru Ceara (7):
>       ovn-northd: Document OVS register usage in logical flows.
>       ovn-northd: Store ETH address of router inport in xreg0.
>       ovn-northd: Refactor ARP/NS responder in router pipeline.
>       ovn-northd: Refactor NAT address parsing.
>       ovn-northd: Minimize number of ARP/NS responder flows for DNAT.
>       ovn-northd: Fix is_chassis_resident() match for DNAT.
>       ovn-northd: Fix logical flows to limit ARP/NS broadcast domain.
>
>
>  northd/ovn-northd.8.xml |   46 +-
>  northd/ovn-northd.c     |  992 ++++++++++++++++++++++++++++++-----------------
>  tests/ovn-northd.at     |  149 +++++++
>  tests/ovn.at            |   82 ++++
>  4 files changed, 893 insertions(+), 376 deletions(-)
>
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Dumitru Ceara Aug. 10, 2020, 11:50 a.m. UTC | #2
On 8/10/20 1:35 PM, Numan Siddique wrote:
> On Mon, Aug 10, 2020 at 4:23 PM Dumitru Ceara <dceara@redhat.com> wrote:
>>
>> This series backports the recent ARP/NS responder fixes that reduce the number
>> of generated logical and OF flows.
>>
>> This has been previously discussed in the OVN community meeting:
>>
>> http://eavesdrop.openstack.org/meetings//ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-07-30-17.14.log.html#l-21
>>
>> CC: Han Zhou <hzhou@ovn.org>
>> CC: Mark Michelson <mmichels@redhat.com>
>> CC: Numan Siddique <numans@ovn.org>
>> Signed-off-by: Dumitru Ceara <dceara@redhat.com>
> 
> Thanks Dumitru. I applied these patches to branch-20.06.
> 
> Regards
> Numan
> 

Thanks Numan!