[ovs-dev] [PATCH 2/2] acinclude: handle dependencies for DPDK's AF_XDP PMD

2020-02-10 Thread Ciara Loftus
If RTE_LIBRTE_AF_XDP is enabled in the DPDK build, OVS must link the libbpf library, otherwise build failures will occur. Signed-off-by: Ciara Loftus --- acinclude.m4 | 4 1 file changed, 4 insertions(+) diff --git a/acinclude.m4 b/acinclude.m4 index 6f284eaf9..366324778 100644

[ovs-dev] [PATCH 1/2] acinclude: handle dependencies for DPDK's PCAP PMD

2020-02-10 Thread Ciara Loftus
If RTE_LIBRTE_PMD_PCAP is enabled in the DPDK build, OVS must link the pcap library, otherwise build failures will occur. Signed-off-by: Ciara Loftus --- acinclude.m4 | 4 1 file changed, 4 insertions(+) diff --git a/acinclude.m4 b/acinclude.m4 index 1212a463e..6f284eaf9 100644

[ovs-dev] [PATCH v3] Documentation: add notes for TSO & i40e

2020-01-20 Thread Ciara Loftus
for performance tuning when testing TSO with the tool iperf. Fixes: 29cf9c1b3b9c ("userspace: Add TCP Segmentation Offload support") Signed-off-by: Ciara Loftus Acked-by: Flavio Leitner --- v3: - Added Fixes tag - Removed unwanted manpages.mk change v2: - rebased to master - changed p

[ovs-dev] [PATCH v2] Documentation: add notes for TSO & i40e

2020-01-20 Thread Ciara Loftus
for performance tuning when testing TSO with the tool iperf. Signed-off-by: Ciara Loftus Acked-by: Flavio Leitner --- v2: - rebased to master - changed patch links from net-next tree to patchwork --- Documentation/topics/userspace-tso.rst | 27 ++ manpages.mk

[ovs-dev] [PATCH] Documentation: add notes for TSO & i40e

2020-01-13 Thread Ciara Loftus
is supported by OVS. Also, document best known methods for performance tuning when testing TSO with the tool iperf. Signed-off-by: Ciara Loftus --- Documentation/topics/dpdk/tso.rst | 27 +++ 1 file changed, 27 insertions(+) diff --git a/Documentation/topics/dpdk/tso.rst b

[ovs-dev] [RFC PATCH] netdev-dpdk: Integrate vHost User PMD

2018-05-21 Thread Ciara Loftus
' and 'dpdkr' ports, but now applies to all DPDK port types including vHost User. Performance (pps) of the different topologies p2p, pvp, pvvp and vv has been measured to remain within a +/- 5% margin of existing performance. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- To function cor

[ovs-dev] [PATCH v13] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-31 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> Acked-by: Ilya Maximets <i.maxim...@samsung.com> --- v13: * Rebase * Update commit message with recent performance measurement Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-user.rst | 73 ++

[ovs-dev] [PATCH v12] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-24 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- v11: * Move assignment of zc_enabled Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-user.rst | 73 NEWS | 1 + lib/netdev-

[ovs-dev] [PATCH v11] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-24 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- v11: * Rebase * Fix mutex Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-user.rst | 73 NEWS | 1 + lib/netdev-dpdk.c

[ovs-dev] [PATCH v10] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-23 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- v10: * Rebase * Fix vhost flags Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-user.rst | 73 NEWS | 1 + lib/netdev-dpdk.c

[ovs-dev] [PATCH v9] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-19 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- v9: * Rebase * Fix docs issue * Move variable asignment inside mutex * Reset dq-zero-copy value if vhost_driver_register fails Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-user.rs

[ovs-dev] [PATCH v8] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2018-01-05 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- v8: * Disallow configurability after vHost device has been registered & update docs accordingly. * Give performance datapoint in commit message. Documentation/intro/install/dpdk.rst | 2 + Documentation/topics/dpdk/vhost-use

[ovs-dev] [PATCH v7] netdev-dpdk: Add support for vHost dequeue zero copy (experimental)

2017-12-19 Thread Ciara Loftus
: The sum of the tx descriptors of all 'dpdk' ports the VM will send to should not exceed 128. Due to this requirement, the feature is considered 'experimental'. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v7: * Remove support for zc for dpdkvhostuser ports & patch 1 in the series

