On 16/09/2021 21:47, Numan Siddique wrote:
> On Thu, Sep 16, 2021 at 3:06 PM Han Zhou <hz...@ovn.org> wrote:
>>
>> Acked-by: Han Zhou <hz...@ovn.org>
> 
> Thanks Mark G (and Mark M and Han).  I applied both the patches to the
> main branch.

Thanks!
> 
> Numan
> 
>>
>> On Thu, Sep 16, 2021 at 8:49 AM Mark Gray <mark.d.g...@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
>>>
>>>
>>>
>> _______________________________________________
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
> 

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to