-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NFV EDPM OVS DPDK with Netoworker OVS DPDK #493
base: main
Are you sure you want to change the base?
NFV EDPM OVS DPDK with Netoworker OVS DPDK #493
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Jaganathancse The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8df810e
to
1b854eb
Compare
This changes are to add the deployment VA for NFV EDPM OVS DPDK with Netoworker OVS DPDK nodes.
1b854eb
to
fb71376
Compare
recheck |
edpm_tuned_profile: "cpu-partitioning-powersave" | ||
edpm_tuned_isolated_cores: "2-11,14-23" | ||
edpm_nova_libvirt_qemu_group: "hugetlbfs" | ||
edpm_ovs_dpdk_pmd_core_list: "1,13,2,14,3,15" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A more general comment is about the pmd core list: they must be part of edpm_tuned_isolated_cores (in this case, I see that core 1 is listed, but core 1 isn't isolated.
members: | ||
- type: ovs_dpdk_port | ||
name: dpdk1 | ||
members: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see a bond configuration for that node, but for a future test, it is interesting to enable LACP and balance-tcp for the bond holding the geneve IP endpoint, to spread the traffic across multiple links.
edpm_kernel_args: "default_hugepagesz=1GB hugepagesz=1G hugepages=64 iommu=pt intel_iommu=on tsx=off isolcpus=2-11,14-23" | ||
edpm_tuned_profile: "cpu-partitioning-powersave" | ||
edpm_tuned_isolated_cores: "2-11,14-23" | ||
edpm_ovs_dpdk_pmd_core_list: "1,13,2,14,3,15" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As no VM will be deployed on a networker node, it is interesting to maximise the number of allocated cores for OVS, within reasonable limits.
In that specific conf, edpm_ovs_dpdk_pmd_core_list should be equal to edpm_tuned_isolated_cores.
A more general comment is about the pmd core list: they must be part of edpm_tuned_isolated_cores (in this case, I see that core 1 is listed, but core 1 isn't isolated.
name: dpdk1 | ||
members: | ||
- type: interface | ||
name: nic2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll want to increase the number of physical queues, to spread the traffic across the many PMD threads we have.
This changes are to add the deployment VA for
NFV EDPM OVS DPDK with Netoworker OVS DPDK nodes.