[ovs-dev] [PATCH v6 2/2] netdev-dpdk: Enable optional dequeue zero copy for vHost User

2017-12-15 Thread Ciara Loftus
descriptors on that 'dpdk' port must be set to a smaller value. 128 is recommended. This can be achieved like so: ovs-vsctl set Interface dpdkport options:n_txq_desc=128 Due to the requirement above, the feature is considered 'experimental'. Signed-off-by: Ciara Loftus <ciara.lof...@intel.

[ovs-dev] [PATCH v6 0/2] vHost Dequeue Zero Copy

2017-12-15 Thread Ciara Loftus
e new feature, so when the VM starts again it doesn't pick up the new device as it hasn't been re-registered in time. Ciara Loftus (2): netdev-dpdk: Helper function for vHost device setup netdev-dpdk: Enable optional dequeue zero copy for vHost User Documentation/howto/dpdk.rst |

[ovs-dev] [PATCH v5 2/2] netdev-dpdk: Enable optional dequeue zero copy for vHost User

2017-12-08 Thread Ciara Loftus
descriptors on that 'dpdk' port must be set to a smaller value. 128 is recommended. This can be achieved like so: ovs-vsctl set Interface dpdkport options:n_txq_desc=128 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v5: * Rebase * Update docs with warning of potential packe

[ovs-dev] [PATCH v5 1/2] netdev-dpdk: Helper function for vHost device setup

2017-12-08 Thread Ciara Loftus
dpdkvhostuser and dpdkvhostuserclient ports share a lot of the same setup & configuration code. Create a common function they can share in order to remove some duplication of code. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dp

[ovs-dev] [PATCH v5 0/2] vHost Dequeue Zero Copy

2017-12-08 Thread Ciara Loftus
ough to allow a vhost device unregister & re-register with the new feature, so when the VM starts again it doesn't pick up the new device as it hasn't been re-registered in time. Ciara Loftus (2): netdev-dpdk: Helper function for vHost device setup netdev-dpdk: Enable optional

[ovs-dev] [PATCH v4 2/2] netdev-dpdk: Enable optional dequeue zero copy for vHost User

2017-11-02 Thread Ciara Loftus
descriptors on that 'dpdk' port must be set to a smaller value. 128 is recommended. This can be achieved like so: ovs-vsctl set Interface dpdkport options:n_txq_desc=128 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v4: * Rebase Documentation/howto/dpdk.rst

[ovs-dev] [PATCH v4 1/2] netdev-dpdk: Helper function for vHost device setup

2017-11-02 Thread Ciara Loftus
dpdkvhostuser and dpdkvhostuserclient ports share a lot of the same setup & configuration code. Create a common function they can share in order to remove some duplication of code. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dp

[ovs-dev] [PATCH v4 0/2] vHost Dequeue Zero Copy

2017-11-02 Thread Ciara Loftus
han a reboot. The reason why this doesn't work is probably because the total downtime during reboot isn't enough to allow a vhost device unregister & re-register with the new feature, so when the VM starts again it doesn't pick up the new device as it hasn't been re-registered in time. Cia

[ovs-dev] [PATCH RFC v2] netdev-dpdk: Allow specification of index for PCI devices

2017-10-17 Thread Ciara Loftus
-devargs option: ovs-vsctl set Interface myport options:dpdk-devargs=:06:00.0,X Where X is an unsigned integer representing one of multiple ports associated with the PCI address provided. This patch has not been tested. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v2: * Si

[ovs-dev] [PATCH v3 2/2] netdev-dpdk: Enable optional dequeue zero copy for vHost User

2017-10-16 Thread Ciara Loftus
descriptors on that 'dpdk' port must be set to a smaller value. 128 is recommended. This can be achieved like so: ovs-vsctl set Interface dpdkport options:n_txq_desc=128 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v3: * Documentation updates: ** Style fixes ** Elaborate on ex

[ovs-dev] [PATCH v3 0/2] vHost Dequeue Zero Copy

2017-10-16 Thread Ciara Loftus
ature, so when the VM starts again it doesn't pick up the new device as it hasn't been re-registered in time. Ciara Loftus (2): netdev-dpdk: Helper function for vHost device setup netdev-dpdk: Enable optional dequeue zero copy for vHost User Documentation/howto/dpdk.rst | 33

