*Quique Llorente*

CNV networking Senior Software Engineer

Red Hat EMEA <https://www.redhat.com/>

ellor...@redhat.com <arxc...@redhat.com>
@RedHat <https://twitter.com/redhat>   Red Hat
<https://www.linkedin.com/company/red-hat>  Red Hat
<https://www.facebook.com/RedHatInc>
<https://www.redhat.com/>


On Wed, 29 Jan 2025 at 09:41, Enrique Llorente Pastora <ellor...@redhat.com>
wrote:

> On Wed, Jan 29, 2025 at 8:53 AM Ales Musil <amu...@redhat.com> wrote:
> >
> > Thank you for the report. After investigation I think I know what is
> triggering the loop. It is part of the patch 4/4 rest of it is fine. Once I
> have a fix for that I will submit a new version.
>
> I just did the same test and now I don't see CPUs going crazy, so the
> fix is working, my next step is to check IPv6, but it looks quite
> promising.


Also the kubevirt lane at ovn-kubernetes is passing so I suppose we are good

https://github.com/ovn-kubernetes/ovn-kubernetes/pull/5014

Tested-by: Enrique Llorente <ellor...@redhat.com>


>
> >
> > On Tue, Jan 28, 2025 at 5:39 PM Enrique Llorente Pastora <
> ellor...@redhat.com> wrote:
> >>
> >> After testing with a ovn-kubernetes poc using transit routers with a
> primary UDN
> >> layer2 topology and migrating a VM with ingress iperf3 traffic I get
> >> almost 100% of some cpu cores bussy
> >> with softirq, even after closing the iperf3 client, so feels like a
> >> loop is there.
> >>
> >>   17 root      20   0       0      0      0 R  99.0   0.0  32:04.56
> >> ksoftirqd/0
> >>   45 root      20   0       0      0      0 R  90.4   0.0  47:50.90
> >> ksoftirqd/4
> >>
> >> the poc ->
> https://github.com/qinqon/ovn-kubernetes/tree/dnm-transit-router-poc
> >> (last two commits)
> >>
> >> The db files are attached to the email:
> >>
> >> On Tue, Jan 21, 2025 at 8:39 AM Ales Musil <amu...@redhat.com> wrote:
> >> >
> >> > In order to have transit router we have to allow peer port connection
> >> > between LR patch port l3gateway types. This allows direct connection
> >> > of two LRs, one of the being transit router and the second one being
> >> > GW router. This is taken care of commit 3/6. To be fair this can
> >> > change can be useful in general even without transit router.
> >> >
> >> > The 2/4 and 4/4 is the actual work needed for transit router. First
> >> > is the way to define remote ports for logical router which on it's
> >> > own is the main component that results in the transit router. The
> >> > LRP can be set remote by setting options:requested-chassis to chassis
> >> > that has is-remote=true. Only one chassis is supported at time. The
> >> > 6/6 is needed in cases when one AZ ARPs port in other AZ, without
> >> > this the ARP wouldn't be delivered to the original AZ, which would
> >> > result in dropped traffic.
> >> >
> >> > Ales Musil (4):
> >> >   physical: Allow l3gateway and patch port to be peers.
> >> >   northd: Introduce the concept of transit routers.
> >> >   actions, physical: Make the MC split action generic.
> >> >   northd, controller: Flood ARP and NA packet on transit router.
> >> >
> >> >  NEWS                      |   3 +
> >> >  controller/lflow.c        |   1 +
> >> >  controller/lflow.h        |   4 +
> >> >  controller/physical.c     | 273
> ++++++++++++++++++++++++++++++++------
> >> >  controller/pinctrl.c      |  21 +--
> >> >  include/ovn/actions.h     |   7 +-
> >> >  lib/actions.c             |  19 ++-
> >> >  lib/ovn-util.c            |   2 +-
> >> >  northd/northd.c           |  52 ++++++--
> >> >  northd/northd.h           |   4 +
> >> >  ovn-nb.xml                |  43 ++++++
> >> >  tests/multinode-macros.at |  48 +++++++
> >> >  tests/multinode.at        | 196 +++++++++++++++++++++++++++
> >> >  tests/ovn-controller.at   | 153 +++++++++++++++++++++
> >> >  tests/ovn-macros.at       |   1 +
> >> >  tests/ovn-northd.at       |  51 +++++++
> >> >  tests/ovn.at              |  10 +-
> >> >  tests/test-ovn.c          |   1 +
> >> >  utilities/ovn-trace.c     |   3 +
> >> >  19 files changed, 826 insertions(+), 66 deletions(-)
> >> >
> >> > --
> >> > 2.47.1
> >> >
> >> > _______________________________________________
> >> > dev mailing list
> >> > d...@openvswitch.org
> >> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >> >
> >>
> >>
> >> --
> >> Quique Llorente
> >>
> >> CNV networking Senior Software Engineer
> >>
> >> Red Hat EMEA
> >>
> >> ellor...@redhat.com
> >>
> >> @RedHat   Red Hat  Red Hat
> >
> >
> > Thanks,
> > Ales
>
>
>
> --
> Quique Llorente
>
> CNV networking Senior Software Engineer
>
> Red Hat EMEA
>
> ellor...@redhat.com
>
> @RedHat   Red Hat  Red Hat
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to