On Thu, Sep 16, 2021 at 3:06 PM Han Zhou <[email protected]> wrote:
>
> Acked-by: Han Zhou <[email protected]>

Thanks Mark G (and Mark M and Han).  I applied both the patches to the
main branch.

Numan

>
> On Thu, Sep 16, 2021 at 8:49 AM Mark Gray <[email protected]> wrote:
>
> > These commits reorganise the northd code base in preparation for potential
> > inclusion
> > of I-P for northd.
> >
> > Please note that this mainly involves moving code around with minimal
> > code changes. However, due to tight coupling between ovn-northd.c and
> > northd.c,
> > some minor changes were needed. For reference, and to help reviews, please
> > examine the following at a minimum:
> >
> > * Configuration of the probe interval in northd.c
> > (ovsdb_idl_set_probe_interval())
> > * Passing of "use_parallel_build" and "lflow_locks" from ovn-northd.c and
> >   northd.c.
> > * Update of "struct northd_context": additon of fields and move to .h file.
> >
> > The commits were (hopefully) structured in a way to make the review
> > easier. As
> > this change touches all of ovn-northd, any change to "master" will make a
> > rebase
> > necessary and probably difficult. Therefore, if the general ideas is OK,
> > then
> > it would be great if this series could be expedited to prevent many
> > rebases!
> >
> > Thanks
> >
> > ---
> > v2: Rebase
> > v3: Rebase. Fixed compile-time error
> > v4: Add additional commits which add framework for incremental processing
> > in northd
> > v5: Remove additional IP commits (
> > https://mail.openvswitch.org/pipermail/ovs-dev/2021-September/387763.html)
> >     Rebase
> >
> > Mark Gray (2):
> >   ovn-northd: Rename ovn-northd.c to northd.c
> >   northd: Split northd.c
> >
> >  Documentation/tutorials/ovn-openstack.rst |   154 +-
> >  northd/automake.mk                        |     2 +
> >  northd/lrouter.dl                         |     2 +-
> >  northd/northd.c                           | 14579 +++++++++++++++++++
> >  northd/northd.h                           |    42 +
> >  northd/ovn-northd.c                       | 14880 +-------------------
> >  northd/ovn.rs                             |     2 +-
> >  northd/ovn_northd.dl                      |     2 +-
> >  tests/ovn-northd.at                       |     2 +-
> >  9 files changed, 14871 insertions(+), 14794 deletions(-)
> >  create mode 100644 northd/northd.c
> >  create mode 100644 northd/northd.h
> >
> > --
> > 2.27.0
> >
> >
> >
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to