[ovs-dev] [PATCH v3 1/2] netdev-dpdk: Helper function for vHost device setup

2017-10-16 Thread Ciara Loftus
dpdkvhostuser and dpdkvhostuserclient ports share a lot of the same setup & configuration code. Create a common function they can share in order to remove some duplication of code. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dp

[ovs-dev] [PATCH v2 2/2] netdev-dpdk: Enable optional dequeue zero copy for vHost User

2017-10-11 Thread Ciara Loftus
descriptors on that 'dpdk' port must be set to a smaller value. 128 is recommended. This can be achieved like so: ovs-vsctl set Interface dpdkport options:n_txq_desc=128 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- Documentation/howto/dpdk.rst

[ovs-dev] [PATCH v2 1/2] netdev-dpdk: Helper function for vHost device setup

2017-10-11 Thread Ciara Loftus
dpdkvhostuser and dpdkvhostuserclient ports share a lot of the same setup & configuration code. Create a common function they can share in order to remove some duplication of code. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dp

[ovs-dev] [PATCH v2 0/2] vHost Dequeue Zero Copy

2017-10-11 Thread Ciara Loftus
egister with the new feature, so when the VM starts again it doesn't pick up the new device as it hasn't been re-registered in time. Ciara Loftus (2): netdev-dpdk: Helper function for vHost device setup netdev-dpdk: Enable optional dequeue zero copy for vHost User Documentation/howt

[ovs-dev] [PATCH RFC] netdev-dpdk: Allow specification of index for PCI devices

2017-10-06 Thread Ciara Loftus
-devargs option: ovs-vsctl set Interface myport options:dpdk-devargs=:06:00.0,X Where X is an unsigned integer representing one of multiple ports associated with the PCI address provided. This patch has not been tested. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-

[ovs-dev] [PATCH 0/2] vHost Dequeue Zero Copy

2017-08-25 Thread Ciara Loftus
stpmd to source (txonly) and sink in the NIC 1C1Q 64B packets: 6.81Mpps -> 7.76Mpps = 13.9% improvement phy -> vhost -> phy No improvement measured Ciara Loftus (2): netdev-dpdk: Helper function for vHost device setup netdev-dpdk: Enable optional dequeue zero copy for vHost User D

[ovs-dev] [PATCH v3] dpif-netdev: Fix insertion probability

2017-06-23 Thread Ciara Loftus
the behavior of the emc_insert_inv_prob setting for high-throughput (large bursts) single-flow cases. Fixes: 4c30b24602c3 ("dpif-netdev: Conditional EMC insert") Reported-by: Kevin Traynor <ktray...@redhat.com> Acked-by: Darrell Ball <dlu...@gmail.com> Signed-off-by: Ciara Loft

[ovs-dev] [PATCH v2] dpif-netdev: Fix insertion probability

2017-06-23 Thread Ciara Loftus
the behavior of the emc_insert_inv_prob setting for high-throughput (large bursts) single-flow cases. Fixes: 4c30b24602c3 ("dpif-netdev: Conditional EMC insert") Reported-by: Kevin Traynor <ktray...@redhat.com> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v2: - Remove

[ovs-dev] [RFC PATCH] netdev-dpdk: Add vHost User PMD

2017-05-30 Thread Ciara Loftus
The vHost PMD allows vHost User ports to be controlled by the DPDK librte_ether API, like 'dpdk' ports and 'dpdkr' ports. This commit integrates this PMD into OVS and removes direct calls to the librte_vhost DPDK library. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/

[ovs-dev] [RFC PATCH] *** SUBJECT HERE ***

2017-05-30 Thread Ciara Loftus
This patch is to be applied on top of the DPDK 17.05 patch: https://mail.openvswitch.org/pipermail/ovs-dev/2017-May/333105.html Previous: https://mail.openvswitch.org/pipermail/ovs-dev/2016-November/325492.html Ciara Loftus (1): netdev-dpdk: Add vHost User PMD lib/dpdk.c| 10 + lib

[ovs-dev] [PATCH v4] dpif-netdev: Change definitions of 'idle' & 'processing' cycles

