On 14/07/2020 16:27, Dumitru Ceara wrote:
On 7/10/20 10:39 AM, anton.iva...@cambridgegreys.com wrote:
Hi all,
This is a series of patches to demo the use in OVN of the parallel
processing of hashes patchset which I submitted to OVS.
The OVS patch series required for this patch can be found here:
https://patchwork.ozlabs.org/project/openvswitch/patch/20200706083650.29443-2-anton.iva...@cambridgegreys.com/
https://patchwork.ozlabs.org/project/openvswitch/patch/20200706083650.29443-3-anton.iva...@cambridgegreys.com/
or here:
https://github.com/kot-begemot-uk/ovs/tree/parallel-submit-final
They improve the performance and stability of ovn in a scale test.
I have used 64+ fake nodes in my scale testing. Your mileage may
vary depending on the performance of the systems used to run the tests.
The patchset covers most parts of ovn-northd which look amenable to
parallel processing. It may be possible to expand it to also cover
initial datapath hash generation and sb_only/nb_only lists, but that
does not look like it is worth it except for extremely large datasets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Hi Anton,
Would you mind rebasing and sending a v2 of this RFC? For some reason
the emails for the first two patches don't show up in patchwork:
I think I know what it is. I will try to find it and fix it in the next
revision. There is something somewhere which is not ASCII clean and I
had UTF8 warnings for these.
https://patchwork.ozlabs.org/project/openvswitch/list/?series=188809
Also, there were changes in OVN master in the meantime and trying to
apply your changes manually also fails due to conflicts in quite a few
places.
That unfortunately will be the case with the next version after a few
commits to master.
As long as the code follows the current layout, any version which tries
to re-organize processing will be very painful to rebase.
It may be a good idea to apply something like patches 1 from this series
(the one that got lost). It splits out the sequential code in the
lswitch and lrouter lflow build into a sequence of procedures.
This makes any future changes much easier to merge.
Thanks,
Dumitru
Brgds,
--
Anton R. Ivanov
Cambridgegreys Limited. Registered in England. Company Number 10273661
https://www.cambridgegreys.com/
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
