[ovs-dev] [PATCH ovn v5 0/2] northd: Split northd

Han Zhou hzhou at ovn.org
Thu Sep 16 19:05:46 UTC 2021


Acked-by: Han Zhou <hzhou at ovn.org>

On Thu, Sep 16, 2021 at 8:49 AM Mark Gray <mark.d.gray at redhat.com> 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
>
>
>


More information about the dev mailing list