2017-02-20 Thread Ciara Loftus
Instead of counting all polling cycles as processing cycles, only count the cycles where packets were received from the polling. Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Co-authored-by: Georg Schmuecking <

[ovs-dev] [PATCH v3] dpif-netdev: Change definitions of 'idle' & 'processing' cycles

2017-02-17 Thread Ciara Loftus
Instead of counting all polling cycles as processing cycles, only count the cycles where packets were received from the polling. Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Co-authored-by: Georg Schmuecking <

[ovs-dev] [PATCH v10] dpif-netdev: Conditional EMC insert

2017-02-16 Thread Ciara Loftus
option. This value sets the average probability of insertion to 1/emc-insert-inv-prob. For example the following command changes the insertion probability to (on average) 1 in every 20 packets ie. 1/20 ie. 5%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-inv-prob=20 Signed-off-by: Ciara

[ovs-dev] [PATCH v9] dpif-netdev: Conditional EMC insert

2017-02-14 Thread Ciara Loftus
option. This value sets the average probability of insertion to 1/emc-insert-inv-prob. For example the following command changes the insertion proability to (on average) 1 in every 20 packets ie. 1/20 ie. 5%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-inv-prob=20 Signed-off-by: Ciara

[ovs-dev] [PATCH v8] dpif-netdev: Conditional EMC insert

2017-02-10 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-authored-by: Georg

[ovs-dev] [PATCH v7] dpif-netdev: Conditional EMC insert

2017-02-09 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-authored-by: Georg

[ovs-dev] [PATCH v5] dpif-netdev: Conditional EMC insert

2017-02-09 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-authored-by: Georg

[ovs-dev] [PATCH v4] dpif-netdev: Conditional EMC insert

2017-02-09 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 1/10 ie. 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-author

[ovs-dev] [PATCH v3] dpif-netdev: Conditional EMC insert

2017-01-31 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 1/10 ie. 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-author

[ovs-dev] [PATCH v2] dpif-netdev: Conditional EMC insert

2017-01-26 Thread Ciara Loftus
option. For example the following command increases the insertion probability to 1/10 ie. 10%. ovs-vsctl set Open_vSwitch . other_config:emc-insert-prob=10 Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Co-author

[ovs-dev] [PATCH v2] dpif-netdev: Change definitions of 'idle' & 'processing' cycles

2017-01-17 Thread Ciara Loftus
Instead of counting all polling cycles as processing cycles, only count the cycles where packets were received from the polling. Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Co-authored-by: Ciara Loftu

[ovs-dev] [PATCH 0/1] dpif-netdev: Conditional EMC insert

2017-01-12 Thread Ciara Loftus
This patch is part of the OVS-DPDK performance optimizations presented on the OVS fall conference (http://openvswitch.org/support/ovscon2016/8/1400-gray.pdf) The Exact Match Cache does not perform well in use cases with a high numbers of parallel packet flows. When the flow count exceeds 8k,

[ovs-dev] [PATCH] dpif-netdev: add EMC entry count and %full figure to pmd-stats-show

2017-01-12 Thread Ciara Loftus
'pmd-stats-show' now reports the number of entries in the EMC as well as the percentage full it is. Eg. For 2048 entries the EMC is reported as 25% full as the maximum capacity is 8192 entries. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Georg Schmuecking <georg

[ovs-dev] [PATCH] dpif-netdev: Change definitions of 'idle' & 'processing' cycles

2017-01-12 Thread Ciara Loftus
Instead of counting all polling cycles as processing cycles, only count the cycles where packets were received from the polling. Signed-off-by: Georg Schmuecking <georg.schmueck...@ericsson.com> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Co-authored-by: Ciara Loftu

[ovs-dev] [PATCH v5 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2017-01-05 Thread Ciara Loftus
now must specify the relevant PCI address as input instead of the port name. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> Co-authored-by: Daniele Di Proietto <diproiet...@vmware.com> Signed-off-by: Kevin Traynor &l

[ovs-dev] [PATCH v5 3/3] netdev-dpdk: Add support for virtual DPDK PMDs (vdevs)

2017-01-05 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- Changelog: * Updated process_vdevargs to work with Daniele's incremental in the previous patch. * Allow vdev detach * Update docs to show af_packet example * Fix af_packet docs example * Fix style issues in docs Documentation/howto/dpdk.rs

[ovs-dev] [PATCH v5 1/3] netdev-dpdk: add hotplug support

2017-01-05 Thread Ciara Loftus
quez B <mauricio.vasquezber...@studenti.polito.it> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Co-authored-by: Ciara Loftus <ciara.lof...@intel.com> --- Changelog: * Use xasprintf instead of snprintf when reporting attach/detach errors. * Updated format of link in docs to comply with ne

[ovs-dev] [PATCH v4 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2017-01-04 Thread Ciara Loftus
now must specify the relevant PCI address as input instead of the port name. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> Co-authored-by: Daniele Di Proietto <diproiet...@vmware.com> Signed-off-by: Kevin Traynor &l

[ovs-dev] [PATCH v3 3/3] netdev-dpdk: Add support for virtual DPDK PMDs (vdevs)

2016-12-15 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- Changelog: * Updated process_vdevargs to work with Daniele's incremental in the previous patch. * Allow vdev detach * Update docs to show af_packet example Documentation/intro/install/dpdk-advanced.rst | 27

[ovs-dev] [PATCH v3 1/3] netdev-dpdk: add hotplug support

2016-12-15 Thread Ciara Loftus
the user attaches a new device, it has to be added to a bridge using the add-port command, similarly, before detaching a device, it has to be removed using the del-port command. Signed-off-by: Mauricio Vasquez B <mauricio.vasquezber...@studenti.polito.it> Signed-off-by: Ciara Loftus <

[ovs-dev] [PATCH v3 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-12-15 Thread Ciara Loftus
has changed in that the user now must specify the relevant PCI address as input instead of the port name. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> Co-authored-by: Daniele Di Proietto <diproiet...@vmware.com

[ovs-dev] [PATCH v2 3/3] netdev-dpdk: Add support for virtual DPDK PMDs (vdevs)

2016-12-14 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- Changelog: * Updated process_vdevargs to work with Daniele's incremental in the previous patch. * Allow vdev detach * Update docs to show af_packet example Documentation/intro/install/dpdk-advanced.rst | 27 ++

[ovs-dev] [PATCH v2 1/3] netdev-dpdk: add hotplug support

2016-12-14 Thread Ciara Loftus
the user attaches a new device, it has to be added to a bridge using the add-port command, similarly, before detaching a device, it has to be removed using the del-port command. Signed-off-by: Mauricio Vasquez B <mauricio.vasquezber...@studenti.polito.it> Signed-off-by: Ciara Loftus <

[ovs-dev] [PATCH 3/3] netdev-dpdk: Add support for virtual DPDK PMDs (vdevs)

2016-12-13 Thread Ciara Loftus
-by: Ciara Loftus <ciara.lof...@intel.com> --- Documentation/intro/install/dpdk-advanced.rst | 22 ++ NEWS | 1 + lib/netdev-dpdk.c | 21 + vswitchd/vswitch.xml

[ovs-dev] [PATCH 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-12-13 Thread Ciara Loftus
-detach command has changed in that the user now must specify the relevant PCI address as input instead of the port name. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- Changelog: * Keep port-detach appctl function - use PCI as input arg * Add requires_mutex to devargs processing fun

[ovs-dev] [PATCH 1/3] netdev-dpdk: add hotplug support

2016-12-13 Thread Ciara Loftus
In order to use dpdk ports in ovs they have to be bound to a DPDK compatible driver before ovs is started. This patch adds the possibility to hotplug (or hot-unplug) a device after ovs has been started. The implementation adds two appctl commands: netdev-dpdk/port-attach and

[ovs-dev] [PATCH RFC v4 3/3] netdev-dpdk: Add new 'dpdkvdev' experimental port type

2016-10-28 Thread Ciara Loftus
: ovs-vsctl set Interface null0 options:dpdk-devargs=eth_null0 All virtual DPDK PMDs haven't been tested so this port type can be considered experimental. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK-ADVANCED.md | 23 ++ NEWS

[ovs-dev] [PATCH RFC v4 1/3] netdev-dpdk: add hotplug support

2016-10-28 Thread Ciara Loftus
In order to use dpdk ports in ovs they have to be bound to a DPDK compatible driver before ovs is started. This patch adds the possibility to hotplug (or hot-unplug) a device after ovs has been started. The implementation adds two appctl commands: netdev-dpdk/port-attach and

[ovs-dev] [PATCH RFC v4 0/3] Port Hotplug, Arbitrary 'dpdk' Port Naming & Experimental vdev PMD Support

2016-10-28 Thread Ciara Loftus
This RFC series consists of 3 patches. 1. Port Hotplug (Mauricio Vasquez) (v8) Previous: http://openvswitch.org/pipermail/dev/2016-July/075350.html 2. Arbitrary Port Naming (Ciara Loftus) (v4) Previous: http://openvswitch.org/pipermail/dev/2016-July/075385.html 3. Experimental vDev PMD (Ciara

[ovs-dev] [PATCH RFC v4 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-10-28 Thread Ciara Loftus
-port options:dpdk-devargs=:06:00.3 The user must no longer hotplug DPDK ports by issuing specific ovs-appctl commands. The hotplug is now automatically invoked when a valid PCI address is set in the dpdk-devargs. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTAL

[ovs-dev] [PATCH RFC v6 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-28 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to no code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus <ciara.

[ovs-dev] [PATCH RFC v6 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-28 Thread Ciara Loftus
Host & update commit message Ciara Loftus (2): netdev-dpdk: Add support for DPDK 16.11 netdev-dpdk: Add vHost User PMD .travis/linux-build.sh | 2 +- INSTALL.DPDK-ADVANCED.md | 6 +- INSTALL.DPDK.rst | 18 +- NEWS | 3 + lib/dpdk.c

[ovs-dev] [PATCH RFC v6 2/2] netdev-dpdk: Add vHost User PMD

2016-10-28 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v6: * Unregister callbacks before detach v5: * change vhost_pmd_id to signed int and use -1 value to indicate an ID from the pool hasn't been alloced for

[ovs-dev] [PATCH RFC v5 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-21 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to only minor code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus

[ovs-dev] [PATCH RFC v5 2/2] netdev-dpdk: Add vHost User PMD

2016-10-21 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v5: * change vhost_pmd_id to signed int and use -1 value to indicate an ID from the pool hasn't been alloced for it yet. * free pool ID if rte_eth_dev_attach

[ovs-dev] [PATCH RFC v5 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-21 Thread Ciara Loftus
tdev_dpdk_init fails * Introduce "out" in client_reconfigure for when txq alloc fails * Remove set_tx_multiq fn for vHost ports v2: * Disable indirect desc for vHost & update commit message Ciara Loftus (2): netdev-dpdk: Add support for DPDK 16.11 netdev-dpdk: Add vHost User PMD .t

[ovs-dev] [PATCH RFC v4 2/2] netdev-dpdk: Add vHost User PMD

2016-10-21 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v4: * Use id-pool implementation for allocating vHost PMD IDs v3: * Added DPDK_DEV_VHOST_CLIENT netdev_dpdk "type" * Reintroduced socket-id logic

[ovs-dev] [PATCH RFC v4 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-21 Thread Ciara Loftus
tdev_dpdk_init fails * Introduce "out" in client_reconfigure for when txq alloc fails * Remove set_tx_multiq fn for vHost ports v2: * Disable indirect desc for vHost & update commit message Ciara Loftus (2): netdev-dpdk: Add support for DPDK 16.11 netdev-dpdk: Add vHost User

[ovs-dev] [PATCH RFC v4 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-21 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to only minor code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus

[ovs-dev] [PATCH RFC v3 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-20 Thread Ciara Loftus
reconfigure for when txq alloc fails * Remove set_tx_multiq fn for vHost ports v2: * Disable indirect desc for vHost & update commit message Ciara Loftus (2): netdev-dpdk: Add support for DPDK 16.11 netdev-dpdk: Add vHost User PMD .travis/linux-build.sh |2 +- INSTALL.DPDK-ADVANCED.md

[ovs-dev] [PATCH RFC v3 2/2] netdev-dpdk: Add vHost User PMD

2016-10-20 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v3: * Added DPDK_DEV_VHOST_CLIENT netdev_dpdk "type" * Reintroduced socket-id logic to correctly set client type sid * Removed magic number &a

[ovs-dev] [PATCH RFC v3 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-20 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to only minor code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus

[ovs-dev] [PATCH RFC v2 2/2] netdev-dpdk: Add vHost User PMD

2016-10-14 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK.md | 10 + NEWS |2 + lib/netdev-dpdk.c | 1103 + 3 files change

[ovs-dev] [PATCH RFC v2 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-14 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to only minor code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus

[ovs-dev] [PATCH RFC v2 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-14 Thread Ciara Loftus
: http://dpdk.org/ml/archives/dev/2016-September/046631.html Early feedback on this patchset is important in case we need to submit changes to DPDK (eg. see above) before the release is made in November. v2: * Disable indirect desc for vHost & update commit message Ciara Loftus (2): netdev-

[ovs-dev] [PATCH RFC 2/2] netdev-dpdk: Add vHost User PMD

2016-10-14 Thread Ciara Loftus
available in previous releases, and thus breaks compatibility with such releases. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK.md | 10 + NEWS |2 + lib/netdev-dpdk.c | 1101 + 3 files change

[ovs-dev] [PATCH RFC 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-14 Thread Ciara Loftus
This commit announces support for DPDK 16.11. No code changes are required, so compaitibilty with DPDK v16.07 is not broken yet. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus <ciara.lof...@intel.

[ovs-dev] [PATCH RFC 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-14 Thread Ciara Loftus
/ml/archives/dev/2016-September/046631.html Early feedback on this patchset is important in case we need to submit changes to DPDK (eg. see above) before the release is made in November. Ciara Loftus (2): netdev-dpdk: Add support for DPDK 16.11 netdev-dpdk: Add vHost User PMD .travis/linux

[ovs-dev] [PATCH] dpdk: Fix DPDK pdump compilation

2016-10-13 Thread Ciara Loftus
The rte_pdump header file was not included in the file that requires it. Fix this. Fixes: 01961bbdd34a ("dpdk: New module with some code from netdev-dpdk.") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/dpdk.c| 4 lib/netdev-dpdk.c | 3 --- 2

[ovs-dev] [PATCH] NEWS: Move entry from 'v2.6.0' to 'Post-v2.6.0'

2016-10-04 Thread Ciara Loftus
Configurable queue sizing for DPDK ports did not make the 2.6 release however the NEWS entry suggests it did. Fix this. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- NEWS | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 725754a..04943d4

[ovs-dev] [PATCH v6] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-30 Thread Ciara Loftus
the NIC to restart when changed. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Acked-by: Yunhong Jiang <yunhong.ji...@linux.intel.com> Acked-by: Ilya Maximets <i.maxim...@samsung.com> --- v6: * Reintroduce check for queue_size >= 0 v5: * Rebase * If invalid value is specifi

[ovs-dev] [PATCH v5] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-30 Thread Ciara Loftus
the NIC to restart when changed. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Acked-by: Yunhong Jiang <yunhong.ji...@linux.intel.com> Acked-by: Ilya Maximets <i.maxim...@samsung.com> --- v5: * Rebase * If invalid value is specified, request a default value rather than th

[ovs-dev] [PATCH v4] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-21 Thread Ciara Loftus
the NIC to restart when changed. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Acked-by: Yunhong Jiang <yunhong.ji...@linux.intel.com> --- v4: * Fix typos in vswitchd.xml * Use internal OVS function is_pow2 rather than DPDK function * Additional documentation v3: * Make queue si

[ovs-dev] [PATCH v3] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-20 Thread Ciara Loftus
the NIC to restart when changed. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v3: * Make queue sizes per-port rather than global * Check if queue size is power of 2 - fail if so. v2: * Rebase INSTALL.DPDK-ADVANCED.md | 16 ++-- NEWS | 2 ++ lib/

[ovs-dev] [PATCH v2] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-20 Thread Ciara Loftus
as changing the queue size requires the NIC to restart. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- v2 * Rebase INSTALL.DPDK-ADVANCED.md | 16 ++-- NEWS | 3 +++ lib/netdev-dpdk.c| 35 +++ vswitchd/vswitch.xml

[ovs-dev] [PATCH] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-09 Thread Ciara Loftus
as changing the queue size requires the NIC to restart. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK-ADVANCED.md | 16 ++-- NEWS | 3 +++ lib/netdev-dpdk.c| 35 +++ vswitchd/vswitch.xml

[ovs-dev] [PATCH v2] netdev-dpdk: Add new 'dpdkvhostuserclient' port type

2016-08-19 Thread Ciara Loftus
The 'dpdkvhostuser' port type no longer supports both server and client mode. Instead, 'dpdkvhostuser' ports are always 'server' mode and 'dpdkvhostuserclient' ports are always 'client' mode. Suggested-by: Daniele Di Proietto <diproiet...@vmware.com> Signed-off-by: Ciara Loftus <

[ovs-dev] [PATCH] netdev-dpdk: Add new 'dpdkvhostuserclient' port type

2016-08-18 Thread Ciara Loftus
The 'dpdkvhostuser' port type no longer supports both server and client mode. Instead, 'dpdkvhostuser' ports are always 'server' mode and 'dpdkvhostuserclient' ports are always 'client' mode. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK-ADVANCED.md

[ovs-dev] [PATCH] INSTALL.DPDK-ADVANCED.md: Fix incorrect numbering

2016-08-18 Thread Ciara Loftus
Fixes: 419876444357 ("netdev-dpdk: Remove dpdkvhostcuse ports") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- INSTALL.DPDK-ADVANCED.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/INSTALL.DPDK-ADVANCED.md b/INSTALL.DPDK-ADVANCED.md index 9694003

[ovs-dev] [PATCH] netdev-dpdk: Provide explicit flag to rte_vhost_driver_register

2016-08-18 Thread Ciara Loftus
function is called but happen to be zero. Provide an explicit flag instead for better practice. Fixes: ("c1ff66ac80b5: netdev-dpdk: vHost client mode and reconnect") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dpdk.c | 2 +- 1 file changed, 1 insertio

[ovs-dev] [PATCH] netdev-dpdk: Fix occurance of error log

2016-08-17 Thread Ciara Loftus
If NUMA information can't be derived from a vHost User device, only print an error if the VHOST_NUMA option is enabled in DPDK. Otherwise 'fail' silently. Fixes: 0a0f39df1d5a ("netdev-dpdk: Add support for DPDK 16.07") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Reporte

[ovs-dev] [PATCH v5 3/3] netdev-dpdk: vHost client mode and reconnect

2016-08-15 Thread Ciara Loftus
of its lifetime. QEMU v2.7.0+ is required when using OVS in vHost client mode and QEMU in vHost server mode. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> --- v5: - Added Daniele's incremental change to fix const id - R

[ovs-dev] [PATCH v5 1/3] netdev-dpdk: Remove dpdkvhostcuse ports

2016-08-15 Thread Ciara Loftus
old this should however be a low impact change. Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> Acked-by: Flavio Leitner <f...@sysclose.org> Acked-by: Daniele Di Proietto <diproiet...@vmware.com> Acked-by: Ilya Maximets <i.maxim...@samsung.com> ---

[ovs-dev] [PATCH v5 2/3] netdev-dpdk: Consistent naming for vhost

2016-08-15 Thread Ciara Loftus
ner <f...@sysclose.org> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/netdev-dpdk.c | 25 - 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index 6998452..9866c85 100644 --- a/lib/netdev-dpdk.c +++ b

[ovs-dev] [PATCH v2] netdev-dpdk: Do not attempt to initialise flow control for 'dpdkr' ports

2016-08-15 Thread Ciara Loftus
Only 'dpdk' ports support flow control. This patch stops 'dpdkr' ports from attempting to initialise this feature as this port type does not support it. Fixes: 9fd39370c12c ("netdev-dpdk: Add Flow Control support.") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> --- lib/ne

[ovs-dev] [PATCH] netdev-dpdk: Remove unnecessary 'if' statement

2016-08-12 Thread Ciara Loftus
Only devices of type "DPDK_DEV_ETH" use the netdev_dpdk_set_config function, so no need to check for the device type within the function. Fixes: 9fd39370c12c ("netdev-dpdk: Add Flow Control support.") Signed-off-by: Ciara Loftus <ciara.lof...@intel.com> -

  1   2   3   >